Marche::setCommencerTransport() - Code Metrics - Inspection of "hotfix in flash message" - Piou-piou/Ruined-World - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 341095...d27ad8 )

by Anthony

created 2016-11-28 11:35 UTC

Marche::setCommencerTransport() B

↳ Parent: Marche

Complexity

Conditions	5
Paths	6

Size

Total Lines	59
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	59
rs	8.7117
cc	5
eloc	37
nc	6
nop	7

How to fix Long Method

<?php
	namespace modules\bataille\app\controller;
	
	
	use core\App;
	use core\HTML\flashmessage\FlashMessage;

	class Marche {
		private $id_base_dest;
		private $aller;
		private $ressources;
		private $date_arrivee;
		private $nom_base;
		private $id_marche_transport;
		private $duree_restante_trajet;

		//-------------------------- BUILDER ----------------------------------------------------------------------------//
		public function __construct() {
			$dbc = App::getDb();

			//récupération des trajets en cours
			$query = $dbc->select()->from("_bataille_marche_transport")
				->from("_bataille_base")
				->where("_bataille_marche_transport.ID_base", "=", Bataille::getIdBase(), "AND")
				->where("_bataille_marche_transport.ID_base_dest", "=", "_bataille_base.ID_base", "", true)
				->get();

			if ((is_array($query)) && (count($query) > 0)) {
				foreach ($query as $obj) {
					$this->id_base_dest = $obj->ID_base_dest;
					$this->aller = $obj->aller;
					$this->ressources = $obj->ressources;
					$this->nom_base = $obj->nom_base;
					$this->date_arrivee = $obj->date_arrivee;
					$this->id_marche_transport = $obj->ID_marche_transport;

					$marche = $this->getTransportArrive();
				}

				Bataille::setValues(["marche" => $marche]);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
			}
		}
		//-------------------------- END BUILDER ----------------------------------------------------------------------------//



		//-------------------------- GETTER ----------------------------------------------------------------------------//
		/**
		 * fonction qui permet de savoir si un transport est arrivé à la base de destination
		 * +si arrivé appel la fonction pour ajouter les ressources et passe le trajet sur le retour
		 * ou en trajet fini suivant le temps de la date d'aujourd'hui et la date à laquelle le
		 * trajet aurait du revenir
		 */
		private function getTransportArrive() {
			$today = Bataille::getToday();

			echo "today : ".$today." ++ date arrivee : ";
			echo $this->date_arrivee."<br>";

			//on test si déja arrivé à destination
			if (($this->aller == 1) && (($this->date_arrivee-$today) <= 0)) {
				$this->setLivrerRessource();

				//on calcul la date d'arrivée du retour
				$date_retour = Bataille::getDureeTrajet($this->id_base_dest)+$this->date_arrivee;

				//si le retour du trajet est également arrivé on finit le transport sinon on le place sur le retour
				if ($date_retour < $today) {
					$this->setTerminerTransport();
				}
				else {
					$this->setTrajetRetour($date_retour);
					$this->duree_restante_trajet = $date_retour-$today;
					$set_array = true;
				}
			}
			else if (($this->aller == 0) && (($this->date_arrivee-$today) <= 0)) {
				$this->setTerminerTransport();
			}
			else {
				$this->duree_restante_trajet = $this->date_arrivee-$today;
				$set_array = true;
			}

			if ($set_array === true) {
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
				if ($this->aller == 1) {
					$marche["aller"][] = [
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
						"id_marche_transport" => $this->id_marche_transport,
						"date_arrivee" => $this->duree_restante_trajet,
						"nom_base_dest" => $this->nom_base
					];
				}
				else {
					$marche["retour"][] = [
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
						"id_marche_transport" => $this->id_marche_transport,
						"date_arrivee" => $this->duree_restante_trajet,
						"nom_base_dest" => $this->nom_base
					];
				}

				return $marche;
			}
		}

		/**
		 * @param $all_ressource
		 * @return bool
		 * fonction qui renvoi true si on a assez de marchand pour ce trajet dans la base
		 * sinon on renvoi false
		 */
		private function getAssezMarchand($all_ressource) {
			$dbc = App::getDb();

			//récupération du nombre max du marchand dispo dans la base
			$nombre_max_marchand = Bataille::getBatiment()->getNiveauBatiment("marche");

			//on récupère tous les marchands qui sont en transport
			$query = $dbc->select("nb_marchand")->from("_bataille_marche_transport")
				->where("ID_base", "=", Bataille::getIdBase(), "OR")
				->where("ID_base_dest", "=", Bataille::getIdBase())
				->get();

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

			//on a le nombre de marchand dispo dans la base
			$nombre_marchand_dispo = $nombre_max_marchand-$marchand_transport;

			//on calcul savoir si on en a assez pour transport toutes les ressoures
			//il faut 1 marchand pour 1000 ressource
			$nombre_marchand_trajet = ceil($all_ressource/1000);

			//si on a assez de marchand on revoi true sinon false
			if ($nombre_marchand_dispo >= $nombre_marchand_trajet) {
				return $nombre_marchand_trajet;
			}

			return false;
		}
		//-------------------------- END GETTER ----------------------------------------------------------------------------//



		//-------------------------- SETTER ----------------------------------------------------------------------------//
		/**
		 * fonction qui permet d'ajouter les ressources à la base destinatrice du transport
		 */
		private function setLivrerRessource() {
			$ressource = new Ressource($this->id_base_dest);

			$ressource_transport = unserialize($this->ressources);

			$ressource->setUpdateRessource($ressource_transport['eau'], $ressource_transport['electricite'], $ressource_transport['fer'], $ressource_transport['fuel'], $ressource_transport['nourriture'], "+");
		}

		/**
		 * fonction qui place le trajet en retour
		 */
		private function setTrajetRetour($date_retour) {
			$dbc = App::getDb();

			$dbc->update("ressources", 0)
				->update("aller", 0)
				->update("date_arrivee", $date_retour)
				->from("_bataille_marche_transport")
				->where("ID_marche_transport", "=", $this->id_marche_transport)
				->set();

			$this->aller = 0;
		}

		/**
		 * permet de terminer totallement un transport
		 */
		private function setTerminerTransport() {
			$dbc = App::getDb();

			$dbc->delete()->from("_bataille_marche_transport")->where("ID_marche_transport", "=", $this->id_marche_transport)->del();
		}

		/**
		 * @param $eau
		 * @param $electricite
		 * @param $fer
		 * @param $fuel
		 * @param $nourriture
		 * @param $posx
		 * @param $posy
		 * @return bool
		 * Fonction qui permet d'initialiser un transport de ressources d'une base à une autre
		 */
		public function setCommencerTransport($eau, $electricite, $fer, $fuel, $nourriture, $posx, $posy) {
			$dbc = App::getDb();
			$id_base_dest = Bataille::getBaseExistPosition($posx, $posy);

			if ($id_base_dest != 0) {
				$ressource["eau"] = Bataille::getTestAssezRessourceBase("eau", $eau);
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
				$ressource["electricite"] = Bataille::getTestAssezRessourceBase("electricite", $electricite);
				$ressource["fer"] = Bataille::getTestAssezRessourceBase("fer", $fer);
				$ressource["fuel"] = Bataille::getTestAssezRessourceBase("fuel", $fuel);
				$ressource["nourriture"] = Bataille::getTestAssezRessourceBase("nourriture", $nourriture);

				//si pas assez de ressources dispo dans la base pour l'envoi on renvoi erreur
				foreach ($ressource as $tab) {
					if (in_array("rouge", $tab)) {
						FlashMessage::setFlash("Vous n'avez pas autant de ressources disponibles à l'envoi");
						return false;
					};
				}

				//on check si assez marchand dans la base, si pas assez on return false
				$nb_marchand = $this->getAssezMarchand($eau+$electricite+$fer+$fuel+$nourriture);

				if ($nb_marchand === false) {
					FlashMessage::setFlash("Vous n'avez pas assez de marchans disponibles pour effectuer ce trajet");
					return false;
				}

				//sinon initialise le transport
				//on recup la date d'arrivee dans la base de destintation
				$date_arrivee = Bataille::getDureeTrajet($id_base_dest)+Bataille::getToday();

				$ressource = [
					"eau" => $eau,
					"electricite" => $electricite,
					"fer" => $fer,
					"fuel" => $fuel,
					"nourriture" => $nourriture,
				];

				//on insert le transport dans la table
				$dbc->insert("date_arrivee", $date_arrivee)
					->insert("ressources", serialize($ressource))
					->insert("aller", 1)
					->insert("nb_marchand", $nb_marchand)
					->insert("ID_base_dest", $id_base_dest)
					->insert("ID_base", Bataille::getIdBase())
					->into("_bataille_marche_transport")
					->set();

				//on retire les ressources de la base
				Bataille::getRessource()->setUpdateRessource($eau, $electricite, $fer, $fuel, $nourriture, "-");

				FlashMessage::setFlash("Votre transport vient de partir !", "info");
				return true;
			}

			FlashMessage::setFlash("Aucune base présente aux coordonnées données");
			return false;
		}
		//-------------------------- END SETTER ----------------------------------------------------------------------------//
	}

Push — master ( 341095...d27ad8 )

Marche::setCommencerTransport() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

1			<?php
2			namespace modules\bataille\app\controller;
3
4
5			use core\App;
6			use core\HTML\flashmessage\FlashMessage;
7
8			class Marche {
9			private $id_base_dest;
10			private $aller;
11			private $ressources;
12			private $date_arrivee;
13			private $nom_base;
14			private $id_marche_transport;
15			private $duree_restante_trajet;
16
17			//-------------------------- BUILDER ----------------------------------------------------------------------------//
18			public function __construct() {
19			$dbc = App::getDb();
20
21			//récupération des trajets en cours
22			$query = $dbc->select()->from("_bataille_marche_transport")
23			->from("_bataille_base")
24			->where("_bataille_marche_transport.ID_base", "=", Bataille::getIdBase(), "AND")
25			->where("_bataille_marche_transport.ID_base_dest", "=", "_bataille_base.ID_base", "", true)
26			->get();
27
28			if ((is_array($query)) && (count($query) > 0)) {
29			foreach ($query as $obj) {
30			$this->id_base_dest = $obj->ID_base_dest;
31			$this->aller = $obj->aller;
32			$this->ressources = $obj->ressources;
33			$this->nom_base = $obj->nom_base;
34			$this->date_arrivee = $obj->date_arrivee;
35			$this->id_marche_transport = $obj->ID_marche_transport;
36
37			$marche = $this->getTransportArrive();
38			}
39
40			Bataille::setValues(["marche" => $marche]);
			0 ignored issues – show Bug introduced 2016-11-27 13:02 UTC by Report Bug Copy Issue Report The variable `$marche` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
41			}
42			}
43			//-------------------------- END BUILDER ----------------------------------------------------------------------------//
44
45
46
47			//-------------------------- GETTER ----------------------------------------------------------------------------//
48			/**
49			* fonction qui permet de savoir si un transport est arrivé à la base de destination
50			* +si arrivé appel la fonction pour ajouter les ressources et passe le trajet sur le retour
51			* ou en trajet fini suivant le temps de la date d'aujourd'hui et la date à laquelle le
52			* trajet aurait du revenir
53			*/
54			private function getTransportArrive() {
55			$today = Bataille::getToday();
56
57			echo "today : ".$today." ++ date arrivee : ";
58			echo $this->date_arrivee."<br>";
59
60			//on test si déja arrivé à destination
61			if (($this->aller == 1) && (($this->date_arrivee-$today) <= 0)) {
62			$this->setLivrerRessource();
63
64			//on calcul la date d'arrivée du retour
65			$date_retour = Bataille::getDureeTrajet($this->id_base_dest)+$this->date_arrivee;
66
67			//si le retour du trajet est également arrivé on finit le transport sinon on le place sur le retour
68			if ($date_retour < $today) {
69			$this->setTerminerTransport();
70			}
71			else {
72			$this->setTrajetRetour($date_retour);
73			$this->duree_restante_trajet = $date_retour-$today;
74			$set_array = true;
75			}
76			}
77			else if (($this->aller == 0) && (($this->date_arrivee-$today) <= 0)) {
78			$this->setTerminerTransport();
79			}
80			else {
81			$this->duree_restante_trajet = $this->date_arrivee-$today;
82			$set_array = true;
83			}
84
85			if ($set_array === true) {
			0 ignored issues – show Bug introduced 2016-11-27 13:02 UTC by Report Bug Copy Issue Report The variable `$set_array` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
86			if ($this->aller == 1) {
87			$marche["aller"][] = [
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-11-27 10:16 UTC by Report Bug Copy Issue Report `$marche` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$marche = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
88			"id_marche_transport" => $this->id_marche_transport,
89			"date_arrivee" => $this->duree_restante_trajet,
90			"nom_base_dest" => $this->nom_base
91			];
92			}
93			else {
94			$marche["retour"][] = [
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-11-27 13:02 UTC by Report Bug Copy Issue Report `$marche` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$marche = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
95			"id_marche_transport" => $this->id_marche_transport,
96			"date_arrivee" => $this->duree_restante_trajet,
97			"nom_base_dest" => $this->nom_base
98			];
99			}
100
101			return $marche;
102			}
103			}
104
105			/**
106			* @param $all_ressource
107			* @return bool
108			* fonction qui renvoi true si on a assez de marchand pour ce trajet dans la base
109			* sinon on renvoi false
110			*/
111			private function getAssezMarchand($all_ressource) {
112			$dbc = App::getDb();
113
114			//récupération du nombre max du marchand dispo dans la base
115			$nombre_max_marchand = Bataille::getBatiment()->getNiveauBatiment("marche");
116
117			//on récupère tous les marchands qui sont en transport
118			$query = $dbc->select("nb_marchand")->from("_bataille_marche_transport")
119			->where("ID_base", "=", Bataille::getIdBase(), "OR")
120			->where("ID_base_dest", "=", Bataille::getIdBase())
121			->get();
122
123			$marchand_transport = 0;
124			if ((is_array($query)) && (count($query) > 0)) {
125			foreach ($query as $obj) {
126			$marchand_transport += $obj->nb_marchand;
127			}
128			}
129
130			//on a le nombre de marchand dispo dans la base
131			$nombre_marchand_dispo = $nombre_max_marchand-$marchand_transport;
132
133			//on calcul savoir si on en a assez pour transport toutes les ressoures
134			//il faut 1 marchand pour 1000 ressource
135			$nombre_marchand_trajet = ceil($all_ressource/1000);
136
137			//si on a assez de marchand on revoi true sinon false
138			if ($nombre_marchand_dispo >= $nombre_marchand_trajet) {
139			return $nombre_marchand_trajet;
140			}
141
142			return false;
143			}
144			//-------------------------- END GETTER ----------------------------------------------------------------------------//
145
146
147
148			//-------------------------- SETTER ----------------------------------------------------------------------------//
149			/**
150			* fonction qui permet d'ajouter les ressources à la base destinatrice du transport
151			*/
152			private function setLivrerRessource() {
153			$ressource = new Ressource($this->id_base_dest);
154
155			$ressource_transport = unserialize($this->ressources);
156
157			$ressource->setUpdateRessource($ressource_transport['eau'], $ressource_transport['electricite'], $ressource_transport['fer'], $ressource_transport['fuel'], $ressource_transport['nourriture'], "+");
158			}
159
160			/**
161			* fonction qui place le trajet en retour
162			*/
163			private function setTrajetRetour($date_retour) {
164			$dbc = App::getDb();
165
166			$dbc->update("ressources", 0)
167			->update("aller", 0)
168			->update("date_arrivee", $date_retour)
169			->from("_bataille_marche_transport")
170			->where("ID_marche_transport", "=", $this->id_marche_transport)
171			->set();
172
173			$this->aller = 0;
174			}
175
176			/**
177			* permet de terminer totallement un transport
178			*/
179			private function setTerminerTransport() {
180			$dbc = App::getDb();
181
182			$dbc->delete()->from("_bataille_marche_transport")->where("ID_marche_transport", "=", $this->id_marche_transport)->del();
183			}
184
185			/**
186			* @param $eau
187			* @param $electricite
188			* @param $fer
189			* @param $fuel
190			* @param $nourriture
191			* @param $posx
192			* @param $posy
193			* @return bool
194			* Fonction qui permet d'initialiser un transport de ressources d'une base à une autre
195			*/
196			public function setCommencerTransport($eau, $electricite, $fer, $fuel, $nourriture, $posx, $posy) {
197			$dbc = App::getDb();
198			$id_base_dest = Bataille::getBaseExistPosition($posx, $posy);
199
200			if ($id_base_dest != 0) {
201			$ressource["eau"] = Bataille::getTestAssezRessourceBase("eau", $eau);
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-11-27 16:40 UTC by Report Bug Copy Issue Report `$ressource` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$ressource = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
202			$ressource["electricite"] = Bataille::getTestAssezRessourceBase("electricite", $electricite);
203			$ressource["fer"] = Bataille::getTestAssezRessourceBase("fer", $fer);
204			$ressource["fuel"] = Bataille::getTestAssezRessourceBase("fuel", $fuel);
205			$ressource["nourriture"] = Bataille::getTestAssezRessourceBase("nourriture", $nourriture);
206
207			//si pas assez de ressources dispo dans la base pour l'envoi on renvoi erreur
208			foreach ($ressource as $tab) {
209			if (in_array("rouge", $tab)) {
210			FlashMessage::setFlash("Vous n'avez pas autant de ressources disponibles à l'envoi");
211			return false;
212			};
213			}
214
215			//on check si assez marchand dans la base, si pas assez on return false
216			$nb_marchand = $this->getAssezMarchand($eau+$electricite+$fer+$fuel+$nourriture);
217
218			if ($nb_marchand === false) {
219			FlashMessage::setFlash("Vous n'avez pas assez de marchans disponibles pour effectuer ce trajet");
220			return false;
221			}
222
223			//sinon initialise le transport
224			//on recup la date d'arrivee dans la base de destintation
225			$date_arrivee = Bataille::getDureeTrajet($id_base_dest)+Bataille::getToday();
226
227			$ressource = [
228			"eau" => $eau,
229			"electricite" => $electricite,
230			"fer" => $fer,
231			"fuel" => $fuel,
232			"nourriture" => $nourriture,
233			];
234
235			//on insert le transport dans la table
236			$dbc->insert("date_arrivee", $date_arrivee)
237			->insert("ressources", serialize($ressource))
238			->insert("aller", 1)
239			->insert("nb_marchand", $nb_marchand)
240			->insert("ID_base_dest", $id_base_dest)
241			->insert("ID_base", Bataille::getIdBase())
242			->into("_bataille_marche_transport")
243			->set();
244
245			//on retire les ressources de la base
246			Bataille::getRessource()->setUpdateRessource($eau, $electricite, $fer, $fuel, $nourriture, "-");
247
248			FlashMessage::setFlash("Votre transport vient de partir !", "info");
249			return true;
250			}
251
252			FlashMessage::setFlash("Aucune base présente aux coordonnées données");
253			return false;
254			}
255			//-------------------------- END SETTER ----------------------------------------------------------------------------//
256			}

Piou-piou / Ruined-World

Push — master ( 341095...d27ad8 )

Marche::setCommencerTransport() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like