fixup! JCL-403: SolidClient builds ProblemDetails

Author: NSeydoux

solid/src/main/java/com/inrupt/client/solid/SolidClient.java

@@ -57,22 +57,11 @@ public class SolidClient {
     private final Client client;
     private final Headers defaultHeaders;
     private final boolean fetchAfterWrite;
-    private final JsonService jsonService;
 
     SolidClient(final Client client, final Headers headers, final boolean fetchAfterWrite) {
         this.client = Objects.requireNonNull(client, "Client may not be null!");
         this.defaultHeaders = Objects.requireNonNull(headers, "Headers may not be null!");
         this.fetchAfterWrite = fetchAfterWrite;
-
-        // It is acceptable for a SolidClient instance to be in a classpath without any implementation for
-        // JsonService, in which case the ProblemDetails exceptions will fallback to default and not be parsed.
-        JsonService js;
-        try {
-            js = ServiceProvider.getJsonService();
-        } catch (IllegalStateException e) {
-            js = null;
-        }
-        this.jsonService = js;
     }
 
     /**
@@ -138,16 +127,10 @@ public <T extends Resource> CompletionStage<T> read(final URI identifier, final
         return client.send(request, Response.BodyHandlers.ofByteArray())
             .thenApply(response -> {
                 if (response.statusCode() >= ERROR_STATUS) {
-                    final ProblemDetails pd = ProblemDetails.fromErrorResponse(
-                        response.statusCode(),
-                        response.headers(),
-                        response.body(),
-                        this.jsonService
-                    );
                     throw SolidClientException.handle(
                         "Unable to read resource at " + request.uri(),
-                        pd,
                         response.uri(),
+                        response.statusCode(),
                         response.headers(),
                         new String(response.body())
                     );
@@ -299,16 +282,10 @@ public <T extends Resource> CompletionStage<Void> delete(final T resource, final
             if (isSuccess(res.statusCode())) {
                 return null;
             } else {
-                final ProblemDetails pd = ProblemDetails.fromErrorResponse(
-                    res.statusCode(),
-                    res.headers(),
-                    res.body(),
-                    this.jsonService
-                );
                 throw SolidClientException.handle(
                     "Unable to delete resource",
-                    pd,
                     resource.getIdentifier(),
+                    res.statusCode(),
                     res.headers(),
                     new String(res.body())
                 );
@@ -397,16 +374,10 @@ <T extends Resource> Function<Response<byte[]>, CompletionStage<T>> handleRespon
             final Headers headers, final String message) {
         return res -> {
             if (!isSuccess(res.statusCode())) {
-                final ProblemDetails pd = ProblemDetails.fromErrorResponse(
-                    res.statusCode(),
-                    res.headers(),
-                    res.body(),
-                    this.jsonService
-                );
                 throw SolidClientException.handle(
                     message,
-                    pd,
                     resource.getIdentifier(),
+                    res.statusCode(),
                     res.headers(),
                     new String(res.body())
                 );