@@ -55,9 +55,9 @@ discard block |
||
55 | 55 | */ |
56 | 56 | public function get_settings_fields() { |
57 | 57 | |
58 | - $this->settings_fields['points']['label'] = _x( 'Points', 'form label', 'wordpoints' ); |
|
59 | - $this->settings_fields['log_text']['label'] = _x( 'Log Text', 'form label', 'wordpoints' ); |
|
60 | - $this->settings_fields['description']['label'] = _x( 'Description', 'form label', 'wordpoints' ); |
|
58 | + $this->settings_fields['points']['label'] = _x('Points', 'form label', 'wordpoints'); |
|
59 | + $this->settings_fields['log_text']['label'] = _x('Log Text', 'form label', 'wordpoints'); |
|
60 | + $this->settings_fields['description']['label'] = _x('Description', 'form label', 'wordpoints'); |
|
61 | 61 | |
62 | 62 | return parent::get_settings_fields(); |
63 | 63 | } |
@@ -71,23 +71,23 @@ discard block |
||
71 | 71 | WordPoints_Hook_Event_Args $event_args |
72 | 72 | ) { |
73 | 73 | |
74 | - if ( ! isset( $settings['points'] ) || false === wordpoints_int( $settings['points'] ) ) { |
|
75 | - $validator->add_error( __( 'Points must be an integer.', 'wordpoints' ), 'points' ); |
|
74 | + if ( ! isset($settings['points']) || false === wordpoints_int($settings['points'])) { |
|
75 | + $validator->add_error(__('Points must be an integer.', 'wordpoints'), 'points'); |
|
76 | 76 | } |
77 | 77 | |
78 | - if ( ! isset( $settings['points_type'] ) || ! wordpoints_is_points_type( $settings['points_type'] ) ) { |
|
79 | - $validator->add_error( __( 'Invalid points type.', 'wordpoints' ), 'points_type' ); |
|
78 | + if ( ! isset($settings['points_type']) || ! wordpoints_is_points_type($settings['points_type'])) { |
|
79 | + $validator->add_error(__('Invalid points type.', 'wordpoints'), 'points_type'); |
|
80 | 80 | } |
81 | 81 | |
82 | - if ( ! isset( $settings['description'] ) ) { |
|
83 | - $validator->add_error( __( 'Description is required.', 'wordpoints' ), 'description' ); |
|
82 | + if ( ! isset($settings['description'])) { |
|
83 | + $validator->add_error(__('Description is required.', 'wordpoints'), 'description'); |
|
84 | 84 | } |
85 | 85 | |
86 | - if ( ! isset( $settings['log_text'] ) ) { |
|
87 | - $validator->add_error( __( 'Log Text is required.', 'wordpoints' ), 'log_text' ); |
|
86 | + if ( ! isset($settings['log_text'])) { |
|
87 | + $validator->add_error(__('Log Text is required.', 'wordpoints'), 'log_text'); |
|
88 | 88 | } |
89 | 89 | |
90 | - return parent::validate_settings( $settings, $validator, $event_args ); |
|
90 | + return parent::validate_settings($settings, $validator, $event_args); |
|
91 | 91 | } |
92 | 92 | |
93 | 93 | /** |
@@ -98,12 +98,12 @@ discard block |
||
98 | 98 | array $settings |
99 | 99 | ) { |
100 | 100 | |
101 | - parent::update_settings( $reaction, $settings ); |
|
101 | + parent::update_settings($reaction, $settings); |
|
102 | 102 | |
103 | - $reaction->update_meta( 'points', $settings['points'] ); |
|
104 | - $reaction->update_meta( 'points_type', $settings['points_type'] ); |
|
105 | - $reaction->update_meta( 'description', $settings['description'] ); |
|
106 | - $reaction->update_meta( 'log_text', $settings['log_text'] ); |
|
103 | + $reaction->update_meta('points', $settings['points']); |
|
104 | + $reaction->update_meta('points_type', $settings['points_type']); |
|
105 | + $reaction->update_meta('description', $settings['description']); |
|
106 | + $reaction->update_meta('log_text', $settings['log_text']); |
|
107 | 107 | } |
108 | 108 | |
109 | 109 | /** |
@@ -115,33 +115,33 @@ discard block |
||
115 | 115 | ) { |
116 | 116 | |
117 | 117 | $target = $event_args->get_from_hierarchy( |
118 | - $reaction->get_meta( 'target' ) |
|
118 | + $reaction->get_meta('target') |
|
119 | 119 | ); |
120 | 120 | |
121 | - if ( ! $target instanceof WordPoints_Entity ) { |
|
121 | + if ( ! $target instanceof WordPoints_Entity) { |
|
122 | 122 | return; |
123 | 123 | } |
124 | 124 | |
125 | 125 | $meta = array(); |
126 | 126 | |
127 | - foreach ( $event_args->get_entities() as $entity ) { |
|
128 | - $meta[ $entity->get_slug() ] = $entity->get_the_id(); |
|
127 | + foreach ($event_args->get_entities() as $entity) { |
|
128 | + $meta[$entity->get_slug()] = $entity->get_the_id(); |
|
129 | 129 | } |
130 | 130 | |
131 | 131 | wordpoints_alter_points( |
132 | 132 | $target->get_the_id() |
133 | - , $reaction->get_meta( 'points' ) |
|
134 | - , $reaction->get_meta( 'points_type' ) |
|
133 | + , $reaction->get_meta('points') |
|
134 | + , $reaction->get_meta('points_type') |
|
135 | 135 | , $reaction->get_event_slug() |
136 | 136 | , $meta |
137 | - , $reaction->get_meta( 'log_text' ) |
|
137 | + , $reaction->get_meta('log_text') |
|
138 | 138 | ); |
139 | 139 | } |
140 | 140 | |
141 | 141 | /** |
142 | 142 | * @since 1.0.0 |
143 | 143 | */ |
144 | - public function spam_hits( $event_slug, WordPoints_Hook_Event_Args $event_args ) { |
|
144 | + public function spam_hits($event_slug, WordPoints_Hook_Event_Args $event_args) { |
|
145 | 145 | |
146 | 146 | $meta_queries = array( |
147 | 147 | array( |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | ), |
153 | 153 | ); |
154 | 154 | |
155 | - foreach ( $event_args->get_entities() as $slug => $entity ) { |
|
155 | + foreach ($event_args->get_entities() as $slug => $entity) { |
|
156 | 156 | |
157 | 157 | $meta_queries[] = array( |
158 | 158 | 'key' => $slug, |
@@ -169,35 +169,35 @@ discard block |
||
169 | 169 | |
170 | 170 | $logs = $query->get(); |
171 | 171 | |
172 | - if ( ! $logs ) { |
|
172 | + if ( ! $logs) { |
|
173 | 173 | return; |
174 | 174 | } |
175 | 175 | |
176 | 176 | global $wpdb; |
177 | 177 | |
178 | - add_filter( 'wordpoints_points_log', '__return_false' ); |
|
178 | + add_filter('wordpoints_points_log', '__return_false'); |
|
179 | 179 | |
180 | - foreach ( $logs as $log ) { |
|
180 | + foreach ($logs as $log) { |
|
181 | 181 | |
182 | 182 | wordpoints_alter_points( |
183 | 183 | $log->user_id |
184 | 184 | , -$log->points |
185 | 185 | , $log->points_type |
186 | 186 | , "spam-{$log->log_type}" |
187 | - , array( 'original_log_id' => $log->id ) |
|
187 | + , array('original_log_id' => $log->id) |
|
188 | 188 | ); |
189 | 189 | |
190 | - wordpoints_points_log_delete_all_metadata( $log->id ); |
|
190 | + wordpoints_points_log_delete_all_metadata($log->id); |
|
191 | 191 | |
192 | 192 | // Now delete the log. |
193 | 193 | $wpdb->delete( |
194 | 194 | $wpdb->wordpoints_points_logs |
195 | - , array( 'id' => $log->id ) |
|
195 | + , array('id' => $log->id) |
|
196 | 196 | , '%d' |
197 | 197 | ); // WPCS: cache OK, cleaned by wordpoints_alter_points(). |
198 | 198 | } |
199 | 199 | |
200 | - remove_filter( 'wordpoints_points_log', '__return_false' ); |
|
200 | + remove_filter('wordpoints_points_log', '__return_false'); |
|
201 | 201 | } |
202 | 202 | } |
203 | 203 |
@@ -22,7 +22,7 @@ |
||
22 | 22 | * @param string $event_slug The event slug. |
23 | 23 | * @param WordPoints_Hook_Event_Args $event_args The event args. |
24 | 24 | */ |
25 | - public function reverse_hits( $event_slug, WordPoints_Hook_Event_Args $event_args ); |
|
25 | + public function reverse_hits($event_slug, WordPoints_Hook_Event_Args $event_args); |
|
26 | 26 | } |
27 | 27 | |
28 | 28 | // EOF |
@@ -22,7 +22,7 @@ |
||
22 | 22 | * @param string $event_slug The event slug. |
23 | 23 | * @param WordPoints_Hook_Event_Args $event_args The event args. |
24 | 24 | */ |
25 | - public function spam_hits( $event_slug, WordPoints_Hook_Event_Args $event_args ); |
|
25 | + public function spam_hits($event_slug, WordPoints_Hook_Event_Args $event_args); |
|
26 | 26 | } |
27 | 27 | |
28 | 28 | // EOF |
@@ -8,7 +8,7 @@ |
||
8 | 8 | */ |
9 | 9 | |
10 | 10 | interface WordPoints_Hook_Retroactive_Query_ModifierI { |
11 | - public function modify_retroactive_query( WordPoints_Hook_Retroactive_QueryI $query ); |
|
11 | + public function modify_retroactive_query(WordPoints_Hook_Retroactive_QueryI $query); |
|
12 | 12 | } |
13 | 13 | |
14 | 14 | // EOF |
@@ -50,7 +50,7 @@ discard block |
||
50 | 50 | |
51 | 51 | interface WordPoints_Hook_Retroactive_QueryI { |
52 | 52 | |
53 | - public function __construct( WordPoints_Hook_ReactionI $reaction ); |
|
53 | + public function __construct(WordPoints_Hook_ReactionI $reaction); |
|
54 | 54 | |
55 | 55 | /** |
56 | 56 | * |
@@ -61,13 +61,13 @@ discard block |
||
61 | 61 | public function get_reaction(); |
62 | 62 | public function get_validator(); |
63 | 63 | |
64 | - public function arg_hierarchy_push( $slug ); |
|
64 | + public function arg_hierarchy_push($slug); |
|
65 | 65 | public function arg_hierarchy_pop(); |
66 | 66 | public function get_arg(); |
67 | 67 | |
68 | - public function set_target( $target_arg ); |
|
68 | + public function set_target($target_arg); |
|
69 | 69 | public function select_value(); |
70 | - public function add_condition( array $condition ); |
|
70 | + public function add_condition(array $condition); |
|
71 | 71 | |
72 | 72 | public function get_results(); |
73 | 73 | } |
@@ -80,13 +80,13 @@ discard block |
||
80 | 80 | /** |
81 | 81 | * @since 1.0.0 |
82 | 82 | */ |
83 | - public function __call( $name, $args ) { |
|
83 | + public function __call($name, $args) { |
|
84 | 84 | |
85 | - $this->route_action( $name, $args ); |
|
85 | + $this->route_action($name, $args); |
|
86 | 86 | |
87 | 87 | // Return the first value, in case it is hooked to a filter. |
88 | 88 | $return = null; |
89 | - if ( isset( $args[0] ) ) { |
|
89 | + if (isset($args[0])) { |
|
90 | 90 | $return = $args[0]; |
91 | 91 | } |
92 | 92 | |
@@ -103,13 +103,13 @@ discard block |
||
103 | 103 | * action name and the priority. |
104 | 104 | * @param array $args The args the action was fired with. |
105 | 105 | */ |
106 | - protected function route_action( $name, $args ) { |
|
106 | + protected function route_action($name, $args) { |
|
107 | 107 | |
108 | - if ( ! isset( $this->action_index[ $name ] ) ) { |
|
108 | + if ( ! isset($this->action_index[$name])) { |
|
109 | 109 | return; |
110 | 110 | } |
111 | 111 | |
112 | - if ( ! isset( $this->actions ) ) { |
|
112 | + if ( ! isset($this->actions)) { |
|
113 | 113 | |
114 | 114 | $hooks = wordpoints_hooks(); |
115 | 115 | |
@@ -118,41 +118,41 @@ discard block |
||
118 | 118 | $this->firers = $hooks->firers; |
119 | 119 | } |
120 | 120 | |
121 | - foreach ( $this->action_index[ $name ]['actions'] as $slug => $data ) { |
|
121 | + foreach ($this->action_index[$name]['actions'] as $slug => $data) { |
|
122 | 122 | |
123 | - if ( ! isset( $this->event_index[ $slug ] ) ) { |
|
123 | + if ( ! isset($this->event_index[$slug])) { |
|
124 | 124 | continue; |
125 | 125 | } |
126 | 126 | |
127 | - $action_object = $this->actions->get( $slug, $args, $data ); |
|
127 | + $action_object = $this->actions->get($slug, $args, $data); |
|
128 | 128 | |
129 | - if ( ! ( $action_object instanceof WordPoints_Hook_ActionI ) ) { |
|
129 | + if ( ! ($action_object instanceof WordPoints_Hook_ActionI)) { |
|
130 | 130 | continue; |
131 | 131 | } |
132 | 132 | |
133 | - if ( ! $action_object->should_fire() ) { |
|
133 | + if ( ! $action_object->should_fire()) { |
|
134 | 134 | continue; |
135 | 135 | } |
136 | 136 | |
137 | - foreach ( $this->event_index[ $slug ] as $type => $events ) { |
|
138 | - foreach ( $events as $event_slug => $unused ) { |
|
137 | + foreach ($this->event_index[$slug] as $type => $events) { |
|
138 | + foreach ($events as $event_slug => $unused) { |
|
139 | 139 | |
140 | - if ( ! $this->events->is_registered( $event_slug ) ) { |
|
140 | + if ( ! $this->events->is_registered($event_slug)) { |
|
141 | 141 | continue; |
142 | 142 | } |
143 | 143 | |
144 | - $event_args = $this->events->args->get_children( $event_slug, array( $action_object ) ); |
|
144 | + $event_args = $this->events->args->get_children($event_slug, array($action_object)); |
|
145 | 145 | |
146 | - if ( empty( $event_args ) ) { |
|
146 | + if (empty($event_args)) { |
|
147 | 147 | continue; |
148 | 148 | } |
149 | 149 | |
150 | - $event_args = new WordPoints_Hook_Event_Args( $event_args ); |
|
150 | + $event_args = new WordPoints_Hook_Event_Args($event_args); |
|
151 | 151 | |
152 | - $firer = $this->firers->get( $type ); |
|
152 | + $firer = $this->firers->get($type); |
|
153 | 153 | |
154 | - if ( $firer instanceof WordPoints_Hook_FirerI ) { |
|
155 | - $firer->do_event( $event_slug, $event_args ); |
|
154 | + if ($firer instanceof WordPoints_Hook_FirerI) { |
|
155 | + $firer->do_event($event_slug, $event_args); |
|
156 | 156 | } |
157 | 157 | } |
158 | 158 | } |
@@ -182,56 +182,56 @@ discard block |
||
182 | 182 | * } |
183 | 183 | * } |
184 | 184 | */ |
185 | - public function add_action( $slug, array $args ) { |
|
185 | + public function add_action($slug, array $args) { |
|
186 | 186 | |
187 | 187 | $priority = 10; |
188 | - if ( isset( $args['priority'] ) ) { |
|
188 | + if (isset($args['priority'])) { |
|
189 | 189 | $priority = $args['priority']; |
190 | 190 | } |
191 | 191 | |
192 | - if ( ! isset( $args['action'] ) ) { |
|
192 | + if ( ! isset($args['action'])) { |
|
193 | 193 | return; |
194 | 194 | } |
195 | 195 | |
196 | 196 | $method = "{$args['action']},{$priority}"; |
197 | 197 | |
198 | - $this->action_index[ $method ]['actions'][ $slug ] = array(); |
|
198 | + $this->action_index[$method]['actions'][$slug] = array(); |
|
199 | 199 | |
200 | 200 | $arg_number = 1; |
201 | 201 | |
202 | - if ( isset( $args['data'] ) ) { |
|
202 | + if (isset($args['data'])) { |
|
203 | 203 | |
204 | - if ( isset( $args['data']['arg_index'] ) ) { |
|
205 | - $arg_number = 1 + max( $args['data']['arg_index'] ); |
|
204 | + if (isset($args['data']['arg_index'])) { |
|
205 | + $arg_number = 1 + max($args['data']['arg_index']); |
|
206 | 206 | } |
207 | 207 | |
208 | - if ( isset( $args['data']['requirements'] ) ) { |
|
209 | - $requirements = 1 + max( array_keys( $args['data']['requirements'] ) ); |
|
208 | + if (isset($args['data']['requirements'])) { |
|
209 | + $requirements = 1 + max(array_keys($args['data']['requirements'])); |
|
210 | 210 | |
211 | - if ( $requirements > $arg_number ) { |
|
211 | + if ($requirements > $arg_number) { |
|
212 | 212 | $arg_number = $requirements; |
213 | 213 | } |
214 | 214 | } |
215 | 215 | |
216 | - $this->action_index[ $method ]['actions'][ $slug ] = $args['data']; |
|
216 | + $this->action_index[$method]['actions'][$slug] = $args['data']; |
|
217 | 217 | } |
218 | 218 | |
219 | - if ( isset( $args['arg_number'] ) ) { |
|
219 | + if (isset($args['arg_number'])) { |
|
220 | 220 | $arg_number = $args['arg_number']; |
221 | 221 | } |
222 | 222 | |
223 | 223 | // If this action is already being routed, and will have enough args, we |
224 | 224 | // don't need to hook to it again. |
225 | 225 | if ( |
226 | - isset( $this->action_index[ $method ]['arg_number'] ) |
|
227 | - && $this->action_index[ $method ]['arg_number'] >= $arg_number |
|
226 | + isset($this->action_index[$method]['arg_number']) |
|
227 | + && $this->action_index[$method]['arg_number'] >= $arg_number |
|
228 | 228 | ) { |
229 | 229 | return; |
230 | 230 | } |
231 | 231 | |
232 | - $this->action_index[ $method ]['arg_number'] = $arg_number; |
|
232 | + $this->action_index[$method]['arg_number'] = $arg_number; |
|
233 | 233 | |
234 | - add_action( $args['action'], array( $this, $method ), $priority, $arg_number ); |
|
234 | + add_action($args['action'], array($this, $method), $priority, $arg_number); |
|
235 | 235 | } |
236 | 236 | |
237 | 237 | /** |
@@ -241,20 +241,20 @@ discard block |
||
241 | 241 | * |
242 | 242 | * @param string $slug The action slug. |
243 | 243 | */ |
244 | - public function remove_action( $slug ) { |
|
244 | + public function remove_action($slug) { |
|
245 | 245 | |
246 | - foreach ( $this->action_index as $method => $data ) { |
|
247 | - if ( isset( $data['actions'][ $slug ] ) ) { |
|
246 | + foreach ($this->action_index as $method => $data) { |
|
247 | + if (isset($data['actions'][$slug])) { |
|
248 | 248 | |
249 | - unset( $this->action_index[ $method ]['actions'][ $slug ] ); |
|
249 | + unset($this->action_index[$method]['actions'][$slug]); |
|
250 | 250 | |
251 | - if ( empty( $this->action_index[ $method ]['actions'] ) ) { |
|
251 | + if (empty($this->action_index[$method]['actions'])) { |
|
252 | 252 | |
253 | - unset( $this->action_index[ $method ] ); |
|
253 | + unset($this->action_index[$method]); |
|
254 | 254 | |
255 | - list( $action, $priority ) = explode( ',', $method ); |
|
255 | + list($action, $priority) = explode(',', $method); |
|
256 | 256 | |
257 | - remove_action( $action, array( $this, $method ), $priority ); |
|
257 | + remove_action($action, array($this, $method), $priority); |
|
258 | 258 | } |
259 | 259 | } |
260 | 260 | } |
@@ -269,8 +269,8 @@ discard block |
||
269 | 269 | * @param string $action_slug The slug of the action. |
270 | 270 | * @param string $action_type The type of action. Default is 'fire'. |
271 | 271 | */ |
272 | - public function add_event_to_action( $event_slug, $action_slug, $action_type = 'fire' ) { |
|
273 | - $this->event_index[ $action_slug ][ $action_type ][ $event_slug ] = true; |
|
272 | + public function add_event_to_action($event_slug, $action_slug, $action_type = 'fire') { |
|
273 | + $this->event_index[$action_slug][$action_type][$event_slug] = true; |
|
274 | 274 | } |
275 | 275 | |
276 | 276 | /** |
@@ -282,8 +282,8 @@ discard block |
||
282 | 282 | * @param string $action_slug The slug of the action. |
283 | 283 | * @param string $action_type The type of action. Default is 'fire'. |
284 | 284 | */ |
285 | - public function remove_event_from_action( $event_slug, $action_slug, $action_type = 'fire' ) { |
|
286 | - unset( $this->event_index[ $action_slug ][ $action_type ][ $event_slug ] ); |
|
285 | + public function remove_event_from_action($event_slug, $action_slug, $action_type = 'fire') { |
|
286 | + unset($this->event_index[$action_slug][$action_type][$event_slug]); |
|
287 | 287 | } |
288 | 288 | } |
289 | 289 |
@@ -9,6 +9,6 @@ |
||
9 | 9 | |
10 | 10 | global $wpdb; |
11 | 11 | |
12 | -$wpdb->wordpoints_hook_periods = $wpdb->base_prefix . 'wordpoints_hook_periods'; |
|
12 | +$wpdb->wordpoints_hook_periods = $wpdb->base_prefix.'wordpoints_hook_periods'; |
|
13 | 13 | |
14 | 14 | // EOF |
@@ -26,7 +26,7 @@ |
||
26 | 26 | /** |
27 | 27 | * @since 1.0.0 |
28 | 28 | */ |
29 | - public function __construct( $slug ) { |
|
29 | + public function __construct($slug) { |
|
30 | 30 | |
31 | 31 | $this->slug = $slug; |
32 | 32 | } |
@@ -33,7 +33,7 @@ |
||
33 | 33 | * @since 1.0.0 |
34 | 34 | */ |
35 | 35 | public function get_title() { |
36 | - return __( 'Author', 'wordpoints' ); |
|
36 | + return __('Author', 'wordpoints'); |
|
37 | 37 | } |
38 | 38 | } |
39 | 39 |