Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chr.fritsch’s picture

Assigned: Unassigned » chr.fritsch
chr.fritsch’s picture

Assigned: chr.fritsch » Unassigned
Status: Active » Needs review
FileSize
7.38 KB

Status: Needs review » Needs work
aprice42’s picture

Assigned: Unassigned » aprice42
chr.fritsch’s picture

Status: Needs work » Needs review
aprice42’s picture

Assigned: aprice42 » Unassigned

Status: Needs review » Needs work
Issue tags: -Novice, -VDC
jasonyarrington’s picture

Status: Needs work » Needs review

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

Status: Needs work » Needs review
dawehner’s picture

Status: Needs review » Needs work
Issue tags: -VDC +Needs reroll

Test failure was random. ... though the patch needs a rerole.

aaronott’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
7.33 KB

This is a reroll of #2

dawehner’s picture

Status: Needs review » Needs work

You missed the one on core/modules/node/lib/Drupal/node/Plugin/views/field/RevisionLink.php

aaronott’s picture

Cool... i just ran a grep and grabbed a comment as well, looks like this should cover them all. Thanks!

Status: Needs review » Needs work
Issue tags: -Novice
somepal’s picture

Status: Needs work » Needs review
Issue tags: +Novice
dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Thank you!

somepal’s picture

no problem. reviewed patch and good test result too. gtg with #14 views-Rename_Views_method_get_entity_to_getEntity-2002906-14.patch

webchick’s picture

Status: Reviewed & tested by the community » Needs work

Sorry, doesn't apply. :(

jibran’s picture

Status: Needs work » Needs review
Issue tags: +VDC
FileSize
8.29 KB

Reroll
Conflict

++<<<<<<< HEAD
 +    $vid = $this->getValue($values, 'node_vid');
 +    $node = $this->get_entity($values);
++=======
+     $vid = $this->get_value($values, 'node_vid');
+     $node = $this->getEntity($values);
++>>>>>>> 14

Resolved

 -    $vid = $this->get_value($values, 'node_vid');
 +    $vid = $this->getValue($values, 'node_vid');
-     $node = $this->get_entity($values);
+     $node = $this->getEntity($values);
aspilicious’s picture

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

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Needs another reroll

curl https://drupal.org/files/2002906-20.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  8487  100  8487    0     0   6940      0  0:00:01  0:00:01 --:--:--  8106
error: patch failed: core/modules/action/lib/Drupal/action/Plugin/views/field/BulkForm.php:112
error: core/modules/action/lib/Drupal/action/Plugin/views/field/BulkForm.php: patch does not apply
error: patch failed: core/modules/field/lib/Drupal/field/Plugin/views/field/Field.php:644
error: core/modules/field/lib/Drupal/field/Plugin/views/field/Field.php: patch does not apply
jibran’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
7.57 KB

reroll

Status: Needs review » Needs work

The last submitted patch, 2002906-22.patch, failed testing.

jibran’s picture

Status: Needs work » Needs review
FileSize
7.57 KB

Let's try one more time.

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

The last submitted patch, 2002906-25.patch, failed testing.

aspilicious’s picture

Status: Needs work » Needs review

#25: 2002906-25.patch queued for re-testing.

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

The last submitted patch, 2002906-25.patch, failed testing.

pwieck’s picture

Status: Needs work » Needs review
FileSize
2.53 KB

Re-roll

dawehner’s picture

Status: Needs review » Needs work

@pwieck
You touched a yml file and removed all renames, I don't think this will pass ;)

pwieck’s picture

Status: Needs work » Needs review
FileSize
3.34 KB

Re-roll. Made a nub mistake

heddn’s picture

Status: Needs review » Needs work

@pwieck, the patch #31 seems to be missing function renames that were included #20. And also, the scope of this issue is to simply rename the functions.

+++ b/core/modules/views/lib/Drupal/views/Tests/Entity/FieldEntityTest.php
@@ -19,7 +19,7 @@ class FieldEntityTest extends ViewTestBase {
+  public static $testViews = array('test_field_getEntity');

No need to rename variables, etc.

pwieck’s picture

@heddn sorry I'm new will try again

pwieck’s picture

Status: Needs work » Needs review
FileSize
2.19 KB

Re-re roll. I hope I got it this time.

aspilicious’s picture

Status: Needs review » Needs work

Look at the first patches, this one doens't include everything

pwieck’s picture

@aspilicious - I'm a nub. Just to get this straight next time. I take all the 'passed' patches apply them first them search for renames then make patch... Right? I don't want to keep wasting everyones time

pwieck’s picture

@aspilicious - I just read the re-roll docs. Sorry. Will re-roll tonight

chrisguindon’s picture

chrisguindon’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work
pwieck’s picture

I can't seem to find a point in the commit that #20 works on. It keeps failing on RevisionLink.php. Started on June 4th and worked backwards till June 3rd. Am I doing something wrong?

chrisguindon’s picture

Status: Needs work » Needs review
chrisguindon’s picture

@pwieck I was able to apply #20 from commit 3a72626107fb906d41de73aed701ca5104a2c550

Status: Needs review » Needs work
pwieck’s picture

Status: Needs work » Needs review
FileSize
8.31 KB

My last try at this re-roll

pwieck’s picture

Holly Cow! #45 passed

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

i think this is OK now

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 2b51e78 and pushed to 8.x. Thanks!

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