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

CommentFileSizeAuthor
#6 views-core-rename-group_form-2003506-6.patch1.41 KBSpartyDan
#3 drupal-camelcase-replace-2003506.3.patch1.41 KBbaldwinlouie
#2 drupal-camelcase-replace-2003506-7451478.patch3.74 KBbaldwinlouie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
Assigned: Unassigned » baldwinlouie

taking this with sillygwailo

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
Status: Active » Needs review
FileSize
drupal-camelcase-replace-2003506-7451478.patch3.74 KB

attaching patch

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
FileSize
drupal-camelcase-replace-2003506.3.patch1.41 KB

rerolling. Above patch is not going to run.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Patch in #3 seems reasonable.

Status: Reviewed & tested by the community » Needs work

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

SpartyDan’s picture

SpartyDan CreditAttribution: SpartyDan commented
Assigned: baldwinlouie » Unassigned
Status: Needs work » Needs review
FileSize
views-core-rename-group_form-2003506-6.patch1.41 KB

reroll

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Perfect!

alexpott’s picture

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

Committed 4b23474 and pushed to 8.x. Thanks!

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