cmen / CMENGoogleChartsBundle

GoogleCharts/Chart.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,8 +2,8 @@
 

 namespace CMEN\GoogleChartsBundle\GoogleCharts;

 

-use CMEN\GoogleChartsBundle\Exception\GoogleChartsException;

-use CMEN\GoogleChartsBundle\GoogleCharts\Charts\Diff\DiffChart;

+use CMEN\GoogleChartsBundle\Exception\GoogleChartsException;
+use CMEN\GoogleChartsBundle\GoogleCharts\Charts\Diff\DiffChart;
 use CMEN\GoogleChartsBundle\GoogleCharts\Options\ChartOptionsDraw;

 

 /**

Spacing +12 added lines, -12 removed lines

@@ -46,7 +46,7 @@ 
      */

     public function getName()

     {

-        return 'chart' . ucfirst($this->elementID);

+        return 'chart'.ucfirst($this->elementID);

     }

 

     /**

@@ -56,7 +56,7 @@ 
      */

     public function getDataName()

     {

-        return 'data'. ucfirst($this->getName());

+        return 'data'.ucfirst($this->getName());

     }

 

     /**

@@ -66,7 +66,7 @@ 
      */

     public function getOptionsName()

     {

-        return 'options'. ucfirst($this->getName());

+        return 'options'.ucfirst($this->getName());

     }

 

     /**

@@ -117,18 +117,18 @@ 
             throw new GoogleChartsException('Container is not defined.');

         }

 

-        $js = 'var ' . $this->getName() . ' = new google.' . $this->getLibrary() . '.' . $this->getType() .

-            '(document.getElementById("' . $this->elementID . '"));';

+        $js = 'var '.$this->getName().' = new google.'.$this->getLibrary().'.'.$this->getType().

+            '(document.getElementById("'.$this->elementID.'"));';

 

         if (!$this instanceof DiffChart) {

             $js .= $this->data->draw($this->getDataName());

 

         } else {

-            $js .= $this->getOldChart()->getData()->draw('old_' . $this->getDataName()) .

-                $this->getNewChart()->getData()->draw('new_' . $this->getDataName()) .

-                'var ' . $this->getDataName() . ' = ' . $this->getName() .

-                    '.computeDiff(old_' . $this->getDataName() . ',

-                 new_' . $this->getDataName() . ');';

+            $js .= $this->getOldChart()->getData()->draw('old_'.$this->getDataName()).

+                $this->getNewChart()->getData()->draw('new_'.$this->getDataName()).

+                'var '.$this->getDataName().' = '.$this->getName().

+                    '.computeDiff(old_'.$this->getDataName().',

+                 new_' . $this->getDataName().');';

         }

 

         $js .= $this->options->draw($this->getOptionsName());

@@ -143,8 +143,8 @@ 
      */

     public function endDraw()

     {

-        return $this->events->draw(). $this->getName() .

-            '.draw('. $this->getDataName() . ', '. $this->getOptionsName() .');';

+        return $this->events->draw().$this->getName().

+            '.draw('.$this->getDataName().', '.$this->getOptionsName().');';

     }

 

     /**

GoogleCharts/Data.php

Spacing +2 added lines, -2 removed lines

@@ -55,8 +55,8 @@
             $lastKeyValue = key($row);

             foreach ($row as $key => $value) {

                 if ($value instanceof \DateTime) {

-                    $js .= 'new Date(' . $value->format('Y') . ', ' . ($value->format('n')-1) . ', ' .

-                        $value->format('d') . ')';

+                    $js .= 'new Date('.$value->format('Y').', '.($value->format('n')-1).', '.

+                        $value->format('d').')';

                 } else {

                     $js .= json_encode($value);

                 }

GoogleCharts/Events.php

Spacing +2 added lines, -2 removed lines

@@ -46,8 +46,8 @@
         $js = '';
 
         foreach ($this->listeners as $listener) {
-            $js .= "google.visualization.events.addListener(" . $this->chart->getName() . ", '" . $listener->getType() .
-                "', " . $listener->getFunction() . ");";
+            $js .= "google.visualization.events.addListener(".$this->chart->getName().", '".$listener->getType().
+                "', ".$listener->getFunction().");";
         }
 
         return $js;

GoogleCharts/Options/ChartOptionsDraw.php

Spacing +1 added lines, -1 removed lines

@@ -40,6 +40,6 @@
 
         $options = $this->optionsFormat->renameRecursivelyKeys($options);
 
-        return "var $optionsName = " . json_encode($options) . ";\n";
+        return "var $optionsName = ".json_encode($options).";\n";
     }
 }

GoogleCharts/Options/Timeline/TimelineOptions.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         parent::__construct();
 
         $this->timeline = new Timeline();
-        $this->tooltip =  new BasicTooltip();
+        $this->tooltip = new BasicTooltip();
     }
 
 

GoogleCharts/Tools/OptionsFormat.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      */
     public function removeRecursivelyNullValue(&$options)
     {
-        $options = array_filter((array)$options, function ($val) {
+        $options = array_filter((array)$options, function($val) {
             return !is_null($val);
         });
 

Twig/GoogleChartsExtension.php

Spacing +10 added lines, -10 removed lines

@@ -159,7 +159,7 @@ 
                     );
                 }
 
-                $callback = function ($item) {
+                $callback = function($item) {
                     if (!is_string($item)) {
                         throw new GoogleChartsException('A string is expected for HTML element ID.');
                     }
@@ -185,7 +185,7 @@ 
      */
     private function checkChartType($charts)
     {
-        $callback = function ($item) {
+        $callback = function($item) {
             if (!$item instanceof Chart) {
                 throw new GoogleChartsException('An instance of Chart or an array of Chart is expected.');
             }
@@ -202,19 +202,19 @@ 
      */
     private function loadLibraries($packages)
     {
-        array_walk($packages, function (&$item) {
-            $item = '"' . $item . '"';
+        array_walk($packages, function(&$item) {
+            $item = '"'.$item.'"';
         });
 
-        ($this->language) ? $language = ', language: "' . $this->language . '"' : $language = '';
+        ($this->language) ? $language = ', language: "'.$this->language.'"' : $language = '';
 
-        $load = '"' . $this->version . '", {packages:[' . implode(',', $packages) . ']' . $language . '}';
+        $load = '"'.$this->version.'", {packages:['.implode(',', $packages).']'.$language.'}';
 
         if ($this->version == '1' || $this->version == '1.1') {
-            return 'google.load("visualization", ' . $load . ');';
+            return 'google.load("visualization", '.$load.');';
 
         } else {
-            return 'google.charts.load(' . $load . ');';
+            return 'google.charts.load('.$load.');';
         }
     }
 
@@ -266,7 +266,7 @@ 
 
         $js = $this->loadLibraries($packages);
 
-        $js .= $this->startCallback('drawChart' . ucfirst(md5($drawChartName)));
+        $js .= $this->startCallback('drawChart'.ucfirst(md5($drawChartName)));
 
         foreach ($charts as $chart) {
             $js .= $chart->startDraw();
@@ -305,7 +305,7 @@ 
      */
     public function drawCharts($charts, $elementsID = null)
     {
-        return $this->startCharts($charts, $elementsID) . $this->endCharts($charts);
+        return $this->startCharts($charts, $elementsID).$this->endCharts($charts);
     }
 
     /**