Completed
Push — develop ( 93df73...26cf30 )
by Kenji
06:04 queued 04:32
created
application/controllers/Bbs.php 3 patches
Braces   +6 added lines, -12 removed lines patch added patch discarded remove patch
@@ -53,8 +53,7 @@  discard block
 block discarded – undo
53 53
 		{
54 54
 			$config['full_tag_open']  = '<tr><td bgcolor="#EEEEEE">';
55 55
 			$config['full_tag_close'] = '</td></tr>';
56
-		}
57
-		else
56
+		} else
58 57
 		{
59 58
 			$config['full_tag_open']  = '<p class="pagination">';
60 59
 			$config['full_tag_close'] = '</p>';
@@ -87,8 +86,7 @@  discard block
 block discarded – undo
87 86
 		if ($this->form_validation->run() == FALSE)
88 87
 		{
89 88
 			$this->_show_post_page();
90
-		}
91
-		else
89
+		} else
92 90
 		{
93 91
 			$data = [];
94 92
 			$data['name']       = $this->input->post('name');
@@ -152,8 +150,7 @@  discard block
 block discarded – undo
152 150
 		if ($password == '')
153 151
 		{
154 152
 			$this->_load_view('bbs_delete_error');
155
-		}
156
-		else
153
+		} else
157 154
 		{
158 155
 # 記事IDと削除パスワードを条件として、bbsテーブルを検索します。
159 156
 			$this->db->where('id', $id);
@@ -206,8 +203,7 @@  discard block
 block discarded – undo
206 203
 		if ($this->agent->is_mobile())
207 204
 		{
208 205
 			$this->form_validation->set_error_delimiters('<div>', '</div>');
209
-		}
210
-		else
206
+		} else
211 207
 		{
212 208
 			$this->form_validation->set_error_delimiters('<div class="error">', '</div>');
213 209
 		}
@@ -248,8 +244,7 @@  discard block
 block discarded – undo
248 244
 		if ($this->form_validation->run() == FALSE)
249 245
 		{
250 246
 			$this->_show_post_page();
251
-		}
252
-		else
247
+		} else
253 248
 		{
254 249
 # 検証にパスした場合は、送られたデータとIPアドレスをbbsテーブルに登録します。
255 250
 			$data = [];
@@ -314,8 +309,7 @@  discard block
 block discarded – undo
314 309
 		if ($this->agent->is_mobile())
315 310
 		{
316 311
 			$this->load->view($file . '_mobile', $data);
317
-		}
318
-		else
312
+		} else
319 313
 		{
320 314
 			$this->load->view($file, $data);
321 315
 		}
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -311,6 +311,9 @@
 block discarded – undo
311 311
 	}
312 312
 
313 313
 # 携帯端末かどうかを判定し、ビューをロードするプライベートメソッドです。
314
+	/**
315
+	 * @param string $file
316
+	 */
314 317
 	private function _load_view($file, $data = '')
315 318
 	{
316 319
 # 携帯端末の場合は、「_mobile」がファイル名に付くビューファイルをロードします。
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 	{
32 32
 # 引数から$offsetに値が渡されます。これは、3番目のURIセグメントの値です。
33 33
 # ユーザが変更可能なデータですので、int型へ変換し、必ず整数値にします。
34
-		$offset = (int) $offset;
34
+		$offset = (int)$offset;
35 35
 
36 36
 # 新しい記事ID順に、limit値とoffset値を指定し、bbsテーブルから記事データ
37 37
 # (オブジェクト)を取得し、$data['query']に代入します。order_by()メソッドは、
@@ -121,8 +121,8 @@  discard block
 block discarded – undo
121 121
 # メソッドを使い、ランダムな4桁の数字を取得します。
122 122
 		$vals = [
123 123
 					'word'      => random_string('numeric', 4),
124
-					'img_path'  => FCPATH . 'captcha/',
125
-					'img_url'   => base_url() . 'captcha/'
124
+					'img_path'  => FCPATH.'captcha/',
125
+					'img_url'   => base_url().'captcha/'
126 126
 				];
127 127
 		$cap = create_captcha($vals);
128 128
 		$data = [
@@ -149,12 +149,12 @@  discard block
 block discarded – undo
149 149
 	public function delete($id = '')
150 150
 	{
151 151
 # 第1引数、つまり、3番目のURIセグメントのデータをint型に変換します。
152
-		$id = (int) $id;
152
+		$id = (int)$id;
153 153
 # POSTされたpasswordフィールドの値を$passwordに代入します。
154 154
 		$password = $this->input->post('password');
155 155
 # POSTされたdeleteフィールドの値を$deleteに代入します。この値が
156 156
 # 1の場合は、削除を実行します。1以外は、削除の確認ページを表示します。
157
-		$delete = (int) $this->input->post('delete');
157
+		$delete = (int)$this->input->post('delete');
158 158
 
159 159
 # 削除パスワードが入力されていない場合は、エラーページを表示します。
160 160
 		if ($password == '')
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
 # 有効期限を2時間に設定し、それ以前に生成されたキャプチャをデータベースから
287 287
 # 削除します。delete()メソッドの第2引数では、「captcha_time <」を配列のキーに
288 288
 # していますが、このように記述することで、WHERE句の条件の演算子を指定できます。
289
-		$expiration = time() - 7200;	// 有効期限 2時間
289
+		$expiration = time() - 7200; // 有効期限 2時間
290 290
 		$this->db->delete('captcha', ['captcha_time <' => $expiration]);
291 291
 
292 292
 # バリデーション(検証)クラスより引数$strに渡された、ユーザからの入力値がデータ
@@ -324,7 +324,7 @@  discard block
 block discarded – undo
324 324
 # 携帯端末の場合は、「_mobile」がファイル名に付くビューファイルをロードします。
325 325
 		if ($this->agent->is_mobile())
326 326
 		{
327
-			$this->load->view($file . '_mobile', $data);
327
+			$this->load->view($file.'_mobile', $data);
328 328
 		}
329 329
 		else
330 330
 		{
Please login to merge, or discard this patch.
application/controllers/Form.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -122,8 +122,7 @@
 block discarded – undo
122 122
 		if ($this->email->send())
123 123
 		{
124 124
 			return TRUE;
125
-		}
126
-		else
125
+		} else
127 126
 		{
128 127
 //			echo $this->email->print_debugger();
129 128
 			return FALSE;
Please login to merge, or discard this patch.
application/controllers/Shop.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -12,8 +12,8 @@  discard block
 block discarded – undo
12 12
  */
13 13
 class Shop extends MY_Controller {
14 14
 
15
-	public $limit;	// 1ページに表示する商品の数
16
-	public $admin;	// 管理者のメールアドレス
15
+	public $limit; // 1ページに表示する商品の数
16
+	public $admin; // 管理者のメールアドレス
17 17
 
18 18
 	public function __construct()
19 19
 	{
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 
68 68
 # モデルよりそのカテゴリの商品数を取得し、ページネーションを生成します。
69 69
 		$this->load->library('generate_pagination');
70
-		$path  = '/shop/index/' . $cat_id;
70
+		$path  = '/shop/index/'.$cat_id;
71 71
 		$total = $this->inventory_model->get_product_count($cat_id);
72 72
 		$data['pagination'] = $this->generate_pagination->get_links($path, $total, 4);
73 73
 
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 		);
150 150
 
151 151
 # 検索キーワードをクエリ文字列から取得します。
152
-		$q = (string) $this->input->get('q');
152
+		$q = (string)$this->input->get('q');
153 153
 # 全角スペースを半角スペースに変換します。
154 154
 		$q = trim(mb_convert_kana($q, 's'));
155 155
 # 検索キーワードを検証します。
@@ -168,13 +168,13 @@  discard block
 block discarded – undo
168 168
 
169 169
 # ページネーションを生成します。
170 170
 		$this->load->library('generate_pagination');
171
-		$path  = '/shop/search';
171
+		$path = '/shop/search';
172 172
 		$data['pagination'] = $this->generate_pagination->get_links($path, $total, 3);
173 173
 
174 174
 		$data['q'] = $q;
175 175
 		$data['total'] = $total;
176 176
 
177
-		$data['main']   = 'shop_search';
177
+		$data['main'] = 'shop_search';
178 178
 		$data['item_count'] = $this->cart_model->count();
179 179
 		$this->twig->display('shop_tmpl_shop', $data);
180 180
 	}
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -216,8 +216,7 @@  discard block
 block discarded – undo
216 216
 
217 217
 			$data['action'] = '注文内容の確認';
218 218
 			$data['main']   = 'shop_confirm';
219
-		}
220
-		else
219
+		} else
221 220
 		{
222 221
 			$data = [
223 222
 				'action' => 'お客様情報の入力',
@@ -245,8 +244,7 @@  discard block
 block discarded – undo
245 244
 			$this->twig->display('shop_tmpl_checkout', $data);
246 245
 # 注文が完了したので、セッションを破棄します。
247 246
 			$this->session->sess_destroy();
248
-		}
249
-		else
247
+		} else
250 248
 		{
251 249
 			echo 'システムエラー';
252 250
 		}
Please login to merge, or discard this patch.
application/hooks/Convert_encoding.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 		}
26 26
 
27 27
 # 携帯端末からのアクセスを判定するためユーザエージェントクラスをロードします。
28
-		$agent =& load_class('User_agent');
28
+		$agent = & load_class('User_agent');
29 29
 
30 30
 # 携帯端末からの入力文字エンコードを変換します。
31 31
 		if (count($_POST) > 1 && $agent->is_mobile())
@@ -41,13 +41,13 @@  discard block
 block discarded – undo
41 41
 			return;
42 42
 		}
43 43
 
44
-		if (! $this->check_route()) {
44
+		if ( ! $this->check_route()) {
45 45
 			return;
46 46
 		}
47 47
 
48 48
 		// load_class()でuser_agentをロードしたため、$this->user_agentとして
49 49
 		// して代入されているので、それを$this->agentに変更する
50
-		$CI =& get_instance();
50
+		$CI = & get_instance();
51 51
 		$CI->agent = $CI->user_agent;
52 52
 		unset($CI->user_agent);
53 53
 	}
Please login to merge, or discard this patch.
application/libraries/Generate_pagination.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
 	public function get_links($path, $total, $uri_segment)
9 9
 	{
10 10
 # ページネーションクラスをロードします。
11
-		$this->CI =& get_instance();
11
+		$this->CI = & get_instance();
12 12
 		$this->CI->load->library('pagination');
13 13
 
14 14
 		$config = [];
Please login to merge, or discard this patch.
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -5,6 +5,12 @@
 block discarded – undo
5 5
 	private $CI;
6 6
 
7 7
 	// ページネーションの生成
8
+
9
+	/**
10
+	 * @param string $path
11
+	 * @param integer $total
12
+	 * @param integer $uri_segment
13
+	 */
8 14
 	public function get_links($path, $total, $uri_segment)
9 15
 	{
10 16
 # ページネーションクラスをロードします。
Please login to merge, or discard this patch.
application/libraries/Seeder.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 
17 17
 	public function __construct()
18 18
 	{
19
-		$this->CI =& get_instance();
19
+		$this->CI = & get_instance();
20 20
 		$this->CI->load->database();
21 21
 		$this->CI->load->dbforge();
22 22
 		$this->db = $this->CI->db;
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
 	public function call($seeder)
32 32
 	{
33
-		$file = APPPATH . 'database/seeds/' . $seeder . '.php';
33
+		$file = APPPATH.'database/seeds/'.$seeder.'.php';
34 34
 		require_once $file;
35 35
 		$obj = new $seeder;
36 36
 		$obj->run();
Please login to merge, or discard this patch.
application/libraries/validation/Custom_validation.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@
 block discarded – undo
6 6
 
7 7
 	public function __construct()
8 8
 	{
9
-		$this->CI =& get_instance();
9
+		$this->CI = & get_instance();
10 10
 		$this->CI->load->library('form_validation');
11 11
 
12 12
 		$this->set_validation_rules();
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -20,6 +20,9 @@
 block discarded – undo
20 20
 		$this->CI->form_validation->set_error_delimiters($prefix, $suffix);
21 21
 	}
22 22
 
23
+	/**
24
+	 * @param string $field
25
+	 */
23 26
 	protected function set_rules(
24 27
 		$field, $label = '', $rules = array(), $errors = array()
25 28
 	)
Please login to merge, or discard this patch.
application/libraries/validation/Field_validation.php 3 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@
 block discarded – undo
6 6
 
7 7
 	public function __construct()
8 8
 	{
9
-		$this->CI =& get_instance();
9
+		$this->CI = & get_instance();
10 10
 		$this->CI->load->library('form_validation');
11 11
 	}
12 12
 
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,8 +24,7 @@
 block discarded – undo
24 24
 		if ($this->CI->form_validation->run() === FALSE)
25 25
 		{
26 26
 			show_error('不正な入力です。');
27
-		}
28
-		else
27
+		} else
29 28
 		{
30 29
 			return TRUE;
31 30
 		}
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -10,6 +10,9 @@
 block discarded – undo
10 10
 		$this->CI->load->library('form_validation');
11 11
 	}
12 12
 
13
+	/**
14
+	 * @param string $rules
15
+	 */
13 16
 	public function validate($value, $rules, $errors = [])
14 17
 	{
15 18
 		$this->CI->form_validation->reset_validation();
Please login to merge, or discard this patch.
application/models/shop/Cart_model.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -25,13 +25,13 @@  discard block
 block discarded – undo
25 25
 # unset_userdata()メソッドで、セッションデータからその商品を削除します。
26 26
 		if ($qty <= 0)
27 27
 		{
28
-			$this->session->unset_userdata('item' . $id);
28
+			$this->session->unset_userdata('item'.$id);
29 29
 		}
30 30
 # 指定の数量が1以上の場合は、その商品が存在するかチェックした後に、商品と数量を
31 31
 # セッションデータに追加します。セッションの項目名は「item+商品ID」とします。
32 32
 		elseif ($this->inventory_model->is_available_product_item($id))
33 33
 		{
34
-			$this->session->set_userdata('item' . $id, $qty);
34
+			$this->session->set_userdata('item'.$id, $qty);
35 35
 		}
36 36
 	}
37 37
 
@@ -42,9 +42,9 @@  discard block
 block discarded – undo
42 42
 	 */
43 43
 	public function get_all()
44 44
 	{
45
-		$items = [];	// 商品情報の配列
46
-		$total = 0;		// 合計金額
47
-		$line  = 0;		// 行数
45
+		$items = []; // 商品情報の配列
46
+		$total = 0; // 合計金額
47
+		$line  = 0; // 行数
48 48
 
49 49
 # セッションクラスのuserdata()メソッドですべてのセッションデータを取得し、
50 50
 # ループで回して必要な情報を取り出します。
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 			{
56 56
 				$line++;
57 57
 # 配列のキーから商品IDを取り出します。
58
-				$id     = (int) substr($key, 4);
58
+				$id     = (int)substr($key, 4);
59 59
 # get_product_item()メソッドを使い、商品データを取得します。
60 60
 				$item   = $this->inventory_model->get_product_item($id);
61 61
 # 単価に数量を掛けて金額を計算します。
@@ -74,9 +74,9 @@  discard block
 block discarded – undo
74 74
 		}
75 75
 
76 76
 		$cart = [];
77
-		$cart['items'] = $items;	// 商品情報の配列
78
-		$cart['line']  = $line;		// 商品アイテム数
79
-		$cart['total'] = $total;	// 合計金額
77
+		$cart['items'] = $items; // 商品情報の配列
78
+		$cart['line']  = $line; // 商品アイテム数
79
+		$cart['total'] = $total; // 合計金額
80 80
 
81 81
 		return $cart;
82 82
 	}
Please login to merge, or discard this patch.