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

CommentFileSizeAuthor
#9 rename_views_method-2002902-9.patch1.38 KBmarlatt
#2 drupal-views-2002902.patch0 bytesrobert.laszlo

Comments

robert.laszlo’s picture

robert.laszlo commented
Assigned: Unassigned » robert.laszlo
robert.laszlo’s picture

robert.laszlo commented
StatusFileSize
new drupal-views-2002902.patch0 bytes

My first patch.

robert.laszlo’s picture

robert.laszlo commented
Assigned: robert.laszlo » Unassigned
Status: Active » Needs review

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

The last submitted patch, drupal-views-2002902.patch, failed testing.

attiks’s picture

attiks commented
Status: Needs work » Needs review

#2: drupal-views-2002902.patch queued for re-testing.

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

The last submitted patch, drupal-views-2002902.patch, failed testing.

aaronott’s picture

aaronott commented

This patch is 0 bytes in size (empty). Please upload again.

Thanks

marlatt’s picture

marlatt commented
Assigned: Unassigned » marlatt

Working..

marlatt’s picture

marlatt commented
Assigned: marlatt » Unassigned
Status: Needs work » Needs review
StatusFileSize
new rename_views_method-2002902-9.patch1.38 KB

Method names changed, access modifier added.

oenie’s picture

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 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed ee5a04c and pushed to 8.x. Thanks!

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