keeko\core\action\base\PackageDeleteActionTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configureParams()	0	3	1
A	run()	0	16	2

<?php
namespace keeko\core\action\base;

use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\OptionsResolver\OptionsResolver;
use keeko\core\model\PackageQuery;
use Symfony\Component\Routing\Exception\ResourceNotFoundException;

/**
 * Base methods for keeko\core\action\PackageDeleteAction
 * 
 * This code is automatically created. Modifications will probably be overwritten.
 * 
 * @author gossi
 */
trait PackageDeleteActionTrait {

	/**
	 * @param OptionsResolver $resolver
	 */
	public function configureParams(OptionsResolver $resolver) {
		$resolver->setRequired(['id']);
	}

	/**
	 * Automatically generated run method
	 * 
	 * @param Request $request
	 * @return Response
	 */
	public function run(Request $request) {
		// read
		$id = $this->getParam('id');

It seems like getParam() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		$package = PackageQuery::create()->findOneById($id);

		// check existence
		if ($package === null) {
			throw new ResourceNotFoundException('package not found.');
		}

		// delete
		$package->delete();

		// run response
		return $this->response->run($request, $package);

The property response does not exist. Did you maybe forget to declare it?

	}
}