mambax7 /
newbb5
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 82 | if (is_object($user) && 0 < $user->getVar('level')) { |
||
| 83 | if (Request::getString('login', '', 'POST')) { |
||
| 84 | $user->setVar('last_login', time()); |
||
| 85 | if (!$memberHandler->insertUser($user)) { |
||
|
|
|||
| 86 | } |
||
| 87 | $_SESSION = []; |
||
| 88 | $_SESSION['xoopsUserId'] = $user->getVar('uid'); |
||
| 99 | $xoopsUserIsAdmin = $GLOBALS['xoopsUser']->isAdmin($xoopsModule->getVar('mid')); |
||
| 100 | } |
||
| 101 | } |
||
| 102 | View Code Duplication | if (!is_object($GLOBALS['xoopsUser'])) { |
|
| 103 | xoops_load('captcha'); |
||
| 104 | $xoopsCaptcha = XoopsCaptcha::getInstance(); |
||
| 105 | if (!$xoopsCaptcha->verify()) { |
||
| 141 | $uid = is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0; |
||
| 142 | |||
| 143 | $topic_status = $topicObject->getVar('topic_status'); |
||
| 144 | if ($topicHandler->getPermission($forumObject, $topic_status, 'edit') |
||
| 145 | && ($isAdmin |
||
| 146 | || ($postObject->checkTimelimit('edit_timelimit') |
||
| 147 | && $postObject->checkIdentity()))) { |
||
| 208 | $post_karma = ('require_karma' === $view_require) ? Request::getInt('post_karma', 0, 'POST') : 0; |
||
| 209 | $require_reply = ('require_reply' === $view_require); |
||
| 210 | $postObject->setVar('subject', $subject); |
||
| 211 | $editwhy = xoops_trim(Request::getString('editwhy', '', 'POST')); // !empty($_POST['editwhy'])) ? xoops_trim($_POST['editwhy']) : ""; |
||
| 212 | |||
| 213 | if ($dohtml && !newbbIsAdmin($forumObject)) { |
||
| 214 | //$message=newbb_textFilter($message); |
||
| 215 | } |
||
| 216 | $postObject->setVar('post_text', $message); |
||
| 217 | $postObject->setVar('post_karma', $post_karma); |
||
| 228 | $postObject->setPostEdit($poster_name, $editwhy); |
||
| 229 | } // is reply |
||
| 230 | |||
| 231 | // $attachments_tmp = array(); |
||
| 232 | // if (!empty($_POST["attachments_tmp"])) { |
||
| 233 | View Code Duplication | if (Request::getString('attachments_tmp', '', 'POST')) { |
|
| 234 | $attachments_tmp = unserialize(base64_decode(Request::getString('attachments_tmp', '', 'POST'))); |
||
| 235 | if (Request::getArray('delete_tmp', null, 'POST') && count(Request::getArray('delete_tmp', null, 'POST')) > 1) { |
||
| 236 | foreach (Request::getArray('delete_tmp', null, 'POST') as $key) { |
||
| 257 | $uploader = new NewbbUploader($uploaddir, $forumObject->getVar('attach_ext'), (int)$maxfilesize, (int)$GLOBALS['xoopsModuleConfig']['max_img_width'], (int)$GLOBALS['xoopsModuleConfig']['max_img_height']); |
||
| 258 | |||
| 259 | if ($_FILES['userfile']['error'] > 0) { |
||
| 260 | View Code Duplication | switch ($_FILES['userfile']['error']) { |
|
| 261 | case 1: |
||
| 262 | $error_message[] = _MD_NEWBB_MAXUPLOADFILEINI; |
||
| 263 | break; |
||
| 320 | } |
||
| 321 | } |
||
| 322 | |||
| 323 | //$postObject->loadFilters(empty($isnew) ? 'update' : 'insert'); |
||
| 324 | |||
| 325 | // Define tags for notification message |
||
| 326 | if (!empty($isnew) && $approved && !empty($GLOBALS['xoopsModuleConfig']['notification_enabled'])) { |
||
| 362 | } elseif (Request::getInt('notify', 0, 'POST') > 0) { |
||
| 363 | $notificationHandler->subscribe('thread', $postObject->getVar('topic_id'), 'new_post'); |
||
| 364 | } |
||
| 365 | // elseif ($_POST['notify']<0) keep it as it is |
||
| 366 | } |
||
| 367 | |||
| 368 | if ($approved) { |
||
| 406 | //$xoopsOption['xoops_module_header']= $xoops_module_header; |
||
| 407 | // irmtfan include header.php after defining $xoopsOption['template_main'] |
||
| 408 | include_once $GLOBALS['xoops']->path('header.php'); |
||
| 409 | //$xoopsTpl->assign('xoops_module_header', $xoops_module_header); |
||
| 410 | |||
| 411 | if (Request::getString('contents_upload', null, 'POST')) { |
||
| 412 | $attachments_tmp = []; |
||
| 413 | View Code Duplication | if (Request::getArray('attachments_tmp', null, 'POST')) { |
|
| 414 | $attachments_tmp = unserialize(base64_decode(Request::getArray('attachments_tmp', [], 'POST'))); |
||
| 415 | if (Request::getArray('delete_tmp', null, 'POST') && count(Request::getArray('delete_tmp', null, 'POST'))) { |
||
| 416 | foreach (Request::getArray('delete_tmp', '', 'POST') as $key) { |
||
| 428 | |||
| 429 | $uploader = new NewbbUploader($uploaddir, $forumObject->getVar('attach_ext'), (int)$maxfilesize, (int)$GLOBALS['xoopsModuleConfig']['max_img_width'], (int)$GLOBALS['xoopsModuleConfig']['max_img_height']); |
||
| 430 | if ($_FILES['userfile']['error'] > 0) { |
||
| 431 | View Code Duplication | switch ($_FILES['userfile']['error']) { |
|
| 432 | case 1: |
||
| 433 | $error_message[] = _MD_NEWBB_MAXUPLOADFILEINI; |
||
| 434 | break; |
||
| 477 | $p_message = $myts->previewTarea($p_message, $dohtml, $dosmiley, $doxcode, 1, $dobr); |
||
| 478 | $p_date = formatTimestamp(time()); |
||
| 479 | if ($postObject->isNew()) { |
||
| 480 | View Code Duplication | if (is_object($GLOBALS['xoopsUser'])) { |
|
| 481 | $p_name = $GLOBALS['xoopsUser']->getVar('uname'); |
||
| 482 | if (!empty($GLOBALS['xoopsModuleConfig']['show_realname']) && $GLOBALS['xoopsUser']->getVar('name')) { |
||
| 483 | $p_name = $GLOBALS['xoopsUser']->getVar('name'); |
||
| 511 | $poster_name = Request::getString('poster_name', '', 'POST'); |
||
| 512 | $hidden = Request::getString('hidden', '', 'POST'); |
||
| 513 | $notify = Request::getInt('notify', 0, 'POST'); |
||
| 514 | $attachsig = Request::getInt('attachsig', 0, 'POST');//!empty($_POST['attachsig']) ? 1 : 0; |
||
| 515 | $isreply = Request::getInt('isreply', 0, 'POST'); //!empty($_POST['isreply']) ? 1 : 0; |
||
| 516 | $isedit = Request::getInt('isedit', 0, 'POST'); //!empty($_POST['isedit']) ? 1 : 0; |
||
| 517 | $icon = (Request::getString('icon', '', 'POST') |
||
| 518 | && is_file($GLOBALS['xoops']->path('images/subject/' . Request::getString('icon', '', 'POST'))) ? Request::getString('icon', '', 'POST') : ''); |
||
| 519 | $view_require = Request::getString('view_require', '', 'POST'); |
||
| 535 |
mambax7
Loading history...
This check looks for the bodies of
ifstatements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete.These
ifbodies can be removed. If you have an empty if but statements in theelsebranch, consider inverting the condition.could be turned into
This is much more concise to read.