Logger::getLogDescription()

Complexity

Conditions	8
Paths	9

Size

Total Lines	74
Code Lines	46

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	72

Importance

Changes

0

<?php
	public static function getLogDescription(Log $entry)
	{
		$text = "Deferred to ";
		if (substr($entry->getAction(), 0, strlen($text)) == $text) {
			// Deferred to a different queue
			// This is exactly what we want to display.
			return $entry->getAction();	
		}
		
		$text = "Closed custom-n";
		if ($entry->getAction() == $text) {
			// Custom-closed
			return "closed (custom reason - account not created)";
		}
		
		$text = "Closed custom-y";
		if ($entry->getAction() == $text) {
			// Custom-closed
			return "closed (custom reason - account created)";
		}
		
		$text = "Closed 0";
		if ($entry->getAction() == $text) {
			// Dropped the request - short-circuit the lookup
			return "dropped request";
		}
		
		$text = "Closed ";
		if (substr($entry->getAction(), 0, strlen($text)) == $text) {
			// Closed with a reason - do a lookup here.
			$id = substr($entry->getAction(), strlen($text));
			/** @var EmailTemplate $template */
			$template = EmailTemplate::getById((int)$id, $entry->getDatabase());
			
			if ($template != false) {

The condition $template != false is always true.

				return "closed (" . $template->getName() . ")";
			}
			
		}
		
		// Fall back to the basic stuff
		$lookup = array(
			'Reserved' => 'reserved',
			'Email Confirmed' => 'email-confirmed',
			'Unreserved' => 'unreserved',
			'Approved' => 'approved',
			'Suspended' => 'suspended',
			'Banned' => 'banned',
			'Edited' => 'edited interface message',
			'Declined' => 'declined',
			'EditComment-c' => 'edited a comment',
			'EditComment-r' => 'edited a comment',
			'Unbanned' => 'unbanned',
			'Promoted' => 'promoted to tool admin',
			'BreakReserve' => 'forcibly broke the reservation',
			'Prefchange' => 'changed user preferences',
			'Renamed' => 'renamed',
			'Demoted' => 'demoted from tool admin',
			'ReceiveReserved' => 'received the reservation',
			'SendReserved' => 'sent the reservation',
			'EditedEmail' => 'edited email',
			'DeletedTemplate' => 'deleted template',
			'EditedTemplate' => 'edited template',
			'CreatedEmail' => 'created email',
			'CreatedTemplate' => 'created template',
			'SentMail' => 'sent an email to the requestor',
			);
		
		if (array_key_exists($entry->getAction(), $lookup)) {
			return $lookup[$entry->getAction()];
		}
		
		// OK, I don't know what this is. Fall back to something sane.
		return "performed an unknown action ({$entry->getAction()})";
	}