Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

closes #2432 #2435

Merged
merged 2 commits into from
May 1, 2017
Merged

closes #2432 #2435

merged 2 commits into from
May 1, 2017

Conversation

crossan007
Copy link
Contributor

@crossan007 crossan007 commented Apr 30, 2017

closes #2432

to test:
try to edit a specific payment on a server with MySQL 5.7 on master branch - it fails hard by printing the failing SQL query in the browser.

it should work with this branch

@ghost ghost assigned crossan007 Apr 30, 2017
@ghost ghost added the In Review label Apr 30, 2017
@crossan007 crossan007 added this to the 2.7.4 milestone Apr 30, 2017
@DawoudIO DawoudIO merged commit 75064ae into master May 1, 2017
@ghost ghost removed the In Review label May 1, 2017
@DawoudIO DawoudIO deleted the bug/2432-fix-payment-editor branch May 1, 2017 02:53
DawoudIO pushed a commit that referenced this pull request May 3, 2017
* log ver

* removed sLogFile setting and updated debug to false by default

* $logger is now alway set

- if debug is on - then use file logger to a local file
- if debug is off - then use a null handler

* new dir for logs but no access from the web

* Split logs into 3

- ORM
- APP
- PHP

* always log PHP and APP Logs

* NullHandler is no longer used

* fixed typo

* added "sLogLevel" from #2278

* log ver (#2416)

* added hasValidMailServerSettings

* using logLevel to enable logs for ORM based on level

* fixed hint for Debug and sLogLevel

* removed debug and using sLogLevel

* finish rev

* enabled sEnableGravatarPhotos (#2427)

* closes #2432 (#2435)

* fix directory date localization (#2444)

* exclude logs in the system files

* Update BaseEmail.php

updated to false

* moved delete into ORM as a pre-delete hook

* delete people for a family is now done via orm

* import cleanup

* fixed delete issues

* re-order and better ORM delete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cannot edit payment!
2 participants