FieldConfigEditForm::save should report exceptions using warning or error messages

Keshav Patel → made their first commit to this issue’s fork.

Changed it to be rendered as error message, instead of status message. Please review.

Looks good to me.

This try catch is interesting - we don't normally do that for generic exceptions and this might not be necessary any more since we do a try catch for EntityStorageExceptions above... but there's notthing wrong in this change. Perhaps we could file a follow-up to remove this try catch and ensure that \Drupal\Tests\field_ui\Functional\ManageFieldsFunctionalTest::testExternalDestinations() still passes and explore the history of why this is there and catches such a generic exception.

Committed 36c3154 and pushed to 11.x. Thanks!
Committed 8097da0 and pushed to 10.3.x. Thanks!
Committed f0e32bb and pushed to 10.2.x. Thanks!

Backported to 10.2.x as this is a non-disruptive bug fix.

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