[MGNLUI-3292] Readonly form field property should be case insensitive Created: 10/Dec/14  Updated: 11/Sep/15  Resolved: 11/Sep/15

Status: Closed
Project: Magnolia UI
Component/s: dialogs, forms
Affects Version/s: 5.3.4
Fix Version/s: None

Type: Improvement Priority: Neutral
Reporter: YMC AG Assignee: Unassigned
Resolution: Won't Fix Votes: 0
Labels: dialog, ux
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

OS X Yosemite, JDK 1.7, Tomcat apache-tomcat-7.0.57


Attachments: PNG File Screen Shot 2014-12-10 at 13.38.35.png    
Issue Links:
Relates
relates to MGNLUI-3581 Add the "Definitions" app to the Apps... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Date of First Response:

 Description   

When form fields contain the value "readonly" all in lowercase letters the rendered fields are not displayed as readonly. The user can still enter text inside the field.

When the form fields contain the "readOnly" property then the user can see the field as readonly and cannot enter text which is the correct behavior.

Typing readonly=true or readOnly=true should have the same effect. In both cases the user should not be able to enter text.



 Comments   
Comment by Andreas Weder [ 11/Sep/15 ]

Yes, properties are indeed case sensitive. As a matter of fact, this has nothing to do with JCR, but with node2bean, which maps JCR properties to object properties. Coming up with an algorithm that detects if something was misspelled or is actually still valid could proof to be quite challenging.

We thus think that the solution is not in making properties case insensitive, but in better validating them. We're in the process of introducing a new app, which is going to provide you with an overview over the currently active configuration. This app will show such problems as errors or warnings.

Comment by Andreas Weder [ 11/Sep/15 ]

Since we'll solve this by better validating configuration settings (see my previous comment), I'll close this as "won't fix".

Generated at Mon Feb 12 09:05:09 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.