Tracker_Issue_Model

Complexity

Total Complexity

7

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

16.66%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	report()	0	22	6

<?php declare(strict_types=1); defined('BASEPATH') OR exit('No direct script access allowed');

class Tracker_Issue_Model extends Tracker_Base_Model {
	public function __construct() {
		parent::__construct();
	}


	public function report(string $text, $userID = NULL, $url = NULL) : bool {
		$this->load->library('email');

		//This is pretty barebones issue reporting, and honestly not a great way to do it, but it works for now (until the Github is public).
		$body = "".
		        (!is_null($url) && !empty($url) ? "URL: ".htmlspecialchars(substr($url, 0, 255))."<br>\n" : "").
		        "Submitted by: ".$this->input->ip_address().(!is_null($userID) ? "| {$userID}" : "")."<br>\n".
		        "<br>Bug report: ".htmlspecialchars(substr($text, 0, 1000));

		$success = TRUE;
		$this->email->from('no-reply@trackr.moe', $this->config->item('site_title', 'ion_auth'));
		if($this->User->id) {
			$this->email->reply_to($this->User->email);
		}
		$this->email->to('bug@trackr.moe');
		$this->email->subject($this->config->item('site_title', 'ion_auth')." - Issue Report");
		$this->email->message($body);
		if(!$this->email->send()) {

The expression $this->email->send() of type null|boolean is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

			$success = FALSE;
		}
		return $success;
	}

}