You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@syncope.apache.org by il...@apache.org on 2017/06/12 14:49:49 UTC
[2/2] syncope git commit: Minor fixes;
including TogglePanel icon vertical align and modern veil
Minor fixes; including TogglePanel icon vertical align and modern veil
Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/3a7cf682
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/3a7cf682
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/3a7cf682
Branch: refs/heads/master
Commit: 3a7cf682f2fae3cfa4051545bd532e0ff68207c8
Parents: a90d50e
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Mon Jun 12 16:49:27 2017 +0200
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Mon Jun 12 16:49:39 2017 +0200
----------------------------------------------------------------------
.../client/console/commons/Constants.java | 2 ++
.../syncope/client/console/pages/BasePage.java | 2 +-
.../console/panels/UserDirectoryPanel.java | 2 +-
.../console/topology/TopologyNodePanel.java | 3 ++-
.../wicket/markup/html/form/ActionPanel.java | 5 ++--
.../wicket/markup/html/form/ActionsPanel.java | 2 +-
.../markup/html/link/VeilPopupSettings.java | 3 ++-
.../META-INF/resources/css/syncopeConsole.css | 4 +++
.../core/logic/LogicInvocationHandler.java | 26 ++++++++++----------
.../api/dao/search/OrderByClause.java | 5 +++-
.../api/event/AfterHandlingEvent.java | 16 ------------
.../provisioning/java/AuditManagerImpl.java | 20 +++++++--------
.../notification/NotificationManagerImpl.java | 20 +++++++--------
.../pushpull/AbstractPushResultHandler.java | 2 --
.../pushpull/RealmPushResultHandlerImpl.java | 2 --
15 files changed, 51 insertions(+), 63 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/commons/Constants.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/commons/Constants.java b/client/console/src/main/java/org/apache/syncope/client/console/commons/Constants.java
index e49f10d..588790d 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/commons/Constants.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/commons/Constants.java
@@ -29,6 +29,8 @@ public final class Constants {
public static final String SYNCOPE = "syncope";
+ public static final String VEIL_INDICATOR_MARKUP_ID = "veil";
+
public static final String ACTIVITI_MODELER_CONTEXT = "activiti-modeler";
public static final String FLOWABLE_MODELER_CONTEXT = "flowable-modeler";
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/pages/BasePage.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/pages/BasePage.java b/client/console/src/main/java/org/apache/syncope/client/console/pages/BasePage.java
index 1a94309..55d8f9c 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/pages/BasePage.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/pages/BasePage.java
@@ -404,7 +404,7 @@ public class BasePage extends WebPage implements IAjaxIndicatorAware {
@Override
public String getAjaxIndicatorMarkupId() {
- return "veil";
+ return Constants.VEIL_INDICATOR_MARKUP_ID;
}
public NotificationPanel getNotificationPanel() {
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/panels/UserDirectoryPanel.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/panels/UserDirectoryPanel.java b/client/console/src/main/java/org/apache/syncope/client/console/panels/UserDirectoryPanel.java
index 5a62063..dae614f 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/panels/UserDirectoryPanel.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/panels/UserDirectoryPanel.java
@@ -128,7 +128,7 @@ public class UserDirectoryPanel extends AnyDirectoryPanel<UserTO, UserRestClient
@Override
protected List<IColumn<UserTO, String>> getColumns() {
final List<IColumn<UserTO, String>> columns = new ArrayList<>();
- final List<IColumn<UserTO, String>> prefcolumns = new ArrayList<IColumn<UserTO, String>>();
+ final List<IColumn<UserTO, String>> prefcolumns = new ArrayList<>();
columns.add(new KeyPropertyColumn<UserTO>(
new ResourceModel(Constants.KEY_FIELD_NAME, Constants.KEY_FIELD_NAME), Constants.KEY_FIELD_NAME));
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/topology/TopologyNodePanel.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/topology/TopologyNodePanel.java b/client/console/src/main/java/org/apache/syncope/client/console/topology/TopologyNodePanel.java
index b563cca..2d81c3d 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/topology/TopologyNodePanel.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/topology/TopologyNodePanel.java
@@ -19,6 +19,7 @@
package org.apache.syncope.client.console.topology;
import org.apache.commons.lang3.StringUtils;
+import org.apache.syncope.client.console.commons.Constants;
import org.apache.syncope.client.console.rest.ConnectorRestClient;
import org.apache.syncope.client.console.topology.TopologyNode.Kind;
import org.apache.syncope.client.console.topology.TopologyTogglePanel.UpdateEvent;
@@ -108,7 +109,7 @@ public class TopologyNodePanel extends Panel implements IAjaxIndicatorAware {
@Override
public String getAjaxIndicatorMarkupId() {
- return "veil";
+ return Constants.VEIL_INDICATOR_MARKUP_ID;
}
@Override
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionPanel.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionPanel.java b/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionPanel.java
index 6f147c0..8eb3135 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionPanel.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionPanel.java
@@ -21,6 +21,7 @@ package org.apache.syncope.client.console.wicket.markup.html.form;
import java.io.Serializable;
import org.apache.commons.lang3.StringUtils;
import org.apache.syncope.client.console.SyncopeConsoleSession;
+import org.apache.syncope.client.console.commons.Constants;
import org.apache.syncope.client.console.wicket.markup.html.form.ActionLink.ActionType;
import org.apache.syncope.client.console.wicket.markup.html.link.VeilPopupSettings;
import org.apache.wicket.AttributeModifier;
@@ -108,7 +109,7 @@ public final class ActionPanel<T extends Serializable> extends Panel {
@Override
public String getAjaxIndicatorMarkupId() {
return disableIndicator || !action.getLink().isIndicatorEnabled()
- ? StringUtils.EMPTY : super.getAjaxIndicatorMarkupId();
+ ? StringUtils.EMPTY : Constants.VEIL_INDICATOR_MARKUP_ID;
}
}
: new IndicatingAjaxLink<Void>("action") {
@@ -124,7 +125,7 @@ public final class ActionPanel<T extends Serializable> extends Panel {
@Override
public String getAjaxIndicatorMarkupId() {
return disableIndicator || !action.getLink().isIndicatorEnabled()
- ? StringUtils.EMPTY : super.getAjaxIndicatorMarkupId();
+ ? StringUtils.EMPTY : Constants.VEIL_INDICATOR_MARKUP_ID;
}
};
}
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionsPanel.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionsPanel.java b/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionsPanel.java
index 6c26aa9..71d342c 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionsPanel.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/ActionsPanel.java
@@ -50,7 +50,7 @@ public final class ActionsPanel<T extends Serializable> extends Panel {
@Override
protected void populateItem(final ListItem<Action<T>> item) {
- item.add(new ActionPanel<T>(ActionsPanel.this.model, item.getModelObject()));
+ item.add(new ActionPanel<>(ActionsPanel.this.model, item.getModelObject()));
}
}.setRenderBodyOnly(true));
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/link/VeilPopupSettings.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/link/VeilPopupSettings.java b/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/link/VeilPopupSettings.java
index 6a766be..c85cfda 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/link/VeilPopupSettings.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/link/VeilPopupSettings.java
@@ -18,6 +18,7 @@
*/
package org.apache.syncope.client.console.wicket.markup.html.link;
+import org.apache.syncope.client.console.commons.Constants;
import org.apache.wicket.markup.html.link.PopupSettings;
public class VeilPopupSettings extends PopupSettings {
@@ -26,7 +27,7 @@ public class VeilPopupSettings extends PopupSettings {
@Override
public String getPopupJavaScript() {
- return "document.getElementById('veil').style.display = 'block';"
+ return "document.getElementById('" + Constants.VEIL_INDICATOR_MARKUP_ID + "').style.display = 'block';"
+ super.getPopupJavaScript().substring(0, super.getPopupJavaScript().lastIndexOf("return false;"))
+ "var loop = setInterval(function() { "
+ " if(w.closed) {"
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/client/console/src/main/resources/META-INF/resources/css/syncopeConsole.css
----------------------------------------------------------------------
diff --git a/client/console/src/main/resources/META-INF/resources/css/syncopeConsole.css b/client/console/src/main/resources/META-INF/resources/css/syncopeConsole.css
index a9f3848..da65a20 100644
--- a/client/console/src/main/resources/META-INF/resources/css/syncopeConsole.css
+++ b/client/console/src/main/resources/META-INF/resources/css/syncopeConsole.css
@@ -1137,6 +1137,10 @@ ul.menu li a {
cursor: pointer !important;
}
+ul.menu i {
+ width: 35px;
+}
+
.toggle-menu ul.menu li a {
padding: 0px;
text-align: left;
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/logic/src/main/java/org/apache/syncope/core/logic/LogicInvocationHandler.java
----------------------------------------------------------------------
diff --git a/core/logic/src/main/java/org/apache/syncope/core/logic/LogicInvocationHandler.java b/core/logic/src/main/java/org/apache/syncope/core/logic/LogicInvocationHandler.java
index a70f3c0..38c5e92 100644
--- a/core/logic/src/main/java/org/apache/syncope/core/logic/LogicInvocationHandler.java
+++ b/core/logic/src/main/java/org/apache/syncope/core/logic/LogicInvocationHandler.java
@@ -97,19 +97,19 @@ public class LogicInvocationHandler {
LOG.debug("After throwing {}.{}", clazz.getSimpleName(), event);
throw t;
} finally {
- Map<String, Object> jobMap = new HashMap<>();
- jobMap.put(AfterHandlingEvent.JOBMAP_KEY, new AfterHandlingEvent(
- notificationsAvailable,
- auditRequested,
- AuditElements.EventCategoryType.LOGIC,
- category,
- null,
- event,
- condition,
- before,
- output,
- input));
- AfterHandlingJob.schedule(scheduler, jobMap);
+ if (notificationsAvailable || auditRequested) {
+ Map<String, Object> jobMap = new HashMap<>();
+ jobMap.put(AfterHandlingEvent.JOBMAP_KEY, new AfterHandlingEvent(
+ AuditElements.EventCategoryType.LOGIC,
+ category,
+ null,
+ event,
+ condition,
+ before,
+ output,
+ input));
+ AfterHandlingJob.schedule(scheduler, jobMap);
+ }
}
}
}
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/dao/search/OrderByClause.java
----------------------------------------------------------------------
diff --git a/core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/dao/search/OrderByClause.java b/core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/dao/search/OrderByClause.java
index 46a9d80..8d4b978 100644
--- a/core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/dao/search/OrderByClause.java
+++ b/core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/dao/search/OrderByClause.java
@@ -18,12 +18,15 @@
*/
package org.apache.syncope.core.persistence.api.dao.search;
+import java.io.Serializable;
import org.apache.commons.lang3.builder.EqualsBuilder;
import org.apache.commons.lang3.builder.HashCodeBuilder;
import org.apache.commons.lang3.builder.ReflectionToStringBuilder;
import org.apache.commons.lang3.builder.ToStringStyle;
-public class OrderByClause {
+public class OrderByClause implements Serializable {
+
+ private static final long serialVersionUID = -1741826744085524716L;
public enum Direction {
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/provisioning-api/src/main/java/org/apache/syncope/core/provisioning/api/event/AfterHandlingEvent.java
----------------------------------------------------------------------
diff --git a/core/provisioning-api/src/main/java/org/apache/syncope/core/provisioning/api/event/AfterHandlingEvent.java b/core/provisioning-api/src/main/java/org/apache/syncope/core/provisioning/api/event/AfterHandlingEvent.java
index b207cb9..2e2e44e 100644
--- a/core/provisioning-api/src/main/java/org/apache/syncope/core/provisioning/api/event/AfterHandlingEvent.java
+++ b/core/provisioning-api/src/main/java/org/apache/syncope/core/provisioning/api/event/AfterHandlingEvent.java
@@ -27,10 +27,6 @@ public class AfterHandlingEvent implements Serializable {
public static final String JOBMAP_KEY = "AfterHandlingEvent";
- private final boolean notificationsAvailable;
-
- private final boolean auditRequested;
-
private final AuditElements.EventCategoryType type;
private final String category;
@@ -48,8 +44,6 @@ public class AfterHandlingEvent implements Serializable {
private final Object[] input;
public AfterHandlingEvent(
- final boolean notificationsAvailable,
- final boolean auditRequested,
final AuditElements.EventCategoryType type,
final String category,
final String subcategory,
@@ -59,8 +53,6 @@ public class AfterHandlingEvent implements Serializable {
final Object output,
final Object... input) {
- this.notificationsAvailable = notificationsAvailable;
- this.auditRequested = auditRequested;
this.type = type;
this.category = category;
this.subcategory = subcategory;
@@ -71,14 +63,6 @@ public class AfterHandlingEvent implements Serializable {
this.input = input;
}
- public boolean isNotificationsAvailable() {
- return notificationsAvailable;
- }
-
- public boolean isAuditRequested() {
- return auditRequested;
- }
-
public AuditElements.EventCategoryType getType() {
return type;
}
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/AuditManagerImpl.java
----------------------------------------------------------------------
diff --git a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/AuditManagerImpl.java b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/AuditManagerImpl.java
index 359538b..14180ec 100644
--- a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/AuditManagerImpl.java
+++ b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/AuditManagerImpl.java
@@ -80,17 +80,15 @@ public class AuditManagerImpl implements AuditManager {
@Override
public void audit(final AfterHandlingEvent event) {
- if (event.isAuditRequested()) {
- audit(
- event.getType(),
- event.getCategory(),
- event.getSubcategory(),
- event.getEvent(),
- event.getCondition(),
- event.getBefore(),
- event.getOutput(),
- event.getInput());
- }
+ audit(
+ event.getType(),
+ event.getCategory(),
+ event.getSubcategory(),
+ event.getEvent(),
+ event.getCondition(),
+ event.getBefore(),
+ event.getOutput(),
+ event.getInput());
}
@Override
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/notification/NotificationManagerImpl.java
----------------------------------------------------------------------
diff --git a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/notification/NotificationManagerImpl.java b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/notification/NotificationManagerImpl.java
index c438da3..1e6f531 100644
--- a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/notification/NotificationManagerImpl.java
+++ b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/notification/NotificationManagerImpl.java
@@ -279,17 +279,15 @@ public class NotificationManagerImpl implements NotificationManager {
@Override
public void createTasks(final AfterHandlingEvent event) {
- if (event.isNotificationsAvailable()) {
- createTasks(
- event.getType(),
- event.getCategory(),
- event.getSubcategory(),
- event.getEvent(),
- event.getCondition(),
- event.getBefore(),
- event.getOutput(),
- event.getInput());
- }
+ createTasks(
+ event.getType(),
+ event.getCategory(),
+ event.getSubcategory(),
+ event.getEvent(),
+ event.getCondition(),
+ event.getBefore(),
+ event.getOutput(),
+ event.getInput());
}
@Override
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/AbstractPushResultHandler.java
----------------------------------------------------------------------
diff --git a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/AbstractPushResultHandler.java b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/AbstractPushResultHandler.java
index bbc6eb0..542009b 100644
--- a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/AbstractPushResultHandler.java
+++ b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/AbstractPushResultHandler.java
@@ -367,8 +367,6 @@ public abstract class AbstractPushResultHandler extends AbstractSyncopeResultHan
} finally {
Map<String, Object> jobMap = new HashMap<>();
jobMap.put(AfterHandlingEvent.JOBMAP_KEY, new AfterHandlingEvent(
- true,
- true,
AuditElements.EventCategoryType.PUSH,
any.getType().getKind().name().toLowerCase(),
profile.getTask().getResource().getKey(),
http://git-wip-us.apache.org/repos/asf/syncope/blob/3a7cf682/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/RealmPushResultHandlerImpl.java
----------------------------------------------------------------------
diff --git a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/RealmPushResultHandlerImpl.java b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/RealmPushResultHandlerImpl.java
index 20371b5..313a527 100644
--- a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/RealmPushResultHandlerImpl.java
+++ b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/pushpull/RealmPushResultHandlerImpl.java
@@ -324,8 +324,6 @@ public class RealmPushResultHandlerImpl
} finally {
Map<String, Object> jobMap = new HashMap<>();
jobMap.put(AfterHandlingEvent.JOBMAP_KEY, new AfterHandlingEvent(
- true,
- true,
AuditElements.EventCategoryType.PUSH,
REALM_TYPE.toLowerCase(),
profile.getTask().getResource().getKey(),