[
https://issues.jboss.org/browse/GTNPORTAL-511?page=com.atlassian.jira.plu...
]
Tuyen Nguyen The commented on GTNPORTAL-511:
--------------------------------------------
I have sent a PR:
https://github.com/gatein/gatein-portal/pull/688 for this issue.
In this PR, we will sent request to server in synchronization mode, every change in
userPref will wait to completion of current request.
userPref of gadgets are not sometime not saved correctly
--------------------------------------------------------
Key: GTNPORTAL-511
URL:
https://issues.jboss.org/browse/GTNPORTAL-511
Project: GateIn Portal
Issue Type: Bug
Components: User Interface
Affects Versions: 3.0.0-Beta04
Environment: svn r1330
Reporter: jerem j
Assignee: Tuyen Nguyen The
Fix For: 3.5.7.Final
Attachments: GTNPORTAL-511.js.patch, GTNPORTAL-511.patch, stacktrace.txt
Original Estimate: 2 days
Time Spent: 1 day, 7 hours
Remaining Estimate: 1 hour
Every call to gadgets.Prefs.set generate a call to save the data in the backend (an
action in the dashboard portlet). To save we send all the set of preferences.
The jira gadget call this API multiple times in a very short time. So there is multiple
ajax request sent to the backend but not in a precise order. So it might happen that the
first request arrive the last one, and so old preferences are saved.
To fix:
In the file Gadgets.js, function
"gadgets.IfrGadgetService.prototype.setUserPref", prefs is updated with all the
name/value, and ALL the prefs object is saved. Instead of saving all the preferences, we
should only save the updated preferences.
In org.exoplatform.portal.pom.spi.gadget.Gadget it will be needed to be able to add
preference by name, something like :
public void setUserPref(String name, String value)
code :
Gadgets.js:
http://fisheye.exoplatform.org/browse/projects/portal/trunk/web/eXoResour...
ExoBasedUserPrefStore.js:
http://fisheye.exoplatform.org/browse/projects/portal/trunk/web/eXoResour...
links:
API doc about adgets.Prefs.set:
http://code.google.com/apis/gadgets/docs/reference/#gadgets.Prefs.set
Jira gadget to test:
http://jira4j.exoplatform.org/rest/gadgets/1.0/g/com.atlassian.jira.gadge...
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:
http://www.atlassian.com/software/jira