Files: 
CommentFileSizeAuthor
#10 views-core-usesExposed-2002942-10.patch1.75 KBSpartyDan
PASSED: [[SimpleTest]]: [MySQL] 55,610 pass(es).
[ View ]
#8 2002942-08-views-uses_exposed.patch1.75 KBSpartyDan
PASSED: [[SimpleTest]]: [MySQL] 57,617 pass(es).
[ View ]
#4 2002942-04-views-uses_exposed.patch1.74 KBSpartyDan
FAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/drupal.git].
[ View ]
#2 vdc-2002942.patch1.09 KBjastraat
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch vdc-2002942.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Comments

jastraat’s picture

Assigned:Unassigned» jastraat
jastraat’s picture

Status:Active» Needs review
StatusFileSize
new1.09 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch vdc-2002942.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Status:Needs review» Needs work

The last submitted patch, vdc-2002942.patch, failed testing.

SpartyDan’s picture

StatusFileSize
new1.74 KB
FAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/drupal.git].
[ View ]

On line 212 of /views/Plugin/views/display/DisplayPluginBase.php there is an instance of uses_exposed that the patch did not update.

Here is a new patch.

SpartyDan’s picture

Status:Needs work» Needs review

I am changing to needs review because I forgot to change the status when I uploaded the new patch.

Status:Needs review» Needs work

The last submitted patch, 2002942-04-views-uses_exposed.patch, failed testing.

aspilicious’s picture

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.phpundefined
@@ -245,7 +245,7 @@ function exposed_form_validate(&$form, &$form_state) { }
+  function usesExposed() {

Cab you add public to the function.

==> Public function usesExposed().

(both places)

Btw I think this is a terrible name doesn't tell me anything. What is exposed in a pager?

SpartyDan’s picture

Status:Needs work» Needs review
StatusFileSize
new1.75 KB
PASSED: [[SimpleTest]]: [MySQL] 57,617 pass(es).
[ View ]

keyword public added to function.

oenie’s picture

Status:Needs review» Needs work

Unfortunately we need a reroll, the patch doesn't apply anymore.

SpartyDan’s picture

Status:Needs work» Needs review
StatusFileSize
new1.75 KB
PASSED: [[SimpleTest]]: [MySQL] 55,610 pass(es).
[ View ]

re-rolled

aspilicious’s picture

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

Status:Reviewed & tested by the community» Fixed

Committed 8cf5b02 and pushed to 8.x. Thanks!

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