Files: 
CommentFileSizeAuthor
#18 2002898_rename-views-method-get_current_page_18.patch4.16 KBsomepal
PASSED: [[SimpleTest]]: [MySQL] 55,627 pass(es).
[ View ]
#16 2002898_rename-views-method-get_current_page_16.patch4.15 KBsomepal
PASSED: [[SimpleTest]]: [MySQL] 55,916 pass(es).
[ View ]
#12 2002898-get_current_page-conversion_12.patch6.41 KBrootwork
Test request sent.
[ View ]
#12 interdiff.txt6.56 KBrootwork
#4 2002898-get_current_page-conversion_4.patch2.79 KBkevin.reiss
FAILED: [[SimpleTest]]: [MySQL] 55,923 pass(es), 1 fail(s), and 0 exception(s).
[ View ]
#2 2002898-get_current_page-conversion.patch2.78 KBPsikik
PASSED: [[SimpleTest]]: [MySQL] 55,902 pass(es).
[ View ]

Comments

Psikik’s picture

Assigned:Unassigned» Psikik
Psikik’s picture

StatusFileSize
new2.78 KB
PASSED: [[SimpleTest]]: [MySQL] 55,902 pass(es).
[ View ]

Converts the get_current_page() function to getCurrentPage and adds the public decorator.

Psikik’s picture

Assigned:Psikik» Unassigned
Status:Active» Needs review
kevin.reiss’s picture

StatusFileSize
new2.79 KB
FAILED: [[SimpleTest]]: [MySQL] 55,923 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

Adding public keyword to method.

shoptalk’s picture

Status:Needs review» Reviewed & tested by the community

Looks good

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

The last submitted patch, 2002898-get_current_page-conversion_4.patch, failed testing.

rootwork’s picture

Status:Needs work» Needs review

Status:Needs review» Needs work

The last submitted patch, 2002898-get_current_page-conversion_4.patch, failed testing.

rootwork’s picture

Status:Needs work» Needs review

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

The last submitted patch, 2002898-get_current_page-conversion_4.patch, failed testing.

rootwork’s picture

Assigned:Unassigned» rootwork

working on this

rootwork’s picture

Assigned:rootwork» Unassigned
Status:Needs work» Needs review
StatusFileSize
new6.56 KB
new6.41 KB
Test request sent.
[ View ]

OK, so I'm pretty sure the patch in #4 is failing because while getCurrentPage was set correctly, the relevant test is also relying on setCurrentPage, which was only partially renamed. The related issue is #2003292: Rename Views method set_current_page() to setCurrentPage().

So just for the purposes of seeing if this will pass the test, I've changed all the references to setCurrentPage as well. I'm attaching an interdiff. If this passes, then #4 will likely pass once #2003292: Rename Views method set_current_page() to setCurrentPage() lands (although conversely that might fail until this one lands).

heddn’s picture

Needs a re-roll. Instructions for "re-rolling" a new patch that applies cleanly.

$ git apply --index 2002898-get_current_page-conversion_12.patch
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/field/Counter.php:50
error: core/modules/views/lib/Drupal/views/Plugin/views/field/Counter.php: patch does not apply
error: patch failed: core/modules/views/lib/Drupal/views/ViewExecutable.php:753
error: core/modules/views/lib/Drupal/views/ViewExecutable.php: patch does not apply

aspilicious’s picture

Status:Needs review» Needs work
somepal’s picture

Assigned:Unassigned» somepal

I am taking this.

somepal’s picture

Status:Needs work» Needs review
StatusFileSize
new4.15 KB
PASSED: [[SimpleTest]]: [MySQL] 55,916 pass(es).
[ View ]

I might need to change the setter as well, but want to try this.

dawehner’s picture

Status:Needs review» Needs work

Thank you. Yeah please not do both at the same time.

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.phpundefined
@@ -103,7 +103,7 @@ function set_offset($offset) {
+  function getCurrentPage() {

Should be a public method.

somepal’s picture

Status:Needs work» Needs review
StatusFileSize
new4.16 KB
PASSED: [[SimpleTest]]: [MySQL] 55,627 pass(es).
[ View ]

yeah, sorry I missed that. here's the re-roll, made it public.

dawehner’s picture

Status:Needs review» Reviewed & tested by the community

Thank you very much!

webchick’s picture

Title:Rename Views method get_current_page() to getCurrentPage()» Change notice: Rename Views method get_current_page() to getCurrentPage()
Project:Drupal core» Views
Version:8.x-dev» 8.x-3.x-dev
Component:views.module» Code
Status:Reviewed & tested by the community» Active
Issue tags:+Needs change record

Committed and pushed to 8.x. Thanks!

Moving to the Views queue for the change notice.

xjm’s picture

Title:Change notice: Rename Views method get_current_page() to getCurrentPage()» Rename Views method get_current_page() to getCurrentPage()
Project:Views» Drupal core
Version:8.x-3.x-dev» 8.x-dev
Component:Code» views.module
Assigned:somepal» Unassigned
Status:Active» Fixed
Issue tags:-Needs change record

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