Bug 33290: Fix incorrect variable in http-client.js
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 21 Mar 2023 07:52:42 +0000 (08:52 +0100)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 27 Mar 2023 09:29:56 +0000 (11:29 +0200)
Silly error from
  commit 821808ec31176d662e518418c9ff8a71948e1a4b
  Bug 32939: Use APIClient to replace PATCH requests

+        const body = params.body
+            ? typeof str === "string"
+                ? params.body
+                : JSON.stringify(params.body)
+            : undefined;

  typeof str === "string"
must be
  typeof params.body === "string"

Test plan:
With this patch, bug 33289 is not working properly. The body contains
  "pref_var=value"
instead of
  pref_var=value
and the syspref `var` contains `value"`

Sponsored-by: BULAC - http://www.bulac.fr/
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/js/vue/fetch/http-client.js

index 3745822..ac999e9 100644 (file)
@@ -52,7 +52,7 @@ class HttpClient {
 
     post(params = {}) {
         const body = params.body
-            ? typeof str === "string"
+            ? typeof params.body === "string"
                 ? params.body
                 : JSON.stringify(params.body)
             : undefined;
@@ -65,7 +65,7 @@ class HttpClient {
 
     put(params = {}) {
         const body = params.body
-            ? typeof str === "string"
+            ? typeof params.body === "string"
                 ? params.body
                 : JSON.stringify(params.body)
             : undefined;
@@ -105,7 +105,7 @@ class HttpClient {
 
     patch(params = {}) {
         const body = params.body
-            ? typeof str === "string"
+            ? typeof params.body === "string"
                 ? params.body
                 : JSON.stringify(params.body)
             : undefined;