Browse Source

OnEvent value validation added

master
Nikita 12 years ago
parent
commit
7d8ad39936
  1. 2
      src/main/java/com/corundumstudio/socketio/annotation/OnEvent.java
  2. 3
      src/main/java/com/corundumstudio/socketio/annotation/OnEventScanner.java

2
src/main/java/com/corundumstudio/socketio/annotation/OnEvent.java

@ -28,7 +28,7 @@ import java.lang.annotation.Target;
*
* - SocketIOClient (optional)
* - AckRequest (optional)
* - 'type of Event' (required)
* - Event data (optional)
*
*/
@Target(ElementType.METHOD)

3
src/main/java/com/corundumstudio/socketio/annotation/OnEventScanner.java

@ -39,6 +39,9 @@ public class OnEventScanner implements AnnotationScanner {
@SuppressWarnings("unchecked")
public void addListener(Namespace namespace, final Object object, final Class clazz, final Method method) {
OnEvent annotation = method.getAnnotation(OnEvent.class);
if (annotation.value() == null || annotation.value().trim().length() == 0) {
throw new IllegalArgumentException("OnEvent \"value\" parameter is required");
}
final int socketIOClientIndex = paramIndex(method, SocketIOClient.class);
final int ackRequestIndex = paramIndex(method, AckRequest.class);
final int dataIndex = dataIndex(method);

Loading…
Cancel
Save