DroitAcces::getDroitAccesContenu() - Code Metrics - Inspection of "update DroitAcces class to simplify it" - Piou-piou/ribs-framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 130235...72544b )

by Anthony

created 2016-03-06 12:10 UTC

DroitAcces::getDroitAccesContenu() D

↳ Parent: DroitAcces

Complexity

Conditions	10
Paths	7

Size

Total Lines	46
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	0	Features	0

Metric	Value
c	5
b	0
f	0
dl	0
loc	46
rs	4.983
cc	10
eloc	26
nc	7
nop	2

How to fix Complexity

<?php
	namespace core\admin\droitsacces;

	use core\App;

	class DroitAcces {
		private $logged;

		//pour la table identite
		private $id_identite;
		private $super_admin;

		//pour la table liste_droit_acces
		private $id_liste_droit_acces;

		//pour des droits pour la gestion des contenus
		private $modif_seo;
		private $modif_contenu;
		private $modif_navigation;
		private $supprimer_page;


		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
		public function __construct() {
			$dbc = \core\App::getDb();

			if (isset($_SESSION["idlogin".CLEF_SITE])) {
				$this->logged = true;
				$this->id_identite = $_SESSION["idlogin".CLEF_SITE];

				//on test voir si super admin
				$query = $dbc->query("SELECT super_admin,liste_droit FROM identite WHERE ID_identite=".$this->id_identite);

				if ((is_array($query)) && (count($query) > 0)) {
					foreach ($query as $obj) {
						$this->super_admin = $obj->super_admin;
						$this->id_liste_droit_acces = $obj->liste_droit;
					}
				}
			}
			else {
				$this->logged = false;
			}
		}
		//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
    
    
    
		//-------------------------- GETTER ----------------------------------------------------------------------------//
		public function getLogged(){
			return $this->logged;
		}
		public function getSuperAdmin(){
			return $this->super_admin;
		}
		public function getIdListeDroitAcces(){
			return $this->id_liste_droit_acces;
		}
		public function getModifSeo(){
			return $this->modif_seo;
		}
		public function getModifContenu(){
			return $this->modif_contenu;
		}
		public function getModifNavigation(){
			return $this->modif_navigation;
		}
		public function getSupprimerPage(){
			return $this->supprimer_page;
		}

		/**
		 * @return array
		 */
		private function getListeDroitAcces() {
			$dbc = App::getDb();

			$liste_droit_acces = [];

			$query = $dbc->query("SELECT * FROM droit_acces, liste_droit_acces, liaison_liste_droit WHERE
								droit_acces.ID_droit_acces = liaison_liste_droit.ID_droit_acces AND
								liste_droit_acces.ID_liste_droit_acces = liaison_liste_droit.ID_liste_droit_acces AND
								liste_droit_acces.ID_liste_droit_acces = $this->id_liste_droit_acces
				");

			if ((is_array($query)) && (count($query) > 0)) {
				foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
			}

			return $liste_droit_acces;
		}

		/**
		 * @return array/

		 */
		private function getListeDroitPage() {
			$dbc = App::getDb();
			$droit_acces = [];

			$query = $dbc->query("SELECT droit_acces FROM droit_acces WHERE page LIKE '%$page%'");

			if ((is_array($query)) && (count($query) > 0)) {
				foreach ($query as $obj) $droit_acces = $obj->droit_acces;
			}

			return $droit_acces;
		}

		//autres getter
		/**
		 * pour savoir si en fonction des droits d'accès de l'utilisateur il peu ou non accéder à cete page
		 * on passe outre les test si on est super admin
		 * @param string $page
		 * @return bool
		 */
		public function getDroitAccesPage($page) {
			//page sans droit dans admin
			$all_access = array("gestion-comptes/mon-compte", "index");

			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage(), $this->getListeDroitAcces())) || (($page == "") || ($page == null)) || (in_array($page, $all_access))) {
				return true;
			}
			else {
				return false;
			}
		}

		/**
		 * fonction qui permet de gérer les droits d'accès sur les contenus :
		 * - creer une page
		 * - modifier du contenu (SEO, navigation, contenu)
		 * - supprimer une page
		 * si on est super admin on passe outre tous les tests
		 * @param $droit
		 * @param $id_page
		 * @return bool|null
		 */
		public function getDroitAccesContenu($droit, $id_page) {
			$dbc = \core\App::getDb();

			//récupération de la liste des droits de l'utilisateur si aps super admin
			if ($this->super_admin != 1) {
				$liste_droit_acces = $this->getListeDroitAcces();

				if (in_array($droit, $liste_droit_acces)) {
					//on check si il a le droit de modifier ou supprimer cette page
					$query = $dbc->query("SELECT * FROM droit_acces_page, liste_droit_acces WHERE
									droit_acces_page.ID_liste_droit_acces = liste_droit_acces.ID_liste_droit_acces AND
									droit_acces_page.ID_page = $id_page AND
									liste_droit_acces.ID_liste_droit_acces = $this->id_liste_droit_acces
					");

					//si on a un resultat
					if ((is_array($query)) && (count($query) > 0)) {
						foreach ($query as $obj) {
							$this->modif_seo = $obj->seo;
							$this->modif_contenu = $obj->contenu;
							$this->modif_navigation = $obj->navigation;
							$this->supprimer_page = $obj->supprimer;
						}

						//si les trois sont différent de 0 on renvoit true soinon false
						if (($this->modif_seo != 0) || ($this->modif_contenu != 0) || ($this->modif_navigation != 0) || ($this->supprimer_page != 0)) {
							return true;
						}
						else {
							return false;
						}
					}
				}
				else {
					return false;
				}
			}
			else {
				$this->modif_seo = 1;
				$this->modif_contenu = 1;
				$this->modif_navigation = 1;
				$this->supprimer_page = 1;

				return true;
			}
		}

		/**
		 * pour savoir si un utilisateur à le droit de supprimer, modifier ou ajouter des trucs
		 * @param $droit_acces
		 * @return bool
		 */
		public function getDroitAccesAction($droit_acces) {
			$liste_droit_acces = $this->getListeDroitAcces();

			if (($this->super_admin == 1) || (in_array($droit_acces, $liste_droit_acces))) {
				return true;
			}
			else {
				return false;
			}
		}
		//-------------------------- FIN GETTER ----------------------------------------------------------------------------//
    
    
    
		//-------------------------- SETTER ----------------------------------------------------------------------------//

		//-------------------------- FIN SETTER ----------------------------------------------------------------------------//
	}

1			<?php
2			namespace core\admin\droitsacces;
3
4			use core\App;
5
6			class DroitAcces {
7			private $logged;
8
9			//pour la table identite
10			private $id_identite;
11			private $super_admin;
12
13			//pour la table liste_droit_acces
14			private $id_liste_droit_acces;
15
16			//pour des droits pour la gestion des contenus
17			private $modif_seo;
18			private $modif_contenu;
19			private $modif_navigation;
20			private $supprimer_page;
21
22
23			//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
24			public function __construct() {
25			$dbc = \core\App::getDb();
26
27			if (isset($_SESSION["idlogin".CLEF_SITE])) {
28			$this->logged = true;
29			$this->id_identite = $_SESSION["idlogin".CLEF_SITE];
30
31			//on test voir si super admin
32			$query = $dbc->query("SELECT super_admin,liste_droit FROM identite WHERE ID_identite=".$this->id_identite);
33
34			if ((is_array($query)) && (count($query) > 0)) {
35			foreach ($query as $obj) {
36			$this->super_admin = $obj->super_admin;
37			$this->id_liste_droit_acces = $obj->liste_droit;
38			}
39			}
40			}
41			else {
42			$this->logged = false;
43			}
44			}
45			//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
46
47
48
49			//-------------------------- GETTER ----------------------------------------------------------------------------//
50			public function getLogged(){
51			return $this->logged;
52			}
53			public function getSuperAdmin(){
54			return $this->super_admin;
55			}
56			public function getIdListeDroitAcces(){
57			return $this->id_liste_droit_acces;
58			}
59			public function getModifSeo(){
60			return $this->modif_seo;
61			}
62			public function getModifContenu(){
63			return $this->modif_contenu;
64			}
65			public function getModifNavigation(){
66			return $this->modif_navigation;
67			}
68			public function getSupprimerPage(){
69			return $this->supprimer_page;
70			}
71
72			/**
73			* @return array
74			*/
75			private function getListeDroitAcces() {
76			$dbc = App::getDb();
77
78			$liste_droit_acces = [];
79
80			$query = $dbc->query("SELECT * FROM droit_acces, liste_droit_acces, liaison_liste_droit WHERE
81			droit_acces.ID_droit_acces = liaison_liste_droit.ID_droit_acces AND
82			liste_droit_acces.ID_liste_droit_acces = liaison_liste_droit.ID_liste_droit_acces AND
83			liste_droit_acces.ID_liste_droit_acces = $this->id_liste_droit_acces
84			");
85
86			if ((is_array($query)) && (count($query) > 0)) {
87			foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
88			}
89
90			return $liste_droit_acces;
91			}
92
93			/**
94			* @return array/
			0 ignored issues – show Documentation introduced 2016-03-06 12:13 UTC by Report Bug Copy Issue Report The doc-type `array/` could not be parsed: Unknown type name "array/" at position 0. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
95			*/
96			private function getListeDroitPage() {
97			$dbc = App::getDb();
98			$droit_acces = [];
99
100			$query = $dbc->query("SELECT droit_acces FROM droit_acces WHERE page LIKE '%$page%'");
			0 ignored issues – show Bug introduced 2016-03-06 12:13 UTC by Report Bug Copy Issue Report The variable `$page` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
101			if ((is_array($query)) && (count($query) > 0)) {
102			foreach ($query as $obj) $droit_acces = $obj->droit_acces;
103			}
104
105			return $droit_acces;
106			}
107
108			//autres getter
109			/**
110			* pour savoir si en fonction des droits d'accès de l'utilisateur il peu ou non accéder à cete page
111			* on passe outre les test si on est super admin
112			* @param string $page
113			* @return bool
114			*/
115			public function getDroitAccesPage($page) {
116			//page sans droit dans admin
117			$all_access = array("gestion-comptes/mon-compte", "index");
118
119			if (($this->super_admin == 1) \|\| (in_array($this->getListeDroitPage(), $this->getListeDroitAcces())) \|\| (($page == "") \|\| ($page == null)) \|\| (in_array($page, $all_access))) {
120			return true;
121			}
122			else {
123			return false;
124			}
125			}
126
127			/**
128			* fonction qui permet de gérer les droits d'accès sur les contenus :
129			* - creer une page
130			* - modifier du contenu (SEO, navigation, contenu)
131			* - supprimer une page
132			* si on est super admin on passe outre tous les tests
133			* @param $droit
134			* @param $id_page
135			* @return bool\|null
136			*/
137			public function getDroitAccesContenu($droit, $id_page) {
138			$dbc = \core\App::getDb();
139
140			//récupération de la liste des droits de l'utilisateur si aps super admin
141			if ($this->super_admin != 1) {
142			$liste_droit_acces = $this->getListeDroitAcces();
143
144			if (in_array($droit, $liste_droit_acces)) {
145			//on check si il a le droit de modifier ou supprimer cette page
146			$query = $dbc->query("SELECT * FROM droit_acces_page, liste_droit_acces WHERE
147			droit_acces_page.ID_liste_droit_acces = liste_droit_acces.ID_liste_droit_acces AND
148			droit_acces_page.ID_page = $id_page AND
149			liste_droit_acces.ID_liste_droit_acces = $this->id_liste_droit_acces
150			");
151
152			//si on a un resultat
153			if ((is_array($query)) && (count($query) > 0)) {
154			foreach ($query as $obj) {
155			$this->modif_seo = $obj->seo;
156			$this->modif_contenu = $obj->contenu;
157			$this->modif_navigation = $obj->navigation;
158			$this->supprimer_page = $obj->supprimer;
159			}
160
161			//si les trois sont différent de 0 on renvoit true soinon false
162			if (($this->modif_seo != 0) \|\| ($this->modif_contenu != 0) \|\| ($this->modif_navigation != 0) \|\| ($this->supprimer_page != 0)) {
163			return true;
164			}
165			else {
166			return false;
167			}
168			}
169			}
170			else {
171			return false;
172			}
173			}
174			else {
175			$this->modif_seo = 1;
176			$this->modif_contenu = 1;
177			$this->modif_navigation = 1;
178			$this->supprimer_page = 1;
179
180			return true;
181			}
182			}
183
184			/**
185			* pour savoir si un utilisateur à le droit de supprimer, modifier ou ajouter des trucs
186			* @param $droit_acces
187			* @return bool
188			*/
189			public function getDroitAccesAction($droit_acces) {
190			$liste_droit_acces = $this->getListeDroitAcces();
191
192			if (($this->super_admin == 1) \|\| (in_array($droit_acces, $liste_droit_acces))) {
193			return true;
194			}
195			else {
196			return false;
197			}
198			}
199			//-------------------------- FIN GETTER ----------------------------------------------------------------------------//
200
201
202
203			//-------------------------- SETTER ----------------------------------------------------------------------------//
204
205			//-------------------------- FIN SETTER ----------------------------------------------------------------------------//
206			}

Piou-piou / ribs-framework

Push — master ( 130235...72544b )

DroitAcces::getDroitAccesContenu() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like