Add YAOUS for hiding Patron Reg fields

Bug #1246339 reported by Pasi Kallinen
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Triaged
Wishlist
Unassigned

Bug Description

There are boolean org settings for each of the patron fields in Patron Registration, ui.patron.edit.au.*.show - but setting the value to "false" does not actually hide the field, as I would've expected.

I believe there is need for YAOUS to actually hide unnecessary fields from Patron Reg UI - not to mention the current situation where setting a *.show OUS to "false" actually does nothing, and is confusing.

from IRC discussions:

< eeevil> how about: show=true, display with required; show=false, do not show at all; show=null, default from template
< jeff> while i think it's safe to assume that the user object read from the database can be written back as-is, I think it would be worth adding logic to disable any dijit validation when show=false, to prevent the hidden-but-invalid-to-dojo situation.

Revision history for this message
Mike Rylander (mrylander) wrote :

A followup thought on jeff's suggestion: IMO, that should be a separate !important css rule that keeps certain fields from hiding. That's not completely fool proof, but at least they'll have to actively thwart the intent, and with a comment in the code we can even point a finger at them from the past!

Ben Shum (bshum)
Changed in evergreen:
status: New → Triaged
tags: added: patron registration
Revision history for this message
Jessica Woolford (jwoolford) wrote :

This seems like a duplicate of bug #1815950

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.