Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NonProfit’s picture

Assigned: Unassigned » NonProfit
NonProfit’s picture

Assigned: NonProfit » Unassigned
Status: Active » Needs review
FileSize
8.52 KB

Patch created.

Status: Needs review » Needs work

The last submitted patch, Rename_Views_method-2003368-2.patch, failed testing.

heddn’s picture

Tests failing. Please investigate and re-roll a patch.

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
5.69 KB

re-rolled

SpartyDan’s picture

Previous patch was not complete.

Status: Needs review » Needs work

The last submitted patch, views-core-rename-pre_execute-2003368-6.patch, failed testing.

marlatt’s picture

Status: Needs work » Needs review
FileSize
3.14 KB
5.39 KB
+++ b/core/modules/views/lib/Drupal/views/ViewExecutable.php
@@ -1166,7 +1166,7 @@ public function execute($display_id = NULL) {
     // Let modules modify the view just prior to executing it.
     $module_handler = \Drupal::moduleHandler();
-    $module_handler->invokeAll('views_pre_execute', array($this));
+    $module_handler->invokeAll('views_preExecute', array($this));
 

It looks as though changes were made to another function reference outside the scope of the intended change?

Reversed changes to views_pre_execute references and similiar and re-rolled patch.

marlatt’s picture

I apologize, the above interdiff was incorrect, here is the patch and interdiff corrected.

lokapujya’s picture

Status: Needs review » Reviewed & tested by the community

Patch in #8 looks good.

Status: Reviewed & tested by the community » Needs work
Issue tags: -Novice, -VDC

The last submitted patch, rename_views_method-2003368-8.patch, failed testing.

marlatt’s picture

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

#9: rename_views_method-2003368-8.patch queued for re-testing.

lokapujya’s picture

Status: Needs review » Reviewed & tested by the community

GTG, the re-test passed.

somepal’s picture

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

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

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