code cleanup

Evolveum · May 30, 2016 · 6c3238b · 6c3238b
1 parent bc0ace5
commit 6c3238b
Showing 1 changed file with 1 addition and 45 deletions.
diff --git a/gui/admin-gui/src/main/java/com/evolveum/midpoint/web/component/search/SearchItemPanel.java b/gui/admin-gui/src/main/java/com/evolveum/midpoint/web/component/search/SearchItemPanel.java
@@ -35,7 +35,6 @@
 import com.evolveum.midpoint.web.component.AjaxSubmitButton;
 import com.evolveum.midpoint.web.component.util.VisibleEnableBehaviour;
 import com.evolveum.midpoint.xml.ns._public.common.common_3.LookupTableType;
-import com.evolveum.midpoint.xml.ns._public.common.common_3.ObjectType;
 import org.apache.commons.lang.StringUtils;
 import org.apache.wicket.AttributeModifier;
 import org.apache.wicket.ajax.AjaxRequestTarget;
@@ -71,9 +70,6 @@ public class SearchItemPanel extends BasePanel<SearchItem> {
     private static final String ID_CLOSE = "close";
     private static final String ID_VALUES = "values";
     private static final String ID_VALUE = "value";
-    private static final String ID_TEXT = "text";
-    private static final String ID_COMBO = "combo";
-    private static final String ID_BROWSER = "browser";
 
     private LoadableModel<SearchItemPopoverDto> popoverModel;
 
@@ -135,7 +131,6 @@ public boolean isVisible() {
         });
 
         initPopover();
-        initBrowserPopup();
     }
 
     private SearchItemPopoverDto loadPopoverItems() {
@@ -154,45 +149,6 @@ private SearchItemPopoverDto loadPopoverItems() {
         return dto;
     }
 
-    private void initBrowserPopup() {
-        //todo better browser dialog
-//        ChooseTypeDialog dialog = new ChooseTypeDialog(ID_BROWSER_POPUP, RoleType.class) {
-//
-//            @Override
-//            protected void chooseOperationPerformed(AjaxRequestTarget target, ObjectType object) {
-//                browserSelectObjectPerformed(target, object);
-//            }
-//
-//            @Override
-//            public boolean isSearchEnabled() {
-//                return true;
-//            }
-//
-//            @Override
-//            public QName getSearchProperty() {
-//                return ObjectType.F_NAME;
-//            }
-//        };
-//        add(dialog);
-    }
-
-    private void browserSelectObjectPerformed(AjaxRequestTarget target, ObjectType object) {
-        SearchItem item = getModelObject();
-
-        PrismReferenceValue ref = PrismReferenceValue.createFromTarget(object.asPrismObject());
-        SearchValue value = new SearchValue(ref, object.getName().getOrig());
-//        item.setValue(value); //todo fix
-
-//        ChooseTypeDialog dialog = (ChooseTypeDialog) get(ID_BROWSER_POPUP);
-//        dialog.close(target);
-
-        updateItemPerformed(target);
-    }
-
-    private void referenceDeleted(AjaxRequestTarget target) {
-
-    }
-
     private void initPopover() {
         WebMarkupContainer popover = new WebMarkupContainer(ID_POPOVER);
         popover.setOutputMarkupId(true);
@@ -270,7 +226,7 @@ private SearchPopupPanel createPopoverFragment(IModel<DisplayableValue> data) {
 
     private PrismObject<LookupTableType> findLookupTable(ItemDefinition definition) {
         PrismReferenceValue valueEnumerationRef = definition.getValueEnumerationRef();
-        if (valueEnumerationRef== null) {
+        if (valueEnumerationRef == null) {
             return null;
         }