Ability to Customize Core Error Message

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.

Removing the tests tag as they appear to have been added.
Tagging for framework review to see if this is something they want to allow.
Also usability to see if there will be an issue.

But this needs an issue summary update. Recommend following the default template.

From the looks of the latest patch, #9 is still not addressed. The point isn't to try to implement this without trying to use the Settings class. The point is that calling Settings::get() might throw and exception and explode, and we need to be okay with that. Hence the need for try in here, and fall-back to the hard-coded thing if that function dies.

Reroll for 9.5

Fix for drupal-scaffold test

re-rolled patch for 11.x.

Updated patch to fix the error.
Addressed #61 ✨ Ability to Customize Core Error Message Needs work

Kindly review and suggest.
Thanks.

IS is still incomplete and should follow the standard template for issue. It's adding a value to the settings.php and screenshots should be included.

Also feel the tests should be updated to test that the string is updated

$this->assertStringContainsString($settings->get('custom_error_message'), $response->getContent());

With the catch added and if it's falling back to the default message this would still pass right?

Hope IS is proper and explanatory.

For those using composer and cweagans/composer-patches, "sites/default/default.settings.php" may not exist at time of patching. In this case, we can patch the file before scaffolding at "core/assets/scaffold/files/default.settings.php".

Joining a new patch for this configuration.