Completed
Push — master ( 071e73...82b9ae )
by J.D.
02:50
created
src/includes/classes/hook/extension/periods.php 1 patch
Spacing   +53 added lines, -53 removed lines patch added patch discarded remove patch
@@ -28,27 +28,27 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 	}
Please login to merge, or discard this patch.
src/includes/classes/hook/firer.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -17,14 +17,14 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 	}
Please login to merge, or discard this patch.
src/includes/classes/hook/firer/reverse.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -17,21 +17,21 @@
 block discarded – undo
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
 		}
Please login to merge, or discard this patch.
src/includes/classes/hook/firer/spam.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -17,21 +17,21 @@
 block discarded – undo
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
 		}
Please login to merge, or discard this patch.
src/includes/classes/hook/fireri.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@
 block discarded – undo
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
Please login to merge, or discard this patch.
src/includes/classes/hook/reaction.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -53,9 +53,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/includes/classes/hook/reaction/options.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -25,13 +25,13 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 	}
Please login to merge, or discard this patch.
src/includes/classes/hook/reaction/storage/options.php 1 patch
Spacing   +43 added lines, -43 removed lines patch added patch discarded remove patch
@@ -22,25 +22,25 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/includes/classes/hook/reaction/storage/options/network.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -22,29 +22,29 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.