~ feedback from shapetrees-js@2ef2476

Author: Eric Prud'hommeaux

asTypescript/packages/client-http/src/HttpResourceAccessor.ts

@@ -279,10 +279,11 @@ export class HttpResourceAccessor implements ResourceAccessor {
     if (location.isPresent()) {
       try {
         url = new URL(location.get());
-      } catch (ex) {
- if (ex instanceof MalformedURLException) {
+      } catch (e) {
+ if (e instanceof MalformedURLException) {
          throw new ShapeTreeException(500, "Retrieving <" + url + "> yielded a Location header \"" + location.get() + "\" which doesn't parse as a URL: " + e.getMessage());
-       }
+       }}
+
     }
     // Determine whether the resource exists based on the response. Even if the resource
     // doesn't exist, additional context and processing is done to provide the appropriate
@@ -354,7 +355,8 @@ export class HttpResourceAccessor implements ResourceAccessor {
     } catch (ex) {
  if (ex instanceof Exception) {
        throw new ShapeTreeException(500, ex.getMessage());
-     }
+     }}
+
   }
 
   /**
@@ -453,10 +455,11 @@ export class HttpResourceAccessor implements ResourceAccessor {
     let managerUrlString: string = optManagerString.get();
     try {
       return Optional.of(new URL(url, managerUrlString));
-    } catch (ex) {
- if (ex instanceof MalformedURLException) {
+    } catch (e) {
+ if (e instanceof MalformedURLException) {
        throw new ShapeTreeException(500, "Malformed relative URL <" + managerUrlString + "> (resolved from <" + url + ">)");
-     }
+     }}
+
   }
 
   /**
@@ -482,10 +485,11 @@ export class HttpResourceAccessor implements ResourceAccessor {
     }
     try {
       managedResourceUrl = new URL(managerUrl, managedUrlString);
-    } catch (ex) {
- if (ex instanceof MalformedURLException) {
+    } catch (e) {
+ if (e instanceof MalformedURLException) {
        throw new ShapeTreeException(500, "Can't calculate managed resource for shape tree manager <" + managerUrl + ">");
-     }
+     }}
+
     return managedResourceUrl;
   }
 

asTypescript/packages/core/src/ManageableResource.ts

@@ -44,10 +44,11 @@ export class ManageableResource extends InstanceResource {
     const rel: string = this.isContainer() ? ".." : ".";
     try {
       return new URL(this.getUrl(), rel);
-    } catch (ex) {
- if (ex instanceof MalformedURLException) {
+    } catch (e) {
+ if (e instanceof MalformedURLException) {
        throw new ShapeTreeException(500, "Malformed focus node when resolving <" + rel + "> against <" + this.getUrl() + ">");
-     }
+     }}
+
   }
 
   public getManagerResourceUrl(): URL | null {

asTypescript/packages/core/src/ShapeTree.ts

@@ -115,7 +115,8 @@ export class ShapeTree {
       } catch (ex) {
  if (ex instanceof Exception) {
          throw new ShapeTreeException(500, "Error parsing ShEx schema - " + ex.getMessage());
-       }
+       }}
+
     }
     // Tell ShExJava we want to use Jena as our graph library
     let jenaRDF: JenaRDF = new org.apache.commons.rdf.jena.JenaRDF();
@@ -151,7 +152,8 @@ export class ShapeTree {
           } catch (ex) {
  if (ex instanceof MalformedURLException) {
              throw new ShapeTreeException(500, "Error reporting validation success on malformed URL <" + focusUriString + ">: " + ex.getMessage());
-           }
+           }}
+
           return new ValidationResult(valid, this, this, matchingFocusNode);
         }
       }

asTypescript/packages/core/src/ShapeTreeAssignment.ts

@@ -56,7 +56,8 @@ export class ShapeTreeAssignment {
     } catch (ex) {
  if (ex instanceof NullPointerException || ex instanceof IllegalStateException) {
        throw new ShapeTreeException(500, "Failed to initialize shape tree assignment: " + ex.getMessage());
-     }
+     }}
+
   }
 
   public static getFromGraph(url: URL, managerGraph: Graph): ShapeTreeAssignment /* throws MalformedURLException, ShapeTreeException */ {

asTypescript/packages/core/src/ShapeTreeFactory.ts

@@ -91,7 +91,8 @@ export class ShapeTreeFactory {
     } catch (ex) {
  if (ex instanceof MalformedURLException || ex instanceof ShapeTreeException) {
        throw new ShapeTreeException(500, "List <" + shapeTreeUrl + "> contains malformed URL: " + ex.getMessage());
-     }
+     }}
+
   }
 
   /**
@@ -117,7 +118,8 @@ export class ShapeTreeFactory {
         } catch (ex) {
  if (ex instanceof MalformedURLException) {
            throw new ShapeTreeException(500, "ShapeTree <" + shapeTreeUrl + "> references malformed URL <" + referencedShapeTreeUrlString + ">: " + ex.getMessage());
-         }
+         }}
+
         let viaShapePath: string = getStringValue(resourceModel, referenceResource, ShapeTreeVocabulary.VIA_SHAPE_PATH);
         let viaPredicate: URL = getUrlValue(resourceModel, referenceResource, ShapeTreeVocabulary.VIA_PREDICATE, shapeTreeUrl);
         referencedShapeTree = new ShapeTreeReference(referencedShapeTreeUrl, viaShapePath, viaPredicate);
@@ -147,7 +149,8 @@ export class ShapeTreeFactory {
         } catch (ex) {
  if (ex instanceof MalformedURLException) {
            throw new IllegalStateException("Malformed ShapeTree <" + shapeTreeUrl + ">: Jena URIResource <" + object + "> didn't parse as URL - " + ex.getMessage());
-         }
+         }}
+
       } else {
         throw new ShapeTreeException(500, "Malformed ShapeTree <" + shapeTreeUrl + ">: expected " + object + " to be a URL");
       }

asTypescript/packages/core/src/ShapeTreeManager.ts

@@ -106,7 +106,8 @@ export class ShapeTreeManager {
     } catch (ex) {
  if (ex instanceof MalformedURLException) {
        throw new IllegalStateException("Minted illegal URL <" + assignmentString + "> - " + ex.getMessage());
-     }
+     }}
+
     return assignmentUrl;
   }
 
@@ -164,10 +165,11 @@ export class ShapeTreeManager {
       let assignment: ShapeTreeAssignment = null;
       try {
         assignment = ShapeTreeAssignment.getFromGraph(new URL(assignmentNode.getObject().getURI()), managerGraph);
-      } catch (ex) {
- if (ex instanceof MalformedURLException) {
+      } catch (e) {
+ if (e instanceof MalformedURLException) {
          throw new ShapeTreeException(500, "Object of { " + s + " " + stAssignment + " " + assignmentNode.getObject() + " } must be a URL.");
-       }
+       }}
+
       manager.assignments.add(assignment);
     }
     return manager;

asTypescript/packages/core/src/ShapeTreeManagerDelta.ts

@@ -82,7 +82,8 @@ export class ShapeTreeManagerDelta {
       } catch (ex) {
  if (ex instanceof URISyntaxException) {
          throw new ShapeTreeException(500, "Unable to convert assignment URLs for comparison: " + ex.getMessage());
-       }
+       }}
+
       if (assignmentUri === targetAssignmentUri) {
         return targetAssignment;
       }

asTypescript/packages/core/src/contentloaders/HttpExternalDocumentLoader.ts

@@ -33,6 +33,7 @@ export class HttpExternalDocumentLoader implements ExternalDocumentLoader {
        throw new ShapeTreeException(500, "Error retrieving <" + resourceUrl + ">: " + ex.getMessage());
      } else if (ex instanceof URISyntaxException) {
        throw new ShapeTreeException(500, "Malformed URL <" + resourceUrl + ">: " + ex.getMessage());
-     }
+     }}
+
   }
 }

asTypescript/packages/core/src/helpers/GraphHelper.ts

@@ -77,10 +77,11 @@ export class GraphHelper {
       let reader: StringReader = new StringReader(rawContent);
       RDFDataMgr.read(model.getGraph(), reader, baseURI.toString(), GraphHelper.getLangForContentType(contentType));
       return model;
-    } catch (ex) {
- if (ex instanceof RiotException) {
+    } catch (rex) {
+ if (rex instanceof RiotException) {
        throw new ShapeTreeException(422, "Error processing input - " + rex.getMessage());
-     }
+     }}
+
   }
 
   /**
@@ -164,7 +165,8 @@ export class GraphHelper {
     } catch (ex) {
  if (ex instanceof URISyntaxException) {
        throw new IllegalStateException("can't convert URL <" + url + "> to IRI: " + ex);
-     }
+     }}
+
   }
 
   /**
@@ -183,7 +185,8 @@ export class GraphHelper {
     } catch (ex) {
  if (ex instanceof MalformedURLException || ex instanceof URISyntaxException) {
        throw new IllegalStateException("Unable to remove fragment from URL: " + ex.getMessage());
-     }
+     }}
+
   }
 
   public static knownUrl(urlString: string): URL {
@@ -192,6 +195,7 @@ export class GraphHelper {
     } catch (ex) {
  if (ex instanceof MalformedURLException) {
        throw new IllegalStateException("Expected known URL <" + urlString + "> to parse as valid URL - " + ex.toString());
-     }
+     }}
+
   }
 }

asTypescript/packages/core/src/helpers/RequestHelper.ts

@@ -97,10 +97,11 @@ export class RequestHelper {
         try {
           const focusNodeUrl: URL = new URL(baseUrl, focusNodeUrlString);
           focusNodeUrls.add(focusNodeUrl);
-        } catch (ex) {
- if (ex instanceof MalformedURLException) {
+        } catch (e) {
+ if (e instanceof MalformedURLException) {
            throw new ShapeTreeException(500, "Malformed focus node when resolving <" + focusNodeUrlString + "> against <" + baseUrl + ">");
-         }
+         }}
+
       }
     }
     return focusNodeUrls;
@@ -120,10 +121,11 @@ export class RequestHelper {
         try {
           const targetShapeTreeUrl: URL = new URL(targetShapeTreeUrlString);
           targetShapeTreeUrls.add(targetShapeTreeUrl);
-        } catch (ex) {
- if (ex instanceof MalformedURLException) {
+        } catch (e) {
+ if (e instanceof MalformedURLException) {
            throw new ShapeTreeException(500, "Malformed focus node when resolving <" + targetShapeTreeUrlString + "> against <" + baseUrl + ">");
-         }
+         }}
+
       }
     }
     return targetShapeTreeUrls;
@@ -158,7 +160,8 @@ export class RequestHelper {
     } catch (ex) {
  if (ex instanceof MalformedURLException) {
        throw new ShapeTreeException(500, "normalized to malformed URL <" + urlString + "> - " + ex.getMessage());
-     }
+     }}
+
   }
 
   /**