Lingo\MessageLog

Complexity

Total Complexity

13

Size/Duplication

Total Lines	91
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	addMessage()	0	10	3
A	addError()	0	4	1
A	addWarning()	0	4	1
A	addNotice()	0	3	1
C	getMessagesFormatted()	0	37	7

<?php

/**
 * File holding the Lingo\MessageLog class.
 *
 * This file is part of the MediaWiki extension Lingo.
 *
 * @copyright 2011 - 2016, Stephan Gambke
 * @license   GNU General Public License, version 2 (or any later version)
 *
 * The Lingo extension is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by the Free
 * Software Foundation; either version 2 of the License, or (at your option) any
 * later version.
 *
 * The Lingo extension is distributed in the hope that it will be useful, but
 * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
 * details.
 *
 * You should have received a copy of the GNU General Public License along
 * with this program. If not, see <http://www.gnu.org/licenses/>.
 *
 * @author Stephan Gambke
 *
 * @file
 * @ingroup Lingo
 */

namespace Lingo;
use Html;
use Parser;
use ParserOptions;

/**
 * This class holds messages (errors, warnings, notices) for Lingo
 *
 * Contains a static function to initiate the parsing.
 *
 * @ingroup Lingo
 */
class MessageLog {

	private $mMessages = array();
	private $mParser = null;

	const MESSAGE_ERROR = 1;
	const MESSAGE_WARNING = 2;
	const MESSAGE_NOTICE = 3;

	/**
	 * @param $message
	 * @param int $severity
	 */
	public function addMessage( $message, $severity = self::MESSAGE_NOTICE ) {
		$this->mMessages[] = array( $message, $severity );

		// log errors and warnings in debug log
		if ( $severity == self::MESSAGE_WARNING ||
			$severity == self::MESSAGE_ERROR
		) {
			wfDebug( $message );
		}
	}

	/**
	 * @param $message
	 */
	public function addError( $message ) {
		$this->mMessages[] = array( $message, self::MESSAGE_ERROR );
		wfDebug( "Error: $message\n" );

As per coding-style, please use concatenation or sprintf for the variable $message instead of interpolation.

	}

	/**
	 * @param $message
	 */
	public function addWarning( $message ) {
		$this->mMessages[] = array( $message, self::MESSAGE_WARNING );
		wfDebug( "Warning: $message\n" );

As per coding-style, please use concatenation or sprintf for the variable $message instead of interpolation.

	}

	/**
	 * @param $message
	 */
	public function addNotice( $message ) {
		$this->mMessages[] = array( $message, self::MESSAGE_NOTICE );
	}

	/**
	 * @param int $severity
	 * @param null $header
	 * @return null|string
	 */
	public function getMessagesFormatted( $severity = self::MESSAGE_WARNING, $header = null ) {
		global $wgTitle, $wgUser;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

		$ret = '';

		foreach ( $this->mMessages as $message ) {
			if ( $message[ 1 ] <= $severity ) {
				$ret .= '* ' . $message[ 0 ] . "\n";
			}
		}

		if ( $ret != '' ) {
			if ( !$this->mParser ) {
				$parser = new Parser();
			}

			if ( $header == null ) {
				$header = '';
			} elseif ( $header != '' ) {
				$header = Html::rawElement( 'div', array( 'class' => 'heading' ), $header );
			}

			$ret = Html::rawElement( 'div', array( 'class' => 'messages' ),
				$header . "\n" .
				$ret
			);

			// FIXME: Variable 'parser' might have not been defined

Comment refers to a FIXME task "Variable 'parser' might have not been defined"

			// FIXME: $parser->parse returns ParserOutput, not String

Comment refers to a FIXME task "$parser->parse returns ParserOutput, not String"

			$ret = $parser->parse( $ret, $wgTitle, ParserOptions::newFromUser( $wgUser ) );

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

		} else {
			// FIXME: Should probably return '' (and throw an error if necessary)

Comment refers to a FIXME task "Should probably return '' (and throw an error if necessary"

			$ret = null;
		}

		return $ret;
	}

}