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

Make init functions for CF2 to PHP 5.2 safe #2861

Merged
merged 6 commits into from
Dec 17, 2018
Merged

Make init functions for CF2 to PHP 5.2 safe #2861

merged 6 commits into from
Dec 17, 2018

Conversation

Shelob9
Copy link
Collaborator

@Shelob9 Shelob9 commented Dec 14, 2018

Fixes #2830 by moving the filters, which were anonymous functions used in the one file that must be PHP 5.2 safe to callbacks. Also, I added tests to make sure that the callbacks run.

Josh Pollock added 4 commits December 14, 2018 18:13
* 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
@Shelob9 Shelob9 requested a review from New0 December 14, 2018 23:28
@Shelob9 Shelob9 added this to the 1.8.0 milestone Dec 14, 2018
@Shelob9 Shelob9 self-assigned this Dec 14, 2018
@Shelob9 Shelob9 added the PR (Patch) Is a pull request label Dec 14, 2018
@New0 New0 merged commit 86153a1 into develop Dec 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR (Patch) Is a pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants