Merge pull request #264 from Inchoo/bugfix/#231-edge-module-boolean-value-support

Added check that converts true/false string values to boolean

Author: vvuksan

Diff for: Controller/Adminhtml/FastlyCdn/ImageOptimization/PushImageSettings.php

@@ -150,7 +150,7 @@ public function execute()
                             'id'            => $id,
                             'type'          => 'io_settings',
                             'attributes'    => [
-                                'webp'  	=> true
+                                'webp'      => true
                             ]
                         ]
                     ];
@@ -161,8 +161,10 @@ public function execute()
                         $imageParams['data']['attributes']['jpeg_quality'] = $imageQuality;
                     }
 
-                    $this->api->configureImageOptimizationDefaultConfigOptions(json_encode($imageParams), $clone->number);
-
+                    $this->api->configureImageOptimizationDefaultConfigOptions(
+                        json_encode($imageParams),
+                        $clone->number
+                    );
                 }
             } else {
                 $this->api->deleteRequest($clone->number, $reqName);

Diff for: Model/FrontControllerPlugin.php

@@ -230,10 +230,12 @@ private function processData($data, $tag, $ttl, $limit, $limitingType = "path")
 
             if ($usage >= $limit) {
                 $this->response->setStatusHeader(429, null, 'API limit exceeded');
-                if ( $limitingType == "path" )
+                if ($limitingType == "path") {
                     $this->response->setHeader('Surrogate-Control', 'max-age=' . $ttl);
-                if ( $limitingType == "crawler" )
+                }
+                if ($limitingType == "crawler") {
                     $this->response->setHeader('Fastly-Vary', 'Fastly-Client-IP');
+                }
                 $this->response->setBody('<h1>Request limit exceeded</h1>');
                 $this->response->setNoCacheHeaders();
                 return true;

Diff for: view/adminhtml/web/js/modly.js

@@ -64,6 +64,9 @@ define([
                 $($(this).find('.modly-field')).each(function () {
                     name = $(this).attr('name');
                     value = $(this).val();
+                    if (value === "true" || value === "false") {
+                        value = (value === "true");
+                    }
                     data[name] = value;
                 });
                 groupData.push(data);