Skip to content
New issue

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

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

Already on GitHub? # to your account

Use output buffering to prevent PHP warnings/notices from returning bad JSON #2820

Closed
Shelob9 opened this issue Nov 21, 2018 · 2 comments
Closed
Assignees
Labels
enhancement New feature Has PR Bug/Feature with a PR created for it.
Milestone

Comments

@Shelob9
Copy link
Collaborator

Shelob9 commented Nov 21, 2018

Would prevent issues like #2448 and #2788

@Shelob9 Shelob9 added the enhancement New feature label Nov 21, 2018
@Shelob9 Shelob9 added this to the 1.8.0 milestone Nov 21, 2018
@Shelob9 Shelob9 self-assigned this Nov 21, 2018
@Shelob9 Shelob9 added the Has PR Bug/Feature with a PR created for it. label Nov 21, 2018
@Shelob9
Copy link
Collaborator Author

Shelob9 commented Nov 21, 2018

Will be fixed via #2822

New0 added a commit that referenced this issue Nov 26, 2018
Error "TypeError: "b.jqxhr.responseJSON is undefined"" will seems to be fixed by #2820
@New0
Copy link
Collaborator

New0 commented Dec 7, 2018

Fixed by #2822

@New0 New0 closed this as completed Dec 7, 2018
Shelob9 pushed a commit that referenced this issue Dec 14, 2018
* develop:
  During form submission, start buffer, then flush it. #2820 and #2448 and #2788
  Set window.cf2 if not an object #2821
  #2814 schedule deletes in cf2 fiel fields and cleanup that handler a bit
  #2814 schedule deletes
  #2814 handle deleting directory if empty
  Only try to delete file, if it exists #2814
  Test to prove that the delete file job will make an uncaught exception in some cases #2814
  Make new job for deleting tests work #2814
  New job for deleting files #2814 - failing tests
  Make new utility methods needed for #2814 and #2794 work
  Failing tests for utility methods needed for #2814  and #2794
  #2814 rm dead code and add inline docs to existing delete job

# Conflicts:
#	cf2/functions.php
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature Has PR Bug/Feature with a PR created for it.
Projects
None yet
Development

No branches or pull requests

2 participants