[ http://jira.magnolia.info/browse/MAGNOLIA-1249?page=comments#action_12934 
] 
            
Sameer Charles commented on MAGNOLIA-1249:
------------------------------------------

for the time I used singleton ExclusiveWrite as used in onSave to synchronize

Problem with synchronize on ExclusiveWrite is that it synchronizes on the 
entire system, which means no save operations are allowed (even in other 
workspaces) when you are saving anything with the gui.
I think we should use content.lock() to synchronize between sessions.

atleast it does not break now, but leave this issue open since this is true for 
all dialogs in magnolia.


> User and Roles dialog boxes does not synchronize saving between sessions
> ------------------------------------------------------------------------
>
>                 Key: MAGNOLIA-1249
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-1249
>             Project: Magnolia
>          Issue Type: Bug
>          Components: admininterface
>    Affects Versions: 3.0 Final
>            Reporter: Sameer Charles
>         Assigned To: Philipp Bracher
>            Priority: Blocker
>             Fix For: 3.0.1
>
>
> post save methods in all gui dialogs does not synchronize, as a result either 
> you have a broken session or a broken node when using jackrabbit
> to reproduce simply try to edit single user or role from 2 different authors.
> its the reposibility of the application to synchronize rather then the core 
> api or the repository, 
> either use jcr locking or atleast synchronize on singleton object

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

        

----------------------------------------------------------------
for list details see
http://www.magnolia.info/en/magnolia/developer.html
----------------------------------------------------------------

Reply via email to