You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by ag...@apache.org on 2006/04/17 21:58:20 UTC

svn commit: r394764 - in /incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi: RollerMemberHandler.java RollerUserHandler.java RollerWeblogHandler.java

Author: agilliland
Date: Mon Apr 17 12:58:18 2006
New Revision: 394764

URL: http://svn.apache.org/viewcvs?rev=394764&view=rev
Log:
removing old backend TODO comments from atomadmin classes and adding new Roller.flush() calls.


Modified:
    incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerMemberHandler.java
    incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerUserHandler.java
    incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerWeblogHandler.java

Modified: incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerMemberHandler.java
URL: http://svn.apache.org/viewcvs/incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerMemberHandler.java?rev=394764&r1=394763&r2=394764&view=diff
==============================================================================
--- incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerMemberHandler.java (original)
+++ incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerMemberHandler.java Mon Apr 17 12:58:18 2006
@@ -269,12 +269,12 @@
         try {
             UserManager mgr = getRoller().getUserManager();
             
-            // TODO BACKEND: new transaction method storePermissions(list)?
             for (int i = 0; i < c.getEntries().length; i++) {
                 MemberEntry entry = (MemberEntry)c.getEntries()[i];
                 PermissionsData pd = toPermissionsData(entry);
                 mgr.storePermissions(pd);
             }
+            getRoller().flush();
         } catch (RollerException re) {
             throw new InternalException("ERROR: Could not create members", re);
         }
@@ -316,7 +316,6 @@
     
     private void updateMembers(MemberEntrySet c) throws HandlerException {
         try {
-            // TODO BACKEND: new transaction method storePermissions(list)?
             for (int i = 0; i < c.getEntries().length; i++) {
                 MemberEntry entry = (MemberEntry)c.getEntries()[i];
                 PermissionsData pd = getPermissionsData(entry);
@@ -325,6 +324,7 @@
                 }
                 updatePermissionsData(pd, entry);
             }
+            getRoller().flush();
         } catch (RollerException re) {
             throw new InternalException("ERROR: Could not update members", re);
         }

Modified: incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerUserHandler.java
URL: http://svn.apache.org/viewcvs/incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerUserHandler.java?rev=394764&r1=394763&r2=394764&view=diff
==============================================================================
--- incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerUserHandler.java (original)
+++ incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerUserHandler.java Mon Apr 17 12:58:18 2006
@@ -171,12 +171,12 @@
         try {
             UserManager mgr = getRoller().getUserManager();
             
-            // TODO BACKEND: new transaction method addUsers(list)?
             for (int i = 0; i < c.getEntries().length; i++) {
                 UserEntry entry = (UserEntry)c.getEntries()[i];
                 UserData user = toUserData(entry);
                 mgr.addUser(user);
             }
+            getRoller().flush();
         } catch (RollerException re) {
             throw new InternalException("ERROR: Could not create users: " + c, re);
         }
@@ -186,7 +186,6 @@
         try {
             UserManager mgr = getRoller().getUserManager();
             
-            // TODO BACKEND: new transaction method storeUsers(list)?
             for (int i = 0; i < c.getEntries().length; i++) {
                 UserEntry entry = (UserEntry)c.getEntries()[i];
                 UserData ud = mgr.getUser(entry.getName());
@@ -197,6 +196,7 @@
                 
                 mgr.storeUser(ud);
             }
+            getRoller().flush();
         } catch (RollerException re) {
             throw new InternalException("ERROR: Could not update users: " + c, re);
         }

Modified: incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerWeblogHandler.java
URL: http://svn.apache.org/viewcvs/incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerWeblogHandler.java?rev=394764&r1=394763&r2=394764&view=diff
==============================================================================
--- incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerWeblogHandler.java (original)
+++ incubator/roller/branches/roller-newbackend/sandbox/atomadminprotocol/src/org/roller/presentation/atomadminapi/RollerWeblogHandler.java Mon Apr 17 12:58:18 2006
@@ -183,12 +183,12 @@
             
             HashMap pages = null; //getRollerContext().readThemeMacros(form.getTheme());
             
-            // TODO BACKEND: new transaction method createWeblogs(list)?
             for (int i = 0; i < c.getEntries().length; i++) {
                 WeblogEntry entry = (WeblogEntry)c.getEntries()[i];
                 UserData user = mgr.getUser(entry.getCreatingUser());
                 WebsiteData wd = mgr.createWebsite(user, pages, entry.getHandle(), entry.getName(), entry.getDescription(), entry.getEmailAddress(), DEFAULT_THEME, entry.getLocale(), entry.getTimezone());
             }
+            getRoller().flush();
         } catch (RollerException re) {
             throw new InternalException("ERROR: Could not create weblogs: " + c, re);
         }
@@ -202,7 +202,6 @@
             
             HashMap pages = null;
             
-            // TODO BACKEND: new transaction method storeWeblogs(list)?
             for (int i = 0; i < c.getEntries().length; i++) {
                 WeblogEntry entry = (WeblogEntry)c.getEntries()[i];
                 WebsiteData wd = mgr.getWebsiteByHandle(entry.getHandle());
@@ -211,6 +210,7 @@
                 }
                 updateWebsiteData(wd, entry);
             }
+            getRoller().flush();
         } catch (RollerException re) {
             throw new InternalException("ERROR: Could not update weblogs: " + c, re);
         }