Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sillygwailo’s picture

Assigned: Unassigned » sillygwailo

baldwinlouie and I are taking this one.

sillygwailo’s picture

Here's a patch.

sillygwailo’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2002918-rename-get_items.patch, failed testing.

Anonymous’s picture

Assigned: sillygwailo »

Going to re-roll.

Anonymous’s picture

Assigned: » Unassigned
Status: Needs work » Needs review
FileSize
606 bytes

I rolled a new patch and added the public keyword to the function, per the feedback on this issue: http://drupal.org/node/2002460

Status: Needs review » Needs work

The last submitted patch, core-rename-views-method-get-items-2002918-7.patch, failed testing.

oenie’s picture

Status: Needs work » Needs review
FileSize
2.03 KB

To avoid confusion over the previous patches, i'm rerolling patch #2 with corrections on the acces modifiers for the changed functions.

oenie’s picture

Assigned: Unassigned » oenie
heddn’s picture

Status: Needs review » Needs work

Needs a re-roll.

$ git apply 2002918-rename-get_items-3.patch
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php:1110
error: core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php: patch does not apply

sillygwailo’s picture

Status: Needs work » Needs review

Re-rolled against 8.x. Might need another re-roll when the advanced_render() change is committed.

sillygwailo’s picture

Patch attached.

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

The last submitted patch, 2002918-rename-get_items.patch, failed testing.

sillygwailo’s picture

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

#12: 2002918-rename-get_items.patch queued for re-testing.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

:D thnx!

somepal’s picture

reviewed and checked all the calls. gtg with #12 2002918-rename-get_items.patch

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Needs a reroll

curl https://drupal.org/files/2002918-rename-get_items_0.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2079  100  2079    0     0   2791      0 --:--:-- --:--:-- --:--:--  3640
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php:1106
error: core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php: patch does not apply
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/field/PrerenderList.php:99
error: core/modules/views/lib/Drupal/views/Plugin/views/field/PrerenderList.php: patch does not apply
jibran’s picture

Status: Needs work » Needs review
FileSize
2.03 KB

Re-roll
Conflict

++<<<<<<< HEAD
 +  function get_items($values) {
 +    $field = $this->getValue($values);
++=======
+   protected function getItems($values) {
+     $field = $this->get_value($values);
++>>>>>>> 12

Resolved

-   function get_items($values) {
 -  protected function getItems($values) {
 -    $field = $this->get_value($values);
++  function getItems($values) {
 +    $field = $this->getValue($values);
aspilicious’s picture

protected function getItems($values) {

oenie’s picture

Assigned: oenie » Unassigned
sillygwailo’s picture

Re-rolled with protected in the PrerenderList.php.

Status: Needs review » Needs work

The last submitted patch, 2002918-rename-get_items.patch, failed testing.

sillygwailo’s picture

Status: Needs work » Needs review
FileSize
2.04 KB

Re-roll.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Perfect

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed ac6bbe4 and pushed to 8.x. Thanks!

tim.plunkett’s picture

Status: Fixed » Needs review
FileSize
1.3 KB

I started porting FullCalendar to D8, and this bit me hard.
I remember adding this method to D7 so contrib could use it.

It very much needs to be public.

aspilicious’s picture

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

Status: Reviewed & tested by the community » Fixed

Committed a24345a and pushed to 8.x. Thanks!

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