Currently the code removes the required fields entirely, rather than making them not required. Patch is attached.

CommentFileSizeAuthor
sans_reqd.patch455 bytesdouggreen

Comments

bdragon’s picture

Status: Needs review » Fixed
xolotl’s picture

Status: Fixed » Active

Seems partially fixed.

Required fields now appear on other location groups beyond the first, but they are only required in the first location group.

yesct’s picture

related to #334311: Required fields in multiple locations

Are there three situations to consider?:

1) allow for example 3 locations per node, but only the first location has data filled in, the other two locations are optional (can be blank, or empty, but not allowed to be partially filled in)

2) allow for example 3 locations per node, but only the first location has data filled in, the other two locations are completely optional (can be blank, or empty, and allowed to be partially filled in, dont even require the required fields that are required for the first location)

3) allow for example 3 locations per node, and each location needs to have all the required fields.

Which way does this fix make it work?

Which way are people expecting it to work?

Maybe there needs to be a settings check box that allows the admin to decide which criteria should be applied when nodes can have more than one location.

yesct’s picture

Status: Active » Closed (fixed)

no feedback in more than a year on a need for it to work any way other than the current way. Closing. Please reply with a detailed summary of use cases and how they differ from the current newest official release. If replying, please set the status back to active.