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
StatusFileSize
new1.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
StatusFileSize
new1.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
StatusFileSize
new1.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.