Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards
Comment | File | Size | Author |
---|---|---|---|
#9 | views-2002498-exposedFormSubmit-8.patch | 2.47 KB | diarmy |
#9 | interdiff.txt | 693 bytes | diarmy |
#7 | views-2002498-exposedFormSubmit-7.patch | 2.47 KB | diarmy |
#7 | interdiff.txt | 885 bytes | diarmy |
#6 | views-2002498-exposedFormSubmit-6.patch | 2.46 KB | diarmy |
Comments
Comment #1
phenaproximaComment #2
phenaproximaThis patch passed the general rendering and exposed form tests on my localhost.
Comment #4
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the functions to adher to the new OOP standards.
Comment #5
diarmy CreditAttribution: diarmy commentedAdded public access modifier in front of the exposedFormSubmit() function in views
Comment #6
diarmy CreditAttribution: diarmy commentedWhoops! Forgot patch. :)
Comment #7
diarmy CreditAttribution: diarmy commentedThis patch adds in second public modifier. #6 only amended one of them.
Comment #9
diarmy CreditAttribution: diarmy commentedPublic modifier had been placed after 'function' in line 246 of PagerPluginBase.php. It has now been placed before.
Comment #10
damiankloip CreditAttribution: damiankloip commentedThank you. This looks good to me.
Comment #11
alexpottCommitted 5202f71 and pushed to 8.x. Thanks!