[jboss-svn-commits] JBL Code SVN: r9890 - labs/jbossforums/branches/forums101P24/forums/src/main/org/jboss/portlet/forums/ui.

jboss-svn-commits at lists.jboss.org jboss-svn-commits at lists.jboss.org
Thu Mar 1 17:22:35 EST 2007


Author: unibrew
Date: 2007-03-01 17:22:34 -0500 (Thu, 01 Mar 2007)
New Revision: 9890

Modified:
   labs/jbossforums/branches/forums101P24/forums/src/main/org/jboss/portlet/forums/ui/ForumsJSFPortlet.java
Log:
[JBFORUMS-185] Making VIEW_ID parameters map unmodifiable and not synchronized.

Modified: labs/jbossforums/branches/forums101P24/forums/src/main/org/jboss/portlet/forums/ui/ForumsJSFPortlet.java
===================================================================
--- labs/jbossforums/branches/forums101P24/forums/src/main/org/jboss/portlet/forums/ui/ForumsJSFPortlet.java	2007-03-01 22:21:18 UTC (rev 9889)
+++ labs/jbossforums/branches/forums101P24/forums/src/main/org/jboss/portlet/forums/ui/ForumsJSFPortlet.java	2007-03-01 22:22:34 UTC (rev 9890)
@@ -32,6 +32,7 @@
 import java.util.Iterator;
 import java.util.Map;
 import java.util.TreeMap;
+import java.util.SortedMap;
 
 // JSF
 import javax.faces.component.UIViewRoot;
@@ -74,132 +75,138 @@
     /**
      * This Map contains sorted pairs of view name and view id.
      */
-    public static final Map VIEW_NAME_TO_ID = Collections.synchronizedSortedMap(new TreeMap());
+    public static final Map VIEW_NAME_TO_ID;
     
     /**
      * This Map contains sorted pairs of view id and view name.
      */
-    public static final Map VIEW_ID_TO_NAME = Collections.synchronizedSortedMap(new TreeMap());
+    public static final Map VIEW_ID_TO_NAME;
     
     
     /**
      * Static class initializer will fill VIEW_ID_TO_NAME and VIEW_ID_TO_NAME maps.
      */
     static {
-    
+        
+        SortedMap TEMP_VIEW_ID_TO_NAME = new TreeMap();
+        SortedMap TEMP_VIEW_NAME_TO_ID = new TreeMap();
+        
         // Root views
-        VIEW_ID_TO_NAME.put("i","/views/index.jsf");
-        VIEW_NAME_TO_ID.put("/views/index.jsf","i");
+        TEMP_VIEW_ID_TO_NAME.put("i","/views/index.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/index.jsf","i");
         
-        VIEW_ID_TO_NAME.put("j","/views/jumpbox.jsf");
-        VIEW_NAME_TO_ID.put("/views/jumpbox.jsf","j");
+        TEMP_VIEW_ID_TO_NAME.put("j","/views/jumpbox.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/jumpbox.jsf","j");
         
-        VIEW_ID_TO_NAME.put("m","/views/portal_index.jsf");
-        VIEW_NAME_TO_ID.put("/views/portal_index.jsf","m");
+        TEMP_VIEW_ID_TO_NAME.put("m","/views/portal_index.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/portal_index.jsf","m");
         
         // admin views
-        VIEW_ID_TO_NAME.put("a","/views/admin/index.jsf");
-        VIEW_NAME_TO_ID.put("/views/admin/index.jsf","a");
+        TEMP_VIEW_ID_TO_NAME.put("a","/views/admin/index.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/admin/index.jsf","a");
         
-        VIEW_ID_TO_NAME.put("h","/views/admin/editCategory.jsf");
-        VIEW_NAME_TO_ID.put("/views/admin/editCategory.jsf","h");
+        TEMP_VIEW_ID_TO_NAME.put("h","/views/admin/editCategory.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/admin/editCategory.jsf","h");
         
-        VIEW_ID_TO_NAME.put("k","/views/admin/editForum.jsf");
-        VIEW_NAME_TO_ID.put("/views/admin/editForum.jsf","k");
+        TEMP_VIEW_ID_TO_NAME.put("k","/views/admin/editForum.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/admin/editForum.jsf","k");
         
-        VIEW_ID_TO_NAME.put("l","/views/admin/deleteCategory.jsf");
-        VIEW_NAME_TO_ID.put("/views/admin/deleteCategory.jsf","l");
+        TEMP_VIEW_ID_TO_NAME.put("l","/views/admin/deleteCategory.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/admin/deleteCategory.jsf","l");
         
-        VIEW_ID_TO_NAME.put("d","/views/admin/deleteForum.jsf");
-        VIEW_NAME_TO_ID.put("/views/admin/deleteForum.jsf","d");
+        TEMP_VIEW_ID_TO_NAME.put("d","/views/admin/deleteForum.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/admin/deleteForum.jsf","d");
         
         // category views
-        VIEW_ID_TO_NAME.put("c","/views/category/viewcategory_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/category/viewcategory_body.jsf","c");
+        TEMP_VIEW_ID_TO_NAME.put("c","/views/category/viewcategory_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/category/viewcategory_body.jsf","c");
          
         // common views
-        VIEW_ID_TO_NAME.put("b","/views/common/common.jsf");
-        VIEW_NAME_TO_ID.put("/views/common/common.jsf","b");
+        TEMP_VIEW_ID_TO_NAME.put("b","/views/common/common.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/common/common.jsf","b");
         
-        VIEW_ID_TO_NAME.put("n","/views/common/common_noMenu.jsf");
-        VIEW_NAME_TO_ID.put("/views/common/common_noMenu.jsf","n");
+        TEMP_VIEW_ID_TO_NAME.put("n","/views/common/common_noMenu.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/common/common_noMenu.jsf","n");
         
         // errors views
-        VIEW_ID_TO_NAME.put("e","/views/errors/error_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/errors/error_body.jsf","e");        
+        TEMP_VIEW_ID_TO_NAME.put("e","/views/errors/error_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/errors/error_body.jsf","e");        
         
         // forums views
-        VIEW_ID_TO_NAME.put("f","/views/forums/viewforum_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/forums/viewforum_body.jsf","f");
+        TEMP_VIEW_ID_TO_NAME.put("f","/views/forums/viewforum_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/forums/viewforum_body.jsf","f");
 
         // moderator views
-        VIEW_ID_TO_NAME.put("m","/views/moderator/modcp_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/moderator/modcp_body.jsf","m");
+        TEMP_VIEW_ID_TO_NAME.put("m","/views/moderator/modcp_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/moderator/modcp_body.jsf","m");
 
-        VIEW_ID_TO_NAME.put("v","/views/moderator/delete_topic.jsf");
-        VIEW_NAME_TO_ID.put("/views/moderator/delete_topic.jsf","v");
+        TEMP_VIEW_ID_TO_NAME.put("v","/views/moderator/delete_topic.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/moderator/delete_topic.jsf","v");
         
-        VIEW_ID_TO_NAME.put("q","/views/moderator/modcp_move.jsf");
-        VIEW_NAME_TO_ID.put("/views/moderator/modcp_move.jsf","q");
+        TEMP_VIEW_ID_TO_NAME.put("q","/views/moderator/modcp_move.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/moderator/modcp_move.jsf","q");
         
-        VIEW_ID_TO_NAME.put("o","/views/moderator/modcp_split.jsf");
-        VIEW_NAME_TO_ID.put("/views/moderator/modcp_split.jsf","o");
+        TEMP_VIEW_ID_TO_NAME.put("o","/views/moderator/modcp_split.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/moderator/modcp_split.jsf","o");
         
         // preferences views
-        VIEW_ID_TO_NAME.put("r","/views/pref/index.jsf");
-        VIEW_NAME_TO_ID.put("/views/pref/index.jsf","r");
+        TEMP_VIEW_ID_TO_NAME.put("r","/views/pref/index.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/pref/index.jsf","r");
         
         // profile views
-        VIEW_ID_TO_NAME.put("u","/views/profile/viewprofile_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/profile/viewprofile_body.jsf","u");
+        TEMP_VIEW_ID_TO_NAME.put("u","/views/profile/viewprofile_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/profile/viewprofile_body.jsf","u");
         
-        VIEW_ID_TO_NAME.put("ue","/views/profile/usermessage_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/profile/usermessage_body.jsf","ue");
+        TEMP_VIEW_ID_TO_NAME.put("ue","/views/profile/usermessage_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/profile/usermessage_body.jsf","ue");
         
         // summary views
-        VIEW_ID_TO_NAME.put("s","/views/summary/viewsummary_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/summary/viewsummary_body.jsf","s");
+        TEMP_VIEW_ID_TO_NAME.put("s","/views/summary/viewsummary_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/summary/viewsummary_body.jsf","s");
         
         // topic views
-        VIEW_ID_TO_NAME.put("t","/views/topics/viewtopic_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/viewtopic_body.jsf","t");
+        TEMP_VIEW_ID_TO_NAME.put("t","/views/topics/viewtopic_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/viewtopic_body.jsf","t");
         
-        VIEW_ID_TO_NAME.put("p","/views/topics/posting_new_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/posting_new_body.jsf","p");
+        TEMP_VIEW_ID_TO_NAME.put("p","/views/topics/posting_new_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/posting_new_body.jsf","p");
         
-        VIEW_ID_TO_NAME.put("g","/views/topics/posting_edit_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/posting_edit_body.jsf","g");
+        TEMP_VIEW_ID_TO_NAME.put("g","/views/topics/posting_edit_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/posting_edit_body.jsf","g");
         
-        VIEW_ID_TO_NAME.put("z","/views/topics/posting_reply_body.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/posting_reply_body.jsf","z");
+        TEMP_VIEW_ID_TO_NAME.put("z","/views/topics/posting_reply_body.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/posting_reply_body.jsf","z");
         
-        VIEW_ID_TO_NAME.put("y","/views/topics/delete_poll.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/delete_poll.jsf","y");
+        TEMP_VIEW_ID_TO_NAME.put("y","/views/topics/delete_poll.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/delete_poll.jsf","y");
         
-        VIEW_ID_TO_NAME.put("x","/views/topics/delete_post.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/delete_post.jsf","x");
+        TEMP_VIEW_ID_TO_NAME.put("x","/views/topics/delete_post.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/delete_post.jsf","x");
         
-        VIEW_ID_TO_NAME.put("pa","/views/topics/attachmentsview.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/attachmentsview.jsf","pa");
+        TEMP_VIEW_ID_TO_NAME.put("pa","/views/topics/attachmentsview.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/attachmentsview.jsf","pa");
         
-        VIEW_ID_TO_NAME.put("pe","/views/topics/posting_edit_preview.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/posting_edit_preview.jsf","pe");
+        TEMP_VIEW_ID_TO_NAME.put("pe","/views/topics/posting_edit_preview.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/posting_edit_preview.jsf","pe");
         
-        VIEW_ID_TO_NAME.put("pi","/views/topics/posting_new_preview.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/posting_new_preview.jsf","pi");
+        TEMP_VIEW_ID_TO_NAME.put("pi","/views/topics/posting_new_preview.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/posting_new_preview.jsf","pi");
         
-        VIEW_ID_TO_NAME.put("pr","/views/topics/posting_reply_preview.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/posting_reply_preview.jsf","pr");
+        TEMP_VIEW_ID_TO_NAME.put("pr","/views/topics/posting_reply_preview.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/posting_reply_preview.jsf","pr");
         
-        VIEW_ID_TO_NAME.put("pc","/views/topics/viewtopic_poll_ballot.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/viewtopic_poll_ballot.jsf","pc");
+        TEMP_VIEW_ID_TO_NAME.put("pc","/views/topics/viewtopic_poll_ballot.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/viewtopic_poll_ballot.jsf","pc");
         
-        VIEW_ID_TO_NAME.put("pk","/views/topics/viewtopic_poll_result.jsf");
-        VIEW_NAME_TO_ID.put("/views/topics/viewtopic_poll_result.jsf","pk");
+        TEMP_VIEW_ID_TO_NAME.put("pk","/views/topics/viewtopic_poll_result.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/topics/viewtopic_poll_result.jsf","pk");
         
         // watches views
-        VIEW_ID_TO_NAME.put("w","/views/watches/forumWatch.jsf");
-        VIEW_NAME_TO_ID.put("/views/watches/forumWatch.jsf","w");
+        TEMP_VIEW_ID_TO_NAME.put("w","/views/watches/forumWatch.jsf");
+        TEMP_VIEW_NAME_TO_ID.put("/views/watches/forumWatch.jsf","w");
+        
+        VIEW_ID_TO_NAME = Collections.unmodifiableSortedMap(TEMP_VIEW_ID_TO_NAME);
+        VIEW_NAME_TO_ID = Collections.unmodifiableSortedMap(TEMP_VIEW_NAME_TO_ID);
 
     }
     




More information about the jboss-svn-commits mailing list