@@ -28,27 +28,27 @@ discard block |
||
28 | 28 | * |
29 | 29 | * @return array The validated periods. |
30 | 30 | */ |
31 | - protected function validate_periods( $periods ) { |
|
31 | + protected function validate_periods($periods) { |
|
32 | 32 | |
33 | - if ( ! is_array( $periods ) ) { |
|
33 | + if ( ! is_array($periods)) { |
|
34 | 34 | |
35 | 35 | $this->validator->add_error( |
36 | - __( 'Periods do not match expected format.', 'wordpoints' ) |
|
36 | + __('Periods do not match expected format.', 'wordpoints') |
|
37 | 37 | ); |
38 | 38 | |
39 | 39 | return array(); |
40 | 40 | } |
41 | 41 | |
42 | - foreach ( $periods as $index => $period ) { |
|
42 | + foreach ($periods as $index => $period) { |
|
43 | 43 | |
44 | - $this->validator->push_field( $index ); |
|
44 | + $this->validator->push_field($index); |
|
45 | 45 | |
46 | - $period = $this->validate_period( $period ); |
|
46 | + $period = $this->validate_period($period); |
|
47 | 47 | |
48 | - if ( $period ) { |
|
49 | - $periods[ $index ] = $period; |
|
48 | + if ($period) { |
|
49 | + $periods[$index] = $period; |
|
50 | 50 | } else { |
51 | - unset( $periods[ $index ] ); |
|
51 | + unset($periods[$index]); |
|
52 | 52 | } |
53 | 53 | |
54 | 54 | $this->validator->pop_field(); |
@@ -66,56 +66,56 @@ discard block |
||
66 | 66 | * |
67 | 67 | * @return array|false The validated period, or false if invalid. |
68 | 68 | */ |
69 | - protected function validate_period( $period ) { |
|
69 | + protected function validate_period($period) { |
|
70 | 70 | |
71 | - if ( ! is_array( $period ) ) { |
|
71 | + if ( ! is_array($period)) { |
|
72 | 72 | $this->validator->add_error( |
73 | - __( 'Period does not match expected format.', 'wordpoints' ) |
|
73 | + __('Period does not match expected format.', 'wordpoints') |
|
74 | 74 | ); |
75 | 75 | |
76 | 76 | return false; |
77 | 77 | } |
78 | 78 | |
79 | - if ( ! isset( $period['args'] ) ) { |
|
79 | + if ( ! isset($period['args'])) { |
|
80 | 80 | // TODO is there a default period args? |
81 | 81 | $period['args'] = array(); |
82 | 82 | } |
83 | 83 | |
84 | - if ( ! is_array( $period['args'] ) ) { |
|
84 | + if ( ! is_array($period['args'])) { |
|
85 | 85 | return false; |
86 | 86 | } |
87 | 87 | |
88 | - if ( ! $this->event_args->get_from_hierarchy( $period['args'] ) ) { |
|
88 | + if ( ! $this->event_args->get_from_hierarchy($period['args'])) { |
|
89 | 89 | $this->validator->add_error( |
90 | - __( 'Invalid period.', 'wordpoints' ) // TODO better error message |
|
90 | + __('Invalid period.', 'wordpoints') // TODO better error message |
|
91 | 91 | , 'args' |
92 | 92 | ); |
93 | 93 | |
94 | 94 | return false; |
95 | 95 | } |
96 | 96 | |
97 | - if ( ! isset( $period['settings'] ) ) { |
|
97 | + if ( ! isset($period['settings'])) { |
|
98 | 98 | $this->validator->add_error( |
99 | - __( 'Period settings are missing.', 'wordpoints' ) |
|
99 | + __('Period settings are missing.', 'wordpoints') |
|
100 | 100 | ); |
101 | 101 | |
102 | 102 | return false; |
103 | 103 | } |
104 | 104 | |
105 | - if ( ! isset( $period['settings']['length'] ) ) { |
|
105 | + if ( ! isset($period['settings']['length'])) { |
|
106 | 106 | $this->validator->add_error( |
107 | - __( 'Period length setting is missing.', 'wordpoints' ) |
|
107 | + __('Period length setting is missing.', 'wordpoints') |
|
108 | 108 | , 'settings' |
109 | 109 | ); |
110 | 110 | |
111 | 111 | return false; |
112 | 112 | } |
113 | 113 | |
114 | - $this->validator->push_field( 'settings' ); |
|
114 | + $this->validator->push_field('settings'); |
|
115 | 115 | |
116 | - if ( false === wordpoints_posint( $period['settings']['length'] ) ) { |
|
116 | + if (false === wordpoints_posint($period['settings']['length'])) { |
|
117 | 117 | $this->validator->add_error( |
118 | - __( 'Period length must be a positive integer.', 'wordpoints' ) |
|
118 | + __('Period length must be a positive integer.', 'wordpoints') |
|
119 | 119 | , 'length' |
120 | 120 | ); |
121 | 121 | |
@@ -130,11 +130,11 @@ discard block |
||
130 | 130 | /** |
131 | 131 | * @since 1.0.0 |
132 | 132 | */ |
133 | - public function should_hit( WordPoints_Hook_Reaction_Validator $reaction, WordPoints_Hook_Event_Args $event_args ) { |
|
133 | + public function should_hit(WordPoints_Hook_Reaction_Validator $reaction, WordPoints_Hook_Event_Args $event_args) { |
|
134 | 134 | |
135 | - $periods = $reaction->get_meta( 'periods' ); |
|
135 | + $periods = $reaction->get_meta('periods'); |
|
136 | 136 | |
137 | - if ( empty( $periods ) ) { |
|
137 | + if (empty($periods)) { |
|
138 | 138 | return true; |
139 | 139 | } |
140 | 140 | |
@@ -142,8 +142,8 @@ discard block |
||
142 | 142 | |
143 | 143 | $this->event_args = $event_args; |
144 | 144 | |
145 | - foreach ( $periods as $period ) { |
|
146 | - if ( ! $this->has_period_ended( $period, $reaction_id ) ) { |
|
145 | + foreach ($periods as $period) { |
|
146 | + if ( ! $this->has_period_ended($period, $reaction_id)) { |
|
147 | 147 | return false; |
148 | 148 | } |
149 | 149 | } |
@@ -163,20 +163,20 @@ discard block |
||
163 | 163 | * |
164 | 164 | * @return bool Whether the period has ended. |
165 | 165 | */ |
166 | - final private function has_period_ended( array $settings, $reaction_id ) { |
|
166 | + final private function has_period_ended(array $settings, $reaction_id) { |
|
167 | 167 | |
168 | 168 | $period = $this->get_period_by_reaction( |
169 | - $this->get_arg_values( $settings['args'] ) |
|
169 | + $this->get_arg_values($settings['args']) |
|
170 | 170 | , $reaction_id |
171 | 171 | ); |
172 | 172 | |
173 | 173 | // If the period isn't found, we know that we can still fire. |
174 | - if ( ! $period ) { |
|
174 | + if ( ! $period) { |
|
175 | 175 | return true; |
176 | 176 | } |
177 | 177 | |
178 | - if ( ! empty( $settings['settings']['absolute'] ) ) { |
|
179 | - return ( $period->expiration < current_time( 'timestamp' ) ); |
|
178 | + if ( ! empty($settings['settings']['absolute'])) { |
|
179 | + return ($period->expiration < current_time('timestamp')); |
|
180 | 180 | } else { |
181 | 181 | return false; |
182 | 182 | } |
@@ -191,18 +191,18 @@ discard block |
||
191 | 191 | * |
192 | 192 | * @return array The arg values. |
193 | 193 | */ |
194 | - final private function get_arg_values( array $period_args ) { |
|
194 | + final private function get_arg_values(array $period_args) { |
|
195 | 195 | |
196 | 196 | $values = array(); |
197 | 197 | |
198 | - foreach ( $period_args as $arg_slug => $sub_args ) { |
|
198 | + foreach ($period_args as $arg_slug => $sub_args) { |
|
199 | 199 | |
200 | - $this->event_args->descend( $arg_slug ); |
|
200 | + $this->event_args->descend($arg_slug); |
|
201 | 201 | |
202 | - if ( is_array( $sub_args ) ) { |
|
203 | - $values = array_merge( $values, $this->get_arg_values( $sub_args ) ); |
|
202 | + if (is_array($sub_args)) { |
|
203 | + $values = array_merge($values, $this->get_arg_values($sub_args)); |
|
204 | 204 | } else { |
205 | - $values[ $arg_slug ] = $this->event_args->get_current()->get_the_value(); |
|
205 | + $values[$arg_slug] = $this->event_args->get_current()->get_the_value(); |
|
206 | 206 | } |
207 | 207 | |
208 | 208 | $this->event_args->ascend(); |
@@ -220,11 +220,11 @@ discard block |
||
220 | 220 | * |
221 | 221 | * @return object|false The period data, or false if not found. |
222 | 222 | */ |
223 | - final private function get_period( $period_id ) { |
|
223 | + final private function get_period($period_id) { |
|
224 | 224 | |
225 | - $period = wp_cache_get( $period_id, 'wordpoints_hook_period' ); |
|
225 | + $period = wp_cache_get($period_id, 'wordpoints_hook_period'); |
|
226 | 226 | |
227 | - if ( ! $period ) { |
|
227 | + if ( ! $period) { |
|
228 | 228 | |
229 | 229 | global $wpdb; |
230 | 230 | |
@@ -239,12 +239,12 @@ discard block |
||
239 | 239 | ) |
240 | 240 | ); |
241 | 241 | |
242 | - if ( ! $period ) { |
|
242 | + if ( ! $period) { |
|
243 | 243 | return false; |
244 | 244 | } |
245 | 245 | |
246 | - wp_cache_set( $period->hook_id . $period->arg_hash, $period->id, 'wordpoints_hook_period_ids' ); |
|
247 | - wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' ); |
|
246 | + wp_cache_set($period->hook_id.$period->arg_hash, $period->id, 'wordpoints_hook_period_ids'); |
|
247 | + wp_cache_set($period->id, $period, 'wordpoints_hook_periods'); |
|
248 | 248 | } |
249 | 249 | |
250 | 250 | return $period; |
@@ -260,19 +260,19 @@ discard block |
||
260 | 260 | * |
261 | 261 | * @return object|false The period data, or false if not found. |
262 | 262 | */ |
263 | - final private function get_period_by_reaction( array $args, $reaction_id ) { |
|
263 | + final private function get_period_by_reaction(array $args, $reaction_id) { |
|
264 | 264 | |
265 | 265 | // The periods for a reaction are differentiated by a hash of specific args. |
266 | - $arg_hash = md5( serialize( $args ) ); |
|
266 | + $arg_hash = md5(serialize($args)); |
|
267 | 267 | |
268 | 268 | $cache_key = "{$this->slug}-{$reaction_id}-{$arg_hash}"; |
269 | 269 | |
270 | 270 | // Before we run the query, we try to lookup the ID in the cache. |
271 | - $period_id = wp_cache_get( $cache_key, 'wordpoints_hook_period_ids' ); |
|
271 | + $period_id = wp_cache_get($cache_key, 'wordpoints_hook_period_ids'); |
|
272 | 272 | |
273 | 273 | // If we found it, we can retrieve the period by ID instead. |
274 | - if ( $period_id ) { |
|
275 | - return $this->get_period( $period_id ); |
|
274 | + if ($period_id) { |
|
275 | + return $this->get_period($period_id); |
|
276 | 276 | } |
277 | 277 | |
278 | 278 | global $wpdb; |
@@ -291,12 +291,12 @@ discard block |
||
291 | 291 | ) |
292 | 292 | ); |
293 | 293 | |
294 | - if ( ! $period ) { |
|
294 | + if ( ! $period) { |
|
295 | 295 | return false; |
296 | 296 | } |
297 | 297 | |
298 | - wp_cache_set( $cache_key, $period->id, 'wordpoints_hook_period_ids' ); |
|
299 | - wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' ); |
|
298 | + wp_cache_set($cache_key, $period->id, 'wordpoints_hook_period_ids'); |
|
299 | + wp_cache_set($period->id, $period, 'wordpoints_hook_periods'); |
|
300 | 300 | |
301 | 301 | return $period; |
302 | 302 | } |
@@ -17,14 +17,14 @@ discard block |
||
17 | 17 | /** |
18 | 18 | * @since 1.0.0 |
19 | 19 | */ |
20 | - public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) { |
|
20 | + public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) { |
|
21 | 21 | |
22 | 22 | $hooks = wordpoints_hooks(); |
23 | 23 | |
24 | 24 | /** @var WordPoints_Hook_Reactor $reactor */ |
25 | - foreach ( $hooks->reactors->get_all() as $reactor ) { |
|
25 | + foreach ($hooks->reactors->get_all() as $reactor) { |
|
26 | 26 | |
27 | - foreach ( $reactor->get_all_reactions_to_event( $event_slug ) as $reaction ) { |
|
27 | + foreach ($reactor->get_all_reactions_to_event($event_slug) as $reaction) { |
|
28 | 28 | |
29 | 29 | $validator = new WordPoints_Hook_Reaction_Validator( |
30 | 30 | $reaction |
@@ -34,25 +34,25 @@ discard block |
||
34 | 34 | |
35 | 35 | $validator->validate(); |
36 | 36 | |
37 | - if ( $validator->had_errors() ) { |
|
37 | + if ($validator->had_errors()) { |
|
38 | 38 | continue; |
39 | 39 | } |
40 | 40 | |
41 | - $event_args->set_validator( $validator ); |
|
41 | + $event_args->set_validator($validator); |
|
42 | 42 | $reaction = $validator; |
43 | 43 | |
44 | 44 | /** @var WordPoints_Hook_Extension $extension */ |
45 | - foreach ( $hooks->extensions->get_all() as $extension ) { |
|
45 | + foreach ($hooks->extensions->get_all() as $extension) { |
|
46 | 46 | |
47 | - if ( ! $extension->should_hit( $reaction, $event_args ) ) { |
|
47 | + if ( ! $extension->should_hit($reaction, $event_args)) { |
|
48 | 48 | continue 2; |
49 | 49 | } |
50 | 50 | } |
51 | 51 | |
52 | - $reactor->hit( $event_args, $reaction ); |
|
52 | + $reactor->hit($event_args, $reaction); |
|
53 | 53 | |
54 | 54 | // TODO hook docs (is this hook even needed?) |
55 | - do_action( 'wordpoints_hook_event_hit', $reaction, $event_args, $reactor ); |
|
55 | + do_action('wordpoints_hook_event_hit', $reaction, $event_args, $reactor); |
|
56 | 56 | } |
57 | 57 | } |
58 | 58 | } |
@@ -17,21 +17,21 @@ |
||
17 | 17 | /** |
18 | 18 | * @since 1.0.0 |
19 | 19 | */ |
20 | - public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) { |
|
20 | + public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) { |
|
21 | 21 | |
22 | 22 | $hooks = wordpoints_hooks(); |
23 | 23 | |
24 | - foreach ( $hooks->reactors->get_all() as $reactor ) { |
|
24 | + foreach ($hooks->reactors->get_all() as $reactor) { |
|
25 | 25 | |
26 | - if ( ! ( $reactor instanceof WordPoints_Hook_Reactor_ReverseI ) ) { |
|
26 | + if ( ! ($reactor instanceof WordPoints_Hook_Reactor_ReverseI)) { |
|
27 | 27 | continue; |
28 | 28 | } |
29 | 29 | |
30 | - $reactor->reverse_hits( $event_slug, $event_args ); |
|
30 | + $reactor->reverse_hits($event_slug, $event_args); |
|
31 | 31 | |
32 | - foreach ( $hooks->extensions->get_all() as $extension ) { |
|
33 | - if ( $extension instanceof WordPoints_Hook_Extension_ReverseI ) { |
|
34 | - $extension->after_reverse( $event_slug, $event_args, $reactor ); |
|
32 | + foreach ($hooks->extensions->get_all() as $extension) { |
|
33 | + if ($extension instanceof WordPoints_Hook_Extension_ReverseI) { |
|
34 | + $extension->after_reverse($event_slug, $event_args, $reactor); |
|
35 | 35 | } |
36 | 36 | } |
37 | 37 | } |
@@ -17,21 +17,21 @@ |
||
17 | 17 | /** |
18 | 18 | * @since 1.0.0 |
19 | 19 | */ |
20 | - public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) { |
|
20 | + public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) { |
|
21 | 21 | |
22 | 22 | $hooks = wordpoints_hooks(); |
23 | 23 | |
24 | - foreach ( $hooks->reactors->get_all() as $reactor ) { |
|
24 | + foreach ($hooks->reactors->get_all() as $reactor) { |
|
25 | 25 | |
26 | - if ( ! ( $reactor instanceof WordPoints_Hook_Reactor_SpamI ) ) { |
|
26 | + if ( ! ($reactor instanceof WordPoints_Hook_Reactor_SpamI)) { |
|
27 | 27 | continue; |
28 | 28 | } |
29 | 29 | |
30 | - $reactor->spam_hits( $event_slug, $event_args ); |
|
30 | + $reactor->spam_hits($event_slug, $event_args); |
|
31 | 31 | |
32 | - foreach ( $hooks->extensions->get_all() as $extension ) { |
|
33 | - if ( $extension instanceof WordPoints_Hook_Extension_SpamI ) { |
|
34 | - $extension->after_spam( $event_slug, $event_args, $reactor ); |
|
32 | + foreach ($hooks->extensions->get_all() as $extension) { |
|
33 | + if ($extension instanceof WordPoints_Hook_Extension_SpamI) { |
|
34 | + $extension->after_spam($event_slug, $event_args, $reactor); |
|
35 | 35 | } |
36 | 36 | } |
37 | 37 | } |
@@ -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 do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ); |
|
25 | + public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args); |
|
26 | 26 | } |
27 | 27 | |
28 | 28 | // EOF |
@@ -53,9 +53,9 @@ discard block |
||
53 | 53 | /** |
54 | 54 | * @since 1.0.0 |
55 | 55 | */ |
56 | - public function __construct( $id, WordPoints_Hook_Reaction_StorageI $storage ) { |
|
56 | + public function __construct($id, WordPoints_Hook_Reaction_StorageI $storage) { |
|
57 | 57 | |
58 | - $this->ID = wordpoints_int( $id ); |
|
58 | + $this->ID = wordpoints_int($id); |
|
59 | 59 | $this->storage = $storage; |
60 | 60 | $this->reactor_slug = $this->storage->get_reactor_slug(); |
61 | 61 | $this->network_wide = $this->storage->is_network_wide(); |
@@ -64,9 +64,9 @@ discard block |
||
64 | 64 | /** |
65 | 65 | * @since 1.0.0 |
66 | 66 | */ |
67 | - public function __get( $var ) { |
|
67 | + public function __get($var) { |
|
68 | 68 | |
69 | - if ( 'ID' === $var ) { |
|
69 | + if ('ID' === $var) { |
|
70 | 70 | return $this->ID; |
71 | 71 | } |
72 | 72 |
@@ -25,13 +25,13 @@ discard block |
||
25 | 25 | * @since 1.0.0 |
26 | 26 | */ |
27 | 27 | public function get_event_slug() { |
28 | - return $this->storage->get_reaction_event_from_index( $this->ID ); |
|
28 | + return $this->storage->get_reaction_event_from_index($this->ID); |
|
29 | 29 | } |
30 | 30 | |
31 | 31 | /** |
32 | 32 | * @since 1.0.0 |
33 | 33 | */ |
34 | - public function update_event_slug( $event_slug ) { |
|
34 | + public function update_event_slug($event_slug) { |
|
35 | 35 | return $this->storage->update_reaction_event_in_index( |
36 | 36 | $this->ID |
37 | 37 | , $event_slug |
@@ -41,63 +41,63 @@ discard block |
||
41 | 41 | /** |
42 | 42 | * @since 1.0.0 |
43 | 43 | */ |
44 | - public function get_meta( $key ) { |
|
44 | + public function get_meta($key) { |
|
45 | 45 | |
46 | 46 | $settings = $this->get_settings(); |
47 | 47 | |
48 | - if ( ! is_array( $settings ) || ! isset( $settings[ $key ] ) ) { |
|
48 | + if ( ! is_array($settings) || ! isset($settings[$key])) { |
|
49 | 49 | return false; |
50 | 50 | } |
51 | 51 | |
52 | - return $settings[ $key ]; |
|
52 | + return $settings[$key]; |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | /** |
56 | 56 | * @since 1.0.0 |
57 | 57 | */ |
58 | - public function add_meta( $key, $value ) { |
|
58 | + public function add_meta($key, $value) { |
|
59 | 59 | |
60 | 60 | $settings = $this->get_settings(); |
61 | 61 | |
62 | - if ( ! is_array( $settings ) || isset( $settings[ $key ] ) ) { |
|
62 | + if ( ! is_array($settings) || isset($settings[$key])) { |
|
63 | 63 | return false; |
64 | 64 | } |
65 | 65 | |
66 | - $settings[ $key ] = $value; |
|
66 | + $settings[$key] = $value; |
|
67 | 67 | |
68 | - return $this->update_settings( $settings ); |
|
68 | + return $this->update_settings($settings); |
|
69 | 69 | } |
70 | 70 | |
71 | 71 | /** |
72 | 72 | * @since 1.0.0 |
73 | 73 | */ |
74 | - public function update_meta( $key, $value ) { |
|
74 | + public function update_meta($key, $value) { |
|
75 | 75 | |
76 | 76 | $settings = $this->get_settings(); |
77 | 77 | |
78 | - if ( ! is_array( $settings ) ) { |
|
78 | + if ( ! is_array($settings)) { |
|
79 | 79 | return false; |
80 | 80 | } |
81 | 81 | |
82 | - $settings[ $key ] = $value; |
|
82 | + $settings[$key] = $value; |
|
83 | 83 | |
84 | - return $this->update_settings( $settings ); |
|
84 | + return $this->update_settings($settings); |
|
85 | 85 | } |
86 | 86 | |
87 | 87 | /** |
88 | 88 | * @since 1.0.0 |
89 | 89 | */ |
90 | - public function delete_meta( $key ) { |
|
90 | + public function delete_meta($key) { |
|
91 | 91 | |
92 | 92 | $settings = $this->get_settings(); |
93 | 93 | |
94 | - if ( ! is_array( $settings ) || ! isset( $settings[ $key ] ) ) { |
|
94 | + if ( ! is_array($settings) || ! isset($settings[$key])) { |
|
95 | 95 | return false; |
96 | 96 | } |
97 | 97 | |
98 | - unset( $settings[ $key ] ); |
|
98 | + unset($settings[$key]); |
|
99 | 99 | |
100 | - return $this->update_settings( $settings ); |
|
100 | + return $this->update_settings($settings); |
|
101 | 101 | } |
102 | 102 | |
103 | 103 | /** |
@@ -117,7 +117,7 @@ discard block |
||
117 | 117 | protected function get_settings() { |
118 | 118 | |
119 | 119 | return $this->storage->get_option( |
120 | - $this->storage->get_settings_option_name( $this->ID ) |
|
120 | + $this->storage->get_settings_option_name($this->ID) |
|
121 | 121 | ); |
122 | 122 | } |
123 | 123 | |
@@ -130,10 +130,10 @@ discard block |
||
130 | 130 | * |
131 | 131 | * @return bool Whether the settings were updated successfully. |
132 | 132 | */ |
133 | - protected function update_settings( $settings ) { |
|
133 | + protected function update_settings($settings) { |
|
134 | 134 | |
135 | 135 | return $this->storage->update_option( |
136 | - $this->storage->get_settings_option_name( $this->ID ) |
|
136 | + $this->storage->get_settings_option_name($this->ID) |
|
137 | 137 | , $settings |
138 | 138 | ); |
139 | 139 | } |
@@ -22,25 +22,25 @@ discard block |
||
22 | 22 | /** |
23 | 23 | * @since 1.0.0 |
24 | 24 | */ |
25 | - public function reaction_exists( $id ) { |
|
26 | - return (bool) $this->get_option( $this->get_settings_option_name( $id ) ); |
|
25 | + public function reaction_exists($id) { |
|
26 | + return (bool) $this->get_option($this->get_settings_option_name($id)); |
|
27 | 27 | } |
28 | 28 | |
29 | 29 | /** |
30 | 30 | * @since 1.0.0 |
31 | 31 | */ |
32 | 32 | public function get_reactions() { |
33 | - return $this->create_reaction_objects( $this->get_reaction_index() ); |
|
33 | + return $this->create_reaction_objects($this->get_reaction_index()); |
|
34 | 34 | } |
35 | 35 | |
36 | 36 | /** |
37 | 37 | * @since 1.0.0 |
38 | 38 | */ |
39 | - public function get_reactions_to_event( $event_slug ) { |
|
39 | + public function get_reactions_to_event($event_slug) { |
|
40 | 40 | |
41 | 41 | $index = $this->get_reaction_index(); |
42 | - $index = wp_list_filter( $index, array( 'event' => $event_slug ) ); |
|
43 | - return $this->create_reaction_objects( $index ); |
|
42 | + $index = wp_list_filter($index, array('event' => $event_slug)); |
|
43 | + return $this->create_reaction_objects($index); |
|
44 | 44 | } |
45 | 45 | |
46 | 46 | /** |
@@ -59,9 +59,9 @@ discard block |
||
59 | 59 | */ |
60 | 60 | protected function get_reaction_index() { |
61 | 61 | |
62 | - $index = $this->get_option( $this->get_reaction_index_option_name() ); |
|
62 | + $index = $this->get_option($this->get_reaction_index_option_name()); |
|
63 | 63 | |
64 | - if ( ! is_array( $index ) ) { |
|
64 | + if ( ! is_array($index)) { |
|
65 | 65 | $index = array(); |
66 | 66 | } |
67 | 67 | |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | * |
78 | 78 | * @return bool Whether the index was updated successfully. |
79 | 79 | */ |
80 | - protected function update_reaction_index( $index ) { |
|
80 | + protected function update_reaction_index($index) { |
|
81 | 81 | |
82 | 82 | return $this->update_option( |
83 | 83 | $this->get_reaction_index_option_name() |
@@ -96,15 +96,15 @@ discard block |
||
96 | 96 | * |
97 | 97 | * @return string|false The event slug, or false if not found. |
98 | 98 | */ |
99 | - public function get_reaction_event_from_index( $id ) { |
|
99 | + public function get_reaction_event_from_index($id) { |
|
100 | 100 | |
101 | 101 | $index = $this->get_reaction_index(); |
102 | 102 | |
103 | - if ( ! isset( $index[ $id ]['event'] ) ) { |
|
103 | + if ( ! isset($index[$id]['event'])) { |
|
104 | 104 | return false; |
105 | 105 | } |
106 | 106 | |
107 | - return $index[ $id ]['event']; |
|
107 | + return $index[$id]['event']; |
|
108 | 108 | } |
109 | 109 | |
110 | 110 | /** |
@@ -119,17 +119,17 @@ discard block |
||
119 | 119 | * |
120 | 120 | * @return bool Whether the event was updated successfully. |
121 | 121 | */ |
122 | - public function update_reaction_event_in_index( $id, $event ) { |
|
122 | + public function update_reaction_event_in_index($id, $event) { |
|
123 | 123 | |
124 | 124 | $index = $this->get_reaction_index(); |
125 | 125 | |
126 | - if ( ! isset( $index[ $id ] ) ) { |
|
126 | + if ( ! isset($index[$id])) { |
|
127 | 127 | return false; |
128 | 128 | } |
129 | 129 | |
130 | - $index[ $id ]['event'] = $event; |
|
130 | + $index[$id]['event'] = $event; |
|
131 | 131 | |
132 | - return $this->update_reaction_index( $index ); |
|
132 | + return $this->update_reaction_index($index); |
|
133 | 133 | } |
134 | 134 | |
135 | 135 | /** |
@@ -141,15 +141,15 @@ discard block |
||
141 | 141 | * |
142 | 142 | * @return WordPoints_Hook_Reaction_Options[] The objects for the reactions. |
143 | 143 | */ |
144 | - protected function create_reaction_objects( $index ) { |
|
144 | + protected function create_reaction_objects($index) { |
|
145 | 145 | |
146 | 146 | $reactions = array(); |
147 | 147 | |
148 | - foreach ( $index as $reaction ) { |
|
148 | + foreach ($index as $reaction) { |
|
149 | 149 | |
150 | - $object = $this->get_reaction( $reaction['id'] ); |
|
150 | + $object = $this->get_reaction($reaction['id']); |
|
151 | 151 | |
152 | - if ( ! $object ) { |
|
152 | + if ( ! $object) { |
|
153 | 153 | continue; |
154 | 154 | } |
155 | 155 | |
@@ -162,53 +162,53 @@ discard block |
||
162 | 162 | /** |
163 | 163 | * @since 1.0.0 |
164 | 164 | */ |
165 | - public function delete_reaction( $id ) { |
|
165 | + public function delete_reaction($id) { |
|
166 | 166 | |
167 | - if ( ! $this->reaction_exists( $id ) ) { |
|
167 | + if ( ! $this->reaction_exists($id)) { |
|
168 | 168 | return false; |
169 | 169 | } |
170 | 170 | |
171 | - $result = $this->delete_option( $this->get_settings_option_name( $id ) ); |
|
171 | + $result = $this->delete_option($this->get_settings_option_name($id)); |
|
172 | 172 | |
173 | - if ( ! $result ) { |
|
173 | + if ( ! $result) { |
|
174 | 174 | return false; |
175 | 175 | } |
176 | 176 | |
177 | 177 | $index = $this->get_reaction_index(); |
178 | 178 | |
179 | - unset( $index[ $id ] ); |
|
179 | + unset($index[$id]); |
|
180 | 180 | |
181 | - return $this->update_reaction_index( $index ); |
|
181 | + return $this->update_reaction_index($index); |
|
182 | 182 | } |
183 | 183 | |
184 | 184 | /** |
185 | 185 | * @since 1.0.0 |
186 | 186 | */ |
187 | - protected function _create_reaction( $event_slug ) { |
|
187 | + protected function _create_reaction($event_slug) { |
|
188 | 188 | |
189 | 189 | $index = $this->get_reaction_index(); |
190 | 190 | |
191 | 191 | $id = 1; |
192 | 192 | |
193 | 193 | // TODO this is fragile when the newest reaction gets deleted. |
194 | - if ( ! empty( $index ) ) { |
|
195 | - $id = 1 + max( array_keys( $index ) ); |
|
194 | + if ( ! empty($index)) { |
|
195 | + $id = 1 + max(array_keys($index)); |
|
196 | 196 | } |
197 | 197 | |
198 | - $settings = array( 'event' => $event_slug ); |
|
198 | + $settings = array('event' => $event_slug); |
|
199 | 199 | |
200 | 200 | $result = $this->add_option( |
201 | - $this->get_settings_option_name( $id ) |
|
201 | + $this->get_settings_option_name($id) |
|
202 | 202 | , $settings |
203 | 203 | ); |
204 | 204 | |
205 | - if ( ! $result ) { |
|
205 | + if ( ! $result) { |
|
206 | 206 | return false; |
207 | 207 | } |
208 | 208 | |
209 | - $index[ $id ] = array( 'event' => $event_slug, 'id' => $id ); |
|
209 | + $index[$id] = array('event' => $event_slug, 'id' => $id); |
|
210 | 210 | |
211 | - if ( ! $this->update_reaction_index( $index ) ) { |
|
211 | + if ( ! $this->update_reaction_index($index)) { |
|
212 | 212 | return false; |
213 | 213 | } |
214 | 214 | |
@@ -226,8 +226,8 @@ discard block |
||
226 | 226 | * |
227 | 227 | * @return mixed The option value, or false. |
228 | 228 | */ |
229 | - public function get_option( $name ) { |
|
230 | - return get_option( $name ); |
|
229 | + public function get_option($name) { |
|
230 | + return get_option($name); |
|
231 | 231 | } |
232 | 232 | |
233 | 233 | /** |
@@ -240,8 +240,8 @@ discard block |
||
240 | 240 | * |
241 | 241 | * @return bool Whether the option was added successfully. |
242 | 242 | */ |
243 | - protected function add_option( $name, $value ) { |
|
244 | - return add_option( $name, $value ); |
|
243 | + protected function add_option($name, $value) { |
|
244 | + return add_option($name, $value); |
|
245 | 245 | } |
246 | 246 | |
247 | 247 | /** |
@@ -256,8 +256,8 @@ discard block |
||
256 | 256 | * |
257 | 257 | * @return bool Whether the option was updated successfully. |
258 | 258 | */ |
259 | - public function update_option( $name, $value ) { |
|
260 | - return update_option( $name, $value ); |
|
259 | + public function update_option($name, $value) { |
|
260 | + return update_option($name, $value); |
|
261 | 261 | } |
262 | 262 | |
263 | 263 | /** |
@@ -269,8 +269,8 @@ discard block |
||
269 | 269 | * |
270 | 270 | * @return bool Whether the option was deleted successfully. |
271 | 271 | */ |
272 | - protected function delete_option( $name ) { |
|
273 | - return delete_option( $name ); |
|
272 | + protected function delete_option($name) { |
|
273 | + return delete_option($name); |
|
274 | 274 | } |
275 | 275 | |
276 | 276 | /** |
@@ -282,7 +282,7 @@ discard block |
||
282 | 282 | * |
283 | 283 | * @return string The name of the option where the settings are stored. |
284 | 284 | */ |
285 | - public function get_settings_option_name( $id ) { |
|
285 | + public function get_settings_option_name($id) { |
|
286 | 286 | return "wordpoints_{$this->reactor_slug}_hook_reaction-{$id}"; |
287 | 287 | } |
288 | 288 |
@@ -22,29 +22,29 @@ |
||
22 | 22 | /** |
23 | 23 | * @since 1.0.0 |
24 | 24 | */ |
25 | - public function get_option( $name ) { |
|
26 | - return get_site_option( $name ); |
|
25 | + public function get_option($name) { |
|
26 | + return get_site_option($name); |
|
27 | 27 | } |
28 | 28 | |
29 | 29 | /** |
30 | 30 | * @since 1.0.0 |
31 | 31 | */ |
32 | - protected function add_option( $name, $value ) { |
|
33 | - return add_site_option( $name, $value ); |
|
32 | + protected function add_option($name, $value) { |
|
33 | + return add_site_option($name, $value); |
|
34 | 34 | } |
35 | 35 | |
36 | 36 | /** |
37 | 37 | * @since 1.0.0 |
38 | 38 | */ |
39 | - public function update_option( $name, $value ) { |
|
40 | - return update_site_option( $name, $value ); |
|
39 | + public function update_option($name, $value) { |
|
40 | + return update_site_option($name, $value); |
|
41 | 41 | } |
42 | 42 | |
43 | 43 | /** |
44 | 44 | * @since 1.0.0 |
45 | 45 | */ |
46 | - protected function delete_option( $name ) { |
|
47 | - return delete_site_option( $name ); |
|
46 | + protected function delete_option($name) { |
|
47 | + return delete_site_option($name); |
|
48 | 48 | } |
49 | 49 | } |
50 | 50 |