@@ -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 |
@@ -549,11 +549,11 @@ |
||
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 |
@@ -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(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 |
@@ -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( |
@@ -768,12 +768,12 @@ |
||
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); |
@@ -116,13 +116,13 @@ |
||
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'])); |