[MAGNOLIA-6077] DefinitionRawView's simple properties should operate on Objects rather than Strings. Created: 16/Feb/15 Updated: 09/May/16 Resolved: 15/Apr/16 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | configuration |
| Affects Version/s: | None |
| Fix Version/s: | 5.4.6 |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Aleksandr Pchelintcev | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 1d | ||
| Original Estimate: | Not Specified | ||
| 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: | |||||||||||||||||
| Epic Link: | Definitions app | ||||||||||||||||
| Sprint: | Basel 39 | ||||||||||||||||
| Story Points: | 3 | ||||||||||||||||
| Description |
|
Could be useful when generating rich raw-view-based config UI where value type actually matters. |
| Comments |
| Comment by Mikaël Geljić [ 10/Sep/15 ] |
|
| Comment by Aleksandr Pchelintcev [ 13/Apr/16 ] |
|
mgeljic yeah, but what if the original data isn't a string itself? =) Returning to this issue as a part of decoration story follow-ups: both in JCR and in Yaml it is possible to specify the type of e.g. property (e.g. numbers is the most obvious example and supported in Yaml without additional hassle). Hence, I propose not to change the things as they are now - by default return string representation of the simple properties, but store values in raw views as just objects and add a method getRawSimpleValue which would return an original parsed object when it is important. |