[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: |
|
||||||||
| Issue Links: |
|
||||||||
| 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". |