AdminModel

Complexity

Total Complexity

9

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

3 Methods

Rating	Name	Duplication	Size	Complexity
B	setAccountSuspensionAndDeletionStatus()	0	24	5
A	writeDeleteAndSuspensionInfoToDatabase()	0	16	2
A	resetUserSession()	0	15	2

<?php

/**
 * Handles all data manipulation of the admin part
 */
class AdminModel

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
	/**
	 * Sets the deletion and suspension values
	 *
	 * @param $suspensionInDays
	 * @param $softDelete
	 * @param $userId
	 */
	public static function setAccountSuspensionAndDeletionStatus($suspensionInDays, $softDelete, $userId)
	{
		if ($suspensionInDays > 0) {
			$suspensionTime = time() + ($suspensionInDays * 60 * 60 * 24);
		} else {
			$suspensionTime = null;
		}

        // FYI "on" is what a checkbox delivers by default when submitted. Didn't know that for a long time :)
		if ($softDelete == "on") {
			$delete = 1;
		} else {
			$delete = 0;
		}

		// write the above info to the database
		self::writeDeleteAndSuspensionInfoToDatabase($userId, $suspensionTime, $delete);

		// if suspension or deletion should happen, then also kick user out of the application instantly by resetting
		// the user's session :)
		if ($suspensionTime != null OR $delete = 1) {

Using logical operators such as or instead of || is generally not recommended.

			self::resetUserSession($userId);
		}
	}

	/**
	 * Simply write the deletion and suspension info for the user into the database, also puts feedback into session
	 *
	 * @param $userId
	 * @param $suspensionTime
	 * @param $delete
	 * @return bool
	 */
	private static function writeDeleteAndSuspensionInfoToDatabase($userId, $suspensionTime, $delete)
	{
		$database = DatabaseFactory::getFactory()->getConnection();

		$query = $database->prepare("UPDATE users SET user_suspension_timestamp = :user_suspension_timestamp, user_deleted = :user_deleted  WHERE user_id = :user_id LIMIT 1");
		$query->execute(array(
				':user_suspension_timestamp' => $suspensionTime,
				':user_deleted' => $delete,
				':user_id' => $userId
		));

		if ($query->rowCount() == 1) {
			Session::add('feedback_positive', Text::get('FEEDBACK_ACCOUNT_SUSPENSION_DELETION_STATUS'));
			return true;
		}
	}

	/**
	 * Kicks the selected user out of the system instantly by resetting the user's session.
	 * This means, the user will be "logged out".
	 *
	 * @param $userId
	 * @return bool
	 */
	private static function resetUserSession($userId)
	{
		$database = DatabaseFactory::getFactory()->getConnection();

		$query = $database->prepare("UPDATE users SET session_id = :session_id  WHERE user_id = :user_id LIMIT 1");
		$query->execute(array(
				':session_id' => null,
				':user_id' => $userId
		));

		if ($query->rowCount() == 1) {
			Session::add('feedback_positive', Text::get('FEEDBACK_ACCOUNT_USER_SUCCESSFULLY_KICKED'));
			return true;
		}
	}
}