Pages\Deprecated\PageMessage::viewMessageList()

Complexity

Conditions	13
Paths	30

Size

Total Lines	88
Code Lines	60

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	182

Importance

Changes

0

<?php
  protected function viewMessageList() {
    require_once('includes/includes/coe_simulator_helpers.php');

    $pager = null;

    if ($this->current_class == MSG_TYPE_OUTBOX) {
      $message_query = "SELECT {{messages}}.message_id, {{messages}}.message_owner, {{users}}.id AS message_sender, {{messages}}.message_time,
          {{messages}}.message_type, {{users}}.username AS message_from, {{messages}}.message_subject, {{messages}}.message_text
       FROM
         {{messages}} LEFT JOIN {{users}} ON {{users}}.id = {{messages}}.message_owner WHERE `message_sender` = '{$this->playerId}' AND `message_type` = 1
       ORDER BY `message_time` DESC;";
    } else {
      if ($this->current_class == MSG_TYPE_NEW) {
        $SubUpdateQry = array();
        foreach ($this->messageClassList as $message_class_id => $message_class) {
          if ($message_class_id != MSG_TYPE_OUTBOX) {
            $SubUpdateQry[] = "`{$message_class['name']}` = '0'";
            $this->user[$message_class['name']] = 0;
          }
        }
        $SubUpdateQry = implode(',', $SubUpdateQry);
      } else {
        $messageClassNameNew = $this->messageClassList[MSG_TYPE_NEW]['name'];
        $messageClassNameCurrent = $this->messageClassList[$this->current_class]['name'];
        $SubUpdateQry = "`{$messageClassNameCurrent}` = '0', `{$messageClassNameNew}` = `{$messageClassNameNew}` - '{$this->user[$messageClassNameCurrent]}'";
        $SubSelectQry = "AND `message_type` = '{$this->current_class}'";

        $this->user[$messageClassNameNew] -= $this->user[$messageClassNameCurrent];
        $this->user[$messageClassNameCurrent] = 0;
      }

      db_user_set_by_id($this->playerId, $SubUpdateQry);

The function db_user_set_by_id() has been deprecated.

      $message_query =
        "SELECT * 
        FROM `{{messages}}` 
        WHERE `message_owner` = '{$this->playerId}' {$SubSelectQry} 

The variable $SubSelectQry does not seem to be defined for all execution paths leading up to this point.

        ORDER BY `message_time` DESC;";
    }

    if($this->showAll) {
      $message_query = $this->db->selectIterator($message_query);
    } else {
      $message_query = new DbSqlPaging($message_query, PAGING_PAGE_SIZE_DEFAULT_MESSAGES, sys_get_param_int(PagingRenderer::KEYWORD));
      $pager = new PagingRenderer($message_query, 'messages.php?mode=show&message_class=' . $this->current_class);
    }

    $template = gettemplate('msg_message_list', true);

true of type true is incompatible with the type null|template expected by parameter $template of gettemplate().

    foreach ($message_query as $message_row) {
      $text = $message_row['message_text'];
      if ($message_row['message_json']) {
        switch ($message_row['message_type']) {
          case MSG_TYPE_SPY:
            $text = DecodeEspionage::decode(MissionEspionageReport::fromJson($text));
          break;
          default:
            $text = '{ Unauthorised access - please contact Administration! }';
          break;

        }
      } else {
        if (in_array($message_row['message_type'], array(MSG_TYPE_PLAYER, MSG_TYPE_ALLIANCE)) && $message_row['message_sender']) {
          $text = htmlspecialchars($message_row['message_text']);
        }
        $text = nl2br($text);
      }

      $template->assign_block_vars('messages', array(
        'ID'   => $message_row['message_id'],
        'DATE' => date(FMT_DATE_TIME, $message_row['message_time'] + SN_CLIENT_TIME_DIFF),
        'FROM' => htmlspecialchars($message_row['message_from']),
        'SUBJ' => htmlspecialchars($message_row['message_subject']),
        'TEXT' => $text,

        'FROM_ID'        => $message_row['message_sender'],
        'SUBJ_SANITIZED' => htmlspecialchars($message_row['message_subject']),
        'STYLE'          => $this->current_class == MSG_TYPE_OUTBOX ? $this->messageClassList[MSG_TYPE_OUTBOX]['name'] : $this->messageClassList[$message_row['message_type']]['name'],
      ));
    }

    $current_class_text = $this->lang['msg_class'][$this->current_class];

    $template->assign_vars(array(
      "MESSAGE_CLASS"      => $this->current_class,
      "MESSAGE_CLASS_TEXT" => $current_class_text,
      "PAGER_MESSAGES"     => $pager ? $pager->render() : '',
    ));

    return $template;
  }