Author: solomax
Date: Tue May 28 11:49:49 2013
New Revision: 1486889

URL: http://svn.apache.org/r1486889
Log:
[OPENMEETINGS-650] room creation/editing works as expected

Modified:
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/AdminBaseForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/configurations/ConfigForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/groups/GroupForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/labels/LabelsForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/ldaps/LdapForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomsPanel.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/servers/ServerForm.java
    
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/users/UserForm.java

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/AdminBaseForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/AdminBaseForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/AdminBaseForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/AdminBaseForm.java
 Tue May 28 11:49:49 2013
@@ -30,7 +30,7 @@ import org.apache.wicket.model.IModel;
  * 
  * @param <T>
  */
-public class AdminBaseForm<T> extends Form<T> {
+public abstract class AdminBaseForm<T> extends Form<T> {
        private static final long serialVersionUID = 1L;
        private AdminSavePanel<T> savePanel;
 
@@ -106,8 +106,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onSaveSubmit(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onSaveSubmit(AjaxRequestTarget target, Form<?> 
form);
        
        /**
         * invoked when save has error
@@ -115,8 +114,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onSaveError(AjaxRequestTarget target, Form<?> 
form);
        
        /**
         * invoked when new button is pressed
@@ -124,8 +122,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onNewSubmit(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onNewSubmit(AjaxRequestTarget target, Form<?> 
form);
        
        /**
         * invoked if new has error
@@ -133,8 +130,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onNewError(AjaxRequestTarget target, Form<?> 
form);
        
        /**
         * invoked when refresh button is pressed
@@ -142,8 +138,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onRefreshSubmit(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onRefreshSubmit(AjaxRequestTarget target, 
Form<?> form);
        
        /**
         * invoked when refresh has error
@@ -151,8 +146,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onRefreshError(AjaxRequestTarget target, 
Form<?> form);
        
        /**
         * invoked when delete button is pressed
@@ -160,8 +154,7 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onDeleteSubmit(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onDeleteSubmit(AjaxRequestTarget target, 
Form<?> form);
        
        /**
         * invoked when delete has error
@@ -169,7 +162,6 @@ public class AdminBaseForm<T> extends Fo
         * @param target
         * @param form
         */
-       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
-       }
+       protected abstract void onDeleteError(AjaxRequestTarget target, Form<?> 
form);
        
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/configurations/ConfigForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/configurations/ConfigForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/configurations/ConfigForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/configurations/ConfigForm.java
 Tue May 28 11:49:49 2013
@@ -114,4 +114,24 @@ public class ConfigForm extends AdminBas
                target.add(listContainer);
                refresh(target);
        }
+
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/groups/GroupForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/groups/GroupForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/groups/GroupForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/groups/GroupForm.java
 Tue May 28 11:49:49 2013
@@ -18,11 +18,12 @@
  */
 package org.apache.openmeetings.web.admin.groups;
 
+import static org.apache.openmeetings.web.app.Application.getBean;
+import static org.apache.openmeetings.web.app.WebSession.getUserId;
+
 import org.apache.openmeetings.data.user.dao.OrganisationDao;
 import org.apache.openmeetings.persistence.beans.domain.Organisation;
 import org.apache.openmeetings.web.admin.AdminBaseForm;
-import org.apache.openmeetings.web.app.Application;
-import org.apache.openmeetings.web.app.WebSession;
 import org.apache.openmeetings.web.common.OmAjaxFormValidatingBehavior;
 import org.apache.wicket.ajax.AjaxRequestTarget;
 import org.apache.wicket.markup.html.WebMarkupContainer;
@@ -70,7 +71,7 @@ public class GroupForm extends AdminBase
        protected void onRefreshSubmit(AjaxRequestTarget target, Form<?> form) {
                Organisation org = getModelObject();
                if (org.getOrganisation_id() != null) {
-                       org = 
Application.getBean(OrganisationDao.class).get(org.getOrganisation_id());
+                       org = 
getBean(OrganisationDao.class).get(org.getOrganisation_id());
                } else {
                        org = new Organisation();
                }
@@ -80,16 +81,36 @@ public class GroupForm extends AdminBase
        
        @Override
        protected void onDeleteSubmit(AjaxRequestTarget target, Form<?> form) {
-               
Application.getBean(OrganisationDao.class).delete(getModelObject(), 
WebSession.getUserId());
+               getBean(OrganisationDao.class).delete(getModelObject(), 
getUserId());
                target.add(groupList);
                target.appendJavaScript("groupsInit();");
        }
        
        @Override
        protected void onSaveSubmit(AjaxRequestTarget target, Form<?> form) {
-               
Application.getBean(OrganisationDao.class).update(getModelObject(), 
WebSession.getUserId());
+               getBean(OrganisationDao.class).update(getModelObject(), 
getUserId());
                hideNewRecord();
                target.add(groupList);
                target.appendJavaScript("groupsInit();");
        }
+
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/labels/LabelsForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/labels/LabelsForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/labels/LabelsForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/labels/LabelsForm.java
 Tue May 28 11:49:49 2013
@@ -96,4 +96,23 @@ public class LabelsForm extends AdminBas
                target.appendJavaScript("labelsInit();");
        }
 
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/ldaps/LdapForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/ldaps/LdapForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/ldaps/LdapForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/ldaps/LdapForm.java
 Tue May 28 11:49:49 2013
@@ -113,4 +113,23 @@ public class LdapForm extends AdminBaseF
                target.appendJavaScript("omLdapPanelInit();");
        }
 
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomForm.java
 Tue May 28 11:49:49 2013
@@ -18,10 +18,15 @@
  */
 package org.apache.openmeetings.web.admin.rooms;
 
+import static org.apache.openmeetings.web.app.Application.getBean;
+import static org.apache.openmeetings.web.app.WebSession.getUserId;
+
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;
 
 import org.apache.openmeetings.data.conference.RoomManager;
+import org.apache.openmeetings.data.conference.dao.RoomDao;
 import org.apache.openmeetings.data.user.OrganisationManager;
 import org.apache.openmeetings.persistence.beans.domain.Organisation;
 import org.apache.openmeetings.persistence.beans.room.Room;
@@ -31,9 +36,12 @@ import org.apache.openmeetings.web.admin
 import org.apache.openmeetings.web.app.Application;
 import org.apache.openmeetings.web.app.WebSession;
 import org.apache.openmeetings.web.common.OmAjaxFormValidatingBehavior;
+import org.apache.wicket.ajax.AjaxRequestTarget;
+import org.apache.wicket.markup.html.WebMarkupContainer;
 import org.apache.wicket.markup.html.form.CheckBox;
 import org.apache.wicket.markup.html.form.ChoiceRenderer;
 import org.apache.wicket.markup.html.form.DropDownChoice;
+import org.apache.wicket.markup.html.form.Form;
 import org.apache.wicket.markup.html.form.IChoiceRenderer;
 import org.apache.wicket.markup.html.form.ListMultipleChoice;
 import org.apache.wicket.markup.html.form.RequiredTextField;
@@ -43,19 +51,15 @@ import org.apache.wicket.model.CompoundP
 import org.apache.wicket.model.Model;
 import org.apache.wicket.util.time.Duration;
 
-import edu.emory.mathcs.backport.java.util.Arrays;
-
 public class RoomForm extends AdminBaseForm<Room> {
-
        private static final long serialVersionUID = 1L;
+       private final static List<Long> DROPDOWN_NUMBER_OF_PARTICIPANTS = 
Arrays.asList(2L, 4L, 6L, 8L, 10L, 12L, 14L, 16L, 20L, 25L, 32L, 50L,
+                       100L, 150L, 200L, 500L, 1000L);
+       private final WebMarkupContainer roomList;
 
-       Object[] array = { 2L, 4L, 6L, 8L, 10L, 12L, 14L, 16L, 20L, 25L, 32L, 
50L,
-                       100L, 150L, 200L, 500L, 1000L };
-       @SuppressWarnings("unchecked")
-       List<Long> DROPDOWN_NUMBER_OF_PARTICIPANTS = Arrays.asList(array);
-
-       public RoomForm(String id, final Room room) {
+       public RoomForm(String id, WebMarkupContainer roomList, final Room 
room) {
                super(id, new CompoundPropertyModel<Room>(room));
+               this.roomList = roomList;
                setOutputMarkupId(true);
                RequiredTextField<String> name = new 
RequiredTextField<String>("name");
                name.setLabel(new Model<String>(WebSession.getString(193)));
@@ -129,4 +133,62 @@ public class RoomForm extends AdminBaseF
                // event and throttle it down to once per second
                OmAjaxFormValidatingBehavior.addToAllFormComponents(this, 
"keydown", Duration.ONE_SECOND);
        }
+
+       private void updateView(AjaxRequestTarget target) {
+               target.add(this);
+               target.add(roomList);
+               target.appendJavaScript("omRoomPanelInit();");
+       }
+       
+       @Override
+       protected void onSaveSubmit(AjaxRequestTarget target, Form<?> form) {
+               getBean(RoomDao.class).update(getModelObject(), getUserId());
+               hideNewRecord();
+               updateView(target);
+       }
+
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewSubmit(AjaxRequestTarget target, Form<?> form) {
+               setModelObject(new Room());
+               updateView(target);
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshSubmit(AjaxRequestTarget target, Form<?> form) {
+               Room r = getModelObject();
+               if (r.getRooms_id() != null) {
+                       r = getBean(RoomDao.class).get(r.getRooms_id());
+               } else {
+                       r = new Room();
+               }
+               setModelObject(r);
+               updateView(target);
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteSubmit(AjaxRequestTarget target, Form<?> form) {
+               getBean(RoomDao.class).delete(getModelObject(), getUserId());
+               target.add(roomList);
+               updateView(target);
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomsPanel.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomsPanel.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomsPanel.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/rooms/RoomsPanel.java
 Tue May 28 11:49:49 2013
@@ -24,8 +24,8 @@ import org.apache.openmeetings.web.admin
 import org.apache.openmeetings.web.admin.SearchableDataView;
 import org.apache.openmeetings.web.common.PagedEntityListPanel;
 import org.apache.openmeetings.web.data.DataViewContainer;
-import org.apache.openmeetings.web.data.SearchableDataProvider;
 import org.apache.openmeetings.web.data.OrderByBorder;
+import org.apache.openmeetings.web.data.SearchableDataProvider;
 import org.apache.wicket.AttributeModifier;
 import org.apache.wicket.ajax.AjaxEventBehavior;
 import org.apache.wicket.ajax.AjaxRequestTarget;
@@ -85,7 +85,7 @@ public class RoomsPanel extends AdminPan
                        }
                });
                
-               form = new RoomForm("form", new Room());
+               form = new RoomForm("form", listContainer, new Room());
                form.showNewRecord();
         add(form);
                

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/servers/ServerForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/servers/ServerForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/servers/ServerForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/servers/ServerForm.java
 Tue May 28 11:49:49 2013
@@ -121,4 +121,23 @@ public class ServerForm extends AdminBas
                target.appendJavaScript("omServerPanelInit();");
        }
 
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }

Modified: 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/users/UserForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/users/UserForm.java?rev=1486889&r1=1486888&r2=1486889&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/users/UserForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/web/admin/users/UserForm.java
 Tue May 28 11:49:49 2013
@@ -199,4 +199,24 @@ public class UserForm extends AdminBaseF
                        error(WebSession.getString(105));
                }
        }
+
+       @Override
+       protected void onSaveError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onNewError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onRefreshError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
+
+       @Override
+       protected void onDeleteError(AjaxRequestTarget target, Form<?> form) {
+               // TODO Auto-generated method stub
+       }
 }


Reply via email to