Inspection of "Refactored Document Storage" - jenskooij/cloudcontrol - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( e3baf2...b0eb3e )

by Jens

created 2017-03-17 11:55 UTC

Status

Indentation +45 added lines, -45 removed lines patch added patch discarded remove patch

@@ -12,58 +12,58 @@
 block discarded – undo
 
 class DocumentRouting implements CmsRouting
 {
-    /**
-     * DocumentRouting constructor.
-     * @param $request
-     * @param $relativeCmsUri
-     * @param CmsComponent $cmsComponent
-     */
-    public function __construct($request, $relativeCmsUri, $cmsComponent)
-    {
-        if ($relativeCmsUri == '/documents') {
-            $cmsComponent->subTemplate = 'cms/documents';
-            $cmsComponent->setParameter(CmsComponent::PARAMETER_DOCUMENTS, $cmsComponent->storage->getDocuments()->getDocuments());
-            $cmsComponent->setParameter(CmsComponent::PARAMETER_MAIN_NAV_CLASS, CmsComponent::PARAMETER_DOCUMENTS);
-        }
-        $this->documentRouting($request, $relativeCmsUri, $cmsComponent);
-        $this->folderRouting($request, $relativeCmsUri, $cmsComponent);
-    }
-
-
-    /**
-     * @param $request
-     * @param $relativeCmsUri
-     * @param CmsComponent $cmsComponent
-     * @throws \Exception
-     */
-    private function documentRouting($request, $relativeCmsUri, $cmsComponent)
-    {
-        if ($relativeCmsUri == '/documents/new-document' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
+	/**
+	 * DocumentRouting constructor.
+	 * @param $request
+	 * @param $relativeCmsUri
+	 * @param CmsComponent $cmsComponent
+	 */
+	public function __construct($request, $relativeCmsUri, $cmsComponent)
+	{
+		if ($relativeCmsUri == '/documents') {
+			$cmsComponent->subTemplate = 'cms/documents';
+			$cmsComponent->setParameter(CmsComponent::PARAMETER_DOCUMENTS, $cmsComponent->storage->getDocuments()->getDocuments());
+			$cmsComponent->setParameter(CmsComponent::PARAMETER_MAIN_NAV_CLASS, CmsComponent::PARAMETER_DOCUMENTS);
+		}
+		$this->documentRouting($request, $relativeCmsUri, $cmsComponent);
+		$this->folderRouting($request, $relativeCmsUri, $cmsComponent);
+	}
+
+
+	/**
+	 * @param $request
+	 * @param $relativeCmsUri
+	 * @param CmsComponent $cmsComponent
+	 * @throws \Exception
+	 */
+	private function documentRouting($request, $relativeCmsUri, $cmsComponent)
+	{
+		if ($relativeCmsUri == '/documents/new-document' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
 			$this->documentOverviewRoute($request, $cmsComponent);
-        } elseif ($relativeCmsUri == '/documents/edit-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
+		} elseif ($relativeCmsUri == '/documents/edit-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
 			$this->editDocumentRoute($request, $cmsComponent);
-        } elseif ($relativeCmsUri == '/documents/get-brick' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
+		} elseif ($relativeCmsUri == '/documents/get-brick' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
 			$this->getBrickRoute($request, $cmsComponent);
-        } else if ($relativeCmsUri == '/documents/delete-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
+		} else if ($relativeCmsUri == '/documents/delete-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
 			$this->deleteDocumentRoute($request, $cmsComponent);
-        }
-    }
-
-    /**
-     * @param $request
-     * @param $relativeCmsUri
-     * @param CmsComponent $cmsComponent
-     */
-    private function folderRouting($request, $relativeCmsUri, $cmsComponent)
-    {
-        if ($relativeCmsUri == '/documents/new-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
+		}
+	}
+
+	/**
+	 * @param $request
+	 * @param $relativeCmsUri
+	 * @param CmsComponent $cmsComponent
+	 */
+	private function folderRouting($request, $relativeCmsUri, $cmsComponent)
+	{
+		if ($relativeCmsUri == '/documents/new-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
 			$this->newFolderRoute($request, $cmsComponent);
-        } else if ($relativeCmsUri == '/documents/edit-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
+		} else if ($relativeCmsUri == '/documents/edit-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
 			$this->editFolderRoute($request, $cmsComponent);
-        } else if ($relativeCmsUri == '/documents/delete-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
+		} else if ($relativeCmsUri == '/documents/delete-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
 			$this->deleteFolderRoute($request, $cmsComponent);
-        }
-    }
+		}
+	}
 
 	/**
 	 * @param $request

Please login to merge, or discard this patch.

		@@ -12,58 +12,58 @@
		block discarded – undo
12	12
13	13	class DocumentRouting implements CmsRouting
14	14	{
15		- /**
16		- * DocumentRouting constructor.
17		- * @param $request
18		- * @param $relativeCmsUri
19		- * @param CmsComponent $cmsComponent
20		- */
21		- public function __construct($request, $relativeCmsUri, $cmsComponent)
22		- {
23		- if ($relativeCmsUri == '/documents') {
24		- $cmsComponent->subTemplate = 'cms/documents';
25		- $cmsComponent->setParameter(CmsComponent::PARAMETER_DOCUMENTS, $cmsComponent->storage->getDocuments()->getDocuments());
26		- $cmsComponent->setParameter(CmsComponent::PARAMETER_MAIN_NAV_CLASS, CmsComponent::PARAMETER_DOCUMENTS);
27		- }
28		- $this->documentRouting($request, $relativeCmsUri, $cmsComponent);
29		- $this->folderRouting($request, $relativeCmsUri, $cmsComponent);
30		- }
31		-
32		-
33		- /**
34		- * @param $request
35		- * @param $relativeCmsUri
36		- * @param CmsComponent $cmsComponent
37		- * @throws \Exception
38		- */
39		- private function documentRouting($request, $relativeCmsUri, $cmsComponent)
40		- {
41		- if ($relativeCmsUri == '/documents/new-document' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
	15	+ /**
	16	+ * DocumentRouting constructor.
	17	+ * @param $request
	18	+ * @param $relativeCmsUri
	19	+ * @param CmsComponent $cmsComponent
	20	+ */
	21	+ public function __construct($request, $relativeCmsUri, $cmsComponent)
	22	+ {
	23	+ if ($relativeCmsUri == '/documents') {
	24	+ $cmsComponent->subTemplate = 'cms/documents';
	25	+ $cmsComponent->setParameter(CmsComponent::PARAMETER_DOCUMENTS, $cmsComponent->storage->getDocuments()->getDocuments());
	26	+ $cmsComponent->setParameter(CmsComponent::PARAMETER_MAIN_NAV_CLASS, CmsComponent::PARAMETER_DOCUMENTS);
	27	+ }
	28	+ $this->documentRouting($request, $relativeCmsUri, $cmsComponent);
	29	+ $this->folderRouting($request, $relativeCmsUri, $cmsComponent);
	30	+ }
	31	+
	32	+
	33	+ /**
	34	+ * @param $request
	35	+ * @param $relativeCmsUri
	36	+ * @param CmsComponent $cmsComponent
	37	+ * @throws \Exception
	38	+ */
	39	+ private function documentRouting($request, $relativeCmsUri, $cmsComponent)
	40	+ {
	41	+ if ($relativeCmsUri == '/documents/new-document' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
42	42	$this->documentOverviewRoute($request, $cmsComponent);
43		- } elseif ($relativeCmsUri == '/documents/edit-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
	43	+ } elseif ($relativeCmsUri == '/documents/edit-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
44	44	$this->editDocumentRoute($request, $cmsComponent);
45		- } elseif ($relativeCmsUri == '/documents/get-brick' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
	45	+ } elseif ($relativeCmsUri == '/documents/get-brick' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
46	46	$this->getBrickRoute($request, $cmsComponent);
47		- } else if ($relativeCmsUri == '/documents/delete-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
	47	+ } else if ($relativeCmsUri == '/documents/delete-document' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
48	48	$this->deleteDocumentRoute($request, $cmsComponent);
49		- }
50		- }
51		-
52		- /**
53		- * @param $request
54		- * @param $relativeCmsUri
55		- * @param CmsComponent $cmsComponent
56		- */
57		- private function folderRouting($request, $relativeCmsUri, $cmsComponent)
58		- {
59		- if ($relativeCmsUri == '/documents/new-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
	49	+ }
	50	+ }
	51	+
	52	+ /**
	53	+ * @param $request
	54	+ * @param $relativeCmsUri
	55	+ * @param CmsComponent $cmsComponent
	56	+ */
	57	+ private function folderRouting($request, $relativeCmsUri, $cmsComponent)
	58	+ {
	59	+ if ($relativeCmsUri == '/documents/new-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_PATH])) {
60	60	$this->newFolderRoute($request, $cmsComponent);
61		- } else if ($relativeCmsUri == '/documents/edit-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
	61	+ } else if ($relativeCmsUri == '/documents/edit-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
62	62	$this->editFolderRoute($request, $cmsComponent);
63		- } else if ($relativeCmsUri == '/documents/delete-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
	63	+ } else if ($relativeCmsUri == '/documents/delete-folder' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
64	64	$this->deleteFolderRoute($request, $cmsComponent);
65		- }
66		- }
	65	+ }
	66	+ }
67	67
68	68	/**
69	69	* @param $request

jenskooij / cloudcontrol

Push — develop ( e3baf2...b0eb3e )

Status

Category

Indentation +45 added lines, -45 removed lines patch added patch discarded remove patch