Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, views-add_additional_fields-rename.patch, failed testing.

alejandrard’s picture

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

Status: Needs review » Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.phpundefined
@@ -115,7 +115,7 @@ public function query() {
+  function addAdditionalFields($fields = NULL) {

Add protected access modifier in front of the method to adher to the new OOP standards.

SpartyDan’s picture

Status: Needs work » Needs review
SpartyDan’s picture

added protected keyword

Status: Needs review » Needs work

The last submitted patch, 2001318-05-views-add_additional_fields-rename.patch, failed testing.

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
6.28 KB

Found additional instances of add_additional_fields that needed to be fixed.

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

The last submitted patch, views-core-add_additional_fields-2001318-7.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 !

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

webchick’s picture

Title: Rename Views method add_additional_fields() to addAdditionalFields() » [Change notice] Rename Views method add_additional_fields() to addAdditionalFields()
Project: Drupal core » Views (for Drupal 7)
Version: 8.x-dev » 8.x-3.x-dev
Component: views.module » Code
Status: Fixed » Active
Issue tags: +Needs change record

Oops. Needs a change notice.

xjm’s picture

Title: [Change notice] Rename Views method add_additional_fields() to addAdditionalFields() » Rename Views method add_additional_fields() to addAdditionalFields()
Project: Views (for Drupal 7) » Drupal core
Version: 8.x-3.x-dev » 8.x-dev
Component: Code » views.module
Assigned: chertzog » Unassigned
Status: Active » Fixed
Issue tags: -Needs change record

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