Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards
Comment | File | Size | Author |
---|---|---|---|
#9 | rename_views_method-2003368-8.patch | 5.39 KB | marlatt |
#9 | interdiff.txt | 3.14 KB | marlatt |
#8 | rename_views_method-2003368-8.patch | 5.39 KB | marlatt |
#8 | interdiff.txt | 3.14 KB | marlatt |
#6 | views-core-rename-pre_execute-2003368-6.patch | 8.53 KB | SpartyDan |
Comments
Comment #1
NonProfit CreditAttribution: NonProfit commentedComment #2
NonProfit CreditAttribution: NonProfit commentedPatch created.
Comment #4
heddnTests failing. Please investigate and re-roll a patch.
Comment #5
SpartyDan CreditAttribution: SpartyDan commentedre-rolled
Comment #6
SpartyDan CreditAttribution: SpartyDan commentedPrevious patch was not complete.
Comment #8
marlatt CreditAttribution: marlatt commentedIt 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.
Comment #9
marlatt CreditAttribution: marlatt commentedI apologize, the above interdiff was incorrect, here is the patch and interdiff corrected.
Comment #10
lokapujyaPatch in #8 looks good.
Comment #12
marlatt CreditAttribution: marlatt commented#9: rename_views_method-2003368-8.patch queued for re-testing.
Comment #13
lokapujyaGTG, the re-test passed.
Comment #14
somepal CreditAttribution: somepal commentedreviewed, gtg with #8 rename_views_method-2003368-8.patch
Comment #15
alexpottCommitted 022531a and pushed to 8.x. Thanks!