@@ -29,7 +29,7 @@ |
||
| 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 |
@@ -548,11 +548,11 @@ |
||
| 548 | 548 | $faqid = isset($_GET['faqid']) ? (int)$_GET['faqid'] : 0; |
| 549 | 549 | xoops_cp_header(); |
| 550 | 550 | xoops_confirm([ |
| 551 | - 'op' => 'del', |
|
| 552 | - 'faqid' => $faqObj->faqid(), |
|
| 553 | - 'confirm' => 1, |
|
| 554 | - 'name' => $faqObj->question() |
|
| 555 | - ], 'faq.php', _AM_SF_DELETETHISARTICLE . " <br>'" . $faqObj->question() . "'. <br> <br>", _AM_SF_DELETE); |
|
| 551 | + 'op' => 'del', |
|
| 552 | + 'faqid' => $faqObj->faqid(), |
|
| 553 | + 'confirm' => 1, |
|
| 554 | + 'name' => $faqObj->question() |
|
| 555 | + ], 'faq.php', _AM_SF_DELETETHISARTICLE . " <br>'" . $faqObj->question() . "'. <br> <br>", _AM_SF_DELETE); |
|
| 556 | 556 | xoops_cp_footer(); |
| 557 | 557 | } |
| 558 | 558 | |
@@ -120,9 +120,9 @@ discard block |
||
| 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 |
||
| 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 | */ |
@@ -379,20 +379,20 @@ |
||
| 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']); |
@@ -26,7 +26,7 @@ |
||
| 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) |
@@ -284,11 +284,11 @@ |
||
| 284 | 284 | $faqid = isset($_GET['faqid']) ? (int)$_GET['faqid'] : 0; |
| 285 | 285 | xoops_cp_header(); |
| 286 | 286 | xoops_confirm([ |
| 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 | |
@@ -27,7 +27,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 | */ |
@@ -167,7 +167,7 @@ discard block |
||
| 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 |
||
| 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 | * |
@@ -25,7 +25,7 @@ |
||
| 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 |
@@ -509,7 +509,7 @@ discard block |
||
| 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 |
||
| 582 | 582 | } |
| 583 | 583 | |
| 584 | 584 | /** |
| 585 | - * @param $name |
|
| 585 | + * @param string $name |
|
| 586 | 586 | * @param bool $optional |
| 587 | 587 | * @return bool |
| 588 | 588 | */ |
@@ -243,7 +243,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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( |
@@ -769,7 +769,7 @@ discard block |
||
| 769 | 769 | if ($faq->isNew()) { |
| 770 | 770 | $sql = sprintf( |
| 771 | 771 | '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)', |
| 772 | - $this->db->prefix('smartfaq_faq'), |
|
| 772 | + $this->db->prefix('smartfaq_faq'), |
|
| 773 | 773 | $categoryid, |
| 774 | 774 | $this->db->quoteString($question), |
| 775 | 775 | $this->db->quoteString($howdoi), |
@@ -785,7 +785,7 @@ discard block |
||
| 785 | 785 | $cancomment, |
| 786 | 786 | $comments, |
| 787 | 787 | $notifypub, |
| 788 | - $this->db->quoteString($modulelink), |
|
| 788 | + $this->db->quoteString($modulelink), |
|
| 789 | 789 | $this->db->quoteString($contextpage), |
| 790 | 790 | $exacturl, |
| 791 | 791 | $partialview |
@@ -793,7 +793,7 @@ discard block |
||
| 793 | 793 | } else { |
| 794 | 794 | $sql = sprintf( |
| 795 | 795 | '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"', |
| 796 | - $this->db->prefix('smartfaq_faq'), |
|
| 796 | + $this->db->prefix('smartfaq_faq'), |
|
| 797 | 797 | $categoryid, |
| 798 | 798 | $this->db->quoteString($question), |
| 799 | 799 | $this->db->quoteString($howdoi), |
@@ -809,7 +809,7 @@ discard block |
||
| 809 | 809 | $cancomment, |
| 810 | 810 | $comments, |
| 811 | 811 | $notifypub, |
| 812 | - $this->db->quoteString($modulelink), |
|
| 812 | + $this->db->quoteString($modulelink), |
|
| 813 | 813 | $this->db->quoteString($contextpage), |
| 814 | 814 | $exacturl, |
| 815 | 815 | $partialview, |
@@ -270,7 +270,7 @@ discard block |
||
| 270 | 270 | * create a new category |
| 271 | 271 | * |
| 272 | 272 | * @param bool $isNew flag the new objects as "new"? |
| 273 | - * @return object sfCategory |
|
| 273 | + * @return sfCategory sfCategory |
|
| 274 | 274 | */ |
| 275 | 275 | public function create($isNew = true) |
| 276 | 276 | { |
@@ -312,7 +312,7 @@ discard block |
||
| 312 | 312 | /** |
| 313 | 313 | * insert a new category in the database |
| 314 | 314 | * |
| 315 | - * @param XoopsObject $category reference to the {@link sfCategory} object |
|
| 315 | + * @param sfCategory $category reference to the {@link sfCategory} object |
|
| 316 | 316 | * @param bool $force |
| 317 | 317 | * @return bool FALSE if failed, TRUE if already present and unchanged or successful |
| 318 | 318 | */ |
@@ -336,7 +336,7 @@ discard block |
||
| 336 | 336 | $sql = sprintf( |
| 337 | 337 | 'INSERT INTO %s (categoryid, parentid, name, description, total, weight, created) VALUES (NULL, %u, %s, %s, %u, %u, %u)', |
| 338 | 338 | $this->db->prefix('smartfaq_categories'), |
| 339 | - $parentid, |
|
| 339 | + $parentid, |
|
| 340 | 340 | $this->db->quoteString($name), |
| 341 | 341 | $this->db->quoteString($description), |
| 342 | 342 | $total, |
@@ -348,7 +348,7 @@ discard block |
||
| 348 | 348 | 'UPDATE %s SET parentid = %u, name = %s, description = %s, total = %s, weight = %u, created = %u WHERE categoryid = %u', |
| 349 | 349 | $this->db->prefix('smartfaq_categories'), |
| 350 | 350 | $parentid, |
| 351 | - $this->db->quoteString($name), |
|
| 351 | + $this->db->quoteString($name), |
|
| 352 | 352 | $this->db->quoteString($description), |
| 353 | 353 | $total, |
| 354 | 354 | $weight, |