If a permission's machine name is the same as its module's machine name, the Extend UI is visually broken

Adding patch for 10.1.x.

Patch #27 is working as accepted. Attaching screenshot.

Still needs tests

dcam → changed the visibility of the branch 2845241-if-a-permissions to hidden.

dcam → changed the visibility of the branch 11.x to hidden.

The patch in #27 was converted to an MR. I also added a test.

Interesting issue. The bug indeed is still around. I can reproduce the issue with the attached test module.

Attached are the screenshots of before and after the MR.

The patch looks straightforward and pipeline looks good as well.

Test looks solid, I think that's good. I wonder if we need a change record for this, but I don't think we do. RTBC++

Thanks small nit on the test documentation

Good catch @nod_

I updated the docblock, but I put the reason why I wrote it the way that I did in an MR comment.

While I agree that adding more functional tests is not great for the total test time duration, having them well documented is important.
We can always change the documentation when we want to add more testing to this.

To me, this change seems like it answers the question by @nod_.

I copied and pasted the $next_row variable name from elsewhere in the test class. I forgot to change it and that name isn't relevant to the new test. It bothered me, so I updated it. This is a minor change that doesn't really affect anything, so I'm leaving the issue at RTBC.

nod_ → closed merge request !11492

Committed 19a37c3 and pushed to 11.x. Thanks!

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