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 |
---|---|---|---|
#8 | 2003322-8.patch | 1.24 KB | jibran |
#4 | views.core-renamed_render_trim_text-2003322-04.patch | 1.24 KB | borisson_ |
#2 | views.core-renamed_render_trim_text-2003322-02.patch | 1.23 KB | toddtomlinson |
Comments
Comment #1
toddtomlinson CreditAttribution: toddtomlinson commentedComment #2
toddtomlinson CreditAttribution: toddtomlinson commentedPatch to rename render_trim_text attached.
Comment #3
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 #4
borisson_Added "public" modifier to function as mentioned in the parent issue: http://drupal.org/node/1856630
Comment #5
borisson_Changed assignee
Comment #6
elvis2 CreditAttribution: elvis2 commentedLooks good.
Reviewed patch based on the following criteria: http://drupal.org/node/1856630#comment-7450696
Comment #7
alexpottNeeds a re-roll
Comment #8
jibranreroll
Comment #9
heddnLooks good. All mentions of render_trim_text are changed.
Comment #10
heddnComment #11
alexpottCommitted cf185ea and pushed to 8.x. Thanks!