PrepareModulesEntityUninstallForm::formTitle does not exist

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

Left a few comments in the MR.
Also needs issue summary update per #33

larowlan → credited dscl → .

@dscl and I worked on this at DrupalSouth
it is ready for review now

Left some comments on the MR

FWIW the test passes without the fix. That's because we're not actually calling any code results in the title callback being used.

Maybe we should go back to #3. Does this page actually need a title callback - hmmm maybe we do as I guess Easy Breadcrumb is using it.

Anyway to test this we need to use the title resolver service on this route. Visiting the page won't trigger the code because of the #title in the form.

Addressed feedback

Only moving to NW for the issue summary update

Remove the _title_callback for this route and just replace it with a static title. The confirm form overrides this value anyway.

Is that still true?

Thanks! Summary now appears to match the MR.

Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!

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