Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NonProfit’s picture

Assigned: Unassigned » NonProfit
NonProfit’s picture

Assigned: NonProfit » Unassigned
Status: Active » Needs review
FileSize
1.51 KB

Patch created

Status: Needs review » Needs work

The last submitted patch, Rename_Views_method-2003332-2.patch, failed testing.

FredericoT’s picture

Status: Needs work » Needs review
FileSize
1.5 KB
oenie’s picture

Status: Needs review » Needs work

Unfortunately we need a reroll, the patch doesn't apply anymore.

FredericoT’s picture

This is my first time attempting a reroll, but while following the instructions on the link you provided. Step 7 didn't give any conflicts on my patch.
Wouldn't that mean it's ok? Or perhaps more likely, I didn't follow the instructions correctly then?

jibran’s picture

Status: Needs work » Needs review
FileSize
1.49 KB

reroll

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

The last submitted patch, 2003332-6.patch, failed testing.

SpartyDan’s picture

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

#7: 2003332-6.patch queued for re-testing.

oenie’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me now !

Apart from a minor issue with a comment reference:

core/modules/views/lib/Drupal/views/Plugin/views/style/StylePluginBase.php, line 93:
* @see StylePluginBase::render_grouping_sets()

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed bf36b09 and pushed to 8.x. Thanks!

Hmmm... due to #10 this should have gone to needs work since that is API documentation that would have broke api.d.o - fixed during commit in interests of getting stuff done(tm)...

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