curl https://drupal.org/files/2003292-9-rename-set_current_page.patch | git a
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 3633 100 3633 0 0 3838 0 --:--:-- --:--:-- --:--:-- 4693
error: patch failed: core/modules/views/lib/Drupal/views/ViewExecutable.php:727
error: core/modules/views/lib/Drupal/views/ViewExecutable.php: patch does not apply
curl https://drupal.org/files/2003292-12-rename-set_current_page.patch | git a
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 3632 100 3632 0 0 3964 0 --:--:-- --:--:-- --:--:-- 4888
error: patch failed: core/modules/views/lib/Drupal/views/Tests/Plugin/PagerTest.php:311
error: core/modules/views/lib/Drupal/views/Tests/Plugin/PagerTest.php: patch does not apply
Comments
Comment #1
ricky.middaugh commentedI'll take this one.
Comment #2
ricky.middaugh commentedIssue is ready for review.
All instances of set_current_page() have been updated to the proper convention.
Comment #3
ricky.middaugh commentedOops... included the wrong patch. Here's the new one.
Comment #5
heddnFailing tests. Please test and re-roll.
Comment #6
elvis2 commentedFailing is due to an error on the test server side... Re-testing...
Comment #7
elvis2 commented#3: core-rename-set-current-page-2003292-3.patch queued for re-testing.
Comment #8
oenie commentedAdd public access modifier in front of the function(s) to adhere to the new OOP standards.
All occurences seem to have been replaced.
Comment #9
elvis2 commentedRe-rolled.
Comment #10
heddnLooks good now.
Comment #11
alexpottNeeds a re-roll
Comment #12
elvis2 commentedRe-rolled based on most recent pull or 8.x branch.
Comment #13
dawehnerComment #14
alexpottNeeds a re-roll
Comment #15
jibranReroll
Conflict
Resolved
Comment #16
aspilicious commentedComment #17
alexpottCommitted cf23ee6 and pushed to 8.x. Thanks!