#Needs reroll

Renamed (rerolled) patch file to be consistent with the others.

Re-rolled patch.

This is a longish issue, I tried following but different things were going on. I happened to work on a fix about url access and I wanted to share my work here.

My recommendation would be to maybe try to get the access bit first (which is easier) and then something more robust for the breadcrumbs in general?

In anyway, the two MR can co-exists and I will be using on a product of mind.

This is pretty much the same fix that the one #3319445-26: Edit link in breadcrumb region shows even when user has no permission → , and they likely should be applied together.

This could be a setting on the text format. We already have a setting for file size, but it's currently not being applied.
Let's add this for the DG4.0 release.

@deviantintegral thanks very much for the reply!

We've done many, many Drupal sites since this was worked on, and obviously it hasn't come up since. I think this may have been a fixed in core since, or that at the time hook_post_update_NAME wasn't nearly as well known (even though it was in 8.0.0): https://www.drupal.org/node/2563673 →

Let's close this. Thanks for checking in!

Should be an MR

Rerolled the patch for contrib module only.

Checked the methods and yes this still applies to phpunit, as per #32 and #5 we should throw an exception in createRole instead of returning FALSE.

We should start with a reroll of #5 on an MR against 11.x

Needs reroll for contrib only.