Support from Acquia helps fund testing for Drupal Acquia logo

Comments

illeace’s picture

Working on this at drupalcon code sprints.

illeace’s picture

Status: Active » Needs review
FileSize
1.49 KB
oenie’s picture

Status: Needs review » Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.phpundefined
@@ -246,7 +246,7 @@ function element_label_type($none_supported = FALSE, $default_empty = FALSE) {
+  function elementWrapperType($none_supported = FALSE, $default_empty = FALSE) {

Add public access modifier in front of the function to adher to the new OOP standards.

JeroenT’s picture

Status: Needs work » Needs review
FileSize
1.5 KB

added public to elementWrapperType function

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 960eb82 and pushed to 8.x. Thanks!

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