Skip to content

Commit

Permalink
MID-4851 checkbox selecting in the popup fix
Browse files Browse the repository at this point in the history
  • Loading branch information
@KaterynaHonchar
KaterynaHonchar committed Sep 11, 2018
1 parent b010397 commit 52621e1
Show file tree
Hide file tree
Showing 4 changed files with 74 additions and 18 deletions.
13 changes: 4 additions & 9 deletions gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/component/AssignmentPopup.java
View file
Expand Up @@ -22,6 +22,7 @@
import com.evolveum.midpoint.web.component.AjaxButton;
import com.evolveum.midpoint.web.component.TabbedPanel;
import com.evolveum.midpoint.web.component.dialog.Popupable;
import com.evolveum.midpoint.web.component.util.EnableBehaviour;
import com.evolveum.midpoint.web.component.util.VisibleBehaviour;
import com.evolveum.midpoint.web.component.util.VisibleEnableBehaviour;
import com.evolveum.midpoint.xml.ns._public.common.common_3.*;
Expand Down Expand Up @@ -107,14 +108,7 @@ public void onClick(AjaxRequestTarget target) {
}
};
addButton.add(AttributeAppender.append("title", getAddButtonTitleModel()));
addButton.add(new VisibleEnableBehaviour(){
private static final long serialVersionUID = 1L;

@Override
public boolean isEnabled(){
return isAssignButtonEnabled();
}
});
addButton.add(new EnableBehaviour(() -> isAssignButtonEnabled()));
addButton.setOutputMarkupId(true);
form.add(addButton);
}
Expand Down Expand Up @@ -289,7 +283,8 @@ private int getTabPanelSelectedCount(WebMarkupContainer panel){
}

private void tabLabelPanelUpdate(AjaxRequestTarget target){
target.add(AssignmentPopup.this);
getTabbedPanel().reloadCountLabels(target);
target.add(get(ID_FORM).get(ID_ASSIGN_BUTTON));
}

private TabbedPanel getTabbedPanel(){
Expand Down
57 changes: 51 additions & 6 deletions gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/component/ChooseMemberPopup.java
View file
Expand Up @@ -16,6 +16,7 @@
package com.evolveum.midpoint.gui.api.component;

import com.evolveum.midpoint.gui.api.component.tabs.CountablePanelTab;
import com.evolveum.midpoint.gui.api.model.LoadableModel;
import com.evolveum.midpoint.gui.api.util.WebComponentUtil;
import com.evolveum.midpoint.prism.delta.ObjectDelta;
import com.evolveum.midpoint.prism.query.InOidFilter;
Expand All @@ -32,13 +33,16 @@
import com.evolveum.midpoint.web.component.AjaxButton;
import com.evolveum.midpoint.web.component.TabbedPanel;
import com.evolveum.midpoint.web.component.dialog.Popupable;
import com.evolveum.midpoint.web.component.util.EnableBehaviour;
import com.evolveum.midpoint.web.component.util.VisibleEnableBehaviour;
import com.evolveum.midpoint.xml.ns._public.common.common_3.*;
import org.apache.wicket.Component;
import org.apache.wicket.ajax.AjaxRequestTarget;
import org.apache.wicket.behavior.AttributeAppender;
import org.apache.wicket.extensions.markup.html.tabs.ITab;
import org.apache.wicket.markup.html.WebMarkupContainer;
import org.apache.wicket.markup.html.form.Form;
import org.apache.wicket.model.IModel;
import org.apache.wicket.model.StringResourceModel;

import javax.xml.namespace.QName;
Expand Down Expand Up @@ -100,19 +104,34 @@ public void onClick(AjaxRequestTarget target) {

@Override
public void onClick(AjaxRequestTarget target) {
tabs.forEach(panelTab -> {
boolean orgPanelProcessed = false;
for (ITab panelTab : tabs){
WebMarkupContainer tabPanel = ((CountablePanelTab)panelTab).getPanel();
if (tabPanel == null){
return;
continue;
}

MemberPopupTabPanel memberPanel = (MemberPopupTabPanel) tabPanel;
executeMemberOperation(memberPanel.getObjectType().getTypeQName(), createInOidQuery(memberPanel.getSelectedObjectsList()),
if (memberPanel.getObjectType().equals(ObjectTypes.ORG) && orgPanelProcessed){
continue;
}
List<ObjectType> selectedObjects = memberPanel.getObjectType().equals(ObjectTypes.ORG) ? memberPanel.getPreselectedObjects() :
memberPanel.getSelectedObjectsList();

if (selectedObjects == null || selectedObjects.size() == 0){
continue;
}
executeMemberOperation(memberPanel.getObjectType().getTypeQName(),
createInOidQuery(selectedObjects),
memberPanel.prepareDelta(), target);
});
if (memberPanel.getObjectType().equals(ObjectTypes.ORG)){
orgPanelProcessed = true;
}
}
ChooseMemberPopup.this.getPageBase().hideMainPopup(target);
}
};
addButton.add(AttributeAppender.append("title", getAddButtonTitleModel()));
addButton.add(new EnableBehaviour(() -> isAddButtonEnabled()));
addButton.setOutputMarkupId(true);
form.add(addButton);
}
Expand Down Expand Up @@ -304,7 +323,9 @@ protected int getTabPanelSelectedCount(WebMarkupContainer panel){
}

protected void tabLabelPanelUpdate(AjaxRequestTarget target){
target.add(getTabbedPanel());
getTabbedPanel().reloadCountLabels(target);
target.add(get(ID_FORM).get(ID_ADD_BUTTON));

}

private TabbedPanel getTabbedPanel(){
Expand All @@ -320,6 +341,30 @@ protected ObjectQuery createInOidQuery(List<ObjectType> selectedObjectsList){
return ObjectQuery.createObjectQuery(InOidFilter.createInOid(oids));
}

private IModel<String> getAddButtonTitleModel(){
return new LoadableModel<String>(true) {
@Override
protected String load() {
return !isAddButtonEnabled() ? createStringResource("AssignmentPopup.addButtonTitle").getString() : "";
}
};
}

private boolean isAddButtonEnabled(){
TabbedPanel tabbedPanel = getTabbedPanel();
List<ITab> tabs = (List<ITab>) tabbedPanel.getTabs().getObject();
for (ITab tab : tabs){
WebMarkupContainer memberPanel = ((CountablePanelTab)tab).getPanel();
if (memberPanel == null){
continue;
}
if (((MemberPopupTabPanel) memberPanel).getSelectedObjectsList().size() > 0) {
return true;
}
}
return false;
}

protected void executeMemberOperation(QName type, ObjectQuery memberQuery,
ObjectDelta delta, AjaxRequestTarget target) {

Expand Down
2 changes: 1 addition & 1 deletion ...c/main/java/com/evolveum/midpoint/gui/api/component/FocusTypeAssignmentPopupTabPanel.java
View file
Expand Up @@ -74,7 +74,7 @@ private List<QName> getSupportedRelations() {
protected Map<String, AssignmentType> getSelectedAssignmentsMap(){
Map<String, AssignmentType> assignmentsMap = new HashedMap();

List<F> selectedObjects = getSelectedObjectsList();
List<F> selectedObjects = getObjectType().equals(ObjectTypes.ORG) ? getPreselectedObjects() : getSelectedObjectsList();
QName relation = getRelationValue();
selectedObjects.forEach(selectedObject -> {
assignmentsMap.put(selectedObject.getOid(), ObjectTypeUtil.createAssignmentTo(selectedObject, relation));
Expand Down
20 changes: 18 additions & 2 deletions gui/admin-gui/src/main/java/com/evolveum/midpoint/web/component/TabbedPanel.java
View file
Expand Up @@ -19,6 +19,7 @@
import org.apache.wicket.AttributeModifier;
import org.apache.wicket.Component;
import org.apache.wicket.WicketRuntimeException;
import org.apache.wicket.ajax.AjaxRequestTarget;
import org.apache.wicket.extensions.markup.html.tabs.ITab;
import org.apache.wicket.markup.ComponentTag;
import org.apache.wicket.markup.html.WebMarkupContainer;
Expand All @@ -37,6 +38,7 @@
import com.evolveum.midpoint.gui.api.model.CountModelProvider;

import java.io.Serializable;
import java.util.ArrayList;
import java.util.List;

/**
Expand All @@ -49,6 +51,8 @@ public class TabbedPanel<T extends ITab> extends Panel {
* id used for child panels
*/
public static final String TAB_PANEL_ID = "panel";
public static final String ID_TABS_CONTAINER = "tabs-container";
public static final String ID_TABS = "tabs";
public static final String RIGHT_SIDE_TAB_ITEM_ID = "rightSideTabItem";
public static final String RIGHT_SIDE_TAB_ID = "rightSideTab";

Expand Down Expand Up @@ -106,11 +110,11 @@ public Integer getObject() {
}
};

WebMarkupContainer tabsContainer = newTabsContainer("tabs-container");
WebMarkupContainer tabsContainer = newTabsContainer(ID_TABS_CONTAINER);
add(tabsContainer);

// add the loop used to generate tab names
tabsContainer.add(new Loop("tabs", tabCount) {
tabsContainer.add(new Loop(ID_TABS, tabCount) {
private static final long serialVersionUID = 1L;

@Override
Expand All @@ -131,6 +135,7 @@ protected void populateItem(final LoopItem item) {
}
Label countLabel = new Label(ID_COUNT, countModel);
countLabel.setVisible(countModel != null);
countLabel.setOutputMarkupId(true);
countLabel.add(AttributeModifier.append("class", new AbstractReadOnlyModel<String>() {
private static final long serialVersionUID = 1L;

Expand Down Expand Up @@ -492,4 +497,15 @@ protected void onTabChange(int index) {}
public interface RightSideItemProvider extends Serializable {
Component createRightSideItem(String id);
}

public void reloadCountLabels(AjaxRequestTarget target){
Loop tabbedPanel = ((Loop)get(ID_TABS_CONTAINER).get(ID_TABS));
int tabsCount = tabbedPanel.getIterations();
for (int i = 0; i < tabsCount; i++){
Component countLabel = tabbedPanel.get(Integer.toString(i)).get(ID_LINK).get(ID_COUNT);
if (countLabel != null) {
target.add(countLabel);
}
}
}
}

0 comments on commit 52621e1

Please sign in to comment.