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

CommentFileSizeAuthor
#3 drupal-camelcase-replace-2003452.3.patch1.14 KBbaldwinlouie
#2 drupal-camelcase-replace-2003452-7451380.patch1.13 KBbaldwinlouie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
Assigned: Unassigned » baldwinlouie

taking it with sillygwailo

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
Status: Active » Needs review
FileSize
drupal-camelcase-replace-2003452-7451380.patch1.13 KB

attaching patch

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
FileSize
drupal-camelcase-replace-2003452.3.patch1.14 KB

rerolling with access modifier

Status: Needs review » Needs work
Issue tags: -Novice, -VDC

The last submitted patch, drupal-camelcase-replace-2003452.3.patch, failed testing.

oenie’s picture

oenie CreditAttribution: oenie commented
Status: Needs work » Needs review
Issue tags: +Novice, +VDC

#3: drupal-camelcase-replace-2003452.3.patch queued for re-testing.

If the test returns OK, the patch is RTBC !

oenie’s picture

oenie CreditAttribution: oenie commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

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

Committed 8ee63be and pushed to 8.x. Thanks!

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