PageController - Code Metrics - sofiafahle/anax-mvc-content2 - Measure and Improve Code Quality continuously with Scrutinizer

PageController A
last analyzed 2016-03-14 20:36 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	283
Duplicated Lines	49.82 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
wmc	22
c	2
b	0
f	1
lcom	1
cbo	5
dl	141
loc	283
rs	10

15 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	5	1
A	setupAction()	0	6	1
A	indexAction()	0	4	1
A	listAction()	10	10	1
A	viewAction()	15	15	2
A	addAction()	15	15	1
A	updateAction()	21	21	2
A	deleteAction()	0	10	2
A	softDeleteAction()	15	15	2
A	restoreAction()	13	13	2
A	activateAction()	0	13	2
A	inactivateAction()	15	15	2
A	activeAction()	13	13	1
A	inactiveAction()	12	12	1
A	trashAction()	12	12	1

How to fix Duplicated Code

<?php

namespace Anax\Page;
 
/**
 * A controller for editable pages.
 *
 */
class PageController implements \Anax\DI\IInjectionAware
{
    use \Anax\DI\TInjectable,
        \Anax\MVC\TRedirectHelpers;
		
	public $page;
	
	/**
	 * Initialize the controller.
	 *
	 * @return void
	 */
	public function initialize()
	{
		$this->page = new \Anax\Page\Page();
		$this->page->setDI($this->di);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
	}
	
	public function setupAction()
	{
		require('setup.php');
		
		$this->redirectTo('page');
	}
	
	public function indexAction()
	{
		$this->listAction();
	}

	
	/**
	 * List all pages.
	 *
	 * @return void
	 */
	public function listAction()

	{
		$all = $this->page->findAll();
	 
		$this->theme->setTitle("List all pages");
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		$this->views->add('page/list', [
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			'pages' => $all,
			'title' => "Editable pages",
		]);
	}
	
	/**
	 * Views page with with slug.
	 *
	 * @param int $slug of page to display
	 *
	 * @return void
	 */
	public function viewAction($slug = null)

	{
		$pages = $this->page->findWhere('slug', $slug);
		
		if (empty($pages)) {
			die('No such page!');
		}
		
		$page = $pages[0];
		
		$this->theme->setTitle($page->title);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		$this->views->add('page/view', [
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			'page' => $page,
		]);
	}
	
	/**
	 * Add new page.
	 *
	 *
	 * @return void
	 */
	public function addAction()

	{
		
		$form = new \Anax\Page\CFormAddPage();
        $form->setDI($this->di);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
		
		// Check the status of the form
        $form->check();
	 
		$this->di->theme->setTitle("Add page");
        $this->di->views->add('default/page', [
            'title' => "Add a page",
            'content' => $form->getHTML()
        ]);
	}
	
	/**
	 * Edit a page.
	 *
	 * @param string $id of page to edit.
	 *
	 * @return void
	 */
	public function updateAction($id = null)

	{
		
		if (!$id) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$this->redirectTo('page');
		}
		
		$page = $this->page->find($id);
		
		$form = new \Anax\Page\CFormUpdatePage($page);
        $form->setDI($this->di);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
		
		// Check the status of the form
        $form->check();
	 
		$this->di->theme->setTitle("Update page");
        $this->di->views->add('default/page', [
            'title' => "Update a page",
            'content' => $form->getHTML()
        ]);
	}
	
	/**
	 * Delete page.
	 *
	 * @param integer $id of page to delete.
	 *
	 * @return void
	 */
	public function deleteAction($id = null)
	{
		if (!isset($id)) {
			die("Missing id");
		}
	 
		$res = $this->page->delete($id);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
	 
		$this->redirectTo('page');
	}
	
	/**
	 * Delete (soft) page.
	 *
	 * @param integer $id of page to delete.
	 *
	 * @return void
	 */
	public function softDeleteAction($id = null)

	{
		if (!isset($id)) {
			die("Missing id");
		}
	 
		$now = gmdate('Y-m-d H:i:s');
	 
		$page = $this->page->find($id);
	 
		$page->deleted = $now;
		$page->save();
	
		$this->redirectTo('page/trash');
	}
	
	/**
	 * Restore (soft) deleted page.
	 *
	 * @param integer $id of page to restore.
	 *
	 * @return void
	 */
	public function restoreAction($id = null)

	{
		if (!isset($id)) {
			die("Missing id");
		}
	 
		$page = $this->page->find($id);
	 
		$page->deleted = null;
		$page->save();
	 
		$this->redirectTo('page');
	}
	
	/**
	 * Activate page.
	 *
	 * @param integer $id of page to activate.
	 *
	 * @return void
	 */
	public function activateAction($id = null)
	{
		if (!isset($id)) {
			die("Missing id");
		}

		$page = $this->page->find($id);
	 
		$page->inactivated =  null;
		$page->save();
	 
		$this->redirectTo('page');
	}
	
	/**
	 * Inactivate page.
	 *
	 * @param integer $id of page to inactivate.
	 *
	 * @return void
	 */
	public function inactivateAction($id = null)

	{
		if (!isset($id)) {
			die("Missing id");
		}
		
		$now = gmdate('Y-m-d H:i:s');
	 
		$page = $this->page->find($id);
	 
		$page->inactivated = $now;
		$page->save();
	 
		$this->redirectTo('page/inactive');
	}
	
	/**
	 * List all active and not deleted pages.
	 *
	 * @return void
	 */
	public function activeAction()

	{
		$all = $this->page->query()
			->where('inactivated IS NULL')
			->andWhere('deleted is NULL')
			->execute();
	 
		$this->theme->setTitle("Pages that are active");
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		$this->views->add('page/active', [
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			'pages' => $all,
			'title' => "Pages that are active",
		]);
	}
	
	/**
	 * List all inactive pages.
	 *
	 * @return void
	 */
	public function inactiveAction()

	{
		$all = $this->page->query()
			->where('inactivated IS NOT NULL')
			->execute();
	 
		$this->theme->setTitle("Pages that are inactive");
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		$this->views->add('page/inactive', [
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			'pages' => $all,
			'title' => "Pages that are inactive",
		]);
	}
	
	/**
	 * List all deleted pages.
	 *
	 * @return void
	 */
	public function trashAction()

	{
		$all = $this->page->query()
			->where('deleted IS NOT NULL')
			->execute();
	 
		$this->theme->setTitle("Pages that are deleted");
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		$this->views->add('page/deleted', [
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			'pages' => $all,
			'title' => "Pages that are deleted",
		]);
	}
	 
}

1			<?php
2
3			namespace Anax\Page;
4
5			/**
6			* A controller for editable pages.
7			*
8			*/
9			class PageController implements \Anax\DI\IInjectionAware
10			{
11			use \Anax\DI\TInjectable,
12			\Anax\MVC\TRedirectHelpers;
13
14			public $page;
15
16			/**
17			* Initialize the controller.
18			*
19			* @return void
20			*/
21			public function initialize()
22			{
23			$this->page = new \Anax\Page\Page();
24			$this->page->setDI($this->di);
			0 ignored issues – show Bug introduced 2016-03-13 11:56 UTC by Report Bug Copy Issue Report It seems like `$this->di` can also be of type `array` or `null`; however, `Anax\DI\TInjectable::setDI()` does only seem to accept `object<Anax\DI\class>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
25			}
26
27			public function setupAction()
28			{
29			require('setup.php');
30
31			$this->redirectTo('page');
32			}
33
34			public function indexAction()
35			{
36			$this->listAction();
37			}
38
39

sofiafahle / anax-mvc-content2

PageController A last analyzed 2016-03-14 20:36 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

15 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like

PageController A
last analyzed 2016-03-14 20:36 UTC