Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NonProfit’s picture

Assigned: Unassigned » NonProfit
phenaproxima’s picture

Assigned: NonProfit » phenaproxima
phenaproxima’s picture

Oops - sorry, NonProfit! I think we both took this on at the same moment. I'll take another issue; didn't mean to step on your toes.

phenaproxima’s picture

Assigned: phenaproxima » Unassigned

Durr, forgot to unassign from myself.

NonProfit’s picture

Status: Active » Needs review
FileSize
1001 bytes

Thanks phenaproxima; no worries.

Status: Needs review » Needs work

The last submitted patch, Rename_Views_method-2003288-5.patch, failed testing.

NonProfit’s picture

Assigned: Unassigned » NonProfit
NonProfit’s picture

Status: Needs work » Needs review

#5: Rename_Views_method-2003288-5.patch queued for re-testing.

NonProfit’s picture

Assigned: NonProfit » Unassigned
oenie’s picture

Status: Needs review » Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/query/Sql.phpundefined
@@ -158,7 +158,7 @@ public function init(ViewExecutable $view, DisplayPluginBase $display, array &$o
+  function setDistinct($value = TRUE) {

Add protected access modifier in front of the function to adhere to the new OOP standards.

All occurences have been replaced, so after the correction, the patch should be RTBC...

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
1011 bytes

Added protected access modifier in front of the function to adhere to the new OOP standards.

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

The last submitted patch, views-core-rename-set_distinct-2003288-11.patch, failed testing.

SpartyDan’s picture

Status: Needs work » Needs review
Issue tags: +Novice, +VDC
oenie’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me now !

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 42af4a5 and pushed to 8.x. Thanks!

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