Passed
Push — master ( dd54b3...7daeb2 )
by Joseph
04:02 queued 01:57
created

Metabox   C

Complexity

Total Complexity 56

Size/Duplication

Total Lines 457
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 1

Importance

Changes 0
Metric Value
dl 0
loc 457
rs 5.5199
c 0
b 0
f 0
wmc 56
lcom 1
cbo 1

13 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 8 1
A initMetabox() 0 4 1
A addMetabox() 0 15 2
B visibilityMetabox() 0 85 6
C saveVisibilityMetabox() 0 69 13
A saveMetaboxValues() 0 5 1
A getPostTypes() 0 13 2
A isNonceValid() 0 8 3
A isPostPrivate() 0 23 5
A isPostPublic() 0 12 3
A getAllowedUserRoles() 0 25 4
B showContentToAllowedRoles() 0 29 7
B currentUserCanViewPage() 0 35 8

How to fix   Complexity   

Complex Class

Complex classes like Metabox often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use Metabox, and based on these observations, apply Extract Interface, too.

1
<?php
2
/**
3
 * This file is part of the Subway WordPress Plugin Package.
4
 * This file contains the class which handles the metabox of the plugin.
5
 *
6
 * (c) Joseph G <[email protected]>
7
 *
8
 * For the full copyright and license information, please view the LICENSE
9
 * file that was distributed with this source code.
10
 *
11
 * Props: Jasper Jardin
12
 *
13
 * PHP Version 5.4
14
 *
15
 * @category Subway\Metabox
16
 * @package  Subway
17
 * @author   Joseph G. <[email protected]>
18
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
19
 * @version  GIT:github.com/codehaiku/subway
20
 * @link     github.com/codehaiku/subway The Plugin Repository
21
 */
22
23
namespace Subway;
24
25
if ( ! defined( 'ABSPATH' ) ) {
26
	return;
27
}
28
29
/**
30
 * Subway Metabox methods.
31
 *
32
 * @category Subway\Metabox
33
 * @package  Subway
34
 * @author   Jasper J. <[email protected]>
35
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
36
 * @link     github.com/codehaiku/subway The Plugin Repository
37
 * @since    2.0.9
38
 */
39
final class Metabox {
40
41
42
	/**
43
	 * Subway visibility meta value,
44
	 *
45
	 * @since 2.0.9
46
	 * @const string VISIBILITY_METAKEY
47
	 */
48
	const VISIBILITY_METAKEY = 'subway_visibility_meta_key';
49
50
	/**
51
	 * Registers and update metabox with its intended method below.
52
	 *
53
	 * @since  2.0.9
54
	 * @return void
0 ignored issues
show
Comprehensibility Best Practice introduced by
Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

Adding a @return annotation to a constructor is not recommended, since a constructor does not have a meaningful return value.

Please refer to the PHP core documentation on constructors.

Loading history...
55
	 */
56
	public function __construct() {
57
58
		add_action( 'add_meta_boxes', array( $this, 'addMetabox' ) );
59
		add_action( 'save_post', array( $this, 'saveMetaboxValues' ) );
60
		add_filter( 'the_content', array( $this, 'showContentToAllowedRoles' ) );
61
62
		return $this;
0 ignored issues
show
Bug introduced by
Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?
Loading history...
63
	}
64
65
	/**
66
	 * Initialize metabox
67
	 *
68
	 * @since  2.0.9
69
	 * @access public
70
	 * @return void
71
	 */
72
	public static function initMetabox() {
73
74
		new Metabox();
75
	}
76
77
	/**
78
	 * Initialize metabox
79
	 *
80
	 * @since  2.0.9
81
	 * @access public
82
	 * @return void
83
	 */
84
	public function addMetabox() {
85
86
		$post_types = $this->getPostTypes();
0 ignored issues
show
Bug introduced by
Are you sure the assignment to $post_types is correct as $this->getPostTypes() (which targets Subway\Metabox::getPostTypes()) seems to always return null.

This check looks for function or method calls that always return null and whose return value is assigned to a variable.

class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

The method getObject() can return nothing but null, so it makes no sense to assign that value to a variable.

The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.

Loading history...
87
88
		foreach ( $post_types as $post_type => $value ) {
0 ignored issues
show
Bug introduced by
The expression $post_types of type null is not traversable.
Loading history...
89
			add_meta_box(
90
				'subway_visibility_metabox',
91
				esc_html__( 'Subway: Visibility Option', 'subway' ),
92
				array( $this, 'visibilityMetabox' ),
93
				$post_type,
94
				'side',
95
				'high'
96
			);
97
		}
98
	}
99
100
	/**
101
	 * This method displays the Subway Visibility checkbox.
102
	 *
103
	 * @param object $post Contains data from the current post.
104
	 *
105
	 * @since  2.0.9
106
	 * @access public
107
	 * @return void
108
	 */
109
	public function visibilityMetabox( $post ) {
110
111
		$howto = __(
112
			'Choose the accessibility of this page from the options above.',
113
			'subway'
114
		);
115
116
		$private_setting_label = __( 'Members Only', 'subway' );
0 ignored issues
show
Unused Code introduced by
$private_setting_label is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
117
118
		$is_post_private         = self::isPostPrivate( $post->ID );
119
120
		// Make sure the form request comes from WordPress
121
		wp_nonce_field( basename( __FILE__ ),  'subway_post_visibility_nonce' );
122
123
		// Disable the options (radio) when site is selected as public
124
		?>
125
		<input type="hidden" name="subway-visibility-form-submitted" value="1" />
126
127
		<?php if ( ! Options::isPublicSite() ) :  ?>
128
		<?php // Site is private. Give them some Beer! ?>
129
			<p>
130
				<label class="subway-visibility-settings-checkbox-label" for="subway-visibility-public">
131
					<input type="radio" class="subway-visibility-settings-radio" id="subway-visibility-public" name="subway-visibility-settings" value="public" <?php echo checked( false, $is_post_private, false ); ?>>
132
					<?php esc_html_e( 'Public', 'subway' ) ?>
133
				</label>
134
			</p>
135
			<p>
136
				<label class="subway-visibility-settings-checkbox-label" for="subway-visibility-private">
137
					<input type="radio" class="subway-visibility-settings-radio" id="subway-visibility-private" name="subway-visibility-settings"
138
					value="private" <?php echo checked( true, $is_post_private, false ); ?>>
139
					<?php esc_html_e( 'Members Only', 'subway' ) ?>
140
				 </label>
141
			</p>
142
			<div id="subway-roles-access-visibility-fields" class="hidden">
143
				<dl>
144
					<?php $post_allowed_user_roles = self::getAllowedUserRoles( $post->ID ); ?>
145
					<?php $editable_roles = get_editable_roles(); ?>
146
					<?php // Remove administrator for editable roles. ?>
147
					<?php unset( $editable_roles['administrator'] ); ?>
148
					<?php foreach ( $editable_roles as $role_name => $role_info ) { ?>
149
						<dt>
150
							<?php $id = 'subway-visibility-settings-user-role-' . esc_html( $role_name ); ?>
151
							<label for="<?php echo esc_attr( $id ); ?>">
152
							<?php if ( is_array( $post_allowed_user_roles ) && in_array( $role_name, $post_allowed_user_roles ) ) { ?>
153
								<?php $checked = 'checked'; ?>
154
							<?php } else { ?>
155
								<?php if ( false === $post_allowed_user_roles ) { ?>
156
									<?php $checked = 'checked'; ?>
157
								<?php } else { ?>
158
										<?php $checked = ''; ?>
159
								<?php } ?>
160
							<?php } ?>
161
							<input <?php echo esc_attr( $checked ); ?> id="<?php echo esc_attr( $id ); ?>" type="checkbox" 
162
							name="subway-visibility-settings-user-role[]" class="subway-visibility-settings-role-access" value="<?php echo esc_attr( $role_name ); ?>" />
163
								<?php echo esc_html( $role_info['name'] ); ?>
164
							</label>
165
						</dt>
166
					<?php } ?>
167
					<p class="howto"><?php echo esc_html_e( 'Uncheck the user roles that you do not want to have access to this content','subway' ); ?></p>
168
				</dl>
169
			</div>
170
			<script>
171
				jQuery(document).ready(function(){
172
					'use strict';
173
					if ( $('#subway-visibility-private').is(':checked') ) {
174
						$('#subway-roles-access-visibility-fields').css('display', 'block');
175
					}
176
					$('.subway-visibility-settings-radio').click(function(){
177
						$('#subway-roles-access-visibility-fields').css('display', 'none');
178
						if ( $('#subway-visibility-private').is(':checked') ) {
179
							$('#subway-roles-access-visibility-fields').css('display', 'block');
180
						}
181
					});
182
				});
183
			</script>
184
			<p class="howto"><?php echo esc_html( $howto ); ?></p>
185
		<?php else : ?>
186
			<?php // Site is public! Explain to them ?>
187
			<p><em>
188
				<?php esc_html_e( 'You have chosen to make your site public inside Settings > Subway.  Subway visibility options will be turned off.', 'subway' ); ?>
189
			</em>
190
			</p>
191
		<?php endif; ?>
192
		<?php
193
	}
194
195
	/**
196
	 * This method verify if nonce is valid then updates a post_meta.
197
	 *
198
	 * @param integer $post_id Contains ID of the current post.
199
	 *
200
	 * @since  2.0.9
201
	 * @access public
202
	 * @return boolean false Returns false if nonce is not valid.
203
	 */
204
	public function saveVisibilityMetabox( $post_id = '' ) {
205
206
		if ( wp_is_post_autosave( $post_id ) ) {
207
			return;
208
		}
209
210
		$is_form_submitted = filter_input( INPUT_POST, 'subway-visibility-form-submitted', FILTER_DEFAULT );
211
212
		if ( ! $is_form_submitted ) {
213
			return;
214
		}
215
216
		$public_posts     = Options::getPublicPostsIdentifiers();
217
218
		$posts_implode    = '';
0 ignored issues
show
Unused Code introduced by
$posts_implode is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
219
220
		$visibility_field = 'subway-visibility-settings';
221
222
		$visibility_nonce = filter_input(
223
			INPUT_POST, 'subway_post_visibility_nonce',
224
			FILTER_SANITIZE_STRING
225
		);
226
227
		$post_visibility = filter_input( INPUT_POST,  $visibility_field, FILTER_SANITIZE_STRING );
228
229
		$is_valid_visibility_nonce = self::isNonceValid( $visibility_nonce );
230
231
		$allowed_roles = filter_input( INPUT_POST, 'subway-visibility-settings-user-role', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
232
233
		// verify taxonomies meta box nonce
234
		if ( false === $is_valid_visibility_nonce ) {
235
			return;
236
		}
237
		if ( empty( $allowed_roles ) ) {
238
			$allowed_roles = array();
239
		}
240
241
		// Update user roles.
242
		update_post_meta( $post_id, 'subway-visibility-settings-allowed-user-roles', $allowed_roles );
243
244
		if ( ! empty( $post_visibility ) ) {
245
			if ( ! empty( $post_id ) ) {
246
				if ( 'public' === $post_visibility ) {
247
					if ( ! in_array( $post_id, $public_posts ) ) {
248
						array_push( $public_posts, $post_id );
249
					}
250
				}
251
				if ( 'private' === $post_visibility ) {
252
					unset( $public_posts[ array_search( $post_id, $public_posts ) ] );
253
				}
254
			}
255
		}
256
257
		if ( ! empty( $post_id ) ) {
258
			$posts_implode = implode( ', ', $public_posts );
259
260
			if ( 'inherit' !== get_post_status( $post_id ) ) {
261
262
				if ( true === $is_valid_visibility_nonce ) {
263
					update_option( 'subway_public_post', $posts_implode );
264
					update_post_meta(
265
						$post_id,
266
						self::VISIBILITY_METAKEY,
267
						$post_visibility
268
					);
269
				}
270
			}
271
		}
272
	}
273
274
	/**
275
	 * This method runs the methods that handles the update for a post_meta.
276
	 *
277
	 * @param integer $post_id Contains ID of the current post.
278
	 *
279
	 * @since  2.0.9
280
	 * @access public
281
	 * @return boolean false Returns false if nonce is not valid.
282
	 */
283
	public function saveMetaboxValues( $post_id ) {
284
285
		$this->saveVisibilityMetabox( $post_id );
286
		return;
287
	}
288
289
	/**
290
	 * Initialize metabox arguments.
291
	 *
292
	 * @param array  $args   The arguments for the get_post_types().
293
	 * @param string $output Your desired output for the data.
294
	 *
295
	 * @since  2.0.9
296
	 * @access public
297
	 * @return void
298
	 */
299
	public static function getPostTypes( $args = '', $output = '' ) {
300
301
		if ( empty( $args ) ) {
302
			$args = array(
303
			'public'   => true,
304
			);
305
			$output = 'names';
306
		}
307
308
		$post_types = get_post_types( $args, $output );
309
310
		return $post_types;
311
	}
312
313
	/**
314
	 * This method verify if nonce is valid.
315
	 *
316
	 * @param mixed $nonce the name of a metabox nonce.
317
	 *
318
	 * @since  2.0.9
319
	 * @access public
320
	 * @return boolean true Returns true if nonce is valid.
321
	 */
322
	public function isNonceValid( $nonce ) {
323
324
		if ( ! isset( $nonce ) || ! wp_verify_nonce( $nonce, basename( __FILE__ ) ) ) {
325
			return;
326
		}
327
328
		return true;
329
	}
330
331
	/**
332
	 * Checks if a post is set to private.
333
	 *
334
	 * @param integer $post_id Contains ID of the current post.
335
	 *
336
	 * @since  2.0.9
337
	 * @access public
338
	 * @return boolean true Returns true if post is private. Otherwise false.
339
	 */
340
	public static function isPostPrivate( $post_id ) {
341
342
		$meta_value = '';
0 ignored issues
show
Unused Code introduced by
$meta_value is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
343
344
		if ( ! empty( $post_id ) ) {
345
			$meta_value = get_post_meta( $post_id, self::VISIBILITY_METAKEY, true );
346
347
			// New page or old pages that don't have Subway'\ Visibility Options
348
			if ( empty( $meta_value ) ) {
349
				// Get the value from the general settings (Settings > Subway)
350
				$is_site_public = Options::isPublicSite();
351
				if ( ! $is_site_public ) {
352
					// It's private.
353
					return true;
354
				}
355
			}
356
			if ( 'private' === $meta_value ) {
357
				return true;
358
			}
359
		}
360
361
		return false;
362
	}
363
364
	/**
365
	 * Checks if a post is set to public.
366
	 *
367
	 * @param integer $post_id Contains ID of the current post.
368
	 *
369
	 * @since  2.0.9
370
	 * @access public
371
	 * @return boolean true Returns true if post is public. Otherwise false.
372
	 */
373
	public static function isPostPublic( $post_id ) {
374
375
		$public_post = Options::getPublicPostsIdentifiers();
376
377
		if ( ! empty( $post_id ) ) {
378
			if ( ! in_array( $post_id, $public_post, true ) ) {
379
				return true;
380
			}
381
		}
382
383
		return false;
384
	}
385
386
	/**
387
	 * Get the allowed users roles
388
	 *
389
	 * @param  integer $post_id The post ID.
390
	 * @return mixed Boolean false if metadata does not exists. Otherwise, return the array value of meta.
391
	 */
392
	public static function getAllowedUserRoles( $post_id = 0 ) {
393
394
		$allowed_roles = array();
0 ignored issues
show
Unused Code introduced by
$allowed_roles is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
395
396
		if ( ! empty( $post_id ) ) {
397
398
			// Check if metadata exists for the following post.
399
			if ( metadata_exists( 'post', $post_id, 'subway-visibility-settings-allowed-user-roles' ) ) {
400
401
				$allowed_roles = get_post_meta( $post_id, 'subway-visibility-settings-allowed-user-roles', true );
402
				if ( ! is_null( $allowed_roles ) ) {
403
					return $allowed_roles;
404
				}
405
				return false;
406
				
407
			} else {
408
				return false;
409
			}
410
411
		} else {
412
			return false;
413
		}
414
415
		return $allowed_roles;
0 ignored issues
show
Unused Code introduced by
return $allowed_roles; does not seem to be reachable.

This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed.

Unreachable code is most often the result of return, die or exit statements that have been added for debug purposes.

function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}

In the above example, the last return false will never be executed, because a return statement has already been met in every possible execution path.

Loading history...
416
	}
417
418
	/**
419
	 * Check if the current user has role for the current content.
420
	 *
421
	 * @param  string $content The content of the post.
422
	 * @return string The content of the post.
423
	 */
424
	public function showContentToAllowedRoles( $content ) {
425
426
		$post_id = get_the_ID();
427
		$allowed_user_roles = self::getAllowedUserRoles( $post_id );
428
429
		if ( ! is_singular() && is_main_query() ) {
430
			return $content;
431
		}
432
433
		if ( is_user_logged_in() ) {
434
435
			$no_privilege = '<div class="subway-role-not-allowed"><p>' . apply_filters( 'subway-content-restricted-to-role', esc_html__( 'You do not have the right privilege or role to view this page.', 'subway' ) ) . '</p></div>';
436
437
			// Restrict access to non admins only.
438
			if ( ! current_user_can( 'manage_options' ) ) {
439
				if ( ! self::currentUserCanViewPage( $post_id ) ) {
440
					return $no_privilege;
441
				}	
442
			}
443
444
			// Return the content if the post is not yet saved.
445
			if ( false === $allowed_user_roles ) {
446
				return $content;
447
			}
448
		}
449
450
		return $content;
451
452
	}
453
454
	/**
455
	 * Check to see if current user has a specific roles to view the page
456
	 * 
457
	 * @return boolean True on success. Otherwise, false.
458
	 */
459
	public function currentUserCanViewPage( $post_id = 0 ) {
460
461
		$allowed_roles = self::getAllowedUserRoles( $post_id );
462
		
463
		// if $allowed_roles is not set, it means meta data is not yet available.
464
		// Post roles are checked but are not yet save. So allow viewing.
465
		if ( ! $allowed_roles ) {
466
			return true;
467
		}
468
469
		// Only check for logged-in users.
470
		if ( is_user_logged_in() ) {
471
			
472
			$can_view = false;
473
474
			$user = wp_get_current_user();
475
476
			if ( ! is_array( $user->roles ) ) {
477
				$user->roles = (array) $user->roles;
478
			}
479
480
			if ( ! empty( $user->roles ) && is_array( $user->roles ) ) {
481
				foreach( $user->roles as $current_user_role ) {
482
					if ( in_array( $current_user_role, $allowed_roles ) ) {
483
						$can_view = true;
484
					}
485
				}
486
			}
487
488
			return $can_view;
489
		
490
		}
491
492
		return false;
493
	}
494
495
}
496