Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/AOD_Index/Lib/Zend/Search/Lucene/Search/Similarity.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -407,8 +407,8 @@  discard block
 block discarded – undo
407 407
     /**
408 408
      * Float to byte conversion
409 409
      *
410
-     * @param integer $b
411
-     * @return float
410
+     * @param double $f
411
+     * @return integer
412 412
      */
413 413
     private static function _floatToByte($f)
414 414
     {
@@ -494,7 +494,7 @@  discard block
 block discarded – undo
494 494
      *
495 495
      * @param mixed $input
496 496
      * @param Zend_Search_Lucene_Interface $reader
497
-     * @return a score factor for the term
497
+     * @return double score factor for the term
498 498
      */
499 499
     public function idf($input, Zend_Search_Lucene_Interface $reader)
500 500
     {
Please login to merge, or discard this patch.
modules/AOD_Index/Lib/Zend/Search/Lucene/Storage/Directory/Filesystem.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -322,7 +322,7 @@
 block discarded – undo
322 322
      * Sets the modified time of $filename to now.
323 323
      *
324 324
      * @param string $filename
325
-     * @return void
325
+     * @return boolean
326 326
      */
327 327
     public function touchFile($filename)
328 328
     {
Please login to merge, or discard this patch.
modules/AOD_Index/PdfParser.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
   /**
42 42
    * Convert a PDF into text.
43 43
    *
44
-   * @param string $filename The filename to extract the data from.
44
+   * @param string $data
45 45
    * @return string The extracted text from the PDF
46 46
    */
47 47
   protected static function extractText($data)
Please login to merge, or discard this patch.
modules/AOP_Case_Events/CaseEventsHook.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -31,6 +31,9 @@
 block discarded – undo
31 31
         array('field'=> 'type', 'display_field' => 'type', 'display_name' => 'Type'),
32 32
     );
33 33
 
34
+    /**
35
+     * @param aCase $old
36
+     */
34 37
     private function compareBeans($old, $new){
35 38
         $events = array();
36 39
         foreach($this->diff_fields as $field){
Please login to merge, or discard this patch.
modules/AOP_Case_Updates/CaseUpdatesHook.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -443,6 +443,9 @@
 block discarded – undo
443 443
         }
444 444
     }
445 445
 
446
+    /**
447
+     * @param SugarPHPMailer $mailer
448
+     */
446 449
     private function logEmail($email, $mailer, $caseId = null){
447 450
         require_once('modules/Emails/Email.php');
448 451
         $emailObj = new Email();
Please login to merge, or discard this patch.
modules/AOP_Case_Updates/util.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -93,6 +93,7 @@
 block discarded – undo
93 93
 
94 94
 /**
95 95
  * Custom parse template method since sugars own doesn't deal with custom fields.
96
+ * @param string $string
96 97
  */
97 98
 function aop_parse_template($string, &$bean_arr) {
98 99
     global $beanFiles, $beanList;
Please login to merge, or discard this patch.
modules/AOR_Charts/AOR_Chart.php 1 patch
Doc Comments   +73 added lines patch added patch discarded remove patch
@@ -56,6 +56,9 @@  discard block
 block discarded – undo
56 56
 		parent::Basic();
57 57
 	}
58 58
 
59
+    /**
60
+     * @param string $postKey
61
+     */
59 62
     function save_lines(array $post,AOR_Report $bean,$postKey){
60 63
         $seenIds = array();
61 64
         if(isset($post[$postKey.'id'])) {
@@ -105,12 +108,23 @@  discard block
 block discarded – undo
105 108
         return array('main'=>$main,'highlight'=>$highlight);
106 109
     }
107 110
 
111
+    /**
112
+     * @param pImage $chartPicture
113
+     */
108 114
     function buildChartImageBar($chartPicture,$recordImageMap = false){
109 115
         $scaleSettings = array("DrawSubTicks" => false, "LabelRotation" => 30, 'MinDivHeight' => 50);
110 116
         $chartPicture->drawScale($scaleSettings);
111 117
         $chartPicture->drawBarChart(array("RecordImageMap"=>$recordImageMap));
112 118
     }
113 119
 
120
+    /**
121
+     * @param pImage $chartPicture
122
+     * @param pData $chartData
123
+     * @param integer $imageHeight
124
+     * @param integer $imageWidth
125
+     * @param string $xName
126
+     * @param boolean $recordImageMap
127
+     */
114 128
     function buildChartImagePie($chartPicture,$chartData, $reportData,$imageHeight, $imageWidth, $xName,$recordImageMap){
115 129
         $PieChart = new pPie($chartPicture,$chartData);
116 130
         $x = 0;
@@ -122,12 +136,20 @@  discard block
 block discarded – undo
122 136
         $PieChart->drawPieLegend($imageWidth*0.7,$imageHeight/3, array('FontSize'=>10,"FontName"=>"modules/AOR_Charts/lib/pChart/fonts/verdana.ttf",'BoxSize'=>14));
123 137
     }
124 138
 
139
+    /**
140
+     * @param pImage $chartPicture
141
+     */
125 142
     function buildChartImageLine($chartPicture, $recordImageMap = false){
126 143
         $scaleSettings = array("XMargin"=>10,"YMargin"=>10,"GridR"=>200,"GridG"=>200,"GridB"=>200,'MinDivHeight' => 50,"LabelRotation" => 30);
127 144
         $chartPicture->drawScale($scaleSettings);
128 145
         $chartPicture->drawLineChart(array("RecordImageMap"=>$recordImageMap));
129 146
     }
130 147
 
148
+    /**
149
+     * @param pImage $chartPicture
150
+     * @param pData $chartData
151
+     * @param boolean $recordImageMap
152
+     */
131 153
     function buildChartImageRadar($chartPicture, $chartData,$recordImageMap){
132 154
         $SplitChart = new pRadar();
133 155
         $Options = array("LabelPos"=>RADAR_LABELS_HORIZONTAL,"RecordImageMap"=>$recordImageMap);
@@ -303,6 +325,11 @@  discard block
 block discarded – undo
303 325
         return $chart;
304 326
     }
305 327
 
328
+    /**
329
+     * @param string $chartDataValues
330
+     * @param string $chartLabelValues
331
+     * @param string $chartTooltips
332
+     */
306 333
     private function getRGraphRoseChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
307 334
     {
308 335
         $dataArray = json_decode($chartDataValues);
@@ -340,6 +367,12 @@  discard block
 block discarded – undo
340 367
     //I have not used a parameter for getRGraphBarChart to say whether to group etc, as the future development could be quite different
341 368
     //for both, hence the separate methods.  However, the $grouped parameter allows us to specify whether the chart is grouped (true)
342 369
     //or stacked (false)
370
+
371
+    /**
372
+     * @param string $chartDataValues
373
+     * @param string $chartLabelValues
374
+     * @param string $chartTooltips
375
+     */
343 376
     private function getRGraphGroupedBarChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400, $grouped = false)
344 377
     {
345 378
         //$keys = array_keys($chartTooltips);
@@ -398,6 +431,11 @@  discard block
 block discarded – undo
398 431
 
399 432
 
400 433
 
434
+    /**
435
+     * @param string $chartDataValues
436
+     * @param string $chartLabelValues
437
+     * @param string $chartTooltips
438
+     */
401 439
     private function getRGraphBarChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
402 440
     {
403 441
         $dataArray = json_decode($chartDataValues);
@@ -437,6 +475,11 @@  discard block
 block discarded – undo
437 475
         return $html;
438 476
     }
439 477
 
478
+    /**
479
+     * @param string $chartDataValues
480
+     * @param string $chartLabelValues
481
+     * @param string $chartTooltips
482
+     */
440 483
     private function getRGraphRadarChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
441 484
     {
442 485
         $dataArray = json_decode($chartDataValues);
@@ -469,6 +512,11 @@  discard block
 block discarded – undo
469 512
         return $html;
470 513
     }
471 514
 
515
+    /**
516
+     * @param string $chartDataValues
517
+     * @param string $chartLabelValues
518
+     * @param string $chartTooltips
519
+     */
472 520
     private function getRGraphPieChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
473 521
     {
474 522
         $dataArray = json_decode($chartDataValues);
@@ -505,6 +553,11 @@  discard block
 block discarded – undo
505 553
         return $html;
506 554
     }
507 555
 
556
+    /**
557
+     * @param string $chartDataValues
558
+     * @param string $chartLabelValues
559
+     * @param string $chartTooltips
560
+     */
508 561
     private function getRGraphLineChart($chartDataValues, $chartLabelValues,$chartTooltips, $chartName= '', $chartId, $chartHeight = 400, $chartWidth = 400)
509 562
     {
510 563
         $dataArray = json_decode($chartDataValues);
@@ -637,6 +690,10 @@  discard block
 block discarded – undo
637 690
     }
638 691
 
639 692
 
693
+    /**
694
+     * @param string $xName
695
+     * @param string $yName
696
+     */
640 697
     private function getRGraphGroupedBarChartData($reportData, $xName,$yName, AOR_Field $mainGroupField){
641 698
 
642 699
 
@@ -706,6 +763,10 @@  discard block
 block discarded – undo
706 763
 
707 764
     }
708 765
 
766
+    /**
767
+     * @param string $xName
768
+     * @param string $yName
769
+     */
709 770
     private function getRGraphBarChartData($reportData, $xName,$yName){
710 771
         $chart['labels']=array();
711 772
         $chart['data']=array();
@@ -753,6 +814,10 @@  discard block
 block discarded – undo
753 814
         return '';
754 815
     }
755 816
 
817
+    /**
818
+     * @param string $xName
819
+     * @param string $yName
820
+     */
756 821
     private function getLineChartData($reportData, $xName,$yName){
757 822
         return $this->getBarChartData($reportData, $xName,$yName);
758 823
     }
@@ -778,10 +843,18 @@  discard block
 block discarded – undo
778 843
         return $this->getBarChartConfig();
779 844
     }
780 845
 
846
+    /**
847
+     * @param string $xName
848
+     * @param string $yName
849
+     */
781 850
     private function getRadarChartData($reportData, $xName,$yName){
782 851
         return $this->getBarChartData($reportData, $xName,$yName);
783 852
     }
784 853
 
854
+    /**
855
+     * @param string $xName
856
+     * @param string $yName
857
+     */
785 858
     private function getPolarChartData($reportData, $xName,$yName){
786 859
         return $this->getPieChartData($reportData, $xName,$yName);
787 860
     }
Please login to merge, or discard this patch.
modules/AOR_Charts/lib/pChart/class/pBarcode128.class.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -179,6 +179,10 @@
 block discarded – undo
179 179
 
180 180
    function left($value,$NbChar) { return substr($value,0,$NbChar); }  
181 181
    function right($value,$NbChar) { return substr($value,strlen($value)-$NbChar,$NbChar); }  
182
+
183
+   /**
184
+    * @param integer $NbChar
185
+    */
182 186
    function mid($value,$Depart,$NbChar) { return substr($value,$Depart-1,$NbChar); }  
183 187
   }
184 188
 ?>
185 189
\ No newline at end of file
Please login to merge, or discard this patch.
modules/AOR_Charts/lib/pChart/class/pBarcode39.class.php 1 patch
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -181,6 +181,9 @@  discard block
 block discarded – undo
181 181
       }
182 182
     }
183 183
 
184
+   /**
185
+    * @param string $string
186
+    */
184 187
    function checksum( $string )
185 188
     {
186 189
      $checksum = 0;
@@ -195,6 +198,11 @@  discard block
 block discarded – undo
195 198
 
196 199
    function left($value,$NbChar) { return substr($value,0,$NbChar); }  
197 200
    function right($value,$NbChar) { return substr($value,strlen($value)-$NbChar,$NbChar); }  
201
+
202
+   /**
203
+    * @param integer $Depart
204
+    * @param integer $NbChar
205
+    */
198 206
    function mid($value,$Depart,$NbChar) { return substr($value,$Depart-1,$NbChar); }  
199 207
   }
200 208
 ?>
201 209
\ No newline at end of file
Please login to merge, or discard this patch.