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 |
---|---|---|---|
#6 | VDC-Rename_render_text-2003328-6.patch | 2.4 KB | tvlooy |
#2 | VDC-Rename_render_text-2003328-2.patch | 2.39 KB | seantwalsh |
Comments
Comment #1
seantwalshWorking on this as part of the Portland2013 getting involved with core sprint.
Comment #2
seantwalshReplaced 7 instances of render_text with renderText.
Comment #3
seantwalshUpdated as I forgot to change to needs review.
Comment #4
elvis2 CreditAttribution: elvis2 commentedThanks for working on this. You 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 #5
tvlooy CreditAttribution: tvlooy commentedWork in progress
Comment #6
tvlooy CreditAttribution: tvlooy commentedAdded the access modifiers.
Comment #7
elvis2 CreditAttribution: elvis2 commentedLooks good.
Reviewed patch based on the following criteria: http://drupal.org/node/1856630#comment-7450696
Comment #8
alexpottCommitted f6e4b9b and pushed to 8.x. Thanks!