adamjakab / SuiteCRM

modules/AOS_Products/metadata/sidecreateviewdefs.php

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /**
  * Products, Quotations & Invoices modules.
  * Extensions to SugarCRM
@@ -40,16 +40,16 @@ 
                                             array('label' => '10', 'field' => '30'),
                                          ),
                         ),
- 'panels' =>array (
+ 'panels' =>array(
   'DEFAULT' =>
-  array (
+  array(
     array(
-      array('name'=>'name', 'displayParams'=>array('required'=>true,'size'=>20)),
+      array('name'=>'name', 'displayParams'=>array('required'=>true, 'size'=>20)),
     ),
-    array (
-      array('name'=>'description','displayParams'=>array('rows'=>3, 'cols'=>20)),
+    array(
+      array('name'=>'description', 'displayParams'=>array('rows'=>3, 'cols'=>20)),
     ),
-    array (
+    array(
       array('name'=>'assigned_user_name', 'displayParams'=>array('required'=>true, 'size'=>11, 'selectOnly'=>true)),
     ),
   ),

modules/TableDictionary.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -124,7 +124,7 @@ 
 include('metadata/securitygroups_recordsMetaData.php');
 include('metadata/securitygroups_usersMetaData.php');
 
-if(file_exists('custom/application/Ext/TableDictionary/tabledictionary.ext.php')){
+if (file_exists('custom/application/Ext/TableDictionary/tabledictionary.ext.php')) {
 	include('custom/application/Ext/TableDictionary/tabledictionary.ext.php');
 }
 ?>

modules/AOR_Charts/AOR_Chart.php

Spacing +153 added lines, -153 removed lines

@@ -30,7 +30,7 @@ 
 	var $object_name = 'AOR_Chart';
 	var $table_name = 'aor_charts';
 	var $importable = true;
-	var $disable_row_level_security = true ;
+	var $disable_row_level_security = true;
 	
 	var $id;
 	var $name;
@@ -52,148 +52,148 @@ 
 
 
 
-	function AOR_Chart(){
+	function AOR_Chart() {
 		parent::Basic();
 	}
 
-    function save_lines(array $post,AOR_Report $bean,$postKey){
+    function save_lines(array $post, AOR_Report $bean, $postKey) {
         $seenIds = array();
-        if(isset($post[$postKey.'id'])) {
-            foreach ($post[$postKey . 'id'] as $key => $id) {
+        if (isset($post[$postKey.'id'])) {
+            foreach ($post[$postKey.'id'] as $key => $id) {
                 if ($id) {
                     $aorChart = BeanFactory::getBean('AOR_Charts', $id);
                 } else {
                     $aorChart = BeanFactory::newBean('AOR_Charts');
                 }
-                $aorChart->name = $post[$postKey . 'title'][$key];
-                $aorChart->type = $post[$postKey . 'type'][$key];
-                $aorChart->x_field = $post[$postKey . 'x_field'][$key];
-                $aorChart->y_field = $post[$postKey . 'y_field'][$key];
+                $aorChart->name = $post[$postKey.'title'][$key];
+                $aorChart->type = $post[$postKey.'type'][$key];
+                $aorChart->x_field = $post[$postKey.'x_field'][$key];
+                $aorChart->y_field = $post[$postKey.'y_field'][$key];
                 $aorChart->aor_report_id = $bean->id;
                 $aorChart->save();
                 $seenIds[] = $aorChart->id;
             }
         }
         //Any beans that exist but aren't in $seenIds must have been removed.
-        foreach($bean->get_linked_beans('aor_charts','AOR_Charts') as $chart){
-            if(!in_array($chart->id,$seenIds)){
+        foreach ($bean->get_linked_beans('aor_charts', 'AOR_Charts') as $chart) {
+            if (!in_array($chart->id, $seenIds)) {
                 $chart->mark_deleted($chart->id);
             }
         }
     }
 
-    private function getValidChartTypes(){
-        return array('bar','line','pie','radar','rose', 'grouped_bar', 'stacked_bar');
+    private function getValidChartTypes() {
+        return array('bar', 'line', 'pie', 'radar', 'rose', 'grouped_bar', 'stacked_bar');
     }
 
 
-    private function getColour($seed,$rgbArray = false){
+    private function getColour($seed, $rgbArray = false) {
         $hash = md5($seed);
         $r = hexdec(substr($hash, 0, 2));
         $g = hexdec(substr($hash, 2, 2));
         $b = hexdec(substr($hash, 4, 2));
-        if($rgbArray){
-            return array('R'=>$r,'G'=>$g,'B'=>$b);
+        if ($rgbArray) {
+            return array('R'=>$r, 'G'=>$g, 'B'=>$b);
         }
         $highR = $r + 10;
         $highG = $g + 10;
         $highB = $b + 10;
-        $main = '#'.str_pad(dechex($r),2,'0',STR_PAD_LEFT)
-            .str_pad(dechex($g),2,'0',STR_PAD_LEFT)
-            .str_pad(dechex($b),2,'0',STR_PAD_LEFT);
+        $main = '#'.str_pad(dechex($r), 2, '0', STR_PAD_LEFT)
+            .str_pad(dechex($g), 2, '0', STR_PAD_LEFT)
+            .str_pad(dechex($b), 2, '0', STR_PAD_LEFT);
         $highlight = '#'.dechex($highR).dechex($highG).dechex($highB);
-        return array('main'=>$main,'highlight'=>$highlight);
+        return array('main'=>$main, 'highlight'=>$highlight);
     }
 
-    function buildChartImageBar($chartPicture,$recordImageMap = false){
+    function buildChartImageBar($chartPicture, $recordImageMap = false) {
         $scaleSettings = array("DrawSubTicks" => false, "LabelRotation" => 30, 'MinDivHeight' => 50);
         $chartPicture->drawScale($scaleSettings);
         $chartPicture->drawBarChart(array("RecordImageMap"=>$recordImageMap));
     }
 
-    function buildChartImagePie($chartPicture,$chartData, $reportData,$imageHeight, $imageWidth, $xName,$recordImageMap){
-        $PieChart = new pPie($chartPicture,$chartData);
+    function buildChartImagePie($chartPicture, $chartData, $reportData, $imageHeight, $imageWidth, $xName, $recordImageMap) {
+        $PieChart = new pPie($chartPicture, $chartData);
         $x = 0;
-        foreach($reportData as $row){
-            $PieChart->setSliceColor($x,$this->getColour($row[$xName],true));
+        foreach ($reportData as $row) {
+            $PieChart->setSliceColor($x, $this->getColour($row[$xName], true));
             $x++;
         }
-        $PieChart->draw2DPie($imageWidth/3,$imageHeight/2,array("Border"=>TRUE,'Radius'=>200,''=>true,"RecordImageMap"=>$recordImageMap));
-        $PieChart->drawPieLegend($imageWidth*0.7,$imageHeight/3, array('FontSize'=>10,"FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf",'BoxSize'=>14));
+        $PieChart->draw2DPie($imageWidth / 3, $imageHeight / 2, array("Border"=>TRUE, 'Radius'=>200, ''=>true, "RecordImageMap"=>$recordImageMap));
+        $PieChart->drawPieLegend($imageWidth * 0.7, $imageHeight / 3, array('FontSize'=>10, "FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf", 'BoxSize'=>14));
     }
 
-    function buildChartImageLine($chartPicture, $recordImageMap = false){
-        $scaleSettings = array("XMargin"=>10,"YMargin"=>10,"GridR"=>200,"GridG"=>200,"GridB"=>200,'MinDivHeight' => 50,"LabelRotation" => 30);
+    function buildChartImageLine($chartPicture, $recordImageMap = false) {
+        $scaleSettings = array("XMargin"=>10, "YMargin"=>10, "GridR"=>200, "GridG"=>200, "GridB"=>200, 'MinDivHeight' => 50, "LabelRotation" => 30);
         $chartPicture->drawScale($scaleSettings);
         $chartPicture->drawLineChart(array("RecordImageMap"=>$recordImageMap));
     }
 
-    function buildChartImageRadar($chartPicture, $chartData,$recordImageMap){
+    function buildChartImageRadar($chartPicture, $chartData, $recordImageMap) {
         $SplitChart = new pRadar();
-        $Options = array("LabelPos"=>RADAR_LABELS_HORIZONTAL,"RecordImageMap"=>$recordImageMap);
-        $SplitChart->drawRadar($chartPicture,$chartData,$Options);
+        $Options = array("LabelPos"=>RADAR_LABELS_HORIZONTAL, "RecordImageMap"=>$recordImageMap);
+        $SplitChart->drawRadar($chartPicture, $chartData, $Options);
 
     }
 
-    public function buildChartImage(array $reportData, array $fields,$asDataURI = true, $generateImageMapId = false){
+    public function buildChartImage(array $reportData, array $fields, $asDataURI = true, $generateImageMapId = false) {
         global $current_user;
         require_once 'modules/AOR_Charts/lib/pChart/pChart.php';
 
-        if($generateImageMapId !== false){
+        if ($generateImageMapId !== false) {
             $generateImageMapId = $current_user->id."-".$generateImageMapId;
         }
 
         $html = '';
-        if(!in_array($this->type, $this->getValidChartTypes())){
+        if (!in_array($this->type, $this->getValidChartTypes())) {
             return $html;
         }
         $x = $fields[$this->x_field];
         $y = $fields[$this->y_field];
-        if(!$x || !$y){
+        if (!$x || !$y) {
             //Malformed chart object - missing an axis field
             return '';
         }
-        $xName = str_replace(' ','_',$x->label) . $this->x_field;
-        $yName = str_replace(' ','_',$y->label) . $this->y_field;
+        $xName = str_replace(' ', '_', $x->label).$this->x_field;
+        $yName = str_replace(' ', '_', $y->label).$this->y_field;
 
         $chartData = new pData();
         $chartData->loadPalette("modules/AOR_Charts/lib/pChart/palettes/navy.color", TRUE);
         $labels = array();
-        foreach($reportData as $row){
-            $chartData->addPoints($row[$yName],'data');
-            $chartData->addPoints($row[$xName],'Labels');
+        foreach ($reportData as $row) {
+            $chartData->addPoints($row[$yName], 'data');
+            $chartData->addPoints($row[$xName], 'Labels');
             $labels[] = $row[$xName];
         }
 
-        $chartData->setSerieDescription("Months","Month");
+        $chartData->setSerieDescription("Months", "Month");
         $chartData->setAbscissa("Labels");
 
         $imageHeight = 700;
         $imageWidth = 700;
 
-        $chartPicture = new pImage($imageWidth,$imageHeight,$chartData);
-        if($generateImageMapId){
+        $chartPicture = new pImage($imageWidth, $imageHeight, $chartData);
+        if ($generateImageMapId) {
             $imageMapDir = create_cache_directory('modules/AOR_Charts/ImageMap/'.$current_user->id.'/');
-            $chartPicture->initialiseImageMap($generateImageMapId,IMAGE_MAP_STORAGE_FILE,$generateImageMapId,$imageMapDir);
+            $chartPicture->initialiseImageMap($generateImageMapId, IMAGE_MAP_STORAGE_FILE, $generateImageMapId, $imageMapDir);
         }
 
         $chartPicture->Antialias = True;
 
-        $chartPicture->drawFilledRectangle(0,0,$imageWidth-1,$imageHeight-1,array("R"=>240,"G"=>240,"B"=>240,"BorderR"=>0,"BorderG"=>0,"BorderB"=>0,));
+        $chartPicture->drawFilledRectangle(0, 0, $imageWidth - 1, $imageHeight - 1, array("R"=>240, "G"=>240, "B"=>240, "BorderR"=>0, "BorderG"=>0, "BorderB"=>0,));
 
-        $chartPicture->setFontProperties(array("FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf","FontSize"=>14));
+        $chartPicture->setFontProperties(array("FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf", "FontSize"=>14));
 
-        $chartPicture->drawText($imageWidth/2,20,$this->name,array("R"=>0,"G"=>0,"B"=>0,'Align'=>TEXT_ALIGN_TOPMIDDLE));
-        $chartPicture->setFontProperties(array("FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf","FontSize"=>6));
+        $chartPicture->drawText($imageWidth / 2, 20, $this->name, array("R"=>0, "G"=>0, "B"=>0, 'Align'=>TEXT_ALIGN_TOPMIDDLE));
+        $chartPicture->setFontProperties(array("FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf", "FontSize"=>6));
 
-        $chartPicture->setGraphArea(60,60,$imageWidth-60,$imageHeight-100);
+        $chartPicture->setGraphArea(60, 60, $imageWidth - 60, $imageHeight - 100);
 
-        switch($this->type){
+        switch ($this->type) {
             case 'radar':
                 $this->buildChartImageRadar($chartPicture, $chartData, !empty($generateImageMapId));
                 break;
             case 'pie':
-                $this->buildChartImagePie($chartPicture,$chartData, $reportData,$imageHeight, $imageWidth, $xName, !empty($generateImageMapId));
+                $this->buildChartImagePie($chartPicture, $chartData, $reportData, $imageHeight, $imageWidth, $xName, !empty($generateImageMapId));
                 break;
             case 'line':
                 $this->buildChartImageLine($chartPicture, !empty($generateImageMapId));
@@ -203,50 +203,50 @@ 
                 $this->buildChartImageBar($chartPicture, !empty($generateImageMapId));
                 break;
         }
-        if($generateImageMapId) {
+        if ($generateImageMapId) {
             $chartPicture->replaceImageMapTitle("data", $labels);
         }
         ob_start();
         $chartPicture->render('');
         $img = ob_get_clean();
-        if($asDataURI){
+        if ($asDataURI) {
             return 'data:image/png;base64,'.base64_encode($img);
-        }else{
+        } else {
             return $img;
         }
     }
 
-    public function buildChartHTML(array $reportData, array $fields,$index = 0, $chartType = AOR_Report::CHART_TYPE_PCHART, AOR_Field $mainGroupField = null){
-        switch($chartType){
+    public function buildChartHTML(array $reportData, array $fields, $index = 0, $chartType = AOR_Report::CHART_TYPE_PCHART, AOR_Field $mainGroupField = null) {
+        switch ($chartType) {
             case AOR_Report::CHART_TYPE_PCHART:
-                return $this->buildChartHTMLPChart($reportData,$fields,$index);
+                return $this->buildChartHTMLPChart($reportData, $fields, $index);
             case AOR_Report::CHART_TYPE_CHARTJS:
-                return $this->buildChartHTMLChartJS($reportData,$fields);
+                return $this->buildChartHTMLChartJS($reportData, $fields);
             case AOR_Report::CHART_TYPE_RGRAPH:
-                return $this->buildChartHTMLRGraph($reportData,$fields, $mainGroupField);
+                return $this->buildChartHTMLRGraph($reportData, $fields, $mainGroupField);
         }
         return '';
     }
 
 
-    private function buildChartHTMLRGraph(array $reportData, array $fields, AOR_Field $mainGroupField = null){
+    private function buildChartHTMLRGraph(array $reportData, array $fields, AOR_Field $mainGroupField = null) {
         $html = '';
-        if(!in_array($this->type, $this->getValidChartTypes())){
+        if (!in_array($this->type, $this->getValidChartTypes())) {
             return $html;
         }
         $x = $fields[$this->x_field];
         $y = $fields[$this->y_field];
-        if(!$x || !$y){
+        if (!$x || !$y) {
             //Malformed chart object - missing an axis field
             return '';
         }
-        $xName = str_replace(' ','_',$x->label) . $this->x_field;
-        $yName = str_replace(' ','_',$y->label) . $this->y_field;
+        $xName = str_replace(' ', '_', $x->label).$this->x_field;
+        $yName = str_replace(' ', '_', $y->label).$this->y_field;
 
         $defaultHeight = 500;
         $defaultWidth = 900;
 
-        switch($this->type){
+        switch ($this->type) {
             /*
              //Polar was not implemented for the previous library (it is not in the getValidChartTypes method)
             case 'polar':
@@ -257,56 +257,56 @@ 
             */
             case 'radar':
                 $chartFunction = 'Radar';
-                $data = $this->getRGraphBarChartData($reportData, $xName,$yName);
+                $data = $this->getRGraphBarChartData($reportData, $xName, $yName);
                 $config = $this->getRadarChartConfig();
-                $chart = $this->getRGraphRadarChart(json_encode($data['data']), json_encode($data['labels']),json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight,$defaultWidth);
+                $chart = $this->getRGraphRadarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth);
                 break;
             case 'pie':
                 $chartFunction = 'Pie';
-                $data = $this->getRGraphBarChartData($reportData, $xName,$yName);
+                $data = $this->getRGraphBarChartData($reportData, $xName, $yName);
                 $config = $this->getPieChartConfig();
-                $chart = $this->getRGraphPieChart(json_encode($data['data']), json_encode($data['labels']),json_encode($data['tooltips']), $this->name, $this->id,  $defaultHeight,$defaultWidth);
+                $chart = $this->getRGraphPieChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth);
                 break;
             case 'line':
                 $chartFunction = 'Line';
-                $data = $this->getRGraphBarChartData($reportData, $xName,$yName);
+                $data = $this->getRGraphBarChartData($reportData, $xName, $yName);
                 $config = $this->getLineChartConfig();
-                $chart = $this->getRGraphLineChart(json_encode($data['data']), json_encode($data['labels']),json_encode($data['tooltips']), $this->name, $this->id,  $defaultHeight,$defaultWidth);
+                $chart = $this->getRGraphLineChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth);
                 break;
             case 'rose':
                 $chartFunction = 'Rose';
-                $data = $this->getRGraphBarChartData($reportData, $xName,$yName);
+                $data = $this->getRGraphBarChartData($reportData, $xName, $yName);
                 $config = $this->getRoseChartConfig();
-                $chart = $this->getRGraphRoseChart(json_encode($data['data']), json_encode($data['labels']),json_encode($data['tooltips']), $this->name, $this->id,  $defaultHeight,$defaultWidth);
+                $chart = $this->getRGraphRoseChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth);
                 break;
             case 'grouped_bar':
                 $chartFunction = 'Grouped bar';
-                $data = $this->getRGraphGroupedBarChartData($reportData, $xName,$yName, $mainGroupField);
+                $data = $this->getRGraphGroupedBarChartData($reportData, $xName, $yName, $mainGroupField);
                 $config = $this->getGroupedBarChartConfig();
-                $chart = $this->getRGraphGroupedBarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id,  $defaultHeight,$defaultWidth, true);
+                $chart = $this->getRGraphGroupedBarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth, true);
                 break;
             case 'stacked_bar':
                 $chartFunction = 'Stacked bar';
-                $data = $this->getRGraphGroupedBarChartData($reportData, $xName,$yName, $mainGroupField);
+                $data = $this->getRGraphGroupedBarChartData($reportData, $xName, $yName, $mainGroupField);
                 $config = $this->getStackedBarChartConfig();
-                $chart = $this->getRGraphGroupedBarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id,  $defaultHeight,$defaultWidth, false);
+                $chart = $this->getRGraphGroupedBarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth, false);
                 break;
             case 'bar':
             default:
                 $chartFunction = 'Bar';
-                $data = $this->getRGraphBarChartData($reportData, $xName,$yName);
+                $data = $this->getRGraphBarChartData($reportData, $xName, $yName);
                 $config = $this->getBarChartConfig();
-                $chart = $this->getRGraphBarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id,  $defaultHeight,$defaultWidth);
+                $chart = $this->getRGraphBarChart(json_encode($data['data']), json_encode($data['labels']), json_encode($data['tooltips']), $this->name, $this->id, $defaultHeight, $defaultWidth);
                 break;
         }
 
         return $chart;
     }
 
-    private function getRGraphRoseChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
+    private function getRGraphRoseChart($chartDataValues, $chartLabelValues, $chartTooltips, $chartName = '', $chartId, $chartHeight = 400, $chartWidth = 400)
     {
         $dataArray = json_decode($chartDataValues);
-        if(!is_array($dataArray)||count($dataArray) < 1)
+        if (!is_array($dataArray) || count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
         }
@@ -340,14 +340,14 @@ 
     //I have not used a parameter for getRGraphBarChart to say whether to group etc, as the future development could be quite different
     //for both, hence the separate methods.  However, the $grouped parameter allows us to specify whether the chart is grouped (true)
     //or stacked (false)
-    private function getRGraphGroupedBarChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400, $grouped = false)
+    private function getRGraphGroupedBarChart($chartDataValues, $chartLabelValues, $chartTooltips, $chartName = '', $chartId, $chartHeight = 400, $chartWidth = 400, $grouped = false)
     {
         //$keys = array_keys($chartTooltips);
 
 
-        $i=0;
-        foreach($chartDataValues as $rowKey => $row) {
-            foreach($row as $key => $value) {
+        $i = 0;
+        foreach ($chartDataValues as $rowKey => $row) {
+            foreach ($row as $key => $value) {
                 $_tooltips[$rowKey][$key] = $chartTooltips[$i];
                 $i++;
             }
@@ -356,9 +356,9 @@ 
 
         $dataArray = json_decode($chartDataValues);
         $grouping = 'grouped'; //$mainGroupField->label; //'grouped';
-        if(!$grouped)
-            $grouping='stacked';
-        if(!is_array($dataArray)||count($dataArray) < 1)
+        if (!$grouped)
+            $grouping = 'stacked';
+        if (!is_array($dataArray) || count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
         }
@@ -398,10 +398,10 @@ 
 
 
 
-    private function getRGraphBarChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
+    private function getRGraphBarChart($chartDataValues, $chartLabelValues, $chartTooltips, $chartName = '', $chartId, $chartHeight = 400, $chartWidth = 400)
     {
         $dataArray = json_decode($chartDataValues);
-        if(!is_array($dataArray)||count($dataArray) < 1)
+        if (!is_array($dataArray) || count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
         }
@@ -437,10 +437,10 @@ 
         return $html;
     }
 
-    private function getRGraphRadarChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
+    private function getRGraphRadarChart($chartDataValues, $chartLabelValues, $chartTooltips, $chartName = '', $chartId, $chartHeight = 400, $chartWidth = 400)
     {
         $dataArray = json_decode($chartDataValues);
-        if(!is_array($dataArray)||count($dataArray) < 1)
+        if (!is_array($dataArray) || count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
         }
@@ -469,10 +469,10 @@ 
         return $html;
     }
 
-    private function getRGraphPieChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
+    private function getRGraphPieChart($chartDataValues, $chartLabelValues, $chartTooltips, $chartName = '', $chartId, $chartHeight = 400, $chartWidth = 400)
     {
         $dataArray = json_decode($chartDataValues);
-        if(!is_array($dataArray)||count($dataArray) < 1)
+        if (!is_array($dataArray) || count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
         }
@@ -505,10 +505,10 @@ 
         return $html;
     }
 
-    private function getRGraphLineChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
+    private function getRGraphLineChart($chartDataValues, $chartLabelValues, $chartTooltips, $chartName = '', $chartId, $chartHeight = 400, $chartWidth = 400)
     {
         $dataArray = json_decode($chartDataValues);
-        if(!is_array($dataArray)||count($dataArray) < 1)
+        if (!is_array($dataArray) || count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
         }
@@ -546,45 +546,45 @@ 
         return $html;
     }
 
-    private function buildChartHTMLChartJS(array $reportData, array $fields){
+    private function buildChartHTMLChartJS(array $reportData, array $fields) {
         $html = '';
-        if(!in_array($this->type, $this->getValidChartTypes())){
+        if (!in_array($this->type, $this->getValidChartTypes())) {
             return $html;
         }
         $x = $fields[$this->x_field];
         $y = $fields[$this->y_field];
-        if(!$x || !$y){
+        if (!$x || !$y) {
             //Malformed chart object - missing an axis field
             return '';
         }
-        $xName = str_replace(' ','_',$x->label) . $this->x_field;
-        $yName = str_replace(' ','_',$y->label) . $this->y_field;
+        $xName = str_replace(' ', '_', $x->label).$this->x_field;
+        $yName = str_replace(' ', '_', $y->label).$this->y_field;
 
-        switch($this->type){
+        switch ($this->type) {
             case 'polar':
                 $chartFunction = 'PolarArea';
-                $data = $this->getPolarChartData($reportData, $xName,$yName);
+                $data = $this->getPolarChartData($reportData, $xName, $yName);
                 $config = $this->getPolarChartConfig();
                 break;
             case 'radar':
                 $chartFunction = 'Radar';
-                $data = $this->getRadarChartData($reportData, $xName,$yName);
+                $data = $this->getRadarChartData($reportData, $xName, $yName);
                 $config = $this->getRadarChartConfig();
                 break;
             case 'pie':
                 $chartFunction = 'Pie';
-                $data = $this->getPieChartData($reportData, $xName,$yName);
+                $data = $this->getPieChartData($reportData, $xName, $yName);
                 $config = $this->getPieChartConfig();
                 break;
             case 'line':
                 $chartFunction = 'Line';
-                $data = $this->getLineChartData($reportData, $xName,$yName);
+                $data = $this->getLineChartData($reportData, $xName, $yName);
                 $config = $this->getLineChartConfig();
                 break;
             case 'bar':
             default:
                 $chartFunction = 'Bar';
-                $data = $this->getBarChartData($reportData, $xName,$yName);
+                $data = $this->getBarChartData($reportData, $xName, $yName);
                 $config = $this->getBarChartConfig();
                 break;
         }
@@ -611,9 +611,9 @@ 
         return $html;
     }
 
-    private function buildChartHTMLPChart(array $reportData, array $fields,$index = 0){
+    private function buildChartHTMLPChart(array $reportData, array $fields, $index = 0) {
         $html = '';
-        $imgUri = $this->buildChartImage($reportData,$fields,true,$index);
+        $imgUri = $this->buildChartImage($reportData, $fields, true, $index);
         $img = "<img id='{$this->id}_img' src='{$imgUri}'>";
         $html .= $img;
         $html .= <<<EOF
@@ -628,24 +628,24 @@ 
 
     private function getShortenedLabel($label, $maxLabelSize = 20)
     {
-        if(strlen($label) > $maxLabelSize)
+        if (strlen($label) > $maxLabelSize)
         {
-            return substr($label,0,$maxLabelSize).'...';
+            return substr($label, 0, $maxLabelSize).'...';
         }
         else
             return $label;
     }
 
 
-    private function getRGraphGroupedBarChartData($reportData, $xName,$yName, AOR_Field $mainGroupField){
+    private function getRGraphGroupedBarChartData($reportData, $xName, $yName, AOR_Field $mainGroupField) {
 
 
         // get z-axis name
 
         $zName = null;
-        foreach($reportData[0] as $key => $value) {
+        foreach ($reportData[0] as $key => $value) {
             $field = str_replace(' ', '_', $mainGroupField->label);
-            if (preg_match('/^' . $field . '[0-9]+/', $key)) {
+            if (preg_match('/^'.$field.'[0-9]+/', $key)) {
                 $zName = $key;
                 break;
             }
@@ -659,20 +659,20 @@ 
         $tooltips = array();
 
         $usedKeys = array();
-        foreach($reportData as $key => $row) {
+        foreach ($reportData as $key => $row) {
             $filter = $row[$xName];
-            foreach($reportData as $key2 => $row2) {
-                if($row2[$xName] == $filter && !in_array($key, $usedKeys)) {
-                    $data      [ $row[$xName]  ]   [] = (float) $row[$yName];
-                    $tooltips  [ $row[$xName]  ]   [] = $row[$zName];
+            foreach ($reportData as $key2 => $row2) {
+                if ($row2[$xName] == $filter && !in_array($key, $usedKeys)) {
+                    $data      [$row[$xName]]   [] = (float)$row[$yName];
+                    $tooltips  [$row[$xName]]   [] = $row[$zName];
                     $usedKeys[] = $key;
                 }
             }
         }
 
         $_data = array();
-        foreach($data as $label => $values) {
-            foreach($values as $key => $value) {
+        foreach ($data as $label => $values) {
+            foreach ($values as $key => $value) {
                 $_data[$label][$tooltips[$label][$key]] = $value;
             }
         }
@@ -684,11 +684,11 @@ 
         $_data = array();
         $_labels = array();
         $_tooltips = array();
-        foreach($data as $label => $values) {
-            $_labels[] = $this->getShortenedLabel($label) . $this->getChartDataNameLabel($label);
+        foreach ($data as $label => $values) {
+            $_labels[] = $this->getShortenedLabel($label).$this->getChartDataNameLabel($label);
             $_values = array();
-            foreach($values as $tooltip => $value) {
-                $_tooltips[] = $tooltip . " ($value)";
+            foreach ($values as $tooltip => $value) {
+                $_tooltips[] = $tooltip." ($value)";
                 $_values[] = $value;
             }
             $_data[] = $_values;
@@ -706,13 +706,13 @@ 
 
     }
 
-    private function getRGraphBarChartData($reportData, $xName,$yName){
-        $chart['labels']=array();
-        $chart['data']=array();
-        $chart['tooltips']=array();
-        foreach($reportData as $row){
-            $chart['labels'][] = $this->getShortenedLabel($row[$xName]) . $this->getChartDataNameLabel($row[$xName]);
-            $chart['tooltips'][] = $row[$xName] . $this->getChartDataNameLabel($row[$xName]);
+    private function getRGraphBarChartData($reportData, $xName, $yName) {
+        $chart['labels'] = array();
+        $chart['data'] = array();
+        $chart['tooltips'] = array();
+        foreach ($reportData as $row) {
+            $chart['labels'][] = $this->getShortenedLabel($row[$xName]).$this->getChartDataNameLabel($row[$xName]);
+            $chart['tooltips'][] = $row[$xName].$this->getChartDataNameLabel($row[$xName]);
             $chart['data'][] = (float)$row[$yName];
 
         }
@@ -720,12 +720,12 @@ 
     }
 
 
-    private function getBarChartData($reportData, $xName,$yName){
+    private function getBarChartData($reportData, $xName, $yName) {
         $data = array();
         $data['labels'] = array();
         $datasetData = array();
-        foreach($reportData as $row){
-            $data['labels'][] = $row[$xName] . $this->getChartDataNameLabel($row[$xName]);
+        foreach ($reportData as $row) {
+            $data['labels'][] = $row[$xName].$this->getChartDataNameLabel($row[$xName]);
             $datasetData[] = $row[$yName];
         }
 
@@ -742,7 +742,7 @@ 
     }
 
     private function getChartDataNameLabel($name) {
-        if(isset($GLOBALS['app_list_strings'])) {
+        if (isset($GLOBALS['app_list_strings'])) {
             $keys = array_keys($GLOBALS['app_list_strings']);
             foreach ($keys as $key) {
                 if (isset($GLOBALS['app_list_strings'][$key][$name])) {
@@ -753,14 +753,14 @@ 
         return '';
     }
 
-    private function getLineChartData($reportData, $xName,$yName){
-        return $this->getBarChartData($reportData, $xName,$yName);
+    private function getLineChartData($reportData, $xName, $yName) {
+        return $this->getBarChartData($reportData, $xName, $yName);
     }
 
-    private function getBarChartConfig(){
+    private function getBarChartConfig() {
         return array();
     }
-    private function getLineChartConfig(){
+    private function getLineChartConfig() {
         return $this->getBarChartConfig();
     }
 
@@ -774,42 +774,42 @@ 
         return $this->getBarChartConfig();
     }
 
-    private function getRoseChartConfig(){
+    private function getRoseChartConfig() {
         return $this->getBarChartConfig();
     }
 
-    private function getRadarChartData($reportData, $xName,$yName){
-        return $this->getBarChartData($reportData, $xName,$yName);
+    private function getRadarChartData($reportData, $xName, $yName) {
+        return $this->getBarChartData($reportData, $xName, $yName);
     }
 
-    private function getPolarChartData($reportData, $xName,$yName){
-        return $this->getPieChartData($reportData, $xName,$yName);
+    private function getPolarChartData($reportData, $xName, $yName) {
+        return $this->getPieChartData($reportData, $xName, $yName);
     }
 
-    private function getRadarChartConfig(){
+    private function getRadarChartConfig() {
         return array();
     }
 
-    private function getPolarChartConfig(){
+    private function getPolarChartConfig() {
         return $this->getPieChartConfig();
     }
-    private function getPieChartConfig(){
+    private function getPieChartConfig() {
         $config = array();
         $config['legendTemplate'] = "<ul class=\"<%=name.toLowerCase()%>-legend\"><% for (var i=0; i<segments.length; i++){%><li><span style=\"background-color:<%=segments[i].fillColor%>\">&nbsp;&nbsp;&nbsp;&nbsp;</span>&nbsp;<%if(segments[i].label){%><%=segments[i].label%><%}%></li><%}%></ul>";
         return $config;
     }
 
-    private function getPieChartData($reportData, $xName,$yName){
+    private function getPieChartData($reportData, $xName, $yName) {
         $data = array();
 
-        foreach($reportData as $row){
-            if(!$row[$yName]){
+        foreach ($reportData as $row) {
+            if (!$row[$yName]) {
                 continue;
             }
             $colour = $this->getColour($row[$xName]);
             $data[] = array(
                 'value' => (int)$row[$yName],
-                'label' => $row[$xName] . $this->getChartDataNameLabel($row[$xName]),
+                'label' => $row[$xName].$this->getChartDataNameLabel($row[$xName]),
                 'color' => $colour['main'],
                 'highlight' => $colour['highlight'],
             );

modules/AOS_Products_Quotes/AOS_Utils.php

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /**
  * Products, Quotations & Invoices modules.
  * Extensions to SugarCRM
@@ -25,17 +25,17 @@ 
  * @author Salesagility Ltd <info@salesagility.com>
  */
 
-function perform_aos_save($focus){
+function perform_aos_save($focus) {
     //US DOLLAR
 
-    foreach($focus->field_defs as $field){
+    foreach ($focus->field_defs as $field) {
 
-        if(isset($focus->field_defs[$field['name'].'_usdollar'])){
+        if (isset($focus->field_defs[$field['name'].'_usdollar'])) {
 
             $fieldName = $field['name'].'_usdollar';
 
             $focus->$fieldName = '';
-            if(!number_empty($focus->field_defs[$field['name']])){
+            if (!number_empty($focus->field_defs[$field['name']])) {
                 $currency = new Currency();
                 $currency->retrieve($focus->currency_id);
                 $focus->$fieldName = $currency->convertToDollar(unformat_number($focus->$field['name']));

modules/AOS_Products_Quotes/Line_Items.php

Spacing +36 added lines, -36 removed lines

@@ -22,7 +22,7 @@ 
  * @author SalesAgility <info@salesagility.com>
  */
 
-function display_lines($focus, $field, $value, $view){
+function display_lines($focus, $field, $value, $view) {
 
     global $sugar_config, $locale, $app_list_strings, $mod_strings;
 
@@ -31,10 +31,10 @@ 
 
     $html = '';
 
-    if($view == 'EditView'){
+    if ($view == 'EditView') {
 
         $html .= '<script src="modules/AOS_Products_Quotes/line_items.js"></script>';
-        if(file_exists('custom/modules/AOS_Products_Quotes/line_items.js')){
+        if (file_exists('custom/modules/AOS_Products_Quotes/line_items.js')) {
             $html .= '<script src="custom/modules/AOS_Products_Quotes/line_items.js"></script>';
         }
         $html .= '<script language="javascript">var sig_digits = '.$locale->getPrecision().';';
@@ -45,18 +45,18 @@ 
 
         $html .= "<table border='0' cellspacing='4' width='37.5%' id='lineItems'></table>";
 
-        if($enable_groups){
+        if ($enable_groups) {
             $html .= "<div style='padding-top: 10px; padding-bottom:10px;'>";
             $html .= "<input type=\"button\" tabindex=\"116\" class=\"button\" value=\"".$mod_strings['LBL_ADD_GROUP']."\" id=\"addGroup\" onclick=\"insertGroup(0)\" />";
             $html .= "</div>";
         }
         $html .= '<input type="hidden" name="vathidden" id="vathidden" value="'.get_select_options_with_id($app_list_strings['vat_list'], '').'">
 				  <input type="hidden" name="discounthidden" id="discounthidden" value="'.get_select_options_with_id($app_list_strings['discount_list'], '').'">';
-        if($focus->id != '') {
+        if ($focus->id != '') {
             require_once('modules/AOS_Products_Quotes/AOS_Products_Quotes.php');
             require_once('modules/AOS_Line_Item_Groups/AOS_Line_Item_Groups.php');
 
-            $sql = "SELECT pg.id, pg.group_id FROM aos_products_quotes pg LEFT JOIN aos_line_item_groups lig ON pg.group_id = lig.id WHERE pg.parent_type = '" . $focus->object_name . "' AND pg.parent_id = '" . $focus->id . "' AND pg.deleted = 0 ORDER BY lig.number ASC, pg.number ASC";
+            $sql = "SELECT pg.id, pg.group_id FROM aos_products_quotes pg LEFT JOIN aos_line_item_groups lig ON pg.group_id = lig.id WHERE pg.parent_type = '".$focus->object_name."' AND pg.parent_id = '".$focus->id."' AND pg.deleted = 0 ORDER BY lig.number ASC, pg.number ASC";
 
             $result = $focus->db->query($sql);
             $html .= "<script>
@@ -75,17 +75,17 @@ 
                     $group_item = json_encode($group_item->toArray());
                 }
                 $html .= "<script>
-                        insertLineItems(" . $line_item . "," . $group_item . ");
+                        insertLineItems(" . $line_item.",".$group_item.");
                     </script>";
 
             }
         }
-        if(!$enable_groups){
+        if (!$enable_groups) {
             $html .= '<script>insertGroup();</script>';
         }
 
     }
-    else if($view == 'DetailView'){
+    else if ($view == 'DetailView') {
         $params = array('currency_id' => $focus->currency_id);
 
         $sql = "SELECT pg.id, pg.group_id FROM aos_products_quotes pg LEFT JOIN aos_line_item_groups lig ON pg.group_id = lig.id WHERE pg.parent_type = '".$focus->object_name."' AND pg.parent_id = '".$focus->id."' AND pg.deleted = 0 ORDER BY lig.number ASC, pg.number ASC";
@@ -109,9 +109,9 @@ 
             $line_item->retrieve($row['id']);
 
 
-            if($enable_groups && ($group_id != $row['group_id'] || $i == 0)){
+            if ($enable_groups && ($group_id != $row['group_id'] || $i == 0)) {
                 $html .= $groupStart.$product.$service.$groupEnd;
-                if($i != 0)$html .= "<tr><td colspan='9' nowrap='nowrap'><br></td></tr>";
+                if ($i != 0)$html .= "<tr><td colspan='9' nowrap='nowrap'><br></td></tr>";
                 $groupStart = '';
                 $groupEnd = '';
                 $product = '';
@@ -152,8 +152,8 @@ 
                 $groupEnd .= "<td class='tabDetailViewDL' style='text-align: right;padding:2px;'>".currency_format_number($group_item->total_amount, $params)."</td>";
                 $groupEnd .= "</tr>";
             }
-            if($line_item->product_id != '0' && $line_item->product_id != null){
-                if($productCount == 0)
+            if ($line_item->product_id != '0' && $line_item->product_id != null) {
+                if ($productCount == 0)
                 {
                     $product .= "<tr>";
                     $product .= "<td width='5%' class='tabDetailViewDL' style='text-align: left;padding:2px;' scope='row'>&nbsp;</td>";
@@ -169,25 +169,25 @@ 
                 }
 
                 $product .= "<tr>";
-                $product_note = wordwrap($line_item->description,40,"<br />\n");
+                $product_note = wordwrap($line_item->description, 40, "<br />\n");
                 $product .= "<td class='tabDetailViewDF' style='text-align: left; padding:2px;'>".++$productCount."</td>";
-                $product .= "<td class='tabDetailViewDF' style='padding:2px;'>".rtrim(rtrim(format_number($line_item->product_qty), '0'),$sep[1])."</td>";
+                $product .= "<td class='tabDetailViewDF' style='padding:2px;'>".rtrim(rtrim(format_number($line_item->product_qty), '0'), $sep[1])."</td>";
                 $product .= "<td class='tabDetailViewDF' style='padding:2px;'><a href='index.php?module=AOS_Products&action=DetailView&record=".$line_item->product_id."' class='tabDetailViewDFLink'>".$line_item->name."</a><br />".$product_note."</td>";
-                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_list_price,$params)."</td>";
+                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_list_price, $params)."</td>";
 
                 $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".get_discount_string($line_item->discount, $line_item->product_discount, $params, $locale, $sep)."</td>";
 
-                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_unit_price,$params )."</td>";
-                if($locale->getPrecision()){
-                    $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".rtrim(rtrim(format_number($line_item->vat), '0'),$sep[1])."%</td>";
+                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_unit_price, $params)."</td>";
+                if ($locale->getPrecision()) {
+                    $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".rtrim(rtrim(format_number($line_item->vat), '0'), $sep[1])."%</td>";
                 } else {
                     $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".format_number($line_item->vat)."%</td>";
                 }
-                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->vat_amt,$params )."</td>";
-                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_total_price,$params )."</td>";
+                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->vat_amt, $params)."</td>";
+                $product .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_total_price, $params)."</td>";
                 $product .= "</tr>";
             } else {
-                if($serviceCount == 0)
+                if ($serviceCount == 0)
                 {
                     $service .= "<tr>";
                     $service .= "<td width='5%' class='tabDetailViewDL' style='text-align: left;padding:2px;' scope='row'>&nbsp;</td>";
@@ -204,15 +204,15 @@ 
                 $service .= "<tr>";
                 $service .= "<td class='tabDetailViewDF' style='text-align: left; padding:2px;'>".++$serviceCount."</td>";
                 $service .= "<td class='tabDetailViewDF' style='padding:2px;' colspan='2'>".$line_item->name."</td>";
-                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_list_price,$params)."</td>";
+                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_list_price, $params)."</td>";
 
                 $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".get_discount_string($line_item->discount, $line_item->product_discount, $params, $locale, $sep)."</td>";
 
 
-                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_unit_price,$params)."</td>";
+                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_unit_price, $params)."</td>";
                 $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".rtrim(rtrim(format_number($line_item->vat), '0'), $sep[1])."%</td>";
-                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->vat_amt,$params )."</td>";
-                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_total_price,$params )."</td>";
+                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->vat_amt, $params)."</td>";
+                $service .= "<td class='tabDetailViewDF' style='text-align: right; padding:2px;'>".currency_format_number($line_item->product_total_price, $params)."</td>";
                 $service .= "</tr>";
 
             }
@@ -223,17 +223,17 @@ 
     return $html;
 }
 
-function get_discount_string($type, $amount, $params, $locale, $sep){
-    if($amount != '' && $amount != '0.00')
+function get_discount_string($type, $amount, $params, $locale, $sep) {
+    if ($amount != '' && $amount != '0.00')
     {
-        if($type == 'Amount')
+        if ($type == 'Amount')
         {
-            return currency_format_number($amount,$params )."</td>";
+            return currency_format_number($amount, $params)."</td>";
         }
-        else if($locale->getPrecision())
+        else if ($locale->getPrecision())
         {
-            return rtrim(rtrim(format_number($amount), '0'),$sep[1])."%";
-        } else{
+            return rtrim(rtrim(format_number($amount), '0'), $sep[1])."%";
+        } else {
             return format_number($amount)."%";
         }
     }
@@ -243,12 +243,12 @@ 
     }
 }
 
-function display_shipping_vat($focus, $field, $value, $view){
+function display_shipping_vat($focus, $field, $value, $view) {
 
-    if($view == 'EditView'){
+    if ($view == 'EditView') {
         global $app_list_strings;
 
-        if($value != '') $value = format_number($value);
+        if ($value != '') $value = format_number($value);
 
         $html = "<input id='shipping_tax_amt' type='text' tabindex='0' title='' value='".$value."' maxlength='26,6' size='22' name='shipping_tax_amt' onblur='calculateTotal(\"lineItems\");'>";
         $html .= "<select name='shipping_tax' id='shipping_tax' onchange='calculateTotal(\"lineItems\");' >".get_select_options_with_id($app_list_strings['vat_list'], (isset($focus->shipping_tax) ? $focus->shipping_tax : ''))."</select>";

modules/AOS_Products_Quotes/vardefs.php

Spacing +36 added lines, -36 removed lines

@@ -25,9 +25,9 @@ 
 $dictionary['AOS_Products_Quotes'] = array(
 	'table'=>'aos_products_quotes',
 	'audited'=>true,
-	'fields'=>array (
+	'fields'=>array(
   'name' => 
-    array (
+    array(
       'name' => 'name',
       'vname' => 'LBL_NAME',
       'type' => 'text',
@@ -37,7 +37,7 @@ 
       'importable' => 'required',
     ),
     'currency_id' =>
-     array (
+     array(
       'required' => false,
       'name' => 'currency_id',
       'vname' => 'LBL_CURRENCY',
@@ -54,13 +54,13 @@ 
       'size' => '20',
       'studio' => 'visible',
       'function' =>
-        array (
+        array(
             'name' => 'getCurrencyDropDown',
             'returns' => 'html',
         ),
     ),
   'part_number' =>
-    array (
+    array(
         'required' => false,
         'name' => 'part_number',
         'vname' => 'LBL_PART_NUMBER',
@@ -82,7 +82,7 @@ 
         'id' => 'AOS_Products_Quotespart_number',
     ),
     'item_description' =>
-        array (
+        array(
             'name' => 'item_description',
             'vname' => 'LBL_PRODUCT_DESCRIPTION',
             'type' => 'text',
@@ -91,7 +91,7 @@ 
             'cols' => 80,
         ),
   'number' => 
-  array (
+  array(
     'required' => false,
     'name' => 'number',
     'vname' => 'LBL_LIST_NUM',
@@ -108,7 +108,7 @@ 
     'disable_num_format' => '',
   ),
   'product_qty' => 
-  array (
+  array(
     'required' => false,
     'name' => 'product_qty',
     'vname' => 'LBL_PRODUCT_QTY',
@@ -127,7 +127,7 @@ 
     'precision' => '4',
   ),
   'product_cost_price' => 
-  array (
+  array(
     'required' => false,
     'name' => 'product_cost_price',
     'vname' => 'LBL_PRODUCT_COST_PRICE',
@@ -143,7 +143,7 @@ 
     'len' => '26,6',
   ),
         'product_cost_price_usdollar' =>
-            array (
+            array(
                 'name' => 'product_cost_price_usdollar',
                 'vname' => 'LBL_PRODUCT_COST_PRICE_USDOLLAR',
                 'type' => 'currency',
@@ -160,7 +160,7 @@ 
                 'len' => '26,6',
             ),
   'product_list_price' => 
-  array (
+  array(
     'required' => false,
     'name' => 'product_list_price',
     'vname' => 'LBL_PRODUCT_LIST_PRICE',
@@ -176,7 +176,7 @@ 
     'len' => '26,6',
   ),
         'product_list_price_usdollar' =>
-            array (
+            array(
                 'name' => 'product_list_price_usdollar',
                 'vname' => 'LBL_PRODUCT_LIST_PRICE_USDOLLAR',
                 'type' => 'currency',
@@ -193,7 +193,7 @@ 
                 'len' => '26,6',
             ),
   'product_discount' => 
-  array (
+  array(
     'required' => false,
     'name' => 'product_discount',
     'vname' => 'LBL_PRODUCT_DISCOUNT',
@@ -209,7 +209,7 @@ 
     'len' => '26,6',
   ),
         'product_discount_usdollar' =>
-            array (
+            array(
                 'name' => 'product_discount_usdollar',
                 'vname' => 'LBL_PRODUCT_DISCOUNT_USDOLLAR',
                 'type' => 'currency',
@@ -226,7 +226,7 @@ 
                 'len' => '26,6',
             ),
   'product_discount_amount' => 
-  array (
+  array(
     'required' => false,
     'name' => 'product_discount_amount',
     'vname' => 'LBL_PRODUCT_DISCOUNT_AMOUNT',
@@ -242,7 +242,7 @@ 
     'len' => '26,6',
   ),
         'product_discount_amount_usdollar' =>
-            array (
+            array(
                 'name' => 'product_discount_amount_usdollar',
                 'vname' => 'LBL_PRODUCT_DISCOUNT_AMOUNT_USDOLLAR',
                 'type' => 'currency',
@@ -259,7 +259,7 @@ 
                 'len' => '26,6',
             ),
   'discount' => 
-  array (
+  array(
     'required' => false,
     'name' => 'discount',
     'vname' => 'LBL_DISCOUNT',
@@ -279,7 +279,7 @@ 
   ),
 
   'product_unit_price' => 
-  array (
+  array(
     'required' => '1',
     'name' => 'product_unit_price',
     'vname' => 'LBL_PRODUCT_UNIT_PRICE',
@@ -295,7 +295,7 @@ 
     'len' => '26,6',
   ),
         'product_unit_price_usdollar' =>
-            array (
+            array(
                 'name' => 'product_unit_price_usdollar',
                 'vname' => 'LBL_PRODUCT_UNIT_PRICE_USDOLLAR',
                 'type' => 'currency',
@@ -312,7 +312,7 @@ 
                 'len' => '26,6',
             ),
   'vat_amt' => 
-  array (
+  array(
     'required' => '1',
     'name' => 'vat_amt',
     'vname' => 'LBL_VAT_AMT',
@@ -328,7 +328,7 @@ 
     'len' => '26,6',
   ),
         'vat_amt_usdollar' =>
-            array (
+            array(
                 'name' => 'vat_amt_usdollar',
                 'vname' => 'LBL_VAT_AMT_USDOLLAR',
                 'type' => 'currency',
@@ -345,7 +345,7 @@ 
                 'len' => '26,6',
             ),
   'product_total_price' => 
-  array (
+  array(
     'required' => '1',
     'name' => 'product_total_price',
     'vname' => 'LBL_PRODUCT_TOTAL_PRICE',
@@ -361,7 +361,7 @@ 
     'len' => '26,6',
   ),
         'product_total_price_usdollar' =>
-            array (
+            array(
                 'name' => 'product_total_price_usdollar',
                 'vname' => 'LBL_PRODUCT_TOTAL_PRICE_USDOLLAR',
                 'type' => 'currency',
@@ -378,7 +378,7 @@ 
                 'len' => '26,6',
             ),
   'vat' => 
-  array (
+  array(
     'required' => false,
     'name' => 'vat',
     'vname' => 'LBL_VAT',
@@ -397,7 +397,7 @@ 
     'studio' => 'visible',
   ),
   'parent_name' => 
-  array (
+  array(
     'required' => false,
     'source' => 'non-db',
     'name' => 'parent_name',
@@ -419,7 +419,7 @@ 
     'parent_type' => 'record_type_display',
   ),
   'parent_type' => 
-  array (
+  array(
     'required' => false,
     'name' => 'parent_type',
     'vname' => 'LBL_PARENT_TYPE',
@@ -437,7 +437,7 @@ 
     'studio' => 'hidden',
   ),
   'parent_id' => 
-  array (
+  array(
     'required' => false,
     'name' => 'parent_id',
     'vname' => 'LBL_PARENT_ID',
@@ -453,7 +453,7 @@ 
     'len' => 36,
   ),
   'product_id' => 
-  array (
+  array(
     'required' => false,
     'name' => 'product_id',
     'vname' => '',
@@ -482,7 +482,7 @@ 
     'duplicate_merge'=>'disabled',
   ),
   'group_id' => 
-  array (
+  array(
     'required' => false,
     'name' => 'group_id',
     'vname' => '',
@@ -498,7 +498,7 @@ 
     'len' => 36,
   ),
    'aos_products' =>
-  array (
+  array(
   'name' => 'aos_products',
     'type' => 'link',
     'relationship' => 'aos_product_quotes_aos_products',
@@ -507,16 +507,16 @@ 
     'source'=>'non-db',
  ),
 ),
-'indices' => array (
-    array (
+'indices' => array(
+    array(
     	'name' => 'idx_aospq_par_del',
     	'type' => 'index',
-    	'fields' => array('parent_id','parent_type','deleted')
+    	'fields' => array('parent_id', 'parent_type', 'deleted')
     )
 ),
-'relationships'=>array (
+'relationships'=>array(
 	'aos_product_quotes_aos_products' =>
-	array (
+	array(
 	'lhs_module'=> 'AOS_Products', 
 	'lhs_table'=> 'aos_products', 
 	'lhs_key' => 'id',
@@ -529,4 +529,4 @@ 
 	'optimistic_lock'=>true,
 );
 require_once('include/SugarObjects/VardefManager.php');
-VardefManager::createVardef('AOS_Products_Quotes','AOS_Products_Quotes', array('basic','assignable'));
+VardefManager::createVardef('AOS_Products_Quotes', 'AOS_Products_Quotes', array('basic', 'assignable'));

modules/AOS_Products_Quotes/AOS_Products_Quotes.php

Spacing +10 added lines, -10 removed lines

@@ -25,29 +25,29 @@ 
 require_once('modules/AOS_Products_Quotes/AOS_Products_Quotes_sugar.php');
 class AOS_Products_Quotes extends AOS_Products_Quotes_sugar {
 
-    function AOS_Products_Quotes(){
+    function AOS_Products_Quotes() {
         parent::AOS_Products_Quotes_sugar();
     }
 
-    function save_lines($post_data, $parent, $groups = array(), $key = ''){
+    function save_lines($post_data, $parent, $groups = array(), $key = '') {
 
         $line_count = isset($post_data[$key.'name']) ? count($post_data[$key.'name']) : 0;
         $j = 0;
         for ($i = 0; $i < $line_count; ++$i) {
 
-            if($post_data[$key.'deleted'][$i] == 1){
+            if ($post_data[$key.'deleted'][$i] == 1) {
                 $this->mark_deleted($post_data[$key.'id'][$i]);
             } else {
                 $product_quote = new AOS_Products_Quotes();
-                foreach($this->field_defs as $field_def) {
-                    if(isset($post_data[$key.$field_def['name']][$i])){
+                foreach ($this->field_defs as $field_def) {
+                    if (isset($post_data[$key.$field_def['name']][$i])) {
                         $product_quote->$field_def['name'] = $post_data[$key.$field_def['name']][$i];
                     }
                 }
-                if(isset($post_data[$key.'group_number'][$i])){
+                if (isset($post_data[$key.'group_number'][$i])) {
                     $product_quote->group_id = $groups[$post_data[$key.'group_number'][$i]];
                 }
-                if(trim($product_quote->product_id) != '' && trim($product_quote->name) != '' && trim($product_quote->product_unit_price) != ''){
+                if (trim($product_quote->product_id) != '' && trim($product_quote->name) != '' && trim($product_quote->product_unit_price) != '') {
                     $product_quote->number = ++$j;
                     $product_quote->assigned_user_id = $parent->assigned_user_id;
                     $product_quote->parent_id = $parent->id;
@@ -60,19 +60,19 @@ 
         }
     }
 
-    function mark_lines_deleted($parent){
+    function mark_lines_deleted($parent) {
 
         require_once('modules/Relationships/Relationship.php');
         //$key = Relationship::retrieve_by_modules($parent->object_name, $this->object_name, $this->db);
         //if (!empty($key)) {
         $product_quotes = $parent->get_linked_beans('aos_products_quotes', $this->object_name);
-        foreach($product_quotes as $product_quote){
+        foreach ($product_quotes as $product_quote) {
             $product_quote->mark_deleted($product_quote->id);
         }
         //}
     }
 
-    function save($check_notify = FALSE){
+    function save($check_notify = FALSE) {
         require_once('modules/AOS_Products_Quotes/AOS_Utils.php');
         perform_aos_save($this);
         parent::save($check_notify);

modules/AOS_Products_Quotes/AOS_Products_Quotes_sugar.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
 	var $tracker_visibility = false;
 	var $importable = true;
 
-	var $disable_row_level_security = true ; // to ensure that modules created and deployed under CE will continue to function under team security if the instance is upgraded to PRO
+	var $disable_row_level_security = true; // to ensure that modules created and deployed under CE will continue to function under team security if the instance is upgraded to PRO
 
 		var $id;
 		var $name;
@@ -68,12 +68,12 @@ 
 
 
 
-	function AOS_Products_Quotes_sugar(){	
+	function AOS_Products_Quotes_sugar() {	
 		parent::Basic();
 	}
 	
-	function bean_implements($interface){
-		switch($interface){
+	function bean_implements($interface) {
+		switch ($interface) {
 			case 'ACL': return false;
 		}
 		return false;

modules/Home/About.php

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -52,12 +52,12 @@ 
 <br>
 <b>
     <?php echo $mod_strings['LBL_VERSION']." ".$suitecrm_version;
-    if( is_file( "custom_version.php" ) ){
-        include( "custom_version.php" );
-        print( "&nbsp;&nbsp;&nbsp;" . $custom_version );
+    if (is_file("custom_version.php")) {
+        include("custom_version.php");
+        print("&nbsp;&nbsp;&nbsp;".$custom_version);
     }?>
 </b>
-<p>Sugar <?php echo $mod_strings['LBL_VERSION']." ".$sugar_version." (".$mod_strings['LBL_BUILD']." ".$sugar_build.")";?></p>
+<p>Sugar <?php echo $mod_strings['LBL_VERSION']." ".$sugar_version." (".$mod_strings['LBL_BUILD']." ".$sugar_build.")"; ?></p>
 <br>
 <h1><?php echo $mod_strings['LBL_CONTRIBUTOR_SUITECRM']; ?></h1>
 <table id="about_table" class="contentBox">