Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards

CommentFileSizeAuthor
#6 views-renderTextarea-2003324-6.patch2.03 KBJeroenT
#4 views-renderTextarea-2003324-4.patch3.4 KBJeroenT
#2 views.core-renamed_render_textarea-2003324-02.patch2.02 KBtoddtomlinson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

toddtomlinson’s picture

toddtomlinson CreditAttribution: toddtomlinson commented
Assigned: Unassigned » toddtomlinson
toddtomlinson’s picture

toddtomlinson CreditAttribution: toddtomlinson commented
Assigned: toddtomlinson » Unassigned
Status: Active » Needs review
FileSize
views.core-renamed_render_textarea-2003324-02.patch2.02 KB

Patch attached for render_textarea rename.

Status: Needs review » Needs work

The last submitted patch, views.core-renamed_render_textarea-2003324-02.patch, failed testing.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
FileSize
views-renderTextarea-2003324-4.patch3.4 KB

Rename render_textarea to renderTextarea.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Status: Needs work » Needs review

marking as needs review.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
FileSize
views-renderTextarea-2003324-6.patch2.03 KB

Rename render_textarea to renderTextarea right patch ;-).

oenie’s picture

oenie CreditAttribution: oenie commented
Status: Needs review » Reviewed & tested by the community

Looks good to me now !

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented

Committed 365637a and pushed to 8.x. Thanks!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Automatically closed -- issue fixed for 2 weeks with no activity.