implement JcrPropertyAdapter updating logic
-------------------------------------------

                 Key: MGNLUI-195
                 URL: http://jira.magnolia-cms.com/browse/MGNLUI-195
             Project: Magnolia UI
          Issue Type: Bug
      Security Level: Public
            Reporter: Mikael Geljic
            Assignee: Mikael Geljic
            Priority: Major
             Fix For: 5.0 Alpha1 s009


JcrPropertyAdapter was never used prior to inplace editing, so as in MGNLUI-164 
for reading vaadin Properties out of a JcrPropertyAdapter, we need to be able 
to update the JcrPropertyAdapter back with edited values from these vaadin 
Properties.

First difference to NodeAdapters is that they listen to valueChanges from the 
DefaultProperties they generate. PropertyAdapters should do as well.
 
Then the JcrItem within a PropertyAdapter is currently the parent Node, which 
is wrong. It means we cannot get the JCR Property from the repository after 
init, using the getJcrItem() method from the AbstractJcrAdapter interface.
 
One thing to note is that some refactoring may be necessary to mutualize more 
code between NodeAdapters and PropertyAdapters and review the levels of 
abstraction, which are not clear right now. But this is out of scope for the 
present task.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------

Reply via email to