View GitHub Repository ribs-module-blog

Schedule Inspection
Subscribe
Completed
Push — master ( 2a9184...b88b55 )
by Anthony
02:01
created

AdminCategory::setCategoriesArticle()   B

↳ Parent: AdminCategory

Complexity

Conditions 5
Paths 4

Size

Total Lines 12
Code Lines 7

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
c 0
b 0
f 0
dl 0
loc 12
rs 8.8571
cc 5
eloc 7
nc 4
nop 2
1 
<?php
2 
	namespace modules\blog\admin\controller;
3
4
5 
	use core\App;
6
7 
	class AdminCategory {
8
9
10 
		//-------------------------- BUILDER ----------------------------------------------------------------------------//
11 
		public function __construct() {
12
13 
		}
14 
		//-------------------------- END BUILDER ----------------------------------------------------------------------------//
15
16
17 
		//-------------------------- GETTER ----------------------------------------------------------------------------//
18 
		/**
19 
		 * @param $name
20 
		 * @return bool
21 
		 * test if a category exist
22 
		 */
23 
		private function getTestCategoryExist($name) {
24 
			$dbc = App::getDb();
25
26 
			$query = $dbc->select()->from("_blog_category")->where("category", "=", $name)->get();
27
28 
			if (count($query) > 0) {
29 
				foreach ($query as $obj) {
30
31 
					return $obj->ID_category;
32 
				}
33 
			}
34
35 
			return false;
36 
		}
37 
		//-------------------------- END GETTER ----------------------------------------------------------------------------//
38
39
40 
		//-------------------------- SETTER ----------------------------------------------------------------------------//
41 
		/**
42 
		 * @param $categories
43 
		 * @param $id_article
44 
		 * add list of categories to an article
45 
		 */
46 
		public function setCategoriesArticle($categories, $id_article) {
47 
			$dbc = App::getDb();
48
49 
			$count = count($categories);
50 
			if ((is_array($categories)) && ($count > 0)) {
51 
				for ($i=0 ; $i<$count ; $i++) {
52 
					if ($this->getTestCategoryExist($categories[$i]) != false) {
0 ignored issues
– show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison !== instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
53 
						$dbc->insert("ID_category", $this->getTestCategoryExist($categories[$i]))->insert("ID_article", $id_article)->into("_blog_article_category")->set();
54 
					}
55 
				}
56 
			}
57 
		}
58 
		//-------------------------- END SETTER ----------------------------------------------------------------------------//
59 
	}