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());

     }

 

     /**

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

         }

 

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

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

+        $js = 'var '.$this->getName().' = new google.visualization.'.$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());

@@ -136,8 +136,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/Charts/OrgChart.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * @param $options OrgChartOptions
      *
-     * @return Map
+     * @return OrgChart
      */
     public function setOptions($options)
     {

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/GeoChart/GeoChartOptions.php

Indentation -1 removed lines

@@ -95,7 +95,6 @@
      *    - A country, specified by its ISO 3166-1 alpha-2 code, e.g., 'AU' for Australia.
      *    - A state in the United States, specified by its ISO 3166-2:US code, e.g., 'US-AL' for Alabama. Note that
      *      the resolution option must be set to either 'provinces' or 'metros'.
-
      * @var string
      */
     protected $region;

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);
         });
 

GoogleCharts/Options/AnnotationChart/AnnotationChartOptions.php

Indentation -1 removed lines

@@ -95,7 +95,6 @@
      *    - A country, specified by its ISO 3166-1 alpha-2 code, e.g., 'AU' for Australia.
      *    - A state in the United States, specified by its ISO 3166-2:US code, e.g., 'US-AL' for Alabama. Note that
      *      the resolution option must be set to either 'provinces' or 'metros'.
-
      * @var string
      */
     protected $region;

Twig/GoogleChartsExtension.php

Spacing +8 added lines, -8 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,13 +202,13 @@ 
      */
     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.'}';
 
         return "google.charts.load($load);";
     }
@@ -261,7 +261,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();
@@ -300,7 +300,7 @@ 
      */
     public function drawCharts($charts, $elementsID = null)
     {
-        return $this->startCharts($charts, $elementsID) . $this->endCharts($charts);
+        return $this->startCharts($charts, $elementsID).$this->endCharts($charts);
     }
 
     /**