update szymach/c-pchart (matomo-org#19604)

sgiehl · web-flow · commit a8338460fd98 · 2022-08-08T01:57:03.000+02:00
* update szymach/c-pchart

* fix php8.1 deprecation notice
diff --git a/composer.json b/composer.json
@@ -59,7 +59,7 @@
         "symfony/monolog-bridge": "~2.6.0",
         "symfony/polyfill-iconv": "^1.20",
         "symfony/polyfill-mbstring": "^1.20",
-        "szymach/c-pchart": "3.0.13",
+        "szymach/c-pchart": "~3.0.13",
         "tecnickcom/tcpdf": "~6.0",
         "tedivm/jshrink": "~v1.4.0",
         "twig/twig": "^3.0",
diff --git a/composer.lock b/composer.lock
diff --git a/core/DataTable/Renderer/Csv.php b/core/DataTable/Renderer/Csv.php
@@ -239,13 +239,12 @@ protected function formatValue($value)
 
         $value = $this->formatFormulas($value);
 
-        $value = str_replace(["\t"], ' ', $value);
+        if (is_string($value)) {
+            $value = str_replace(["\t"], ' ', $value);
 
-        if (is_string($value)
-            && (strpos($value, '"') !== false
-                || strpos($value, $this->separator) !== false)
-        ) {
-            $value = '"' . str_replace('"', '""', $value) . '"';
+            if (strpos($value, '"') !== false || strpos($value, $this->separator) !== false) {
+                $value = '"' . str_replace('"', '""', $value) . '"';
+            }
         }
 
         // in some number formats (e.g. German), the decimal separator is a comma
