Revert "Bug 23075: Better error handling in Mana config"
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 17 Jun 2019 15:53:17 +0000 (16:53 +0100)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 17 Jun 2019 15:54:05 +0000 (16:54 +0100)
This reverts commit adb30fb8a0347ab6ff391238a8bd1ce4b6931fcd.

We missed some cases that means this handling was throwing out perfectly
accepable requests.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Koha/SharedContent.pm
koha-tmpl/intranet-tmpl/prog/en/modules/admin/share_content.tt

index 80b4309..580fef2 100644 (file)
@@ -60,13 +60,6 @@ sub process_request {
 
     my $response = $userAgent->request($mana_request);
 
-    if ( $response->code != 200 ) {
-        return {
-            code => $response->code,
-            msg  => $response->message,
-        };
-    }
-
     eval { $result = from_json( $response->decoded_content, { utf8 => 1} ); };
     $result->{code} = $response->code;
     if ( $@ ){
index df8599e..11d4e6c 100644 (file)
@@ -19,7 +19,7 @@
             <div class="col-sm-6">
                 [% IF result.code != 201 && result.msg %]
                     <div class="dialog alert" role="alert">
-                        Error trying to connect or read response from the Mana server (Code: [% result.code | html %]) [% result.msg | html %]
+                        [% result.msg | html %]
                     </div>
                 [% END %]