RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix issue with messages for users

Browse source 
The MessagesForUser component is used for showing information and
error messages to the users. It's designed to disappear once the user
does some action and a minimum of time has passed.

This is done listening to the events being produced in the page. In
previous versions of ZK this was enough to identify user activity, but
in the version currently used, 5.0.11, events not related to user
activity are generated. This causes the messages to disappear without
user action.

With this fix, only when the user clicks on something, or changes the
value of some input the shown message is cleared.
This commit is contained in:
Oscar Gonzalez Fernandez 2014-04-10 01:13:35 +02:00
parent 5530184f8a
commit d45b801ed2
1 changed files with 24 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
libreplan-webapp/src/main/java/org/libreplan/web/common/MessagesForUser.java
View file

@ -35,9 +35,13 @@ import org.apache.commons.logging.LogFactory;
import org.hibernate.validator.InvalidValue;
import org.libreplan.business.common.exceptions.ValidationException;
import org.zkoss.zk.ui.Component;
import org.zkoss.zk.ui.event.CheckEvent;
import org.zkoss.zk.ui.event.Event;
import org.zkoss.zk.ui.event.EventListener;
import org.zkoss.zk.ui.event.Events;
import org.zkoss.zk.ui.event.InputEvent;
import org.zkoss.zk.ui.event.MouseEvent;
import org.zkoss.zk.ui.event.SelectEvent;
import org.zkoss.zk.ui.util.EventInterceptor;
import org.zkoss.zk.ui.util.GenericForwardComposer;
import org.zkoss.zul.Div;
@ -45,7 +49,12 @@ import org.zkoss.zul.Image;
import org.zkoss.zul.Label;
/**
* It shows messages to the user. <br />
* <p>
* It shows messages from the application to the user with different
* {@link Level levels} of severity. Once the user does some action in the page,
* like clicking a button or changing some input the messages automatically
* disappear.
*
* @author Óscar González Fernández <ogonzalez@igalia.com>
*/
public class MessagesForUser extends GenericForwardComposer implements
@ -78,25 +87,32 @@ public class MessagesForUser extends GenericForwardComposer implements
@Override
public Event beforeProcessEvent(Event event) {
MessagesForUser messagesForUser = messagesForUserRef.get();
if (messagesForUser == null) {
if (messagesForUser == null
|| messagesForUser.pendingToDetach.isEmpty()
|| !eventIndicatesUserActivity(event)) {
return event;
}
if (event.getName().equals(DETACH_EVENT_NAME)
|| messagesForUser.pendingToDetach.isEmpty()) {
return event;
}
long currentTime = System.currentTimeMillis();
ComponentHolderTimestamped currrent = null;
while ((currrent = messagesForUser.pendingToDetach.peek()) != null
&& currrent
.minimumVisualizationTimeSurpased(currentTime)) {
&& currrent.minimumVisualizationTimeSurpased(currentTime)) {
currrent.component.detach();
messagesForUser.pendingToDetach.poll();
}
return event;
}
private boolean eventIndicatesUserActivity(Event event) {
if (event instanceof MouseEvent) {
MouseEvent e = (MouseEvent) event;
return e.getName().equals("onClick");
}
return event instanceof InputEvent || event instanceof CheckEvent
|| event instanceof SelectEvent;
}
@Override
public Event beforeSendEvent(Event event) {
return event;