Completed
Push — master ( bb1ebf...0d7d1b )
by Michael
02:07
created
admin/faq.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
 $startfaq = isset($_GET['startfaq']) ? (int)$_GET['startfaq'] : 0;
30 30
 
31 31
 /**
32
- * @param bool $showmenu
32
+ * @param string $showmenu
33 33
  * @param int  $faqid
34 34
  * @param int  $answerid
35 35
  * @param bool $merge
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -549,11 +549,11 @@
 block discarded – undo
549 549
             $faqid = isset($_GET['faqid']) ? (int)$_GET['faqid'] : 0;
550 550
             xoops_cp_header();
551 551
             xoops_confirm(array(
552
-                              'op'      => 'del',
553
-                              'faqid'   => $faqObj->faqid(),
554
-                              'confirm' => 1,
555
-                              'name'    => $faqObj->question()
556
-                          ), 'faq.php', _AM_SF_DELETETHISARTICLE . " <br>'" . $faqObj->question() . "'. <br> <br>", _AM_SF_DELETE);
552
+                                'op'      => 'del',
553
+                                'faqid'   => $faqObj->faqid(),
554
+                                'confirm' => 1,
555
+                                'name'    => $faqObj->question()
556
+                            ), 'faq.php', _AM_SF_DELETETHISARTICLE . " <br>'" . $faqObj->question() . "'. <br> <br>", _AM_SF_DELETE);
557 557
             xoops_cp_footer();
558 558
         }
559 559
 
Please login to merge, or discard this patch.
admin/mygrouppermform.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -120,9 +120,9 @@  discard block
 block discarded – undo
120 120
      * Add appendix
121 121
      *
122 122
      * @access public
123
-     * @param $permName
123
+     * @param string $permName
124 124
      * @param $itemId
125
-     * @param $itemName
125
+     * @param string $itemName
126 126
      */
127 127
     public function addAppendix($permName, $itemId, $itemName)
128 128
     {
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
     /**
257 257
      * Constructor
258 258
      * @param      $caption
259
-     * @param      $name
259
+     * @param      string $name
260 260
      * @param      $groupId
261 261
      * @param null $values
262 262
      */
Please login to merge, or discard this patch.
Indentation   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -379,20 +379,20 @@
 block discarded – undo
379 379
             $tree .= ' checked';
380 380
         }
381 381
         $tree .= '>'
382
-                 . $option['name']
383
-                 . '<input type="hidden" name="'
384
-                 . $this->getName()
385
-                 . '[parents]['
386
-                 . $option['id']
387
-                 . ']" value="'
388
-                 . implode(':', $parentIds)
389
-                 . '"><input type="hidden" name="'
390
-                 . $this->getName()
391
-                 . '[itemname]['
392
-                 . $option['id']
393
-                 . ']" value="'
394
-                 . htmlspecialchars($option['name'])
395
-                 . "\"><br>\n";
382
+                    . $option['name']
383
+                    . '<input type="hidden" name="'
384
+                    . $this->getName()
385
+                    . '[parents]['
386
+                    . $option['id']
387
+                    . ']" value="'
388
+                    . implode(':', $parentIds)
389
+                    . '"><input type="hidden" name="'
390
+                    . $this->getName()
391
+                    . '[itemname]['
392
+                    . $option['id']
393
+                    . ']" value="'
394
+                    . htmlspecialchars($option['name'])
395
+                    . "\"><br>\n";
396 396
         if (isset($option['children'])) {
397 397
             foreach ($option['children'] as $child) {
398 398
                 array_push($parentIds, $option['id']);
Please login to merge, or discard this patch.
admin/question.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
 $startfaq = isset($_GET['startfaq']) ? (int)$_GET['startfaq'] : 0;
27 27
 
28 28
 /**
29
- * @param bool $showmenu
29
+ * @param string $showmenu
30 30
  * @param int  $faqid
31 31
  */
32 32
 function editfaq($showmenu = false, $faqid = -1)
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -284,11 +284,11 @@
 block discarded – undo
284 284
             $faqid = isset($_GET['faqid']) ? (int)$_GET['faqid'] : 0;
285 285
             xoops_cp_header();
286 286
             xoops_confirm(array(
287
-                              'op'      => 'del',
288
-                              'faqid'   => $faqObj->faqid(),
289
-                              'confirm' => 1,
290
-                              'name'    => $faqObj->question()
291
-                          ), 'question.php', _AM_SF_DELETETHISQUESTION . " <br>'" . $faqObj->question() . "'. <br> <br>", _AM_SF_DELETE);
287
+                                'op'      => 'del',
288
+                                'faqid'   => $faqObj->faqid(),
289
+                                'confirm' => 1,
290
+                                'name'    => $faqObj->question()
291
+                            ), 'question.php', _AM_SF_DELETETHISQUESTION . " <br>'" . $faqObj->question() . "'. <br> <br>", _AM_SF_DELETE);
292 292
             xoops_cp_footer();
293 293
         }
294 294
 
Please login to merge, or discard this patch.
class/answer.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
     /**
29 29
      * constructor
30
-     * @param null $id
30
+     * @param integer $id
31 31
      */
32 32
     public function __construct($id = null)
33 33
     {
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
      * create a new answer
369 369
      *
370 370
      * @param  bool $isNew flag the new objects as "new"?
371
-     * @return object sfAnswer
371
+     * @return sfAnswer sfAnswer
372 372
      */
373 373
     public function create($isNew = true)
374 374
     {
@@ -410,7 +410,7 @@  discard block
 block discarded – undo
410 410
     /**
411 411
      * insert a new answer in the database
412 412
      *
413
-     * @param  XoopsObject $answerObj reference to the <a href='psi_element://sfAnswer'>sfAnswer</a> object
413
+     * @param  sfAnswer $answerObj reference to the <a href='psi_element://sfAnswer'>sfAnswer</a> object
414 414
      * @param  bool        $force
415 415
      * @return bool        FALSE if failed, TRUE if already present and unchanged or successful
416 416
      */
Please login to merge, or discard this patch.
class/smartdbupdater.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
     }
168 168
 
169 169
     /**
170
-     * @param $field
170
+     * @param string $field
171 171
      * @return bool
172 172
      */
173 173
     public function fieldExists($field)
@@ -628,7 +628,7 @@  discard block
 block discarded – undo
628 628
     /**
629 629
      * Use to update a table
630 630
      *
631
-     * @param object $table {@link SmartDbTable} that will be updated
631
+     * @param SmartDbTable $table {@link SmartDbTable} that will be updated
632 632
      *
633 633
      * @see SmartDbTable
634 634
      *
Please login to merge, or discard this patch.
class/uploader.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
      * Constructor
26 26
      *
27 27
      * @param string    $uploadDir
28
-     * @param array|int $allowedMimeTypes
28
+     * @param integer $allowedMimeTypes
29 29
      * @param int       $maxFileSize
30 30
      * @param int       $maxWidth
31 31
      * @param int       $maxHeight
Please login to merge, or discard this patch.
include/functions.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -509,7 +509,7 @@  discard block
 block discarded – undo
509 509
 
510 510
 /**
511 511
  * @param  string $url
512
- * @return mixed|string
512
+ * @return string
513 513
  */
514 514
 function sf_getxoopslink($url = '')
515 515
 {
@@ -582,7 +582,7 @@  discard block
 block discarded – undo
582 582
 }
583 583
 
584 584
 /**
585
- * @param       $name
585
+ * @param       string $name
586 586
  * @param  bool $optional
587 587
  * @return bool
588 588
  */
Please login to merge, or discard this patch.
class/faq.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
     }
244 244
 
245 245
     /**
246
-     * @return mixed
246
+     * @return integer
247 247
      */
248 248
     public function uid()
249 249
     {
@@ -754,7 +754,7 @@  discard block
 block discarded – undo
754 754
     /**
755 755
      * insert a new faq in the database
756 756
      *
757
-     * @param  XoopsObject $faq reference to the {@link sfFaq} object
757
+     * @param  sfFaq $faq reference to the {@link sfFaq} object
758 758
      * @param  bool        $force
759 759
      * @return bool        FALSE if failed, TRUE if already present and unchanged or successful
760 760
      */
@@ -1143,7 +1143,7 @@  discard block
 block discarded – undo
1143 1143
      * @param  string $order
1144 1144
      * @param  string $notNullFields
1145 1145
      * @param  bool   $asobject
1146
-     * @param  null   $otherCriteria
1146
+     * @param  null|CriteriaCompo   $otherCriteria
1147 1147
      * @return array
1148 1148
      */
1149 1149
     public function getFaqs(
Please login to merge, or discard this patch.
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -768,12 +768,12 @@
 block discarded – undo
768 768
 
769 769
         if ($faq->isNew()) {
770 770
             $sql = sprintf('INSERT INTO "%s" (faqid, categoryid, question, howdoi, diduno, uid, datesub, `status`, counter, weight, html, smiley, xcodes, cancomment, comments, notifypub, modulelink, contextpage, exacturl, partialview) VALUES (NULL, %u, %s, %s, %s, %u, %u, %u, %u, %u, %u, %u, %u, %u, %u, %u, %s, %s, %u, %u)',
771
-                           $this->db->prefix('smartfaq_faq'), $categoryid, $this->db->quoteString($question), $this->db->quoteString($howdoi), $this->db->quoteString($diduno), $uid, time(), $status, $counter, $weight, $html, $smiley, $xcodes, $cancomment, $comments, $notifypub,
772
-                           $this->db->quoteString($modulelink), $this->db->quoteString($contextpage), $exacturl, $partialview);
771
+                            $this->db->prefix('smartfaq_faq'), $categoryid, $this->db->quoteString($question), $this->db->quoteString($howdoi), $this->db->quoteString($diduno), $uid, time(), $status, $counter, $weight, $html, $smiley, $xcodes, $cancomment, $comments, $notifypub,
772
+                            $this->db->quoteString($modulelink), $this->db->quoteString($contextpage), $exacturl, $partialview);
773 773
         } else {
774 774
             $sql = sprintf('UPDATE "%s" SET categoryid = "%u", question = "%s", howdoi = "%s", diduno = "%s", uid = "%u", datesub = "%u", `status` = "%u", counter = "%u", weight = "%u", html = "%u", smiley = "%u", xcodes = "%u", cancomment = "%u", comments = "%u", notifypub = "%u", modulelink = "%s", contextpage = "%s", exacturl = "%u", partialview = "%u"  WHERE faqid = "%u"',
775
-                           $this->db->prefix('smartfaq_faq'), $categoryid, $this->db->quoteString($question), $this->db->quoteString($howdoi), $this->db->quoteString($diduno), $uid, $datesub, $status, $counter, $weight, $html, $smiley, $xcodes, $cancomment, $comments, $notifypub,
776
-                           $this->db->quoteString($modulelink), $this->db->quoteString($contextpage), $exacturl, $partialview, $faqid);
775
+                            $this->db->prefix('smartfaq_faq'), $categoryid, $this->db->quoteString($question), $this->db->quoteString($howdoi), $this->db->quoteString($diduno), $uid, $datesub, $status, $counter, $weight, $html, $smiley, $xcodes, $cancomment, $comments, $notifypub,
776
+                            $this->db->quoteString($modulelink), $this->db->quoteString($contextpage), $exacturl, $partialview, $faqid);
777 777
         }
778 778
         if (false !== $force) {
779 779
             $result = $this->db->queryF($sql);
Please login to merge, or discard this patch.
open_index.php 1 patch
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -116,13 +116,13 @@
 block discarded – undo
116 116
 // Language constants
117 117
 $moduleName =& $myts->displayTarea($xoopsModule->getVar('name'));
118 118
 $xoopsTpl->assign(array(
119
-                      'lang_on'       => _MD_SF_ON,
120
-                      'lang_postedby' => _MD_SF_POSTEDBY,
121
-                      'lang_total'    => $totalQnasOnPage,
122
-                      'lang_faq'      => _MD_SF_FAQ,
123
-                      'lang_datesub'  => _MD_SF_DATESUB,
124
-                      'lang_hits'     => _MD_SF_HITS
125
-                  ));
119
+                        'lang_on'       => _MD_SF_ON,
120
+                        'lang_postedby' => _MD_SF_POSTEDBY,
121
+                        'lang_total'    => $totalQnasOnPage,
122
+                        'lang_faq'      => _MD_SF_FAQ,
123
+                        'lang_datesub'  => _MD_SF_DATESUB,
124
+                        'lang_hits'     => _MD_SF_HITS
125
+                    ));
126 126
 
127 127
 $moduleName =& $myts->displayTarea($xoopsModule->getVar('name'));
128 128
 $xoopsTpl->assign('lang_mainhead', sprintf(_MD_SF_OPEN_WELCOME, $xoopsConfig['sitename']));
Please login to merge, or discard this patch.