Files: 
CommentFileSizeAuthor
#7 2003332-6.patch1.49 KBjibran
PASSED: [[SimpleTest]]: [MySQL] 55,612 pass(es). View
#4 views-2003332-renderGroupingSets-2003332-04.patch1.5 KBFredericoT
PASSED: [[SimpleTest]]: [MySQL] 57,350 pass(es). View
#2 Rename_Views_method-2003332-2.patch1.51 KBNonProfit
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch Rename_Views_method-2003332-2.patch. Unable to apply patch. See the log in the details link for more information. View

Comments

NonProfit’s picture

Assigned: Unassigned » NonProfit
NonProfit’s picture

Assigned: NonProfit » Unassigned
Status: Active » Needs review
FileSize
1.51 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch Rename_Views_method-2003332-2.patch. Unable to apply patch. See the log in the details link for more information. View

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
PASSED: [[SimpleTest]]: [MySQL] 57,350 pass(es). View
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
PASSED: [[SimpleTest]]: [MySQL] 55,612 pass(es). View

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.