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 |
---|---|---|---|
#10 | view-rename_method-2002974-10.patch | 6.18 KB | danylevskyi |
#7 | VDC-Rename_render_text-2003328-5.patch | 2.4 KB | tvlooy |
#5 | drupal-rename-title-query-2002974-5.patch | 6.18 KB | tvlooy |
#2 | drupal-rename-title-query-2002974-2.patch | 6.11 KB | kenianbei |
Comments
Comment #1
kenianbei CreditAttribution: kenianbei commentedComment #2
kenianbei CreditAttribution: kenianbei commentedComment #3
elvis2 CreditAttribution: elvis2 commentedYou need to add the access modifier (public/private/protected) before the function name. Take a look at the parent issue (http://drupal.org/node/1856630), comments #28, #34, #35 for more details.
You can see an example patch here: http://drupal.org/node/2001672
Comment #4
tvlooy CreditAttribution: tvlooy commentedWork in progress
Comment #5
tvlooy CreditAttribution: tvlooy commentedAdded the access modifiers.
Comment #6
tvlooy CreditAttribution: tvlooy commentedAdded the access modifiers.
Comment #7
tvlooy CreditAttribution: tvlooy commentedWrong post. Please ingore. #5 is the correct one.
Comment #8
elvis2 CreditAttribution: elvis2 commented@tvlooy Please resubmit the correct patch. This will reduce confusion for the person who is approving the patches to be moved into core.
Thanks
Comment #9
danylevskyiComment #10
danylevskyiComment #11
podarok#10 nice conversion
RTBC
Comment #12
heddnApplies cleanly and all occurrences of title_query are now changed.
Comment #13
webchickCommitted and pushed to 8.x. Thanks!
Moving to the Views queue for the change notice.
Comment #14
xjmSingle change notice in the meta.