changeset 102:9140017e8962

fix bug with empty username. add logging for JSON exceptions.
author casties
date Thu, 09 Feb 2017 20:46:15 +0100
parents 7268c3ca025b
children f83eb8b335b1
files src/main/java/de/mpiwg/itgroup/annotations/Person.java src/main/java/de/mpiwg/itgroup/annotations/restlet/AnnotatorAnnotations.java src/main/java/de/mpiwg/itgroup/annotations/restlet/AnnotatorResourceImpl.java
diffstat 3 files changed, 7 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/src/main/java/de/mpiwg/itgroup/annotations/Person.java	Fri Feb 13 18:10:11 2015 +0100
+++ b/src/main/java/de/mpiwg/itgroup/annotations/Person.java	Thu Feb 09 20:46:15 2017 +0100
@@ -99,7 +99,7 @@
      * @return
      */
     public static Person getAnonymous() {
-        return new Person("anonymous");
+        return new Person("anonymous", "Anonymous");
     }
 
     /* (non-Javadoc)
--- a/src/main/java/de/mpiwg/itgroup/annotations/restlet/AnnotatorAnnotations.java	Fri Feb 13 18:10:11 2015 +0100
+++ b/src/main/java/de/mpiwg/itgroup/annotations/restlet/AnnotatorAnnotations.java	Thu Feb 09 20:46:15 2017 +0100
@@ -179,10 +179,12 @@
             jo.remove("id");
             // create Annotation object from posted JSON
             annot = createAnnotation(jo, entity);
-        } catch (IOException e1) {
+        } catch (IOException e) {
+        	logger.warning(e.toString());
             setStatus(Status.SERVER_ERROR_INTERNAL);
             return null;
         } catch (JSONException e) {
+        	logger.warning(e.toString());
             setStatus(Status.CLIENT_ERROR_BAD_REQUEST);
             return null;
         }
--- a/src/main/java/de/mpiwg/itgroup/annotations/restlet/AnnotatorResourceImpl.java	Fri Feb 13 18:10:11 2015 +0100
+++ b/src/main/java/de/mpiwg/itgroup/annotations/restlet/AnnotatorResourceImpl.java	Thu Feb 09 20:46:15 2017 +0100
@@ -638,7 +638,7 @@
                 }
             }
         } catch (JSONException e) {
-            // nothing to do
+            logger.warning(e.toString());
         }
         try {
             if (jo.has("shapes")) {
@@ -655,7 +655,7 @@
                 }
             }
         } catch (JSONException e) {
-            // nothing to do
+            logger.warning(e.toString());
         }
         // deprecated areas type
         try {
@@ -669,7 +669,7 @@
                 }
             }
         } catch (JSONException e) {
-            // nothing to do
+            logger.warning(e.toString());
         }
         // no fragment is an error
         if (annot.getFragmentType() == null || annot.getTargetFragment() == null) {