Problem/Motivation

If a color field ends up containing an invalid value, the whole page crashes with an exception. Normally this shouldn't be possible since there's validation, but if this validation is bypassed/ignored by creating entities programatically, this could happen. It's not a common thing, but I think it wouldn't be a bad thing anyway to handle invalid field values more gracefully.

Proposed resolution

Display an empty value instead of crashing the whole page if a color field ends up containing an invalid color.

Issue fork color_field-3586279

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dieterholvoet created an issue. See original summary.

dieterholvoet opened merge request !42

dieterholvoet’s picture

dieterholvoet commented
Status: Active » Needs review
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Reviewed & tested by the community

Thanks, nice one! Should this get tests? RTBC from my side!

mandclu’s picture

mandclu commented
Version: 3.0.x-dev » 3.1.x-dev

mandclu’s picture

mandclu commented
Status: Reviewed & tested by the community » Fixed

Thanks for the work here. Merged in, and I'll roll a new release with these changes.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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