Dammit, Core!

It looks like I will need to rewrite / duplicate Allowed Values settings code, which will be a HUGE UX fail as there will be TWO form elements describing the same thing (one on the field and one on the widget level). In core, it makes no sense to allow you to remove Allowed Values, but in a select other list it does since you may want to try it out, edit a node, and then get bitten in the butt when you want to take it out.

CommentFileSizeAuthor
#3 1074914-allowed-values.patch875 bytesmradcliffe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mradcliffe’s picture

Or I can remove the validate for that for cck select other widgets?

mradcliffe’s picture

Nope, I am not going to be able to do this either. I think I am screwed either way here.

The only option is to code our own field at this point because core list field is fundamentally broken for contrib modules that want to do funky things like this.

mradcliffe’s picture

FileSize
875 bytes

I guess I can just disable the element_validate for list allowed values. This will allow that functionality, but allow users to remove allowed values from cck_select_other list fields.

It should re-enable the element_validate if the widget changes back.

mradcliffe’s picture

Status: Active » Fixed

Found some more time to run tests in the coder lounge.

Committed.

Status: Fixed » Closed (fixed)

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