Author: [email protected]
Date: Wed Feb 8 10:12:10 2012
New Revision: 2071
Log:
[AMDATUOPENSOCIAL-198] Fixed possible NullPointerException
Modified:
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/rest/DashboardRESTServiceImpl.java
Modified:
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/rest/DashboardRESTServiceImpl.java
==============================================================================
---
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/rest/DashboardRESTServiceImpl.java
(original)
+++
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/rest/DashboardRESTServiceImpl.java
Wed Feb 8 10:12:10 2012
@@ -283,8 +283,10 @@
// posted along with the request, but that doesn't mean we
should delete them.
// We need to think about partial updates, for now we just
copy the user preferences
Dashboard oldDb = findDashboard(dashboards, dashboardid);
- for (Gadget oldGadget : oldDb.getGadgets()) {
- findGadget(dashboard,
oldGadget.getId()).setUserPrefs(oldGadget.getUserPrefs());
+ if (oldDb != null) {
+ for (Gadget oldGadget : oldDb.getGadgets()) {
+ findGadget(dashboard,
oldGadget.getId()).setUserPrefs(oldGadget.getUserPrefs());
+ }
}
// Remove the dashboards with the specified id, if it exists.
Then add it.
_______________________________________________
Amdatu-commits mailing list
[email protected]
http://lists.amdatu.org/mailman/listinfo/amdatu-commits
