Author: solomax
Date: Sun Feb  2 13:08:14 2014
New Revision: 1563609

URL: http://svn.apache.org/r1563609
Log:
[OPENMEETINGS-892] "Add user" button is enabled on group save

Modified:
    
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
    
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
    
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
    
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java

Modified: 
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java?rev=1563609&r1=1563608&r2=1563609&view=diff
==============================================================================
--- 
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
 (original)
+++ 
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
 Sun Feb  2 13:08:14 2014
@@ -62,7 +62,7 @@ public class GroupForm extends AdminComm
        @Override
        public void updateView(AjaxRequestTarget target) {
                usersPanel.update(getOrgId());
-               target.add(this);
+               target.add(this, groupList);
                target.appendJavaScript("groupsInit();");
        }
 
@@ -115,16 +115,15 @@ public class GroupForm extends AdminComm
        @Override
        protected void onDeleteSubmit(AjaxRequestTarget target, Form<?> form) {
                getBean(OrganisationDao.class).delete(getModelObject(), 
getUserId());
-               target.add(groupList);
-               target.appendJavaScript("groupsInit();");
+               this.setModelObject(new Organisation());
+               updateView(target);
        }
        
        @Override
        protected void onSaveSubmit(AjaxRequestTarget target, Form<?> form) {
                getBean(OrganisationDao.class).update(getModelObject(), 
getUserId());
                hideNewRecord();
-               target.add(groupList);
-               target.appendJavaScript("groupsInit();");
+               updateView(target);
        }
 
        @Override

Modified: 
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
URL: 
http://svn.apache.org/viewvc/openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java?rev=1563609&r1=1563608&r2=1563609&view=diff
==============================================================================
--- 
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
 (original)
+++ 
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
 Sun Feb  2 13:08:14 2014
@@ -69,20 +69,25 @@ public class GroupsPanel extends AdminPa
         
                //Adding the Group Form
                form = new GroupForm("form", listContainer, new Organisation()){
-                       private static final long serialVersionUID = 
-3042797340375988889L;
+                       private static final long serialVersionUID = 1L;
 
+                       private AjaxButton processAddClass() {
+                               boolean orgSaved = 
getModelObject().getOrganisation_id() != null;
+                               
addUsersBtn.add(AttributeModifier.replace("class", orgSaved ? "formNewButton" : 
"formNewButton disabled"));
+                               addUsersBtn.setEnabled(orgSaved);
+                               return addUsersBtn;
+                       }
+                       
                        @Override
                        protected void onModelChanged() {
                                super.onModelChanged();
-                               boolean orgEmpty = 
getModelObject().getOrganisation_id() == null;
-                               
addUsersBtn.add(AttributeModifier.replace("class", orgEmpty ? "formNewButton 
disabled" : "formNewButton"));
-                               addUsersBtn.setEnabled(!orgEmpty);
+                               processAddClass();
                        }
                        
                        @Override
                        public void updateView(AjaxRequestTarget target) {
                                super.updateView(target);
-                               target.add(addUsersBtn);
+                               target.add(processAddClass());
                        }
                };
         add(form.add(addUsersBtn.setOutputMarkupId(true)));

Modified: 
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java?rev=1563609&r1=1563608&r2=1563609&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupForm.java
 Sun Feb  2 13:08:14 2014
@@ -62,7 +62,7 @@ public class GroupForm extends AdminComm
        @Override
        public void updateView(AjaxRequestTarget target) {
                usersPanel.update(getOrgId());
-               target.add(this);
+               target.add(this, groupList);
                target.appendJavaScript("groupsInit();");
        }
 
@@ -115,16 +115,15 @@ public class GroupForm extends AdminComm
        @Override
        protected void onDeleteSubmit(AjaxRequestTarget target, Form<?> form) {
                getBean(OrganisationDao.class).delete(getModelObject(), 
getUserId());
-               target.add(groupList);
-               target.appendJavaScript("groupsInit();");
+               this.setModelObject(new Organisation());
+               updateView(target);
        }
        
        @Override
        protected void onSaveSubmit(AjaxRequestTarget target, Form<?> form) {
                getBean(OrganisationDao.class).update(getModelObject(), 
getUserId());
                hideNewRecord();
-               target.add(groupList);
-               target.appendJavaScript("groupsInit();");
+               updateView(target);
        }
 
        @Override

Modified: 
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java?rev=1563609&r1=1563608&r2=1563609&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/src/web/java/org/apache/openmeetings/web/admin/groups/GroupsPanel.java
 Sun Feb  2 13:08:14 2014
@@ -69,20 +69,25 @@ public class GroupsPanel extends AdminPa
         
                //Adding the Group Form
                form = new GroupForm("form", listContainer, new Organisation()){
-                       private static final long serialVersionUID = 
-3042797340375988889L;
+                       private static final long serialVersionUID = 1L;
 
+                       private AjaxButton processAddClass() {
+                               boolean orgSaved = 
getModelObject().getOrganisation_id() != null;
+                               
addUsersBtn.add(AttributeModifier.replace("class", orgSaved ? "formNewButton" : 
"formNewButton disabled"));
+                               addUsersBtn.setEnabled(orgSaved);
+                               return addUsersBtn;
+                       }
+                       
                        @Override
                        protected void onModelChanged() {
                                super.onModelChanged();
-                               boolean orgEmpty = 
getModelObject().getOrganisation_id() == null;
-                               
addUsersBtn.add(AttributeModifier.replace("class", orgEmpty ? "formNewButton 
disabled" : "formNewButton"));
-                               addUsersBtn.setEnabled(!orgEmpty);
+                               processAddClass();
                        }
                        
                        @Override
                        public void updateView(AjaxRequestTarget target) {
                                super.updateView(target);
-                               target.add(addUsersBtn);
+                               target.add(processAddClass());
                        }
                };
         add(form.add(addUsersBtn.setOutputMarkupId(true)));


Reply via email to