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

We need to update the views functions with names that match function naming convention.

I'm starting this off assigned to myself.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ricky.middaugh’s picture

Status: Active » Needs review
FileSize
652 bytes

Issue ready for review. The function name has been updated.

Status: Needs review » Needs work

The last submitted patch, core-rename-get_where_args-function-2003410-1.patch, failed testing.

ricky.middaugh’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, core-rename-get_where_args-function-2003410-1.patch, failed testing.

ricky.middaugh’s picture

Status: Needs work » Needs review
FileSize
652 bytes

Updating with new patch.

Status: Needs review » Needs work

The last submitted patch, core-rename-get_where_args-function-2003410-5.patch, failed testing.

heddn’s picture

There's no calls to get_where_args() so I'm guess that it needs to be public. Otherwise how could contrib module use it? Or maybe it isn't used any more, but that's cruft for another issue. Let's see if the testbot is happy today.

aspilicious’s picture

Status: Needs work » Needs review
SpartyDan’s picture

Assigned: ricky.middaugh » Unassigned

Tested patch to make sure it still applies. Queued patch for testing.

Status: Needs review » Needs work

The last submitted patch, drupal-viewsRename-get_where_args-2003410-7.patch, failed testing.

pcambra’s picture

Status: Needs work » Needs review
FileSize
659 bytes

Rerroll, it should pass?

aspilicious’s picture

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

Status: Reviewed & tested by the community » Fixed

Committed 9791575 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

Adding parent issue