Completed
Pull Request — trunk (#584)
by Juliette
05:07
created

CMB2_hookup::show_on()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 15
Code Lines 4

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2
Metric Value
dl 0
loc 15
ccs 0
cts 4
cp 0
rs 9.4285
cc 1
eloc 4
nc 1
nop 0
crap 2
1
<?php
2
/**
3
 * Handles hooking CMB2 forms/metaboxes into the post/attachement/user screens
4
 * and handles hooking in and saving those fields.
5
 *
6
 * @since  2.0.0
7
 *
8
 * @category  WordPress_Plugin
9
 * @package   CMB2
10
 * @author    WebDevStudios
11
 * @license   GPL-2.0+
12
 * @link      http://webdevstudios.com
13
 */
14
class CMB2_hookup {
0 ignored issues
show
Coding Style Compatibility introduced by
PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

You can fix this by adding a namespace to your class:

namespace YourVendor;

class YourClass { }

When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.

Loading history...
15
16
	/**
17
	 * Array of all hooks done (to be run once)
18
	 * @var   array
19
	 * @since 2.0.0
20
	 */
21
	protected static $hooks_completed = array();
22
23
	/**
24
	 * Only allow JS registration once
25
	 * @var   bool
26
	 * @since 2.0.7
27
	 */
28
	protected static $js_registration_done = false;
29
30
	/**
31
	 * Only allow CSS registration once
32
	 * @var   bool
33
	 * @since 2.0.7
34
	 */
35
	protected static $css_registration_done = false;
36
37
	/**
38
	 * @var   CMB2 object
39
	 * @since 2.0.2
40
	 */
41
	protected $cmb;
42
43
	/**
44
	 * CMB taxonomies array for term meta
45
	 * @var   array
46
	 * @since 2.2.0
47
	 */
48
	protected $taxonomies = array();
49
50
	/**
51
	 * The object type we are performing the hookup for
52
	 * @var   string
53
	 * @since 2.0.9
54
	 */
55
	protected $object_type = 'post';
56
57
	public function __construct( CMB2 $cmb ) {
58
		$this->cmb = $cmb;
59
		$this->object_type = $this->cmb->mb_object_type();
60
61
		$this->universal_hooks();
62
63
		if ( is_admin() ) {
64
65
			switch ( $this->object_type ) {
66
				case 'post':
67
					return $this->post_hooks();
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...
68
				case 'comment':
69
					return $this->comment_hooks();
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...
70
				case 'user':
71
					return $this->user_hooks();
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...
72
				case 'term':
73
					return $this->term_hooks();
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...
74
			}
75
76
		}
77
	}
78
79
	public function universal_hooks() {
80
		// Handle oembed Ajax
81
		$this->once( 'wp_ajax_cmb2_oembed_handler', array( cmb2_ajax(), 'oembed_handler' ) );
82
		$this->once( 'wp_ajax_nopriv_cmb2_oembed_handler', array( cmb2_ajax(), 'oembed_handler' ) );
83
84
		foreach ( get_class_methods( 'CMB2_Show_Filters' ) as $filter ) {
85
			add_filter( 'cmb2_show_on', array( 'CMB2_Show_Filters', $filter ), 10, 3 );
86
		}
87
88
		if ( is_admin() ) {
89
			// register our scripts and styles for cmb
90
			$this->once( 'admin_enqueue_scripts', array( __CLASS__, 'register_scripts' ), 8 );
91
		}
92
	}
93
94
	public function post_hooks() {
95
		add_action( 'add_meta_boxes', array( $this, 'add_metaboxes' ) );
96
		add_action( 'add_attachment', array( $this, 'save_post' ) );
97
		add_action( 'edit_attachment', array( $this, 'save_post' ) );
98
		add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
99
100
		$this->once( 'admin_enqueue_scripts', array( $this, 'do_scripts' ) );
101
	}
102
103
	public function comment_hooks() {
104
		add_action( 'add_meta_boxes_comment', array( $this, 'add_metaboxes' ) );
105
		add_action( 'edit_comment', array( $this, 'save_comment' ) );
106
107
		$this->once( 'admin_enqueue_scripts', array( $this, 'do_scripts' ) );
108
	}
109
110
	public function user_hooks() {
111
		$priority = $this->get_priority();
112
113
		add_action( 'show_user_profile', array( $this, 'user_metabox' ), $priority );
114
		add_action( 'edit_user_profile', array( $this, 'user_metabox' ), $priority );
115
		add_action( 'user_new_form', array( $this, 'user_new_metabox' ), $priority );
116
117
		add_action( 'personal_options_update', array( $this, 'save_user' ) );
118
		add_action( 'edit_user_profile_update', array( $this, 'save_user' ) );
119
		add_action( 'user_register', array( $this, 'save_user' ) );
120
	}
121
122
	public function term_hooks() {
123
		if ( ! function_exists( 'get_term_meta' ) ) {
124
			wp_die( __( 'Term Metadata is a WordPress > 4.4 feature. Please upgrade your WordPress install.', 'cmb2' ) );
125
		}
126
127
		if ( ! $this->cmb->prop( 'taxonomies' ) ) {
128
			wp_die( __( 'Term metaboxes configuration requires a \'taxonomies\' parameter', 'cmb2' ) );
129
		}
130
131
		$this->taxonomies = (array) $this->cmb->prop( 'taxonomies' );
132
		$show_on_term_add = $this->cmb->prop( 'new_term_section' );
133
		$priority         = $this->get_priority( 8 );
134
135
		foreach ( $this->taxonomies as $taxonomy ) {
136
			// Display our form data
137
			add_action( "{$taxonomy}_edit_form", array( $this, 'term_metabox' ), $priority, 2 );
138
139
			$show_on_add = is_array( $show_on_term_add )
140
				? in_array( $taxonomy, $show_on_term_add )
141
				: (bool) $show_on_term_add;
142
143
			$show_on_add = apply_filters( "cmb2_show_on_term_add_form_{$this->cmb->cmb_id}", $show_on_add, $this->cmb );
144
145
			// Display form in add-new section (unless specified not to)
146
			if ( $show_on_add ) {
147
				add_action( "{$taxonomy}_add_form_fields", array( $this, 'term_metabox' ), $priority, 2 );
148
			}
149
150
		}
151
152
		add_action( 'created_term', array( $this, 'save_term' ), 10, 3 );
153
		add_action( 'edited_terms', array( $this, 'save_term' ), 10, 2 );
154
155
		add_action( 'delete_term', array( $this, 'delete_term' ), 10, 3 );
156
	}
157
158
	/**
159
	 * Registers styles for CMB2
160
	 * @since 2.0.7
161
	 */
162 1
	protected static function register_styles() {
163 1
		if ( self::$css_registration_done ) {
164
			return;
165
		}
166
167
		// Only use minified files if SCRIPT_DEBUG is off
168 1
		$min   = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
169 1
		$front = is_admin() ? '' : '-front';
170 1
		$rtl = is_rtl() ? '-rtl' : '';
171
172
		// Filter required styles and register stylesheet
173 1
		$styles = apply_filters( 'cmb2_style_dependencies', array() );
174 1
		wp_register_style( 'cmb2-styles', cmb2_utils()->url( "css/cmb2{$front}{$rtl}{$min}.css" ), $styles );
175
176 1
		self::$css_registration_done = true;
177 1
	}
178
179
	/**
180
	 * Registers scripts for CMB2
181
	 * @since  2.0.7
182
	 */
183 1
	protected static function register_js() {
184 1
		if ( self::$js_registration_done ) {
185
			return;
186
		}
187
188 1
		$hook = is_admin() ? 'admin_footer' : 'wp_footer';
189 1
		add_action( $hook, array( 'CMB2_JS', 'enqueue' ), 8 );
190
191 1
		self::$js_registration_done = true;
192 1
	}
193
194
	/**
195
	 * Registers scripts and styles for CMB2
196
	 * @since  1.0.0
197
	 */
198
	public static function register_scripts() {
199
		self::register_styles();
200
		self::register_js();
201
	}
202
203
	/**
204
	 * Enqueues scripts and styles for CMB2
205
	 * @since  1.0.0
206
	 */
207
	public function do_scripts() {
208
		$screen = get_current_screen();
209
210
		if ( ! property_exists( $screen, 'base' ) || ! property_exists( $screen, 'post_type' ) ) {
211
			return;
212
		}
213
214
		// Only enqueue our scripts/styles on the proper pages.
215
		if ( ( 'post' === $screen->base && ( is_string( $screen->post_type ) && '' !== $screen->post_type ) ) ||
216
			'comment' === $screen->base ) {
217
			if ( $this->cmb->prop( 'cmb_styles' ) ) {
218
				self::enqueue_cmb_css();
219
			}
220
			if ( $this->cmb->prop( 'enqueue_js' ) ) {
221
				self::enqueue_cmb_js();
222
			}
223
		}
224
	}
225
226
	/**
227
	 * Add metaboxes (to 'post' or 'comment' object types)
228
	 * @since 1.0.0
229
	 */
230
	public function add_metaboxes() {
231
232
		if ( ! $this->show_on() ) {
233
			return;
234
		}
235
236
		foreach ( $this->cmb->prop( 'object_types' ) as $post_type ) {
237
			/**
238
			 * To keep from registering an actual post-screen metabox,
239
			 * omit the 'title' attribute from the metabox registration array.
240
			 *
241
			 * (WordPress will not display metaboxes without titles anyway)
242
			 *
243
			 * This is a good solution if you want to output your metaboxes
244
			 * Somewhere else in the post-screen
245
			 */
246
			if ( $this->cmb->prop( 'title' ) ) {
247
248
				if ( $this->cmb->prop( 'closed' ) ) {
249
					add_filter( "postbox_classes_{$post_type}_{$this->cmb->cmb_id}", array( $this, 'close_metabox_class' ) );
250
				}
251
252
				add_meta_box( $this->cmb->cmb_id, $this->cmb->prop( 'title' ), array( $this, 'metabox_callback' ), $post_type, $this->cmb->prop( 'context' ), $this->cmb->prop( 'priority' ) );
253
			}
254
		}
255
	}
256
257
	/**
258
	 * Add 'closed' class to metabox
259
	 * @since  2.0.0
260
	 * @param  array  $classes Array of classes
261
	 * @return array           Modified array of classes
262
	 */
263
	public function close_metabox_class( $classes ) {
264
		$classes[] = 'closed';
265
		return $classes;
266
	}
267
268
	/**
269
	 * Display metaboxes for a post or comment object
270
	 * @since  1.0.0
271
	 */
272
	public function metabox_callback() {
273
		$object_id = 'comment' == $this->object_type ? get_comment_ID() : get_the_ID();
274
		$this->cmb->show_form( $object_id, $this->object_type );
275
	}
276
277
	/**
278
	 * Display metaboxes for new user page
279
	 * @since  1.0.0
280
	 */
281
	public function user_new_metabox( $section ) {
0 ignored issues
show
Coding Style introduced by
user_new_metabox uses the super-global variable $_REQUEST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
282
		if ( $section == $this->cmb->prop( 'new_user_section' ) ) {
283
			$object_id = $this->cmb->object_id();
284
			$this->cmb->object_id( isset( $_REQUEST['user_id'] ) ? $_REQUEST['user_id'] : $object_id );
285
			$this->user_metabox();
286
		}
287
	}
288
289
	/**
290
	 * Display metaboxes for a user object
291
	 * @since  1.0.0
292
	 */
293
	public function user_metabox() {
294
		$this->show_form_for_type( 'user' );
295
	}
296
297
	/**
298
	 * Display metaboxes for a taxonomy term object
299
	 * @since  2.2.0
300
	 */
301
	public function term_metabox() {
302
		$this->show_form_for_type( 'term' );
303
	}
304
305
	/**
306
	 * Display metaboxes for an object type
307
	 * @since  2.2.0
308
	 * @param  string $type Object type
309
	 * @return void
310
	 */
311
	public function show_form_for_type( $type ) {
312
		if ( $type != $this->cmb->mb_object_type() ) {
313
			return;
314
		}
315
316
		if ( ! $this->show_on() ) {
317
			return;
318
		}
319
320
		if ( $this->cmb->prop( 'cmb_styles' ) ) {
321
			self::enqueue_cmb_css();
322
		}
323
		if ( $this->cmb->prop( 'enqueue_js' ) ) {
324
			self::enqueue_cmb_js();
325
		}
326
327
		$this->cmb->show_form( 0, $type );
328
	}
329
330
	/**
331
	 * Determines if metabox should be shown in current context
332
	 * @since  2.0.0
333
	 * @return bool Whether metabox should be added/shown
334
	 */
335
	public function show_on() {
336
		// If metabox is requesting to be conditionally shown
337
		$show = $this->cmb->should_show();
338
339
		/**
340
		 * Filter to determine if metabox should show. Default is true
341
		 *
342
		 * @param array  $show          Default is true, show the metabox
343
		 * @param mixed  $meta_box_args Array of the metabox arguments
344
		 * @param mixed  $cmb           The CMB2 instance
345
		 */
346
		$show = (bool) apply_filters( 'cmb2_show_on', $show, $this->cmb->meta_box, $this->cmb );
347
348
		return $show;
349
	}
350
351
	/**
352
	 * Get the CMB priority property set to numeric hook priority.
353
	 * @since  2.2.0
354
	 * @param  integer $default Default display hook priority.
355
	 * @return integer          Hook priority.
356
	 */
357
	public function get_priority( $default = 10 ) {
358
		$priority = $this->cmb->prop( 'priority' );
359
360
		if ( ! is_numeric( $priority ) ) {
361
			switch ( $priority ) {
362
363
				case 'high':
364
					$priority = 5;
365
					break;
366
367
				case 'low':
368
					$priority = 20;
369
					break;
370
371
				default:
372
					$priority = $default;
373
					break;
374
			}
375
		}
376
377
		return $priority;
378
	}
379
380
	/**
381
	 * Save data from post metabox
382
	 * @since  1.0.0
383
	 * @param  int    $post_id Post ID
384
	 * @param  mixed  $post    Post object
385
	 * @return null
386
	 */
387
	public function save_post( $post_id, $post = false ) {
0 ignored issues
show
Coding Style introduced by
save_post uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
388
389
		$post_type = $post ? $post->post_type : get_post_type( $post_id );
390
391
		$do_not_pass_go = (
392
			! $this->can_save( $post_type )
393
			// check user editing permissions
394
			|| ( 'page' == $post_type && ! current_user_can( 'edit_page', $post_id ) )
395
			|| ! current_user_can( 'edit_post', $post_id )
396
		);
397
398
		if ( $do_not_pass_go ) {
399
			// do not collect $200
400
			return;
401
		}
402
403
		// take a trip to reading railroad – if you pass go collect $200
404
		$this->cmb->save_fields( $post_id, 'post', $_POST );
405
	}
406
407
	/**
408
	 * Save data from comment metabox
409
	 * @since  2.0.9
410
	 * @param  int    $comment_id Comment ID
411
	 * @return null
412
	 */
413
	public function save_comment( $comment_id ) {
0 ignored issues
show
Coding Style introduced by
save_comment uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
414
415
		$can_edit = current_user_can( 'moderate_comments', $comment_id );
416
417
		if ( $this->can_save( get_comment_type( $comment_id ) ) && $can_edit ) {
418
			$this->cmb->save_fields( $comment_id, 'comment', $_POST );
419
		}
420
	}
421
422
	/**
423
	 * Save data from user fields
424
	 * @since  1.0.x
425
	 * @param  int   $user_id  User ID
426
	 * @return null
427
	 */
428
	public function save_user( $user_id ) {
0 ignored issues
show
Coding Style introduced by
save_user uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
429
		// check permissions
430
		if ( $this->can_save( 'user' ) ) {
431
			$this->cmb->save_fields( $user_id, 'user', $_POST );
432
		}
433
	}
434
435
	/**
436
	 * Save data from term fields
437
	 * @since  1.0.x
438
	 * @param  int    $term_id  Term ID
439
	 * @param  int    $tt_id    Term Taxonomy ID
440
	 * @param  string $taxonomy Taxonomy
441
	 * @return null
442
	 */
443
	public function save_term( $term_id, $tt_id, $taxonomy = '' ) {
0 ignored issues
show
Coding Style introduced by
save_term uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
444
		$taxonomy = $taxonomy ? $taxonomy : $tt_id;
445
446
		// check permissions
447
		if ( $this->taxonomy_can_save( $taxonomy ) && $this->can_save( 'term' ) ) {
448
			$this->cmb->save_fields( $term_id, 'term', $_POST );
449
		}
450
	}
451
452
	/**
453
	 * Delete term meta when a term is deleted.
454
	 * @since  1.0.x
455
	 * @param  int    $term_id  Term ID
456
	 * @param  int    $tt_id    Term Taxonomy ID
457
	 * @param  string $taxonomy Taxonomy
458
	 * @return null
459
	 */
460
	public function delete_term( $term_id, $tt_id, $taxonomy = '' ) {
461
		if ( $this->taxonomy_can_save( $taxonomy ) ) {
462
463
			foreach ( $this->cmb->prop( 'fields' ) as $field ) {
464
				$data_to_delete[ $field['id'] ] = '';
0 ignored issues
show
Coding Style Comprehensibility introduced by
$data_to_delete was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data_to_delete = array(); before regardless.

Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code.

Let’s take a look at an example:

foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

As you can see in this example, the array $myArray is initialized the first time when the foreach loop is entered. You can also see that the value of the bar key is only written conditionally; thus, its value might result from a previous iteration.

This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization $myArray = array() either outside or inside the foreach loop.

Loading history...
465
			}
466
467
			$this->cmb->save_fields( $term_id, 'term', $data_to_delete );
0 ignored issues
show
Bug introduced by
The variable $data_to_delete does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
468
		}
469
	}
470
471
	/**
472
	 * Determines if the current object is able to be saved
473
	 * @since  2.0.9
474
	 * @param  string  $type Current post_type or comment_type
475
	 * @return bool          Whether object can be saved
476
	 */
477
	public function can_save( $type = '' ) {
0 ignored issues
show
Coding Style introduced by
can_save uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
478
		return (
479
			$this->cmb->prop( 'save_fields' )
480
			// check nonce
481
			&& isset( $_POST[ $this->cmb->nonce() ] )
482
			&& wp_verify_nonce( $_POST[ $this->cmb->nonce() ], $this->cmb->nonce() )
483
			// check if autosave
484
			&& ! ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
485
			// get the metabox types & compare it to this type
486
			&& ( $type && in_array( $type, $this->cmb->prop( 'object_types' ) ) )
487
		);
488
	}
489
490
	/**
491
	 * Determine if taxonomy of term being modified is cmb2-editable.
492
	 * @since  2.2.0
493
	 * @param  string $taxonomy Taxonomy of term being modified.
494
	 * @return bool             Whether taxonomy is editable.
495
	 */
496
	public function taxonomy_can_save( $taxonomy ) {
497
		$taxonomy = $taxonomy ? $taxonomy : $tt_id;
0 ignored issues
show
Bug introduced by
The variable $tt_id does not exist. Did you forget to declare it?

This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug.

Loading history...
498
499
		if ( empty( $this->taxonomies ) || ! in_array( $taxonomy, $this->taxonomies ) ) {
500
			return false;
501
		}
502
503
		$taxonomy_object = get_taxonomy( $taxonomy );
504
		// Can the user edit this term?
505
		if ( ! isset( $taxonomy_object->cap ) || ! current_user_can( $taxonomy_object->cap->edit_terms ) ) {
506
			return false;
507
		}
508
509
		return true;
510
	}
511
512
	/**
513
	 * Ensures WordPress hook only gets fired once
514
	 * @since  2.0.0
515
	 * @param string   $action        The name of the filter to hook the $hook callback to.
516
	 * @param callback $hook          The callback to be run when the filter is applied.
517
	 * @param integer  $priority      Order the functions are executed
518
	 * @param int      $accepted_args The number of arguments the function accepts.
519
	 */
520
	public function once( $action, $hook, $priority = 10, $accepted_args = 1 ) {
521
		$key = md5( serialize( func_get_args() ) );
522
523
		if ( in_array( $key, self::$hooks_completed ) ) {
524
			return;
525
		}
526
527
		self::$hooks_completed[] = $key;
528
		add_filter( $action, $hook, $priority, $accepted_args );
529
	}
530
531
	/**
532
	 * Includes CMB2 styles
533
	 * @since  2.0.0
534
	 */
535 1
	public static function enqueue_cmb_css() {
536 1
		if ( ! apply_filters( 'cmb2_enqueue_css', true ) ) {
537
			return false;
538
		}
539
540 1
		self::register_styles();
541 1
		return wp_enqueue_style( 'cmb2-styles' );
542
	}
543
544
	/**
545
	 * Includes CMB2 JS
546
	 * @since  2.0.0
547
	 */
548 1
	public static function enqueue_cmb_js() {
549 1
		if ( ! apply_filters( 'cmb2_enqueue_js', true ) ) {
550
			return false;
551
		}
552
553 1
		self::register_js();
554 1
		return true;
555
	}
556
557
}
558