Completed
Push — #12 ( cef8fd...5132ea )
by Erwan
02:09
created
acp/main_module.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -27,18 +27,18 @@  discard block
 block discarded – undo
27 27
 	{
28 28
 		global $phpbb_container, $db, $user, $phpbb_log, $template, $cache, $request, $table_prefix;
29 29
 
30
-		$this->qte						= $phpbb_container->get('ernadoo.qte');
31
-		$this->migrator_tool_permission	= $phpbb_container->get('migrator.tool.permission');
30
+		$this->qte = $phpbb_container->get('ernadoo.qte');
31
+		$this->migrator_tool_permission = $phpbb_container->get('migrator.tool.permission');
32 32
 
33 33
 		$action			= $request->variable('action', '');
34 34
 		$submit			= $request->is_set_post('submit');
35 35
 		$attr_id		= $request->variable('id', 0);
36
-		$attr_auth_id	= $request->variable('attr_auth_id', 0);
36
+		$attr_auth_id = $request->variable('attr_auth_id', 0);
37 37
 
38 38
 		$error = array();
39 39
 		$clear_dest_perms = false;
40 40
 
41
-		$this->tpl_name		= 'acp_attributes';
41
+		$this->tpl_name = 'acp_attributes';
42 42
 		$this->page_title	= 'QTE_MANAGE_TITLE';
43 43
 
44 44
 		$user->add_lang_ext('ernadoo/qte', array('attributes', 'attributes_acp'));
@@ -156,14 +156,14 @@  discard block
 block discarded – undo
156 156
 							$db->sql_query($sql);
157 157
 							$attr_id = $db->sql_nextid();
158 158
 
159
-							$this->migrator_tool_permission->add('f_qte_attr_'.$attr_id);
159
+							$this->migrator_tool_permission->add('f_qte_attr_' . $attr_id);
160 160
 
161 161
 							$message = 'ADDED';
162 162
 						}
163 163
 
164 164
 						if ($attr_auth_id)
165 165
 						{
166
-							$this->_copy_permission('f_qte_attr_'.$attr_id, 'f_qte_attr_'.$attr_auth_id, $clear_dest_perms);
166
+							$this->_copy_permission('f_qte_attr_' . $attr_id, 'f_qte_attr_' . $attr_auth_id, $clear_dest_perms);
167 167
 						}
168 168
 
169 169
 						$cache->destroy('_attr');
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
 
272 272
 					$phpbb_log->add('admin', $user->data['user_id'], $user->ip, 'LOG_ATTRIBUTE_REMOVED', time(), array($attr_name));
273 273
 
274
-					$this->migrator_tool_permission->remove('f_qte_attr_'.$attr_id, false);
274
+					$this->migrator_tool_permission->remove('f_qte_attr_' . $attr_id, false);
275 275
 
276 276
 					$sql = 'DELETE FROM ' . $table_prefix . 'topics_attr
277 277
 						WHERE attr_id = ' . (int) $attr_id;
Please login to merge, or discard this patch.
event/acp_listener.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -36,11 +36,11 @@  discard block
 block discarded – undo
36 36
 
37 37
 	public function __construct(\phpbb\request\request $request, \phpbb\cache\driver\driver_interface $cache, \phpbb\db\driver\driver_interface $db, \phpbb\user $user, \ernadoo\qte\qte $qte, $table_prefix)
38 38
 	{
39
-		$this->request	= $request;
39
+		$this->request = $request;
40 40
 		$this->cache	= $cache;
41
-		$this->db		= $db;
41
+		$this->db = $db;
42 42
 		$this->user		= $user;
43
-		$this->qte		= $qte;
43
+		$this->qte = $qte;
44 44
 
45 45
 		$this->table_prefix = $table_prefix;
46 46
 	}
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	{
66 66
 		$categories				= $event['categories'];
67 67
 		$categories				= array_merge($categories, array('qte' => 'ACL_CAT_QTE'));
68
-		$event['categories']	= $categories;
68
+		$event['categories'] = $categories;
69 69
 
70 70
 		$permissions = $event['permissions'];
71 71
 
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 		while ($row = $this->db->sql_fetchrow($result))
83 83
 		{
84 84
 			$permissions += array(
85
-				'f_qte_attr_'.$row['attr_id'] => array('lang' => $this->user->lang('QTE_CAN_USE_ATTR', $row['attr_name']), 'cat' => 'qte'),
85
+				'f_qte_attr_' . $row['attr_id'] => array('lang' => $this->user->lang('QTE_CAN_USE_ATTR', $row['attr_name']), 'cat' => 'qte'),
86 86
 			);
87 87
 		}
88 88
 		$event['permissions'] = $permissions;
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
 		$dest_forum_ids = array();
230 230
 		while ($row = $this->db->sql_fetchrow($result))
231 231
 		{
232
-			$dest_forum_ids[]	= (int) $row['forum_id'];
232
+			$dest_forum_ids[] = (int) $row['forum_id'];
233 233
 		}
234 234
 		$this->db->sql_freeresult($result);
235 235
 
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
 		$acl_options_ids = array();
248 248
 		while ($row = $this->db->sql_fetchrow($result))
249 249
 		{
250
-			$acl_options_ids[]	= (int) $row['auth_option_id'];
250
+			$acl_options_ids[] = (int) $row['auth_option_id'];
251 251
 		}
252 252
 
253 253
 		// From the mysql documentation:
Please login to merge, or discard this patch.
event/search_listener.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -30,9 +30,9 @@  discard block
 block discarded – undo
30 30
 
31 31
 	public function __construct(\phpbb\request\request $request, \ernadoo\qte\qte $qte, \ernadoo\qte\search\fulltext_attribute $qte_search)
32 32
 	{
33
-		$this->request		= $request;
34
-		$this->qte			= $qte;
35
-		$this->qte_search	= $qte_search;
33
+		$this->request = $request;
34
+		$this->qte = $qte;
35
+		$this->qte_search = $qte_search;
36 36
 	}
37 37
 
38 38
 	static public function getSubscribedEvents()
@@ -119,8 +119,8 @@  discard block
 block discarded – undo
119 119
 	{
120 120
 		if ($this->searc_attr)
121 121
 		{
122
-			$keywords	= utf8_normalize_nfc($this->request->variable('keywords', '', true));
123
-			$author		= $this->request->variable('author', '', true);
122
+			$keywords = utf8_normalize_nfc($this->request->variable('keywords', '', true));
123
+			$author = $this->request->variable('author', '', true);
124 124
 
125 125
 			if (!$keywords && !$author)
126 126
 			{
Please login to merge, or discard this patch.
event/main_listener.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -33,9 +33,9 @@  discard block
 block discarded – undo
33 33
 
34 34
 	public function __construct(\phpbb\request\request $request, \phpbb\template\template $template, \phpbb\user $user, \phpbb\log\log $log, \ernadoo\qte\qte $qte)
35 35
 	{
36
-		$this->request	= $request;
37
-		$this->template	= $template;
38
-		$this->user		= $user;
36
+		$this->request = $request;
37
+		$this->template = $template;
38
+		$this->user = $user;
39 39
 		$this->log		= $log;
40 40
 		$this->qte		= $qte;
41 41
 	}
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 
120 120
 				if ($topic_attribute != \ernadoo\qte\qte::KEEP)
121 121
 				{
122
-					$post_data['topic_attr_id']		= (int) $topic_attribute;
122
+					$post_data['topic_attr_id'] = (int) $topic_attribute;
123 123
 					$post_data['topic_attr_user']	= (int) $this->user->data['user_id'];
124 124
 					$post_data['topic_attr_time']	= time();
125 125
 
@@ -142,11 +142,11 @@  discard block
 block discarded – undo
142 142
 
143 143
 		if ($event['post_data']['force_attr'])
144 144
 		{
145
-			if ($post_data['attr_id'] == \ernadoo\qte\qte::REMOVE && ($event['mode'] == 'post' || ($event['mode'] == 'edit' && $event['post_data']['topic_first_post_id'] == $event['post_id'])) )
145
+			if ($post_data['attr_id'] == \ernadoo\qte\qte::REMOVE && ($event['mode'] == 'post' || ($event['mode'] == 'edit' && $event['post_data']['topic_first_post_id'] == $event['post_id'])))
146 146
 			{
147 147
 				$error = $event['error'];
148 148
 				$error[] = $this->user->lang['QTE_ATTRIBUTE_UNSELECTED'];
149
-				$event['error'] = $error ;
149
+				$event['error'] = $error;
150 150
 
151 151
 				// init the value
152 152
 				$post_data['attr_id'] = 0;
Please login to merge, or discard this patch.
event/mcp_listener.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,8 +30,8 @@  discard block
 block discarded – undo
30 30
 	*/
31 31
 	public function __construct(\phpbb\request\request $request, \ernadoo\qte\qte $qte)
32 32
 	{
33
-		$this->request	= $request;
34
-		$this->qte		= $qte;
33
+		$this->request = $request;
34
+		$this->qte = $qte;
35 35
 	}
36 36
 
37 37
 	static public function getSubscribedEvents()
@@ -56,8 +56,8 @@  discard block
 block discarded – undo
56 56
 
57 57
 	public function mcp_select_assign_attributes($event)
58 58
 	{
59
-		$attr_id	= (int) $this->request->variable('attr_id', 0);
60
-		$forum_id	= (int) $event['forum_info']['forum_id'];
59
+		$attr_id = (int) $this->request->variable('attr_id', 0);
60
+		$forum_id = (int) $event['forum_info']['forum_id'];
61 61
 
62 62
 		if ($attr_id)
63 63
 		{
Please login to merge, or discard this patch.
qte.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -69,16 +69,16 @@  discard block
 block discarded – undo
69 69
 	*/
70 70
 	public function __construct(\phpbb\request\request $request, \phpbb\cache\driver\driver_interface $cache, \phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\user $user, \phpbb\log\log $log, \phpbb\auth\auth $auth, $root_path, $php_ext, $table_prefix)
71 71
 	{
72
-		$this->request		= $request;
72
+		$this->request = $request;
73 73
 		$this->cache		= $cache;
74
-		$this->db			= $db;
75
-		$this->template		= $template;
74
+		$this->db = $db;
75
+		$this->template = $template;
76 76
 		$this->user			= $user;
77
-		$this->log			= $log;
77
+		$this->log = $log;
78 78
 		$this->auth			= $auth;
79 79
 
80
-		$this->root_path	= $root_path;
81
-		$this->php_ext		= $php_ext;
80
+		$this->root_path = $root_path;
81
+		$this->php_ext = $php_ext;
82 82
 		$this->table_prefix = $table_prefix;
83 83
 
84 84
 		$this->_get_attributes();
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 					AND t.topic_attr_user <> ' . ANONYMOUS;
104 104
 			$result = $this->db->sql_query($sql);
105 105
 
106
-			while($row = $this->db->sql_fetchrow($result))
106
+			while ($row = $this->db->sql_fetchrow($result))
107 107
 			{
108 108
 				$this->_name[$row['user_id']] = array(
109 109
 					'user_id'		=> (int) $row['user_id'],
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 				WHERE user_id = ' . (int) $user_id;
144 144
 			$result = $this->db->sql_query($sql);
145 145
 
146
-			while($row = $this->db->sql_fetchrow($result) )
146
+			while ($row = $this->db->sql_fetchrow($result))
147 147
 			{
148 148
 				$this->_name[$row['user_id']] = array(
149 149
 					'user_id'		=> (int) $row['user_id'],
@@ -168,10 +168,10 @@  discard block
 block discarded – undo
168 168
 	public function attr_select($forum_id, $author_id = 0, $attribute_id = 0, $viewtopic_url = '')
169 169
 	{
170 170
 		$show_select	= false;
171
-		$current_time	= time();
172
-		$can_edit		= $this->auth->acl_get('m_qte_attr_edit', $forum_id);
171
+		$current_time = time();
172
+		$can_edit = $this->auth->acl_get('m_qte_attr_edit', $forum_id);
173 173
 		$can_remove		= $this->auth->acl_get('m_qte_attr_del', $forum_id);
174
-		$is_author		= $this->user->data['is_registered'] && $this->user->data['user_id'] == $author_id;
174
+		$is_author = $this->user->data['is_registered'] && $this->user->data['user_id'] == $author_id;
175 175
 
176 176
 		// Basic auth
177 177
 		if (!$can_remove && !$can_edit && !$is_author)
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
 
182 182
 		foreach ($this->_attr as $attr)
183 183
 		{
184
-			if (!$this->auth->acl_get('f_qte_attr_'.$attr['attr_id'], $forum_id))
184
+			if (!$this->auth->acl_get('f_qte_attr_' . $attr['attr_id'], $forum_id))
185 185
 			{
186 186
 				continue;
187 187
 			}
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
 	{
280 280
 		foreach ($this->_attr as $attr)
281 281
 		{
282
-			$forum_allowed = $this->auth->acl_getf('f_qte_attr_'.$attr['attr_id'], true);
282
+			$forum_allowed = $this->auth->acl_getf('f_qte_attr_' . $attr['attr_id'], true);
283 283
 
284 284
 			if (isset($forum_allowed[$forum_id]))
285 285
 			{
@@ -390,8 +390,8 @@  discard block
 block discarded – undo
390 390
 			return;
391 391
 		}
392 392
 
393
-		$can_edit		= $this->auth->acl_get('m_qte_attr_edit', $forum_id) || $this->auth->acl_get('f_qte_attr_'.$attribute_id, $forum_id) && $this->user->data['is_registered'] && $this->user->data['user_id'] == $author_id;
394
-		$can_remove		= $this->auth->acl_get('m_qte_attr_del', $forum_id);
393
+		$can_edit = $this->auth->acl_get('m_qte_attr_edit', $forum_id) || $this->auth->acl_get('f_qte_attr_' . $attribute_id, $forum_id) && $this->user->data['is_registered'] && $this->user->data['user_id'] == $author_id;
394
+		$can_remove = $this->auth->acl_get('m_qte_attr_del', $forum_id);
395 395
 
396 396
 		if (!$can_edit && $attribute_id != self::REMOVE || !$can_remove && $attribute_id == self::REMOVE)
397 397
 		{
@@ -466,8 +466,8 @@  discard block
 block discarded – undo
466 466
 	*/
467 467
 	public function mcp_attr_apply($attribute_id = 0, $forum_id = 0, $topic_ids = array())
468 468
 	{
469
-		$can_edit		= $this->auth->acl_get('m_qte_attr_edit', $forum_id);
470
-		$can_remove		= $this->auth->acl_get('m_qte_attr_del', $forum_id);
469
+		$can_edit = $this->auth->acl_get('m_qte_attr_edit', $forum_id);
470
+		$can_remove = $this->auth->acl_get('m_qte_attr_del', $forum_id);
471 471
 
472 472
 		if (!$can_edit && $attribute_id != self::REMOVE || !$can_remove && $attribute_id == self::REMOVE)
473 473
 		{
@@ -578,7 +578,7 @@  discard block
 block discarded – undo
578 578
 			$a_class_name = preg_replace('#[^a-z0-9 _-]#', '', strtolower($a_name));
579 579
 		}
580 580
 
581
-		return ' class="qte-attr ' . (isset($a_class_name) ?  $a_class_name : '') . '"' . (!empty($a_colour) ? ' style="color:#' . $a_colour . '; font-weight:bold;"' : '');
581
+		return ' class="qte-attr ' . (isset($a_class_name) ? $a_class_name : '') . '"' . (!empty($a_colour) ? ' style="color:#' . $a_colour . '; font-weight:bold;"' : '');
582 582
 	}
583 583
 
584 584
 	/**
Please login to merge, or discard this patch.