Completed
Branch BUG-9622-fix-broken-cpt-crud (e3e588)
by
unknown
574:45 queued 558:10
created

EE_Admin_Page::__construct()   A

Complexity

Conditions 3
Paths 4

Size

Total Lines 47
Code Lines 21

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 1
Metric Value
c 1
b 0
f 1
dl 0
loc 47
rs 9.0303
cc 3
eloc 21
nc 4
nop 1
1
<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
2
/**
3
 * Event Espresso
4
 *
5
 * Event Registration and Management Plugin for WordPress
6
 *
7
 * @ package			Event Espresso
8
 * @ author				Seth Shoultes
9
 * @ copyright		(c) 2008-2011 Event Espresso  All Rights Reserved.
10
 * @ license			{@link http://eventespresso.com/support/terms-conditions/}   * see Plugin Licensing *
11
 * @ link				{@link http://www.eventespresso.com}
12
 * @ since		 		4.0
13
 *
14
 * ------------------------------------------------------------------------
15
 */
16
17
18
19
/**
20
 * EE_Admin_Page class
21
 *
22
 * @package		Event Espresso
23
 * @subpackage		includes/core/admin/EE_Admin_Page.core.php
24
 * @abstract
25
 * @author		Brent Christensen, Darren Ethier
26
 *
27
 * ------------------------------------------------------------------------
28
 */
29
abstract class EE_Admin_Page extends EE_BASE {
30
31
32
	//set in _init_page_props()
33
	public $page_slug;
34
	public $page_label;
35
	public $page_folder;
36
37
	//set in define_page_props()
38
	protected $_admin_base_url;
39
	protected $_admin_base_path;
40
	protected $_admin_page_title;
41
	protected $_labels;
42
43
44
	//set early within EE_Admin_Init
45
	protected $_wp_page_slug;
46
47
	//navtabs
48
	protected $_nav_tabs;
49
	protected $_default_nav_tab_name;
50
51
	//helptourstops
52
	protected $_help_tour = array();
53
54
55
	//template variables (used by templates)
56
	protected $_template_path;
57
	protected $_column_template_path;
58
59
	/**
60
	 * @var array $_template_args
61
	 */
62
	protected $_template_args = array();
63
64
	//this will hold the list table object for a given view.
65
	protected $_list_table_object;
66
67
	//bools
68
	protected $_is_UI_request = NULL; //this starts at null so we can have no header routes progress through two states.
69
	protected $_routing;
70
71
	//list table args
72
	protected $_view;
73
	protected $_views;
74
75
76
	//action => method pairs used for routing incoming requests
77
	protected $_page_routes;
78
	protected $_page_config;
79
80
	//the current page route and route config
81
	protected $_route;
82
	protected $_route_config;
83
84
	/**
85
	 * Used to hold default query args for list table routes to help preserve stickiness of filters for carried out
86
	 * actions.
87
	 *
88
	 * @since 4.6.x
89
	 * @var array.
90
	 */
91
	protected $_default_route_query_args;
92
93
	//set via request page and action args.
94
	protected $_current_page;
95
	protected $_current_view;
96
	protected $_current_page_view_url;
97
98
	//sanitized request action (and nonce)
99
	/**
100
	 * @var string $_req_action
101
	 */
102
	protected $_req_action;
103
104
	/**
105
	 * @var string $_req_nonce
106
	 */
107
	protected $_req_nonce;
108
109
	//search related
110
	protected $_search_btn_label;
111
	protected $_search_box_callback;
112
113
	/**
114
	 * WP Current Screen object
115
	 *
116
	 * @var WP_Screen
117
	 */
118
	protected $_current_screen;
119
120
	//for holding EE_Admin_Hooks object when needed (set via set_hook_object())
121
	protected $_hook_obj;
122
123
	//for holding incoming request data
124
	protected $_req_data;
125
126
	// yes / no array for admin form fields
127
	protected $_yes_no_values = array();
128
129
	//some default things shared by all child classes
130
	protected $_default_espresso_metaboxes;
131
132
	/**
133
	 * 	EE_Registry Object
134
	 *	@var 	EE_Registry
135
	 * 	@access 	protected
136
	 */
137
	protected $EE = NULL;
138
139
140
141
	/**
142
	 * This is just a property that flags whether the given route is a caffeinated route or not.
143
	 * @var boolean
144
	 */
145
	protected $_is_caf = FALSE;
146
147
148
149
150
151
	/**
152
	 * 		@Constructor
153
	 *
154
	 * 		@param bool $routing indicate whether we want to just load the object and handle routing or just load the object.
155
	 * 		@access public
156
	 */
157
	public function __construct( $routing = TRUE ) {
158
		EE_Admin::debug_log(
159
			__METHOD__,
160
			array(
161
				'class_name' => get_class( $this ),
162
				'spl_object_hash' => spl_object_hash( $this )
163
			)
164
		);
165
		if ( strpos( $this->_get_dir(), 'caffeinated' ) !== false )
166
			$this->_is_caf = TRUE;
167
168
		$this->_yes_no_values = array(
169
			array('id' => TRUE, 'text' => __('Yes', 'event_espresso')),
170
			array('id' => FALSE, 'text' => __('No', 'event_espresso'))
171
		);
172
173
174
175
		//set the _req_data property.
176
		$this->_req_data = array_merge( $_GET, $_POST );
177
178
179
		//routing enabled?
180
		$this->_routing = $routing;
181
182
		//set initial page props (child method)
183
		$this->_init_page_props();
184
185
		//set global defaults
186
		$this->_set_defaults();
187
188
		//set early because incoming requests could be ajax related and we need to register those hooks.
189
		$this->_global_ajax_hooks();
190
		$this->_ajax_hooks();
191
192
193
		//other_page_hooks have to be early too.
194
		$this->_do_other_page_hooks();
195
196
		//This just allows us to have extending classes do something specific before the parent constructor runs _page_setup.
197
		if ( method_exists( $this, '_before_page_setup' ) )
198
			$this->_before_page_setup();
0 ignored issues
show
Bug introduced by
The method _before_page_setup() does not exist on EE_Admin_Page. Did you maybe mean _page_setup()?

This check marks calls to methods that do not seem to exist on an object.

This is most likely the result of a method being renamed without all references to it being renamed likewise.

Loading history...
199
200
		//set up page dependencies
201
		$this->_page_setup();
202
203
	}
204
205
206
207
208
	/**
209
	 * _init_page_props
210
	 * Child classes use to set at least the following properties:
211
	 * $page_slug.
212
	 * $page_label.
213
	 *
214
	 * @abstract
215
	 * @access protected
216
	 * @return void
217
	 */
218
	abstract protected function _init_page_props();
219
220
221
222
	/**
223
	 * _ajax_hooks
224
	 * child classes put all their add_action('wp_ajax_{name_of_hook}') hooks in here.
225
	 * Note: within the ajax callback methods.
226
	 *
227
	 * @abstract
228
	 * @access protected
229
	 * @return void
230
	 */
231
	abstract protected function _ajax_hooks();
232
233
234
235
236
	/**
237
	 * _define_page_props
238
	 * child classes define page properties in here.  Must include at least:
239
	 * $_admin_base_url = base_url for all admin pages
240
	 * $_admin_page_title = default admin_page_title for admin pages
241
	 * $_labels = array of default labels for various automatically generated elements:
242
	 * 	array(
243
	 * 		'buttons' => array(
244
	 * 			'add' => __('label for add new button'),
245
	 * 	 		'edit' => __('label for edit button'),
246
	 * 	  		'delete' => __('label for delete button')
247
	 * 	  	 	)
248
	 * 	  	)
249
	 *
250
	 * @abstract
251
	 * @access protected
252
	 * @return void
253
	 */
254
	abstract protected function _define_page_props();
255
256
257
258
259
260
	/**
261
	 * _set_page_routes
262
	 * child classes use this to define the page routes for all subpages handled by the class.  Page routes are assigned to a action => method pairs in an array and to the $_page_routes property.  Each page route must also have a 'default' route. Here's the format
263
	 * $this->_page_routes = array(
264
	 * 		'default' => array(
265
	 * 			'func' => '_default_method_handling_route',
266
	 * 			'args' => array('array','of','args'),
267
	 * 			'noheader' => true, //add this in if this page route is processed before any headers are loaded (i.e. ajax request, backend processing)
268
	 *			'headers_sent_route'=>'headers_route_reference', //add this if noheader=>true, and you want to load a headers route after.  The string you enter here should match the defined route reference for a headers sent route.
269
	 *			'capability' => 'route_capability', //indicate a string for minimum capability required to access this route.
270
	 *			'obj_id' => 10 // if this route has an object id, then this can include it (used for capability checks).
271
	 * 		),
272
	 * 		'insert_item' => '_method_for_handling_insert_item' //this can be used if all we need to have is a handling method.
273
	 * 		)
274
	 *
275
	 * )
276
	 *
277
	 *
278
	 * @abstract
279
	 * @access protected
280
	 * @return void
281
	 */
282
	abstract protected function _set_page_routes();
283
284
285
286
287
288
289
	/**
290
	 * _set_page_config
291
	 * child classes use this to define the _page_config array for all subpages handled by the class. Each key in the array corresponds to the page_route for the loaded page.
292
	 * Format:
293
	 * $this->_page_config = array(
294
	 * 		'default' => array(
295
	 * 			'labels' => array(
296
	 * 				'buttons' => array(
297
	 * 					'add' => __('label for adding item'),
298
	 * 				 	'edit' => __('label for editing item'),
299
	 * 				  	'delete' => __('label for deleting item')
300
	 * 			    ),
301
	 * 			    'publishbox' => __('Localized Title for Publish metabox', 'event_espresso')
302
	 * 			), //optional an array of custom labels for various automatically generated elements to use on the page. If this isn't present then the defaults will be used as set for the $this->_labels in _define_page_props() method
303
	 * 			'nav' => array(
304
	 * 				'label' => __('Label for Tab', 'event_espresso').
305
	 *     			'url' => 'http://someurl', //automatically generated UNLESS you define
306
	 *     			'css_class' => 'css-class', //automatically generated UNLESS you define
307
	 *     			'order' => 10, //required to indicate tab position.
308
	 *     			'persistent' => false //if you want the nav tab to ONLY display when the specific route is displayed then add this parameter.
309
	 *     		'list_table' => 'name_of_list_table' //string for list table class to be loaded for this admin_page.
310
	 *     		'metaboxes' => array('metabox1', 'metabox2'), //if present this key indicates we want to load metaboxes set for eventespresso admin pages.
311
	 *     		'has_metaboxes' => true, //this boolean flag can simply be used to indicate if the route will have metaboxes.  Typically this is used if the 'metaboxes' index is not used because metaboxes are added later.  We just use this flag to make sure the necessary js gets enqueued on page load.
312
	 *     		'has_help_popups' => false //defaults(true) //this boolean flag can simply be used to indicate if the given route has help popups setup and if it does then we need to make sure thickbox is enqueued.
313
	 *     		'columns' => array(4, 2), //this key triggers the setup of a page that uses columns (metaboxes).  The array indicates the max number of columns (4) and the default number of columns on page load (2).  There is an option in the "screen_options" dropdown that is setup so users can pick what columns they want to display.
314
	 *     		'help_tabs' => array( //this is used for adding help tabs to a page
315
	 *     			'tab_id' => array(
316
	 *     				'title' => 'tab_title',
317
	 *     				'filename' => 'name_of_file_containing_content', //this is the primary method for setting help tab content.  The fallback if it isn't present is to try a the callback.  Filename should match a file in the admin folder's "help_tabs" dir (ie.. events/help_tabs/name_of_file_containing_content.help_tab.php)
318
	 *     				'callback' => 'callback_method_for_content', //if 'filename' isn't present then system will attempt to use the callback which should match the name of a method in the class
319
	 *     				),
320
	 *     			'tab2_id' => array(
321
	 *     			 	'title' => 'tab2 title',
322
	 *     			 	'filename' => 'file_name_2'
323
	 *     			 	'callback' => 'callback_method_for_content',
324
	 *     			 ),
325
	 *     	   	'help_sidebar' => 'callback_for_sidebar_content', //this is used for setting up the sidebar in the help tab area on an admin page. @link http://make.wordpress.org/core/2011/12/06/help-and-screen-api-changes-in-3-3/
326
	 *     		'help_tour' => array(
327
	 *     			'name_of_help_tour_class', //all help tours shoudl be a child class of EE_Help_Tour and located in a folder for this admin page named "help_tours", a file name matching the key given here (name_of_help_tour_class.class.php), and class matching key given here (name_of_help_tour_class)
328
	 *     		),
329
	 *     		'require_nonce' => TRUE //this is used if you want to set a route to NOT require a nonce (default is true if it isn't present).  To remove the requirement for a nonce check when this route is visited just set 'require_nonce' to FALSE
330
	 *     		)
331
	 *
332
	 * )
333
	 *
334
	 *
335
	 * @abstract
336
	 * @access protected
337
	 * @return void
338
	 */
339
	abstract protected function _set_page_config();
340
341
342
343
344
345
	/** end sample help_tour methods **/
346
347
348
	/**
349
	 * _add_screen_options
350
	 * Child classes can add any extra wp_screen_options within this method using built-in WP functions/methods for doing so.
351
	 * Note child classes can also define _add_screen_options_($this->_current_view) to limit screen options to a particular view.
352
	 * @link http://chrismarslender.com/wp-tutorials/wordpress-screen-options-tutorial/
353
	 * see also WP_Screen object documents...
354
	 * @link http://codex.wordpress.org/Class_Reference/WP_Screen
355
	 *
356
	 * @abstract
357
	 * @access protected
358
	 * @return void
359
	 */
360
	abstract protected function _add_screen_options();
361
362
363
364
365
366
367
368
	/**
369
	 * _add_feature_pointers
370
	 * Child classes should use this method for implementing any "feature pointers" (using built-in WP styling js).
371
	 * Note child classes can also define _add_feature_pointers_($this->_current_view) to limit screen options to a particular view.
372
	 * Note: this is just a placeholder for now.  Implementation will come down the road
373
	 * See: WP_Internal_Pointers class in wp-admin/includes/template.php for example (its a final class so can't be extended) also see:
374
	 * @link http://eamann.com/tech/wordpress-portland/
375
	 *
376
	 * @abstract
377
	 * @access protected
378
	 * @return void
379
	 */
380
	abstract protected function _add_feature_pointers();
381
382
383
384
385
386
	/**
387
	 * load_scripts_styles
388
	 * child classes put their wp_enqueue_script and wp_enqueue_style hooks in here for anything they need loaded for their pages/subpages.  Note this is for all pages/subpages of the system.  You can also load only specific scripts/styles per view by putting them in a dynamic function in this format (load_scripts_styles_{$this->_current_view}) which matches your page route (action request arg)
389
	 *
390
	 * @abstract
391
	 * @access public
392
	 * @return void
393
	 */
394
	abstract public function load_scripts_styles();
395
396
397
398
399
400
401
	/**
402
	 * admin_init
403
	 * Anything that should be set/executed at 'admin_init' WP hook runtime should be put in here.  This will apply to all pages/views loaded by child class.
404
	 *
405
	 * @abstract
406
	 * @access public
407
	 * @return void
408
	 */
409
	abstract public function admin_init();
410
411
412
413
414
415
416
	/**
417
	 * admin_notices
418
	 * Anything triggered by the 'admin_notices' WP hook should be put in here.  This particular method will apply to all pages/views loaded by child class.
419
	 *
420
	 * @abstract
421
	 * @access public
422
	 * @return void
423
	 */
424
	abstract public function admin_notices();
425
426
427
428
429
430
	/**
431
	 * admin_footer_scripts
432
	 * Anything triggered by the 'admin_print_footer_scripts' WP hook should be put in here. This particular method will apply to all pages/views loaded by child class.
433
	 *
434
	 * @access public
435
	 * @return void
436
	 */
437
	abstract public function admin_footer_scripts();
438
439
440
441
442
443
444
	/**
445
	 * admin_footer
446
	 * anything triggered by the 'admin_footer' WP action hook should be added to here. This particular method will apply to all pages/views loaded by child class.
447
	 *
448
	 * @access  public
449
	 * @return void
450
	 */
451
	public function admin_footer() {}
452
453
454
455
456
	/**
457
	 * _global_ajax_hooks
458
	 * all global add_action('wp_ajax_{name_of_hook}') hooks in here.
459
	 * Note: within the ajax callback methods.
460
	 *
461
	 * @abstract
462
	 * @access protected
463
	 * @return void
464
	 */
465
	protected function _global_ajax_hooks() {
466
		//for lazy loading of metabox content
467
		add_action( 'wp_ajax_espresso-ajax-content', array( $this, 'ajax_metabox_content'), 10 );
468
	}
469
470
471
472
	public function ajax_metabox_content() {
473
		$contentid = isset( $this->_req_data['contentid'] ) ? $this->_req_data['contentid'] : '';
474
		$url = isset( $this->_req_data['contenturl'] ) ? $this->_req_data['contenturl'] : '';
475
476
		self::cached_rss_display( $contentid, $url );
477
		wp_die();
478
	}
479
480
481
482
	/**
483
	 * _page_setup
484
	 * Makes sure any things that need to be loaded early get handled.  We also escape early here if the page requested doesn't match the object.
485
	 *
486
	 * @final
487
	 * @access protected
488
	 * @return void
489
	 */
490
	final protected function _page_setup() {
491
492
		//requires?
493
		EE_Registry::instance()->load_helper('Template');
494
495
496
		//admin_init stuff - global - we're setting this REALLY early so if EE_Admin pages have to hook into other WP pages they can.  But keep in mind, not everything is available from the EE_Admin Page object at this point.
497
		add_action( 'admin_init', array( $this, 'admin_init_global' ), 5 );
498
499
500
		//next verify if we need to load anything...
501
		$this->_current_page = !empty( $_GET['page'] ) ? sanitize_key( $_GET['page'] ) : FALSE;
502
		$this->page_folder = strtolower( str_replace( '_Admin_Page', '', str_replace( 'Extend_', '', get_class($this) ) ) );
503
504
		global $ee_menu_slugs;
505
		$ee_menu_slugs = (array) $ee_menu_slugs;
506
507
		if ( ( !$this->_current_page || ! isset( $ee_menu_slugs[$this->_current_page] ) ) && !defined( 'DOING_AJAX') ) return FALSE;
508
509
510
		// becuz WP List tables have two duplicate select inputs for choosing bulk actions, we need to copy the action from the second to the first
511
		if ( isset( $this->_req_data['action2'] ) && $this->_req_data['action'] == -1 ) {
512
			$this->_req_data['action'] = ! empty( $this->_req_data['action2'] ) && $this->_req_data['action2'] != -1 ? $this->_req_data['action2'] : $this->_req_data['action'];
513
		}
514
		// then set blank or -1 action values to 'default'
515
		$this->_req_action = isset( $this->_req_data['action'] ) && ! empty( $this->_req_data['action'] ) && $this->_req_data['action'] != -1 ? sanitize_key( $this->_req_data['action'] ) : 'default';
516
517
		//if action is 'default' after the above BUT we have  'route' var set, then let's use the route as the action.  This covers cases where we're coming in from a list table that isn't on the default route.
518
		$this->_req_action = $this->_req_action == 'default' && isset( $this->_req_data['route'] ) ? $this->_req_data['route'] : $this->_req_action;
519
520
		//however if we are doing_ajax and we've got a 'route' set then that's what the req_action will be
521
		$this->_req_action = defined('DOING_AJAX') && isset($this->_req_data['route']) ? $this->_req_data['route'] : $this->_req_action;
522
523
		$this->_current_view = $this->_req_action;
524
		$this->_req_nonce = $this->_req_action . '_nonce';
525
		$this->_define_page_props();
526
527
		$this->_current_page_view_url = add_query_arg( array( 'page' => $this->_current_page, 'action' => $this->_current_view ),  $this->_admin_base_url );
528
529
		//default things
530
		$this->_default_espresso_metaboxes = array('_espresso_news_post_box', '_espresso_links_post_box', '_espresso_ratings_request', '_espresso_sponsors_post_box' );
531
532
		//set page configs
533
		$this->_set_page_routes();
534
		$this->_set_page_config();
535
536
		//let's include any referrer data in our default_query_args for this route for "stickiness".
537
		if ( isset( $this->_req_data['wp_referer'] ) ) {
538
			$this->_default_route_query_args['wp_referer'] = $this->_req_data['wp_referer'];
539
		}
540
541
		//for caffeinated and other extended functionality.  If there is a _extend_page_config method then let's run that to modify the all the various page configuration arrays
542
		if ( method_exists( $this, '_extend_page_config' ) )
543
			$this->_extend_page_config();
0 ignored issues
show
Bug introduced by
It seems like you code against a specific sub-type and not the parent class EE_Admin_Page as the method _extend_page_config() does only exist in the following sub-classes of EE_Admin_Page: Extend_About_Admin_Page, Extend_Events_Admin_Page, Extend_General_Settings_Admin_Page, Extend_Messages_Admin_Page, Extend_Registration_Form_Admin_Page, Extend_Registrations_Admin_Page, Extend_Support_Admin_Page, Extend_Transactions_Admin_Page. Maybe you want to instanceof check for one of these explicitly?

Let’s take a look at an example:

abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break.

Available Fixes

  1. Change the type-hint for the parameter:

    class AuthSystem
    {
        public function authenticate(MyUser $user) { /* ... */ }
    }
    
  2. Add an additional type-check:

    class AuthSystem
    {
        public function authenticate(User $user)
        {
            if ($user instanceof MyUser) {
                $this->logger->info(/** ... */);
            }
    
            // or alternatively
            if ( ! $user instanceof MyUser) {
                throw new \LogicException(
                    '$user must be an instance of MyUser, '
                   .'other instances are not supported.'
                );
            }
    
        }
    }
    
Note: PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case.
  1. Add the method to the parent class:

    abstract class User
    {
        /** @return string */
        abstract public function getPassword();
    
        /** @return string */
        abstract public function getDisplayName();
    }
    
Loading history...
544
545
		//for CPT and other extended functionality. If there is an _extend_page_config_for_cpt then let's run that to modify all the various page configuration arrays.
546
		if ( method_exists( $this, '_extend_page_config_for_cpt' ) )
547
			$this->_extend_page_config_for_cpt();
0 ignored issues
show
Bug introduced by
It seems like you code against a specific sub-type and not the parent class EE_Admin_Page as the method _extend_page_config_for_cpt() does only exist in the following sub-classes of EE_Admin_Page: EE_Admin_Page_CPT, Events_Admin_Page, Extend_Events_Admin_Page, Extend_Registrations_Admin_Page, Registrations_Admin_Page, Venues_Admin_Page. Maybe you want to instanceof check for one of these explicitly?

Let’s take a look at an example:

abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break.

Available Fixes

  1. Change the type-hint for the parameter:

    class AuthSystem
    {
        public function authenticate(MyUser $user) { /* ... */ }
    }
    
  2. Add an additional type-check:

    class AuthSystem
    {
        public function authenticate(User $user)
        {
            if ($user instanceof MyUser) {
                $this->logger->info(/** ... */);
            }
    
            // or alternatively
            if ( ! $user instanceof MyUser) {
                throw new \LogicException(
                    '$user must be an instance of MyUser, '
                   .'other instances are not supported.'
                );
            }
    
        }
    }
    
Note: PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case.
  1. Add the method to the parent class:

    abstract class User
    {
        /** @return string */
        abstract public function getPassword();
    
        /** @return string */
        abstract public function getDisplayName();
    }
    
Loading history...
548
549
		//filter routes and page_config so addons can add their stuff. Filtering done per class
550
		$this->_page_routes = apply_filters( 'FHEE__' . get_class($this) . '__page_setup__page_routes', $this->_page_routes, $this );
551
		$this->_page_config = apply_filters( 'FHEE__' . get_class($this) . '__page_setup__page_config', $this->_page_config, $this );
552
553
554
		//if AHEE__EE_Admin_Page__route_admin_request_$this->_current_view method is present then we call it hooked into the AHEE__EE_Admin_Page__route_admin_request action
555
		if ( method_exists( $this, 'AHEE__EE_Admin_Page__route_admin_request_' . $this->_current_view ) ) {
556
			add_action( 'AHEE__EE_Admin_Page__route_admin_request', array( $this, 'AHEE__EE_Admin_Page__route_admin_request_' . $this->_current_view ), 10, 2 );
557
		}
558
559
560
		//next route only if routing enabled
561
		if ( $this->_routing && !defined('DOING_AJAX') ) {
562
563
			$this->_verify_routes();
564
565
			//next let's just check user_access and kill if no access
566
			$this->check_user_access();
567
568
			if ( $this->_is_UI_request ) {
569
				//admin_init stuff - global, all views for this page class, specific view
570
				add_action( 'admin_init', array( $this, 'admin_init' ), 10 );
571
				if ( method_exists( $this, 'admin_init_' . $this->_current_view )) {
572
					add_action( 'admin_init', array( $this, 'admin_init_' . $this->_current_view ), 15 );
573
				}
574
575
			} else {
576
				//hijack regular WP loading and route admin request immediately
577
				@ini_set( 'memory_limit', apply_filters( 'admin_memory_limit', WP_MAX_MEMORY_LIMIT ) );
0 ignored issues
show
Security Best Practice introduced by
It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

If you suppress an error, we recommend checking for the error condition explicitly:

// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
Loading history...
578
				$this->route_admin_request();
579
			}
580
		}
581
	}
582
583
584
585
586
587
	/**
588
	 * Provides a way for related child admin pages to load stuff on the loaded admin page.
589
	 *
590
	 * @access private
591
	 * @return void
592
	 */
593
	private function _do_other_page_hooks() {
594
		$registered_pages = apply_filters( 'FHEE_do_other_page_hooks_' . $this->page_slug, array() );
595
596
		foreach ( $registered_pages as $page ) {
597
598
			//now let's setup the file name and class that should be present
599
			$classname = str_replace('.class.php', '', $page);
600
601
			//autoloaders should take care of loading file
602 View Code Duplication
			if ( !class_exists( $classname ) ) {
603
				$error_msg[] = sprintf( __('Something went wrong with loading the %s admin hooks page.', 'event_espresso' ), $page);
0 ignored issues
show
Coding Style Comprehensibility introduced by
$error_msg was never initialized. Although not strictly required by PHP, it is generally a good practice to add $error_msg = 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...
604
				$error_msg[] = $error_msg[0] . "\r\n" . sprintf( __( 'There is no class in place for the %s admin hooks page.%sMake sure you have <strong>%s</strong> defined. If this is a non-EE-core admin page then you also must have an autoloader in place for your class', 'event_espresso'), $page, '<br />', $classname );
605
				throw new EE_Error( implode( '||', $error_msg ));
606
			}
607
608
			$a = new ReflectionClass($classname);
609
610
			//notice we are passing the instance of this class to the hook object.
611
			$hookobj[] = $a->newInstance($this);
0 ignored issues
show
Coding Style Comprehensibility introduced by
$hookobj was never initialized. Although not strictly required by PHP, it is generally a good practice to add $hookobj = 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...
612
		}
613
	}
614
615
616
617
	public function load_page_dependencies() {
618
		try {
619
			$this->_load_page_dependencies();
620
		} catch ( EE_Error $e ) {
621
			$e->get_error();
622
		}
623
	}
624
625
626
627
	/**
628
	 * load_page_dependencies
629
	 * loads things specific to this page class when its loaded.  Really helps with efficiency.
630
	 * @access public
631
	 * @return void
632
	 */
633
	protected function _load_page_dependencies() {
634
		//let's set the current_screen and screen options to override what WP set
635
		$this->_current_screen = get_current_screen();
636
637
		//load admin_notices - global, page class, and view specific
638
		add_action( 'admin_notices', array( $this, 'admin_notices_global'), 5 );
639
		add_action( 'admin_notices', array( $this, 'admin_notices' ), 10 );
640
		if ( method_exists( $this, 'admin_notices_' . $this->_current_view ) ) {
641
			add_action( 'admin_notices', array( $this, 'admin_notices_' . $this->_current_view ), 15 );
642
		}
643
644
		//load network admin_notices - global, page class, and view specific
645
		add_action( 'network_admin_notices', array( $this, 'network_admin_notices_global'), 5 );
646
		if ( method_exists( $this, 'network_admin_notices_' . $this->_current_view ) ) {
647
			add_action( 'network_admin_notices', array( $this, 'network_admin_notices_' . $this->_current_view ) );
648
		}
649
650
		//this will save any per_page screen options if they are present
651
		$this->_set_per_page_screen_options();
652
653
		//setup list table properties
654
		$this->_set_list_table();
655
656
		// child classes can "register" a metabox to be automatically handled via the _page_config array property.  However in some cases the metaboxes will need to be added within a route handling callback.
657
		$this->_add_registered_meta_boxes();
658
		$this->_add_screen_columns();
659
660
		//add screen options - global, page child class, and view specific
661
		$this->_add_global_screen_options();
0 ignored issues
show
Unused Code introduced by
The call to the method EE_Admin_Page::_add_global_screen_options() seems un-needed as the method has no side-effects.

PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left.

Let’s take a look at an example:

class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}

If we look at the getEmail() method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless:

$user = new User();
$user->getEmail(); // This line could safely be removed as it has no effect.

On the hand, if we look at the setEmail(), this method _has_ side-effects. In the following case, we could not remove the method call:

$user = new User();
$user->setEmail('email@domain'); // This line has a side-effect (it changes an
                                 // instance variable).
Loading history...
662
		$this->_add_screen_options();
663
		if ( method_exists( $this, '_add_screen_options_' . $this->_current_view ) )
664
			call_user_func( array( $this, '_add_screen_options_' . $this->_current_view ) );
665
666
667
		//add help tab(s) and tours- set via page_config and qtips.
668
		$this->_add_help_tour();
669
		$this->_add_help_tabs();
670
		$this->_add_qtips();
671
672
		//add feature_pointers - global, page child class, and view specific
673
		$this->_add_feature_pointers();
674
		$this->_add_global_feature_pointers();
0 ignored issues
show
Unused Code introduced by
The call to the method EE_Admin_Page::_add_global_feature_pointers() seems un-needed as the method has no side-effects.

PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left.

Let’s take a look at an example:

class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}

If we look at the getEmail() method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless:

$user = new User();
$user->getEmail(); // This line could safely be removed as it has no effect.

On the hand, if we look at the setEmail(), this method _has_ side-effects. In the following case, we could not remove the method call:

$user = new User();
$user->setEmail('email@domain'); // This line has a side-effect (it changes an
                                 // instance variable).
Loading history...
675
		if ( method_exists( $this, '_add_feature_pointer_' . $this->_current_view ) )
676
			call_user_func( array( $this, '_add_feature_pointer_' . $this->_current_view ) );
677
678
		//enqueue scripts/styles - global, page class, and view specific
679
		add_action('admin_enqueue_scripts', array($this, 'load_global_scripts_styles'), 5 );
680
		add_action('admin_enqueue_scripts', array($this, 'load_scripts_styles'), 10 );
681
		if ( method_exists( $this, 'load_scripts_styles_' . $this->_current_view ) )
682
			add_action('admin_enqueue_scripts', array($this, 'load_scripts_styles_' . $this->_current_view ), 15 );
683
684
		add_action('admin_enqueue_scripts', array( $this, 'admin_footer_scripts_eei18n_js_strings' ), 100 );
685
686
		//admin_print_footer_scripts - global, page child class, and view specific.  NOTE, despite the name, whenever possible, scripts should NOT be loaded using this.  In most cases that's doing_it_wrong().  But adding hidden container elements etc. is a good use case. Notice the late priority we're giving these
687
		add_action('admin_print_footer_scripts', array( $this, 'admin_footer_scripts_global' ), 99 );
688
		add_action('admin_print_footer_scripts', array( $this, 'admin_footer_scripts' ), 100 );
689
		if ( method_exists( $this, 'admin_footer_scripts_' . $this->_current_view ) )
690
			add_action('admin_print_footer_scripts', array( $this, 'admin_footer_scripts_' . $this->_current_view ), 101 );
691
692
		//admin footer scripts
693
		add_action('admin_footer', array( $this, 'admin_footer_global' ), 99 );
694
		add_action('admin_footer', array( $this, 'admin_footer'), 100 );
695
		if ( method_exists( $this, 'admin_footer_' . $this->_current_view ) )
696
			add_action('admin_footer', array( $this, 'admin_footer_' . $this->_current_view ), 101 );
697
698
699
		do_action( 'FHEE__EE_Admin_Page___load_page_dependencies__after_load', $this->page_slug );
700
		//targeted hook
701
		do_action( 'FHEE__EE_Admin_Page___load_page_dependencies__after_load__' . $this->page_slug . '__' . $this->_req_action );
702
703
	}
704
705
706
707
708
709
	/**
710
	 * _set_defaults
711
	 * This sets some global defaults for class properties.
712
	 */
713
	private function _set_defaults() {
714
		$this->_current_screen = $this->_admin_page_title = $this->_req_action = $this->_req_nonce = $this->_event = $this->_template_path = $this->_column_template_path = NULL;
0 ignored issues
show
Documentation introduced by
The property _event does not exist on object<EE_Admin_Page>. Since you implemented __set, maybe consider adding a @property annotation.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
715
716
		$this->_nav_tabs = $this_views = $this->_page_routes = $this->_page_config =  $this->_default_route_query_args = array();
0 ignored issues
show
Unused Code introduced by
$this_views 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...
717
718
		$this->default_nav_tab_name = 'overview';
0 ignored issues
show
Bug introduced by
The property default_nav_tab_name does not seem to exist. Did you mean _default_nav_tab_name?

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
719
720
		//init template args
721
		$this->_template_args = array(
722
			'admin_page_header' => '',
723
			'admin_page_content' => '',
724
			'post_body_content' => '',
725
			'before_list_table' => '',
726
			'after_list_table' => ''
727
		);
728
	}
729
730
731
732
733
	/**
734
	 * route_admin_request
735
	 *
736
	 * @see _route_admin_request()
737
	 * @access public
738
	 * @return void|exception error
739
	 */
740
	public function route_admin_request() {
741
		try {
742
			$this->_route_admin_request();
743
		} catch ( EE_Error $e ) {
744
			$e->get_error();
745
		}
746
	}
747
748
749
750
	public function set_wp_page_slug($wp_page_slug) {
751
		$this->_wp_page_slug = $wp_page_slug;
752
753
		//if in network admin then we need to append "-network" to the page slug. Why? Because that's how WP rolls...
754
		if ( is_network_admin() ) {
755
			$this->_wp_page_slug .= '-network';
756
		}
757
	}
758
759
	/**
760
	 * _verify_routes
761
	 * All this method does is verify the incoming request and make sure that routes exist for it.  We do this early so we know if we need to drop out.
762
	 *
763
	 * @access protected
764
	 * @return void
765
	 */
766
	protected function _verify_routes() {
767
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
768
769
		if ( !$this->_current_page && !defined( 'DOING_AJAX')) return FALSE;
770
771
		$this->_route = FALSE;
772
		$func = FALSE;
0 ignored issues
show
Unused Code introduced by
$func 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...
773
		$args = array();
0 ignored issues
show
Unused Code introduced by
$args 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...
774
775
		// check that the page_routes array is not empty
776
		if ( empty( $this->_page_routes )) {
777
			// user error msg
778
			$error_msg = sprintf( __('No page routes have been set for the %s admin page.', 'event_espresso'), $this->_admin_page_title );
779
			// developer error msg
780
			$error_msg .=  '||' . $error_msg . __( ' Make sure the "set_page_routes()" method exists, and is setting the "_page_routes" array properly.', 'event_espresso' );
781
			throw new EE_Error( $error_msg );
782
		}
783
784
		// and that the requested page route exists
785
		if ( array_key_exists( $this->_req_action, $this->_page_routes )) {
786
			$this->_route = $this->_page_routes[ $this->_req_action ];
787
			$this->_route_config = isset($this->_page_config[$this->_req_action]) ? $this->_page_config[$this->_req_action] : array();
788 View Code Duplication
		} else {
789
			// user error msg
790
			$error_msg =  sprintf( __( 'The requested page route does not exist for the %s admin page.', 'event_espresso' ), $this->_admin_page_title );
791
			// developer error msg
792
			$error_msg .=  '||' . $error_msg . sprintf( __( ' Create a key in the "_page_routes" array named "%s" and set its value to the appropriate method.', 'event_espresso' ), $this->_req_action );
793
			throw new EE_Error( $error_msg );
794
		}
795
796
		// and that a default route exists
797
		if ( ! array_key_exists( 'default', $this->_page_routes )) {
798
			// user error msg
799
			$error_msg = sprintf( __( 'A default page route has not been set for the % admin page.', 'event_espresso' ), $this->_admin_page_title );
800
			// developer error msg
801
			$error_msg .=  '||' . $error_msg . __( ' Create a key in the "_page_routes" array named "default" and set its value to your default page method.', 'event_espresso' );
802
			throw new EE_Error( $error_msg );
803
		}
804
805
806
		//first lets' catch if the UI request has EVER been set.
807
		if ( $this->_is_UI_request === NULL ) {
808
			//lets set if this is a UI request or not.
809
			$this->_is_UI_request = ( ! isset( $this->_req_data['noheader'] ) || $this->_req_data['noheader'] !== TRUE ) ? TRUE : FALSE;
810
811
812
			//wait a minute... we might have a noheader in the route array
813
			$this->_is_UI_request = is_array($this->_route) && isset($this->_route['noheader'] ) && $this->_route['noheader'] ? FALSE : $this->_is_UI_request;
814
		}
815
816
		$this->_set_current_labels();
817
818
	}
819
820
821
822
823
	/**
824
	 * this method simply verifies a given route and makes sure its an actual route available for the loaded page
825
	 * @param  string $route the route name we're verifying
826
	 * @return mixed  (bool|Exception)      we'll throw an exception if this isn't a valid route.
827
	 */
828
	protected function _verify_route( $route ) {
829
		if ( array_key_exists( $this->_req_action, $this->_page_routes )) {
830
			return true;
831 View Code Duplication
		} else {
832
			// user error msg
833
			$error_msg =  sprintf( __( 'The given page route does not exist for the %s admin page.', 'event_espresso' ), $this->_admin_page_title );
834
			// developer error msg
835
			$error_msg .=  '||' . $error_msg . sprintf( __( ' Check the route you are using in your method (%s) and make sure it matches a route set in your "_page_routes" array property', 'event_espresso' ), $route );
836
			throw new EE_Error( $error_msg );
837
		}
838
	}
839
840
841
842
843
	/**
844
	 * perform nonce verification
845
	 * This method has be encapsulated here so that any ajax requests that bypass normal routes can verify their nonces using this method (and save retyping!)
846
	 * @param  string $nonce     The nonce sent
847
	 * @param  string $nonce_ref The nonce reference string (name0)
848
	 * @return mixed (bool|die)
849
	 */
850
	protected function _verify_nonce( $nonce, $nonce_ref ) {
851
		// verify nonce against expected value
852
		if ( ! wp_verify_nonce( $nonce, $nonce_ref) ) {
853
			// these are not the droids you are looking for !!!
854
			$msg = sprintf(__('%sNonce Fail.%s' , 'event_espresso'), '<a href="http://www.youtube.com/watch?v=56_S0WeTkzs">', '</a>' );
855
			if ( WP_DEBUG ) {
856
				$msg .= "\n  " . sprintf( __('In order to dynamically generate nonces for your actions, use the %s::add_query_args_and_nonce() method. May the Nonce be with you!', 'event_espresso' ), __CLASS__  );
857
			}
858
			if ( ! defined( 'DOING_AJAX' )) {
859
				wp_die( $msg );
860
			} else {
861
				EE_Error::add_error( $msg, __FILE__, __FUNCTION__, __LINE__ );
862
				$this->_return_json();
863
			}
864
		}
865
	}
866
867
868
869
870
	/**
871
	 * _route_admin_request()
872
	 * Meat and potatoes of the class.  Basically, this dude checks out what's being requested and sees if theres are some doodads to work the magic and handle the flingjangy.
873
	 * Translation:  Checks if the requested action is listed in the page routes and then will try to load the corresponding method.
874
	 *
875
	 * @access protected
876
	 * @return void
877
	 */
878
	protected function _route_admin_request() {
879
		if (  ! $this->_is_UI_request )
880
			$this->_verify_routes();
881
882
		$nonce_check = isset( $this->_route_config['require_nonce'] ) ? $this->_route_config['require_nonce'] : TRUE;
883
884 View Code Duplication
		if ( $this->_req_action != 'default' && $nonce_check ) {
885
			// set nonce from post data
886
			$nonce = isset($this->_req_data[ $this->_req_nonce  ]) ? sanitize_text_field( $this->_req_data[ $this->_req_nonce  ] ) : '';
887
			$this->_verify_nonce( $nonce, $this->_req_nonce );
888
		}
889
		//set the nav_tabs array but ONLY if this is  UI_request
890
		if ( $this->_is_UI_request )
891
			$this->_set_nav_tabs();
892
893
		// grab callback function
894
		$func = is_array( $this->_route ) ? $this->_route['func'] : $this->_route;
895
896
		// check if callback has args
897
		$args = is_array( $this->_route ) && isset( $this->_route['args'] ) ? $this->_route['args'] : array();
898
899
		$error_msg = '';
900
901
		//action right before calling route (hook is something like 'AHEE__Registrations_Admin_Page__route_admin_request')
902
		if ( !did_action('AHEE__EE_Admin_Page__route_admin_request')) {
903
			do_action( 'AHEE__EE_Admin_Page__route_admin_request', $this->_current_view, $this );
904
		}
905
906
		//right before calling the route, let's remove _wp_http_referer from the $_SERVER[REQUEST_URI] global (its now in _req_data for route processing).
907
		$_SERVER['REQUEST_URI'] = remove_query_arg( '_wp_http_referer', wp_unslash( $_SERVER['REQUEST_URI'] ) );
908
909
		if ( ! empty( $func )) {
910
			$base_call = $addon_call = FALSE;
911
			//try to access page route via this class
912
			if ( ! is_array( $func ) && method_exists( $this, $func ) && ( $base_call = call_user_func_array( array( $this, &$func  ), $args ) ) === FALSE ) {
913
				// user error msg
914
				$error_msg =  __( 'An error occurred. The  requested page route could not be found.', 'event_espresso' );
915
				// developer error msg
916
				$error_msg .= '||' . sprintf( __( 'Page route "%s" could not be called. Check that the spelling for method names and actions in the "_page_routes" array are all correct.', 'event_espresso' ), $func );
917
			}
918
919
			//for pluggability by addons first let's see if just the function exists (this will also work in the case where $func is an array indicating class/method)
920
			$args['admin_page_object'] = $this; //send along this admin page object for access by addons.
921
922
			if ( $base_call === FALSE && ( $addon_call = call_user_func_array( $func, $args ) )=== FALSE ) {
923
				$error_msg = __('An error occurred. The requested page route could not be found', 'event_espresso' );
924
				$error_msg .= '||' . sprintf( __('Page route "%s" could not be called.  Check that the spelling for the function name and action in the "_page_routes" array filtered by your plugin is correct.', 'event_espresso'), $func );
925
			}
926
927
928
			if ( !empty( $error_msg ) && $base_call === FALSE && $addon_call === FALSE )
929
				throw new EE_Error( $error_msg );
930
		}
931
932
		//if we've routed and this route has a no headers route AND a sent_headers_route, then we need to reset the routing properties to the new route.
933
		//now if UI request is FALSE and noheader is true AND we have a headers_sent_route in the route array then let's set UI_request to true because the no header route has a second func after headers have been sent.
934
		if ( $this->_is_UI_request === FALSE && is_array( $this->_route) && ! empty( $this->_route['headers_sent_route'] ) ) {
935
			$this->_reset_routing_properties( $this->_route['headers_sent_route'] );
936
		}
937
	}
938
939
940
941
942
	/**
943
	 * This method just allows the resetting of page properties in the case where a no headers
944
	 * route redirects to a headers route in its route config.
945
	 *
946
	 * @since   4.3.0
947
	 *
948
	 * @param  string    $new_route   New (non header) route to redirect to.
949
	 * @return   void
950
	 */
951
	protected function _reset_routing_properties( $new_route ) {
952
		$this->_is_UI_request = TRUE;
953
		//now we set the current route to whatever the headers_sent_route is set at
954
		$this->_req_data['action'] = $new_route;
955
		//rerun page setup
956
		$this->_page_setup();
957
	}
958
959
960
961
962
	/**
963
	 * _add_query_arg
964
	 * adds nonce to array of arguments then calls WP add_query_arg function
965
	 *(internally just uses EEH_URL's function with the same name)
966
	 * 	@access public
967
	 *	@param array $args
968
	 *	@param string $url
969
	 *	@param bool $sticky if true, then the existing Request params will be appended to the generated
970
	 *	                    		url in an associative array indexed by the key 'wp_referer';
971
	 *	                    		Example usage:
972
	 *
973
	 *	                    		If the current page is:
974
	 *	                    		http://mydomain.com/wp-admin/admin.php?page=espresso_registrations
975
	 *	                    		&action=default&event_id=20&month_range=March%202015
976
	 *	                    		&_wpnonce=5467821
977
	 *
978
	 *	                    		and you call:
979
	 *
980
	 *	                    		EE_Admin_Page::add_query_args_and_nonce(
981
	 *	                    			array(
982
	 *	                    				'action' => 'resend_something',
983
	 *	                    				'page=>espresso_registrations'
984
	 *	                    				),
985
	 *	                    			$some_url,
986
	 *	                    			true
987
	 *	                    		 );
988
	 *
989
	 *	                    		It will produce a url in this structure:
990
	 *
991
	 *	                      		http://{$some_url}/?page=espresso_registrations&action=resend_something
992
	 *	                        	&wp_referer[action]=default&wp_referer[event_id]=20&wpreferer[
993
	 *	                        	month_range]=March%202015
994
	 * @param   bool    $exclude_nonce  If true, the the nonce will be excluded from the generated nonce.
995
	 * 	@return string
996
	 */
997
	public static function add_query_args_and_nonce( $args = array(), $url = false, $sticky = false, $exclude_nonce = false ) {
998
		EE_Registry::instance()->load_helper('URL');
999
1000
		//if there is a _wp_http_referer include the values from the request but only if sticky = true
1001
		if ( $sticky ) {
1002
			$request = $_REQUEST;
1003
			unset( $request['_wp_http_referer'] );
1004
			unset( $request['wp_referer'] );
1005
			foreach ( $request as $key => $value ) {
1006
				//do not add nonces
1007
				if ( strpos( $key, 'nonce' ) !== false ) {
1008
					continue;
1009
				}
1010
				$args['wp_referer[' . $key . ']'] = $value;
1011
			}
1012
		}
1013
1014
		return EEH_URL::add_query_args_and_nonce( $args, $url, $exclude_nonce );
0 ignored issues
show
Bug introduced by
It seems like $url defined by parameter $url on line 997 can also be of type false; however, EEH_URL::add_query_args_and_nonce() does only seem to accept string, maybe add an additional type check?

This check looks at variables that have been passed in as parameters and are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
1015
	}
1016
1017
1018
1019
1020
	/**
1021
	 * This returns a generated link that will load the related help tab.
1022
	 *
1023
	 *
1024
	 * @param  string $help_tab_id the id for the connected help tab
1025
	 * @param  string $icon_style (optional) include css class for the style you want to use for the help icon.
1026
	 * @param  string $help_text (optional) send help text you want to use for the link if default not to be used
1027
	 * @uses EEH_Template::get_help_tab_link()
1028
	 * @return string              generated link
1029
	 */
1030
	protected function _get_help_tab_link( $help_tab_id, $icon_style = FALSE, $help_text = FALSE ) {
1031
		return EEH_Template::get_help_tab_link( $help_tab_id, $this->page_slug, $this->_req_action, $icon_style, $help_text );
1032
	}
1033
1034
1035
1036
1037
	/**
1038
	 * _add_help_tabs
1039
	 *
1040
	 * Note child classes define their help tabs within the page_config array.
1041
	 * @link http://codex.wordpress.org/Function_Reference/add_help_tab
1042
	 *
1043
	 * @access protected
1044
	 * @return void
1045
	 */
1046
	protected function _add_help_tabs() {
1047
		$tour_buttons = '';
1048
		if ( isset( $this->_page_config[$this->_req_action] ) ) {
1049
			$config = $this->_page_config[$this->_req_action];
1050
1051
			//is there a help tour for the current route?  if there is let's setup the tour buttons
1052
			if ( isset( $this->_help_tour[$this->_req_action]) ) {
1053
				$tb = array();
1054
				$tour_buttons = '<div class="ee-abs-container"><div class="ee-help-tour-restart-buttons">';
1055
				foreach ( $this->_help_tour['tours'] as $tour ) {
1056
					//if this is the end tour then we don't need to setup a button
1057
					if ( $tour instanceof EE_Help_Tour_final_stop )
1058
						continue;
1059
					$tb[] = '<button id="trigger-tour-' . $tour->get_slug() . '" class="button-primary trigger-ee-help-tour">' . $tour->get_label() . '</button>';
1060
				}
1061
				$tour_buttons .= implode('<br />', $tb);
1062
				$tour_buttons .= '</div></div>';
1063
			}
1064
1065
			// let's see if there is a help_sidebar set for the current route and we'll set that up for usage as well.
1066
			if ( is_array( $config ) && isset( $config['help_sidebar'] ) ) {
1067
				//check that the callback given is valid
1068
				if ( !method_exists($this, $config['help_sidebar'] ) )
1069
					throw new EE_Error( sprintf( __('The _page_config array has a callback set for the "help_sidebar" option.  However the callback given (%s) is not a valid callback.  Doublecheck the spelling and make sure this method exists for the class %s', 'event_espresso'), $config['help_sidebar'], get_class($this) ) );
1070
1071
				$content = apply_filters( 'FHEE__' . get_class($this) . '__add_help_tabs__help_sidebar', call_user_func( array( $this, $config['help_sidebar'] ) ) );
1072
1073
				$content .= $tour_buttons; //add help tour buttons.
1074
1075
				//do we have any help tours setup?  Cause if we do we want to add the buttons
1076
				$this->_current_screen->set_help_sidebar($content);
1077
			}
1078
1079
			//if we DON'T have config help sidebar and there ARE toure buttons then we'll just add the tour buttons to the sidebar.
1080
			if ( !isset( $config['help_sidebar'] ) && !empty( $tour_buttons ) ) {
1081
				$this->_current_screen->set_help_sidebar($tour_buttons);
1082
			}
1083
1084
			//handle if no help_tabs are set so the sidebar will still show for the help tour buttons
1085
			if ( !isset( $config['help_tabs'] ) && !empty($tour_buttons) ) {
1086
				$_ht['id'] = $this->page_slug;
0 ignored issues
show
Coding Style Comprehensibility introduced by
$_ht was never initialized. Although not strictly required by PHP, it is generally a good practice to add $_ht = 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...
1087
				$_ht['title'] = __('Help Tours', 'event_espresso');
1088
				$_ht['content'] = '<p>' . __('The buttons to the right allow you to start/restart any help tours available for this page', 'event_espresso') . '</p>';
1089
				$this->_current_screen->add_help_tab($_ht);
1090
				}/**/
1091
1092
1093
			if ( !isset( $config['help_tabs'] ) ) return; //no help tabs for this route
1094
1095
			foreach ( (array) $config['help_tabs'] as $tab_id => $cfg ) {
1096
				//we're here so there ARE help tabs!
1097
1098
				//make sure we've got what we need
1099
				if ( !isset( $cfg['title'] ) )
1100
					throw new EE_Error( __('The _page_config array is not set up properly for help tabs.  It is missing a title', 'event_espresso') );
1101
1102
1103
				if ( !isset($cfg['filename']) && !isset( $cfg['callback'] ) && !isset( $cfg['content'] ) )
1104
					throw new EE_Error( __('The _page_config array is not setup properly for help tabs. It is missing a either a filename reference, or a callback reference or a content reference so there is no way to know the content for the help tab', 'event_espresso') );
1105
1106
1107
1108
				//first priority goes to content.
1109
				if ( !empty($cfg['content'] ) ) {
1110
					$content = !empty($cfg['content']) ? $cfg['content'] : NULL;
1111
1112
				//second priority goes to filename
1113
				} else if ( !empty($cfg['filename'] ) ) {
1114
					$file_path = $this->_get_dir() . '/help_tabs/' . $cfg['filename'] . '.help_tab.php';
1115
1116
1117
					//it's possible that the file is located on decaf route (and above sets up for caf route, if this is the case then lets check decaf route too)
1118
					$file_path = !is_readable($file_path) ? EE_ADMIN_PAGES . basename($this->_get_dir()) . '/help_tabs/' . $cfg['filename'] . '.help_tab.php' : $file_path;
1119
1120
					//if file is STILL not readable then let's do a EE_Error so its more graceful than a fatal error.
1121 View Code Duplication
					if ( !is_readable($file_path) && !isset($cfg['callback']) ) {
1122
						EE_Error::add_error( sprintf( __('The filename given for the help tab %s is not a valid file and there is no other configuration for the tab content.  Please check that the string you set for the help tab on this route (%s) is the correct spelling.  The file should be in %s', 'event_espresso'), $tab_id, key($config), $file_path ), __FILE__, __FUNCTION__, __LINE__ );
1123
						return;
1124
					}
1125
					$template_args['admin_page_obj'] = $this;
0 ignored issues
show
Coding Style Comprehensibility introduced by
$template_args was never initialized. Although not strictly required by PHP, it is generally a good practice to add $template_args = 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...
1126
					$content = EEH_Template::display_template($file_path, $template_args, true);
0 ignored issues
show
Bug introduced by
The variable $template_args 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...
1127
				} else {
1128
					$content = '';
1129
				}
1130
1131
1132
				//check if callback is valid
1133
				if ( empty($content) && ( !isset($cfg['callback']) || !method_exists( $this, $cfg['callback'] ) ) ) {
1134
					EE_Error::add_error( sprintf( __('The callback given for a %s help tab on this page does not content OR a corresponding method for generating the content.  Check the spelling or make sure the method is present.', 'event_espresso'), $cfg['title'] ), __FILE__, __FUNCTION__, __LINE__ );
1135
					return;
1136
				}
1137
1138
				//setup config array for help tab method
1139
				$id = $this->page_slug . '-' . $this->_req_action . '-' . $tab_id;
1140
				$_ht = array(
1141
					'id' => $id,
1142
					'title' => $cfg['title'],
1143
					'callback' => isset( $cfg['callback'] ) && empty($content) ? array( $this, $cfg['callback'] ) : NULL,
1144
					'content' => $content
1145
					);
1146
1147
				$this->_current_screen->add_help_tab( $_ht );
1148
			}
1149
		}
1150
	}
1151
1152
1153
1154
	/**
1155
	 * This basically checks loaded $_page_config property to see if there are any help_tours defined.  "help_tours" is an array with properties for setting up usage of the joyride plugin
1156
	 *
1157
	 * @link http://zurb.com/playground/jquery-joyride-feature-tour-plugin
1158
	 * @see instructions regarding the format and construction of the "help_tour" array element is found in the _set_page_config() comments
1159
	 * @access protected
1160
	 * @return void
1161
	 */
1162
	protected function _add_help_tour() {
1163
		$tours = array();
1164
		$this->_help_tour = array();
1165
1166
		//exit early if help tours are turned off globally
1167
		if ( ! EE_Registry::instance()->CFG->admin->help_tour_activation || ( defined( 'EE_DISABLE_HELP_TOURS' ) && EE_DISABLE_HELP_TOURS ) )
1168
			return;
1169
1170
		//loop through _page_config to find any help_tour defined
1171
		foreach ( $this->_page_config as $route => $config ) {
1172
			//we're only going to set things up for this route
1173
			if ( $route !== $this->_req_action )
1174
				continue;
1175
1176
			if ( isset( $config['help_tour'] ) ) {
1177
1178
				foreach( $config['help_tour'] as $tour ) {
1179
					$file_path = $this->_get_dir() . '/help_tours/' . $tour . '.class.php';
1180
					//let's see if we can get that file... if not its possible this is a decaf route not set in caffienated so lets try and get the caffeinated equivalent
1181
					$file_path = !is_readable($file_path) ? EE_ADMIN_PAGES . basename($this->_get_dir()) . '/help_tours/' . $tour . '.class.php' : $file_path;
1182
1183
					//if file is STILL not readable then let's do a EE_Error so its more graceful than a fatal error.
1184
					if ( !is_readable($file_path) ) {
1185
						EE_Error::add_error( sprintf( __('The file path given for the help tour (%s) is not a valid path.  Please check that the string you set for the help tour on this route (%s) is the correct spelling', 'event_espresso'), $file_path, $tour ), __FILE__, __FUNCTION__, __LINE__ );
1186
						return;
1187
					}
1188
1189
					require_once $file_path;
1190 View Code Duplication
					if ( !class_exists( $tour ) ) {
1191
						$error_msg[] = sprintf( __('Something went wrong with loading the %s Help Tour Class.', 'event_espresso' ), $tour);
0 ignored issues
show
Coding Style Comprehensibility introduced by
$error_msg was never initialized. Although not strictly required by PHP, it is generally a good practice to add $error_msg = 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...
1192
						$error_msg[] = $error_msg[0] . "\r\n" . sprintf( __( 'There is no class in place for the %s help tour.%s Make sure you have <strong>%s</strong> defined in the "help_tour" array for the %s route of the % admin page.', 'event_espresso'), $tour, '<br />', $tour, $this->_req_action, get_class($this) );
1193
						throw new EE_Error( implode( '||', $error_msg ));
1194
					}
1195
					$a = new ReflectionClass($tour);
1196
					$tour_obj = $a->newInstance($this->_is_caf);
1197
1198
					$tours[] = $tour_obj;
1199
					$this->_help_tour[$route][] = EEH_Template::help_tour_stops_generator( $tour_obj );
1200
				}
1201
1202
				//let's inject the end tour stop element common to all pages... this will only get seen once per machine.
1203
				$end_stop_tour = new EE_Help_Tour_final_stop($this->_is_caf);
1204
				$tours[] = $end_stop_tour;
1205
				$this->_help_tour[$route][] = EEH_Template::help_tour_stops_generator( $end_stop_tour );
1206
			}
1207
		}
1208
1209
		if ( !empty( $tours ) )
1210
			$this->_help_tour['tours'] = $tours;
1211
1212
		//thats it!  Now that the $_help_tours property is set (or not) the scripts and html should be taken care of automatically.
1213
	}
1214
1215
1216
1217
1218
	/**
1219
	 * This simply sets up any qtips that have been defined in the page config
1220
	 *
1221
	 * @access protected
1222
	 * @return void
1223
	 */
1224
	protected function _add_qtips() {
1225
		if ( isset( $this->_route_config['qtips'] ) ) {
1226
			$qtips = (array) $this->_route_config['qtips'];
1227
			//load qtip loader
1228
			EE_Registry::instance()->load_helper('Qtip_Loader', array(), TRUE);
1229
			$path = array(
1230
				$this->_get_dir() . '/qtips/',
1231
				EE_ADMIN_PAGES . basename($this->_get_dir()) . '/qtips/'
1232
				);
1233
			EEH_Qtip_Loader::instance()->register($qtips, $path);
1234
		}
1235
	}
1236
1237
1238
1239
1240
	/**
1241
	 * _set_nav_tabs
1242
	 * This sets up the nav tabs from the page_routes array.  This method can be overwritten by child classes if you wish to add additional tabs or modify accordingly.
1243
	 *
1244
	 * @access protected
1245
	 * @return void
1246
	 */
1247
	protected function _set_nav_tabs() {
1248
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
1249
		$i = 0;
1250
		foreach ( $this->_page_config as $slug => $config ) {
1251
			if ( !is_array( $config ) || ( is_array($config) && (isset($config['nav']) && !$config['nav'] ) || !isset($config['nav'] ) ) )
1252
				continue; //no nav tab for this config
1253
1254
			//check for persistent flag
1255
			if ( isset( $config['nav']['persistent']) && !$config['nav']['persistent'] && $slug !== $this->_req_action )
1256
				continue; //nav tab is only to appear when route requested.
1257
1258
			if ( ! $this->check_user_access( $slug, TRUE ) )
1259
				continue; //no nav tab becasue current user does not have access.
1260
1261
			$css_class = isset( $config['css_class'] ) ? $config['css_class'] . ' ' : '';
1262
			$this->_nav_tabs[$slug] = array(
1263
				'url' => isset($config['nav']['url']) ? $config['nav']['url'] : self::add_query_args_and_nonce( array( 'action'=>$slug ), $this->_admin_base_url ),
1264
				'link_text' => isset( $config['nav']['label'] ) ? $config['nav']['label'] : ucwords(str_replace('_', ' ', $slug ) ),
1265
				'css_class' => $this->_req_action == $slug ? $css_class . 'nav-tab-active' : $css_class,
1266
				'order' => isset( $config['nav']['order'] ) ? $config['nav']['order'] : $i
1267
				);
1268
			$i++;
1269
		}
1270
1271
		//if $this->_nav_tabs is empty then lets set the default
1272
		if ( empty( $this->_nav_tabs ) ) {
1273
			$this->_nav_tabs[$this->default_nav_tab_name] = array(
0 ignored issues
show
Bug introduced by
The property default_nav_tab_name does not seem to exist. Did you mean _default_nav_tab_name?

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
1274
				'url' => $this->admin_base_url,
0 ignored issues
show
Bug introduced by
The property admin_base_url does not seem to exist. Did you mean _admin_base_url?

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
1275
				'link_text' => ucwords( str_replace( '_', ' ', $this->default_nav_tab_name ) ),
0 ignored issues
show
Bug introduced by
The property default_nav_tab_name does not seem to exist. Did you mean _default_nav_tab_name?

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
1276
				'css_class' => 'nav-tab-active',
1277
				'order' => 10
1278
				);
1279
		}
1280
1281
		//now let's sort the tabs according to order
1282
		usort( $this->_nav_tabs, array($this, '_sort_nav_tabs' ));
1283
1284
	}
1285
1286
1287
1288
1289
1290
	/**
1291
	 * _set_current_labels
1292
	 * This method modifies the _labels property with any optional specific labels indicated in the _page_routes property array
1293
	 *
1294
	 * @access private
1295
	 * @return void
1296
	 */
1297
	private function _set_current_labels() {
1298
		if ( is_array($this->_route_config) && isset($this->_route_config['labels']) ) {
1299
			foreach ( $this->_route_config['labels'] as $label => $text ) {
1300
				if ( is_array($text) ) {
1301
					foreach ( $text as $sublabel => $subtext ) {
1302
						$this->_labels[$label][$sublabel] = $subtext;
1303
					}
1304
				} else {
1305
					$this->_labels[$label] = $text;
1306
				}
1307
			}
1308
		}
1309
	}
1310
1311
1312
1313
1314
1315
1316
	/**
1317
	 * 		verifies user access for this admin page
1318
	 * 		@param string $route_to_check if present then the capability for the route matching this string is checked.
1319
	 * 		@param bool   $verify_only Default is FALSE which means if user check fails then wp_die().  Otherwise just return false if verify fail.
1320
	*		@return 		BOOL|wp_die()
0 ignored issues
show
Documentation introduced by
The doc-type BOOL|wp_die() could not be parsed: Expected "|" or "end of type", but got "(" at position 11. (view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
1321
	*/
1322
	public function check_user_access( $route_to_check = '', $verify_only = FALSE ) {
1323
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
1324
		$route_to_check = empty( $route_to_check ) ? $this->_req_action : $route_to_check;
1325
		$capability = ! empty( $route_to_check ) && isset( $this->_page_routes[$route_to_check] ) && is_array( $this->_page_routes[$route_to_check] ) && ! empty( $this->_page_routes[$route_to_check]['capability'] ) ? $this->_page_routes[$route_to_check]['capability'] : NULL;
1326
1327
		if ( empty( $capability ) && empty( $route_to_check )  ) {
1328
			$capability = is_array( $this->_route ) && empty( $this->_route['capability'] ) ? 'manage_options' : $this->_route['capability'];
1329
		} else {
1330
			$capability = empty( $capability ) ? 'manage_options' : $capability;
1331
		}
1332
1333
		$id = is_array( $this->_route ) && ! empty( $this->_route['obj_id'] ) ? $this->_route['obj_id'] : 0;
1334
1335
		if (( ! function_exists( 'is_admin' ) || ! EE_Registry::instance()->CAP->current_user_can( $capability, $this->page_slug . '_' . $route_to_check, $id ) ) && ! defined( 'DOING_AJAX')) {
1336
			if ( $verify_only ) {
1337
				return FALSE;
1338
			} else {
1339
				wp_die( __('You do not have access to this route.', 'event_espresso' ) );
1340
			}
1341
		}
1342
		return TRUE;
1343
	}
1344
1345
1346
1347
1348
1349
1350
1351
	/**
1352
	 * admin_init_global
1353
	 * This runs all the code that we want executed within the WP admin_init hook.
1354
	 * This method executes for ALL EE Admin pages.
1355
	 *
1356
	 * @access public
1357
	 * @return void
1358
	 */
1359
	public function admin_init_global() {
1360
	}
1361
1362
1363
1364
1365
1366
1367
	/**
1368
	 * wp_loaded_global
1369
	 * This runs all the code that we want executed within the WP wp_loaded hook.  This method is optional for an EE_Admin page and will execute on every EE Admin Page load
1370
	 *
1371
	 * @access public
1372
	 * @return void
1373
	 */
1374
	public function wp_loaded() {}
1375
1376
1377
1378
1379
1380
1381
	/**
1382
	 * admin_notices
1383
	 * Anything triggered by the 'admin_notices' WP hook should be put in here.  This particular method will apply on ALL EE_Admin pages.
1384
	 *
1385
	 * @access public
1386
	 * @return void
1387
	 */
1388
	public function admin_notices_global() {
1389
		$this->_display_no_javascript_warning();
1390
		$this->_display_espresso_notices();
1391
	}
1392
1393
1394
1395
1396
	public function network_admin_notices_global() {
1397
		$this->_display_no_javascript_warning();
1398
		$this->_display_espresso_notices();
1399
	}
1400
1401
1402
1403
1404
	/**
1405
	 * admin_footer_scripts_global
1406
	 * Anything triggered by the 'admin_print_footer_scripts' WP hook should be put in here. This particular method will apply on ALL EE_Admin pages.
1407
	 *
1408
	 * @access public
1409
	 * @return void
1410
	 */
1411
	public function admin_footer_scripts_global() {
1412
		$this->_add_admin_page_ajax_loading_img();
1413
		$this->_add_admin_page_overlay();
0 ignored issues
show
Unused Code introduced by
The call to the method EE_Admin_Page::_add_admin_page_overlay() seems un-needed as the method has no side-effects.

PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left.

Let’s take a look at an example:

class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}

If we look at the getEmail() method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless:

$user = new User();
$user->getEmail(); // This line could safely be removed as it has no effect.

On the hand, if we look at the setEmail(), this method _has_ side-effects. In the following case, we could not remove the method call:

$user = new User();
$user->setEmail('email@domain'); // This line has a side-effect (it changes an
                                 // instance variable).
Loading history...
1414
1415
		//if metaboxes are present we need to add the nonce field
1416
		if ( ( isset($this->_route_config['metaboxes']) || ( isset($this->_route_config['has_metaboxes']) && $this->_route_config['has_metaboxes'] ) || isset($this->_route_config['list_table']) ) ) {
1417
			wp_nonce_field('closedpostboxes', 'closedpostboxesnonce', false);
1418
			wp_nonce_field('meta-box-order', 'meta-box-order-nonce', false);
1419
		}
1420
	}
1421
1422
1423
1424
1425
1426
	/**
1427
	 * admin_footer_global
1428
	 * Anything triggered by the wp 'admin_footer' wp hook should be put in here. This particluar method will apply on ALL EE_Admin Pages.
1429
	 *
1430
	 * @access  public
1431
	 * @return  void
1432
	 */
1433
	public function admin_footer_global() {
1434
		//dialog container for dialog helper
1435
		$d_cont = '<div class="ee-admin-dialog-container auto-hide hidden">' . "\n";
1436
		$d_cont .= '<div class="ee-notices"></div>';
1437
		$d_cont .= '<div class="ee-admin-dialog-container-inner-content"></div>';
1438
		$d_cont .= '</div>';
1439
		echo $d_cont;
1440
1441
		//help tour stuff?
1442
		if ( isset( $this->_help_tour[$this->_req_action] ) ) {
1443
			echo implode('<br />', $this->_help_tour[$this->_req_action]);
1444
		}
1445
1446
		//current set timezone for timezone js
1447
		EE_Registry::instance()->load_helper('DTT_Helper');
1448
		echo '<span id="current_timezone" class="hidden">' . EEH_DTT_Helper::get_timezone() . '</span>';
1449
	}
1450
1451
1452
	/**
1453
	 * This function sees if there is a method for help popup content existing for the given route.  If there is then we'll use the retrieved array to output the content using the template.
1454
	 *
1455
	 * For child classes:
1456
	 * If you want to have help popups then in your templates or your content you set "triggers" for the content using the "_set_help_trigger('help_trigger_id')" where "help_trigger_id" is what you will use later in your custom method for the help popup content on that page.
1457
	 * Then in your Child_Admin_Page class you need to define a help popup method for the content in the format "_help_popup_content_{route_name}()"  So if you are setting help content for the 'edit_event' route you should have a method named "_help_popup_content_edit_route".
1458
	 * In your defined "help_popup_content_..." method.  You must prepare and return an array in the following format
1459
	 * array(
1460
	 * 	'help_trigger_id' => array(
1461
	 * 		'title' => __('localized title for popup', 'event_espresso'),
1462
	 * 		'content' => __('localized content for popup', 'event_espresso')
1463
	 * 	)
1464
	 * );
1465
	 *
1466
	 * Then the EE_Admin_Parent will take care of making sure that is setup properly on the correct route.
1467
	 *
1468
	 *
1469
	 * @access protected
1470
	 * @return string content
1471
	 */
1472
	protected function _set_help_popup_content( $help_array = array(), $display = FALSE ) {
1473
		$content = '';
1474
1475
		$help_array = empty( $help_array ) ? $this->_get_help_content() : $help_array;
1476
		$template_path = EE_ADMIN_TEMPLATE . 'admin_help_popup.template.php';
1477
1478
1479
		//loop through the array and setup content
1480
		foreach ( $help_array as $trigger => $help ) {
1481
			//make sure the array is setup properly
1482
			if ( !isset($help['title']) || !isset($help['content'] ) ) {
1483
				throw new EE_Error( __('Does not look like the popup content array has been setup correctly.  Might want to double check that.  Read the comments for the _get_help_popup_content method found in "EE_Admin_Page" class', 'event_espresso') );
1484
			}
1485
1486
			//we're good so let'd setup the template vars and then assign parsed template content to our content.
1487
			$template_args = array(
1488
				'help_popup_id' => $trigger,
1489
				'help_popup_title' => $help['title'],
1490
				'help_popup_content' => $help['content']
1491
				);
1492
1493
			$content .= EEH_Template::display_template( $template_path, $template_args, TRUE );
1494
		}
1495
1496
		if ( $display )
1497
			echo $content;
1498
		else
1499
			return $content;
1500
	}
1501
1502
1503
1504
1505
	/**
1506
	 * All this does is retrive the help content array if set by the EE_Admin_Page child
1507
	 *
1508
	 * @access private
1509
	 * @return array properly formatted array for help popup content
1510
	 */
1511
	private function _get_help_content() {
1512
		//what is the method we're looking for?
1513
		$method_name = '_help_popup_content_' . $this->_req_action;
1514
1515
		//if method doesn't exist let's get out.
1516
		if ( !method_exists( $this, $method_name ) )
1517
			return array();
1518
1519
		//k we're good to go let's retrieve the help array
1520
		$help_array = call_user_func( array( $this, $method_name ) );
1521
1522
		//make sure we've got an array!
1523
		if ( !is_array($help_array) ) {
1524
			throw new EE_Error( __('Something went wrong with help popup content generation. Expecting an array and well, this ain\'t no array bub.', 'event_espresso' ) );
1525
		}
1526
1527
		return $help_array;
1528
	}
1529
1530
1531
1532
	/**
1533
	 * EE Admin Pages can use this to set a properly formatted trigger for a help popup.
1534
	 *
1535
	 * By default the trigger html is printed.  Otherwise it can be returned if the $display flag is set "false"
1536
	 *
1537
	 * See comments made on the _set_help_content method for understanding other parts to the help popup tool.
1538
	 *
1539
	 *
1540
	 * @access protected
1541
	 * @param string  $trigger_id reference for retrieving the trigger content for the popup
1542
	 * @param boolean $display    if false then we return the trigger string
1543
	 * @param array $dimensions an array of dimensions for the box (array(h,w))
1544
	 * @return string
1545
	 */
1546
	protected function _set_help_trigger( $trigger_id, $display = TRUE, $dimensions = array( '400', '640') ) {
1547
1548
		if ( defined('DOING_AJAX') ) return;
1549
1550
		//let's check and see if there is any content set for this popup.  If there isn't then we'll include a default title and content so that developers know something needs to be corrected
1551
		$help_array = $this->_get_help_content();
1552
		$help_content = '';
1553
1554
		if ( empty( $help_array ) || !isset( $help_array[$trigger_id] ) ) {
1555
			$help_array[$trigger_id] = array(
1556
				'title' => __('Missing Content', 'event_espresso'),
1557
				'content' => __('A trigger has been set that doesn\'t have any corresponding content. Make sure you have set the help content. (see the "_set_help_popup_content" method in the EE_Admin_Page for instructions.)', 'event_espresso')
1558
				);
1559
			$help_content = $this->_set_help_popup_content( $help_array, FALSE );
1560
		}
1561
1562
		//let's setup the trigger
1563
		$content = '<a class="ee-dialog" href="?height='. $dimensions[0] . '&width=' . $dimensions[1] . '&inlineId=' . $trigger_id . '" target="_blank"><span class="question ee-help-popup-question"></span></a>';
1564
		$content = $content . $help_content;
1565
1566
		if ( $display )
1567
			echo $content;
1568
		else
1569
			return $content;
1570
	}
1571
1572
1573
	/**
1574
	 * _add_global_screen_options
1575
	 * Add any extra wp_screen_options within this method using built-in WP functions/methods for doing so.
1576
	 * This particular method will add_screen_options on ALL EE_Admin Pages
1577
	 * @link http://chrismarslender.com/wp-tutorials/wordpress-screen-options-tutorial/
1578
	 * see also WP_Screen object documents...
1579
	 * @link http://codex.wordpress.org/Class_Reference/WP_Screen
1580
	 *
1581
	 * @abstract
1582
	 * @access private
1583
	 * @return void
1584
	 */
1585
	private function _add_global_screen_options() {}
1586
1587
1588
1589
1590
1591
1592
1593
1594
1595
	/**
1596
	 * _add_global_feature_pointers
1597
	 * This method is used for implementing any "feature pointers" (using built-in WP styling js).
1598
	 * This particular method will implement feature pointers for ALL EE_Admin pages.
1599
	 * Note: this is just a placeholder for now.  Implementation will come down the road
1600
	 * @see WP_Internal_Pointers class in wp-admin/includes/template.php for example (its a final class so can't be extended) also see:
1601
	 * @link http://eamann.com/tech/wordpress-portland/
1602
	 *
1603
	 * @abstract
1604
	 * @access protected
1605
	 * @return void
1606
	 */
1607
	private function _add_global_feature_pointers() {}
1608
1609
1610
1611
1612
1613
1614
	/**
1615
	 * load_global_scripts_styles
1616
	 * The scripts and styles enqueued in here will be loaded on every EE Admin page
1617
	 *
1618
	 * @return void
1619
	 */
1620
	public function load_global_scripts_styles() {
1621
		/** STYLES **/
1622
		// add debugging styles
1623
		if ( WP_DEBUG ) {
1624
			add_action('admin_head', array( $this, 'add_xdebug_style' ));
1625
		}
1626
1627
		//register all styles
1628
		wp_register_style( 'espresso-ui-theme', EE_GLOBAL_ASSETS_URL . 'css/espresso-ui-theme/jquery-ui-1.10.3.custom.min.css', array(),EVENT_ESPRESSO_VERSION );
1629
		wp_register_style('ee-admin-css', EE_ADMIN_URL . 'assets/ee-admin-page.css', array(), EVENT_ESPRESSO_VERSION);
1630
		//helpers styles
1631
		wp_register_style('ee-text-links', EE_PLUGIN_DIR_URL . 'core/helpers/assets/ee_text_list_helper.css', array(), EVENT_ESPRESSO_VERSION );
1632
		//enqueue global styles
1633
		wp_enqueue_style('ee-admin-css');
1634
1635
1636
		/** SCRIPTS **/
1637
1638
		//register all scripts
1639
		wp_register_script( 'espresso_core', EE_GLOBAL_ASSETS_URL . 'scripts/espresso_core.js', array('jquery'), EVENT_ESPRESSO_VERSION, TRUE );
1640
		wp_register_script('ee-dialog', EE_ADMIN_URL . 'assets/ee-dialog-helper.js', array('jquery', 'jquery-ui-draggable'), EVENT_ESPRESSO_VERSION, TRUE );
1641
		wp_register_script('ee_admin_js', EE_ADMIN_URL . 'assets/ee-admin-page.js', array( 'espresso_core', 'ee-parse-uri', 'ee-dialog'), EVENT_ESPRESSO_VERSION, true );
1642
1643
		wp_register_script('jquery-ui-timepicker-addon', EE_GLOBAL_ASSETS_URL . 'scripts/jquery-ui-timepicker-addon.js', array('jquery-ui-datepicker', 'jquery-ui-slider'), EVENT_ESPRESSO_VERSION, true );
1644
		// register jQuery Validate - see /includes/functions/wp_hooks.php
1645
		add_filter( 'FHEE_load_jquery_validate', '__return_true' );
1646
		add_filter('FHEE_load_joyride', '__return_true');
1647
1648
		//script for sorting tables
1649
		wp_register_script('espresso_ajax_table_sorting', EE_ADMIN_URL . "assets/espresso_ajax_table_sorting.js", array('ee_admin_js', 'jquery-ui-sortable'), EVENT_ESPRESSO_VERSION, TRUE);
1650
		//script for parsing uri's
1651
		wp_register_script( 'ee-parse-uri', EE_GLOBAL_ASSETS_URL . 'scripts/parseuri.js', array(), EVENT_ESPRESSO_VERSION, TRUE );
1652
		//and parsing associative serialized form elements
1653
		wp_register_script( 'ee-serialize-full-array', EE_GLOBAL_ASSETS_URL . 'scripts/jquery.serializefullarray.js', array('jquery'), EVENT_ESPRESSO_VERSION, TRUE );
1654
		//helpers scripts
1655
		wp_register_script('ee-text-links', EE_PLUGIN_DIR_URL . 'core/helpers/assets/ee_text_list_helper.js', array('jquery'), EVENT_ESPRESSO_VERSION, TRUE );
1656
		wp_register_script( 'ee-moment-core', EE_THIRD_PARTY_URL . 'moment/moment-with-locales.min.js', array(), EVENT_ESPRESSO_VERSION, TRUE );
1657
		wp_register_script( 'ee-moment', EE_THIRD_PARTY_URL . 'moment/moment-timezone-with-data.min.js', array('ee-moment-core'), EVENT_ESPRESSO_VERSION, TRUE );
1658
		wp_register_script( 'ee-datepicker', EE_ADMIN_URL . 'assets/ee-datepicker.js', array('jquery-ui-timepicker-addon','ee-moment'), EVENT_ESPRESSO_VERSION, TRUE );
1659
1660
		//google charts
1661
		wp_register_script( 'google-charts', 'https://www.gstatic.com/charts/loader.js', array(), EVENT_ESPRESSO_VERSION, false );
1662
1663
		//enqueue global scripts
1664
1665
		//taking care of metaboxes
1666
		if ( ( isset($this->_route_config['metaboxes'] ) || isset($this->_route_config['has_metaboxes']) ) && empty( $this->_cpt_route) ) {
0 ignored issues
show
Bug introduced by
The property _cpt_route does not seem to exist. Did you mean _route?

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
1667
			wp_enqueue_script('dashboard');
1668
		}
1669
1670
		//enqueue thickbox for ee help popups.  default is to enqueue unless its explicitly set to false since we're assuming all EE pages will have popups
1671
		if ( ! isset( $this->_route_config['has_help_popups']) || ( isset( $this->_route_config['has_help_popups']) && $this->_route_config['has_help_popups'] ) ) {
1672
			wp_enqueue_script('ee_admin_js');
1673
			wp_enqueue_style('ee-admin-css');
1674
		}
1675
1676
1677
		//localize script for ajax lazy loading
1678
		$lazy_loader_container_ids = apply_filters( 'FHEE__EE_Admin_Page_Core__load_global_scripts_styles__loader_containers', array('espresso_news_post_box_content') );
1679
		wp_localize_script( 'ee_admin_js', 'eeLazyLoadingContainers', $lazy_loader_container_ids);
1680
1681
1682
		/**
1683
		 * help tour stuff
1684
		 */
1685
		if ( !empty( $this->_help_tour ) ) {
1686
1687
			//register the js for kicking things off
1688
			wp_enqueue_script('ee-help-tour', EE_ADMIN_URL . 'assets/ee-help-tour.js', array('jquery-joyride'), EVENT_ESPRESSO_VERSION, TRUE );
1689
1690
			//setup tours for the js tour object
1691
			foreach ( $this->_help_tour['tours'] as $tour ) {
1692
				$tours[] = array(
0 ignored issues
show
Coding Style Comprehensibility introduced by
$tours was never initialized. Although not strictly required by PHP, it is generally a good practice to add $tours = 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...
1693
					'id' => $tour->get_slug(),
1694
					'options' => $tour->get_options()
1695
					);
1696
			}
1697
1698
			wp_localize_script('ee-help-tour', 'EE_HELP_TOUR', array('tours' => $tours ) );
0 ignored issues
show
Bug introduced by
The variable $tours 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...
1699
1700
			//admin_footer_global will take care of making sure our help_tour skeleton gets printed via the info stored in $this->_help_tour
1701
		}
1702
	}
1703
1704
1705
1706
1707
	/**
1708
	*		admin_footer_scripts_eei18n_js_strings
1709
	*
1710
	*		@access 		public
1711
	*		@return 		void
1712
	*/
1713
	public function admin_footer_scripts_eei18n_js_strings() {
1714
1715
		EE_Registry::$i18n_js_strings['ajax_url'] = WP_AJAX_URL;
1716
		EE_Registry::$i18n_js_strings['confirm_delete'] = __( 'Are you absolutely sure you want to delete this item?\nThis action will delete ALL DATA associated with this item!!!\nThis can NOT be undone!!!', 'event_espresso' );
1717
1718
		EE_Registry::$i18n_js_strings['January'] = __( 'January', 'event_espresso' );
1719
		EE_Registry::$i18n_js_strings['February'] = __( 'February', 'event_espresso' );
1720
		EE_Registry::$i18n_js_strings['March'] = __( 'March', 'event_espresso' );
1721
		EE_Registry::$i18n_js_strings['April'] = __( 'April', 'event_espresso' );
1722
		EE_Registry::$i18n_js_strings['May'] = __( 'May', 'event_espresso' );
1723
		EE_Registry::$i18n_js_strings['June'] = __( 'June', 'event_espresso' );
1724
		EE_Registry::$i18n_js_strings['July'] = __( 'July', 'event_espresso' );
1725
		EE_Registry::$i18n_js_strings['August'] = __( 'August', 'event_espresso' );
1726
		EE_Registry::$i18n_js_strings['September'] = __( 'September', 'event_espresso' );
1727
		EE_Registry::$i18n_js_strings['October'] = __( 'October', 'event_espresso' );
1728
		EE_Registry::$i18n_js_strings['November'] = __( 'November', 'event_espresso' );
1729
		EE_Registry::$i18n_js_strings['December'] = __( 'December', 'event_espresso' );
1730
		EE_Registry::$i18n_js_strings['Jan'] = __( 'Jan', 'event_espresso' );
1731
		EE_Registry::$i18n_js_strings['Feb'] = __( 'Feb', 'event_espresso' );
1732
		EE_Registry::$i18n_js_strings['Mar'] = __( 'Mar', 'event_espresso' );
1733
		EE_Registry::$i18n_js_strings['Apr'] = __( 'Apr', 'event_espresso' );
1734
		EE_Registry::$i18n_js_strings['May'] = __( 'May', 'event_espresso' );
1735
		EE_Registry::$i18n_js_strings['Jun'] = __( 'Jun', 'event_espresso' );
1736
		EE_Registry::$i18n_js_strings['Jul'] = __( 'Jul', 'event_espresso' );
1737
		EE_Registry::$i18n_js_strings['Aug'] = __( 'Aug', 'event_espresso' );
1738
		EE_Registry::$i18n_js_strings['Sep'] = __( 'Sep', 'event_espresso' );
1739
		EE_Registry::$i18n_js_strings['Oct'] = __( 'Oct', 'event_espresso' );
1740
		EE_Registry::$i18n_js_strings['Nov'] = __( 'Nov', 'event_espresso' );
1741
		EE_Registry::$i18n_js_strings['Dec'] = __( 'Dec', 'event_espresso' );
1742
1743
		EE_Registry::$i18n_js_strings['Sunday'] = __( 'Sunday', 'event_espresso' );
1744
		EE_Registry::$i18n_js_strings['Monday'] = __( 'Monday', 'event_espresso' );
1745
		EE_Registry::$i18n_js_strings['Tuesday'] = __( 'Tuesday', 'event_espresso' );
1746
		EE_Registry::$i18n_js_strings['Wednesday'] = __( 'Wednesday', 'event_espresso' );
1747
		EE_Registry::$i18n_js_strings['Thursday'] = __( 'Thursday', 'event_espresso' );
1748
		EE_Registry::$i18n_js_strings['Friday'] = __( 'Friday', 'event_espresso' );
1749
		EE_Registry::$i18n_js_strings['Saturday'] = __( 'Saturday', 'event_espresso' );
1750
		EE_Registry::$i18n_js_strings['Sun'] = __( 'Sun', 'event_espresso' );
1751
		EE_Registry::$i18n_js_strings['Mon'] = __( 'Mon', 'event_espresso' );
1752
		EE_Registry::$i18n_js_strings['Tue'] = __( 'Tue', 'event_espresso' );
1753
		EE_Registry::$i18n_js_strings['Wed'] = __( 'Wed', 'event_espresso' );
1754
		EE_Registry::$i18n_js_strings['Thu'] = __( 'Thu', 'event_espresso' );
1755
		EE_Registry::$i18n_js_strings['Fri'] = __( 'Fri', 'event_espresso' );
1756
		EE_Registry::$i18n_js_strings['Sat'] = __( 'Sat', 'event_espresso' );
1757
1758
		//setting on espresso_core instead of ee_admin_js because espresso_core is enqueued by the maintenance
1759
		//admin page when in maintenance mode and ee_admin_js is not loaded then.  This works everywhere else because
1760
		//espresso_core is listed as a dependency of ee_admin_js.
1761
		wp_localize_script( 'espresso_core', 'eei18n', EE_Registry::$i18n_js_strings );
1762
1763
	}
1764
1765
1766
1767
	/**
1768
	*		load enhanced xdebug styles for ppl with failing eyesight
1769
	*
1770
	*		@access 		public
1771
	*		@return 		void
1772
	*/
1773
	public function add_xdebug_style() {
1774
		echo '<style>.xdebug-error { font-size:1.5em; }</style>';
1775
	}
1776
1777
1778
	/************************/
1779
	/** LIST TABLE METHODS **/
1780
	/************************/
1781
1782
1783
1784
1785
1786
	/**
1787
	 * this sets up the list table if the current view requires it.
1788
	 *
1789
	 * @access protected
1790
	 * @return void
1791
	 */
1792
	protected function _set_list_table() {
1793
1794
		//first is this a list_table view?
1795
		if ( !isset($this->_route_config['list_table']) )
1796
			return; //not a list_table view so get out.
1797
1798
		//list table functions are per view specific (because some admin pages might have more than one listtable!)
1799
1800
		if ( call_user_func( array( $this, '_set_list_table_views_' . $this->_req_action ) ) === FALSE ) {
1801
			//user error msg
1802
			$error_msg = __('An error occurred. The requested list table views could not be found.', 'event_espresso' );
1803
			//developer error msg
1804
			$error_msg .= '||' . sprintf( __('List table views for "%s" route could not be setup. Check that you have the corresponding method, "%s" set up for defining list_table_views for this route.', 'event_espresso' ), $this->_req_action, '_set_list_table_views_' . $this->_req_action );
1805
			throw new EE_Error( $error_msg );
1806
		}
1807
1808
		//let's provide the ability to filter the views per PAGE AND ROUTE, per PAGE, and globally
1809
		$this->_views = apply_filters( 'FHEE_list_table_views_' . $this->page_slug . '_' . $this->_req_action, $this->_views );
1810
		$this->_views = apply_filters( 'FHEE_list_table_views_' . $this->page_slug, $this->_views );
1811
		$this->_views = apply_filters( 'FHEE_list_table_views', $this->_views );
1812
1813
		$this->_set_list_table_view();
1814
		$this->_set_list_table_object();
1815
1816
	}
1817
1818
1819
1820
1821
1822
1823
1824
1825
1826
1827
	/**
1828
	 * 		set current view for List Table
1829
	*		@access public
1830
	*		@return array
1831
	*/
1832
	protected function _set_list_table_view() {
1833
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
1834
1835
1836
		// looking at active items or dumpster diving ?
1837
		if ( ! isset( $this->_req_data['status'] ) || ! array_key_exists( $this->_req_data['status'], $this->_views )) {
1838
			$this->_view = isset( $this->_views['in_use'] ) ? 'in_use' : 'all';
1839
		} else {
1840
			$this->_view = sanitize_key( $this->_req_data['status'] );
1841
		}
1842
	}
1843
1844
1845
1846
	/**
1847
	 * _set_list_table_object
1848
	 * WP_List_Table objects need to be loaded fairly early so automatic stuff WP does is taken care of.
1849
	 */
1850
	protected function _set_list_table_object() {
1851
		if ( isset($this->_route_config['list_table'] ) ) {
1852 View Code Duplication
			if ( !class_exists( $this->_route_config['list_table'] ) )
1853
				throw new EE_Error( sprintf( __('The %s class defined for the list table does not exist.  Please check the spelling of the class ref in the $_page_config property on %s.', 'event_espresso'), $this->_route_config['list_table'], get_class($this) ) );
1854
			$a = new ReflectionClass($this->_route_config['list_table']);
1855
			$this->_list_table_object = $a->newInstance($this);
1856
		}
1857
	}
1858
1859
1860
1861
1862
1863
	/**
1864
	 * get_list_table_view_RLs - get it? View RL ?? VU-RL???  URL ??
1865
	 *
1866
	 * @param array $extra_query_args Optional. An array of extra query args to add to the generated
1867
	 *                                		          	urls.  The array should be indexed by the view it is being
1868
	 *                                		          	added to.
1869
	 *
1870
	 * @return array
1871
	 */
1872
	public function get_list_table_view_RLs( $extra_query_args = array() ) {
1873
1874
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
1875
1876
		if ( empty( $this->_views )) {
1877
			$this->_views = array();
1878
		}
1879
1880
		// cycle thru views
1881
		foreach ( $this->_views as $key => $view ) {
1882
			$query_args = array();
1883
			// check for current view
1884
			$this->_views[ $key ]['class'] = $this->_view == $view['slug'] ? 'current' : '';
1885
			$query_args['action'] = $this->_req_action;
1886
			$query_args[$this->_req_action.'_nonce'] = wp_create_nonce( $query_args['action'] . '_nonce' );
1887
			$query_args['status'] = $view['slug'];
1888
			//merge any other arguments sent in.
1889
			if ( isset( $extra_query_args[$view['slug']] ) ) {
1890
				$query_args = array_merge( $query_args, $extra_query_args[$view['slug']] );
1891
			}
1892
			$this->_views[ $key ]['url'] = EE_Admin_Page::add_query_args_and_nonce( $query_args, $this->_admin_base_url );
1893
		}
1894
1895
		return $this->_views;
1896
	}
1897
1898
1899
1900
1901
	/**
1902
	 * _entries_per_page_dropdown
1903
	 * generates a drop down box for selecting the number of visiable rows in an admin page list table
1904
	 * @todo: Note: ideally this should be added to the screen options dropdown as that would be consistent with how WP does it.
1905
	 * @access protected
1906
	 * @param int $max_entries total number of rows in the table
1907
	 * @return string
1908
	*/
1909
	protected function _entries_per_page_dropdown( $max_entries = FALSE ) {
1910
1911
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
1912
		$values = array( 10, 25, 50, 100 );
1913
		$per_page = ( ! empty( $this->_req_data['per_page'] )) ? absint( $this->_req_data['per_page'] ) : 10;
1914
1915
		if ( $max_entries ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $max_entries of type false|integer is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For integer values, zero is a special case, in particular the following results might be unexpected:

0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
Loading history...
1916
			$values[] = $max_entries;
1917
			sort( $values );
1918
		}
1919
1920
		$entries_per_page_dropdown = '
1921
			<div id="entries-per-page-dv" class="alignleft actions">
1922
				<label class="hide-if-no-js">
1923
					Show
1924
					<select id="entries-per-page-slct" name="entries-per-page-slct">';
1925
1926
		foreach ( $values as $value ) {
1927
			if ( $value < $max_entries ) {
1928
				$selected = $value == $per_page ?  ' selected="' . $per_page . '"' : '';
1929
				$entries_per_page_dropdown .= '
1930
						<option value="'.$value.'"'.$selected.'>'.$value.'&nbsp;&nbsp;</option>';
1931
			}
1932
		}
1933
1934
		$selected = $max_entries == $per_page ?  ' selected="' . $per_page . '"' : '';
1935
		$entries_per_page_dropdown .= '
1936
						<option value="'.$max_entries.'"'.$selected.'>All&nbsp;&nbsp;</option>';
1937
1938
		$entries_per_page_dropdown .= '
1939
					</select>
1940
					entries
1941
				</label>
1942
				<input id="entries-per-page-btn" class="button-secondary" type="submit" value="Go" >
1943
			</div>
1944
		';
1945
		return $entries_per_page_dropdown;
1946
	}
1947
1948
1949
1950
1951
	/**
1952
	 * 		_set_search_attributes
1953
	*		@access 		protected
1954
	*		@return 		void
1955
	*/
1956
	public function _set_search_attributes() {
1957
		$this->_template_args['search']['btn_label'] = sprintf( __( 'Search %s', 'event_espresso' ), empty( $this->_search_btn_label ) ? $this->page_label : $this->_search_btn_label );
1958
		$this->_template_args['search']['callback'] = 'search_' . $this->page_slug;
1959
	}
1960
1961
	/*** END LIST TABLE METHODS **/
1962
	/*****************************/
1963
1964
1965
1966
1967
1968
	/**
1969
	 * 		_add_registered_metaboxes
1970
	 * 		this loads any registered metaboxes via the 'metaboxes' index in the _page_config property array.
1971
	 *
1972
	 * @link http://codex.wordpress.org/Function_Reference/add_meta_box
1973
	 * @access private
1974
	 * @return void
1975
	*/
1976
	private function _add_registered_meta_boxes() {
1977
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
1978
1979
		//we only add meta boxes if the page_route calls for it
1980
		if ( is_array($this->_route_config) && isset( $this->_route_config['metaboxes'] ) && is_array($this->_route_config['metaboxes']) ) {
1981
1982
1983
			//this simply loops through the callbacks provided and checks if there is a corresponding callback registered by the child - if there is then we go ahead and process the metabox loader.
1984
			foreach ( $this->_route_config['metaboxes'] as $metabox_callback ) {
1985
				if ( call_user_func( array($this, &$metabox_callback) ) === FALSE ) {
1986
					// user error msg
1987
				$error_msg =  __( 'An error occurred. The  requested metabox could not be found.', 'event_espresso' );
1988
				// developer error msg
1989
				$error_msg .= '||' . sprintf( __( 'The metabox with the string "%s" could not be called. Check that the spelling for method names and actions in the "_page_config[\'metaboxes\']" array are all correct.', 'event_espresso' ), $metabox_callback );
1990
				throw new EE_Error( $error_msg );
1991
				}
1992
			}
1993
		}
1994
	}
1995
1996
1997
1998
1999
	/**
2000
	 * _add_screen_columns
2001
	 * This will check the _page_config array and if there is "columns" key index indicated, we'll set the template as the dynamic column template and we'll setup the column options for the page.
2002
	 *
2003
	 * @access private
2004
	 * @return void
2005
	 */
2006
	private function _add_screen_columns() {
2007
		if ( is_array($this->_route_config) && isset( $this->_route_config['columns'] ) && is_array($this->_route_config['columns']) && count( $this->_route_config['columns'] == 2 ) ) {
2008
2009
			add_screen_option('layout_columns', array('max' => (int) $this->_route_config['columns'][0], 'default' => (int) $this->_route_config['columns'][1] ) );
2010
			$this->_template_args['num_columns'] = $this->_route_config['columns'][0];
2011
			$screen_id = $this->_current_screen->id;
2012
			$screen_columns = (int) get_user_option("screen_layout_$screen_id");
2013
			$total_columns = !empty($screen_columns) ? $screen_columns : $this->_route_config['columns'][1];
2014
			$this->_template_args['current_screen_widget_class'] = 'columns-' . $total_columns;
2015
			$this->_template_args['current_page'] = $this->_wp_page_slug;
2016
			$this->_template_args['screen'] = $this->_current_screen;
2017
			$this->_column_template_path = EE_ADMIN_TEMPLATE . 'admin_details_metabox_column_wrapper.template.php';
2018
2019
			//finally if we don't have has_metaboxes set in the route config let's make sure it IS set other wise the necessary hidden fields for this won't be loaded.
2020
			$this->_route_config['has_metaboxes'] = TRUE;
2021
		}
2022
	}
2023
2024
2025
2026
	/**********************************/
2027
	/** GLOBALLY AVAILABLE METABOXES **/
2028
2029
	/**
2030
	 * In this section we put any globally available EE metaboxes for all EE Admin pages.  They are called by simply referencing the callback in the _page_config array property.  This way you can be very specific about what pages these get loaded on.
2031
	 */
2032
2033
	private function _espresso_news_post_box() {
2034
		$news_box_title = apply_filters( 'FHEE__EE_Admin_Page___espresso_news_post_box__news_box_title', __( 'New @ Event Espresso', 'event_espresso' ) );
2035
		add_meta_box( 'espresso_news_post_box', $news_box_title, array(
2036
			$this,
2037
			'espresso_news_post_box'
2038
		), $this->_wp_page_slug, 'side' );
2039
	}
2040
2041
2042
	/**
2043
	 * Code for setting up espresso ratings request metabox.
2044
	 */
2045
	protected function _espresso_ratings_request() {
2046
		if ( ! apply_filters( 'FHEE_show_ratings_request_meta_box', true ) ) {
2047
			return '';
2048
		}
2049
		$ratings_box_title = apply_filters( 'FHEE__EE_Admin_Page___espresso_news_post_box__news_box_title', __('Keep Event Espresso Decaf Free', 'event_espresso') );
2050
		add_meta_box( 'espresso_ratings_request', $ratings_box_title, array(
2051
			$this,
2052
			'espresso_ratings_request'
2053
		), $this->_wp_page_slug, 'side' );
2054
	}
2055
2056
2057
	/**
2058
	 * Code for setting up espresso ratings request metabox content.
2059
	 */
2060
	public function espresso_ratings_request() {
2061
		$template_path = EE_ADMIN_TEMPLATE . 'espresso_ratings_request_content.template.php';
2062
		EE_Registry::instance()->load_helper( 'Template' );
2063
		EEH_Template::display_template( $template_path, array() );
2064
	}
2065
2066
2067
2068
2069
	public static function cached_rss_display( $rss_id, $url ) {
2070
		$loading = '<p class="widget-loading hide-if-no-js">' . __( 'Loading&#8230;' ) . '</p><p class="hide-if-js">' . __( 'This widget requires JavaScript.' ) . '</p>';
2071
		$doing_ajax = ( defined( 'DOING_AJAX' ) && DOING_AJAX );
2072
		$pre = '<div class="espresso-rss-display">' . "\n\t";
2073
		$pre .= '<span id="' . $rss_id . '_url" class="hidden">' . $url . '</span>';
2074
		$post = '</div>' . "\n";
2075
2076
		$cache_key = 'ee_rss_' . md5( $rss_id );
2077
		if ( FALSE != ( $output = get_transient( $cache_key ) ) ) {
2078
			echo $pre . $output . $post;
2079
			return TRUE;
2080
		}
2081
2082
		if ( ! $doing_ajax ) {
2083
			echo $pre . $loading . $post;
2084
			return FALSE;
2085
		}
2086
2087
		ob_start();
2088
		wp_widget_rss_output($url, array('show_date' => 0, 'items' => 5) );
2089
		set_transient( $cache_key, ob_get_flush(), 12 * HOUR_IN_SECONDS );
2090
		return TRUE;
2091
2092
	}
2093
2094
2095
	public function espresso_news_post_box() {
2096
		?>
2097
	  <div class="padding">
2098
	  	<div id="espresso_news_post_box_content" class="infolinks">
2099
	  		<?php
2100
	  		// Get RSS Feed(s)
2101
	  		$feed_url = apply_filters( 'FHEE__EE_Admin_Page__espresso_news_post_box__feed_url', 'http://eventespresso.com/feed/' );
2102
	  		$url = urlencode($feed_url);
2103
	  		self::cached_rss_display( 'espresso_news_post_box_content', $url );
2104
2105
	  		?>
2106
	  	</div>
2107
	  	<?php do_action( 'AHEE__EE_Admin_Page__espresso_news_post_box__after_content'); ?>
2108
	  </div>
2109
		<?php
2110
	}
2111
2112
2113
	private function _espresso_links_post_box() {
2114
		//Hiding until we actually have content to put in here...
2115
		//add_meta_box('espresso_links_post_box', __('Helpful Plugin Links', 'event_espresso'), array( $this, 'espresso_links_post_box'), $this->_wp_page_slug, 'side');
2116
	}
2117
2118
	public function espresso_links_post_box() {
2119
		   //Hiding until we actually have content to put in here...
2120
		   //$templatepath = EE_ADMIN_TEMPLATE . 'admin_general_metabox_contents_espresso_links.template.php';
2121
			//EEH_Template::display_template( $templatepath );
2122
		}
2123
2124
2125
2126
	protected function _espresso_sponsors_post_box() {
2127
2128
		$show_sponsors = apply_filters( 'FHEE_show_sponsors_meta_box', TRUE );
2129
		if ( $show_sponsors )
2130
			add_meta_box('espresso_sponsors_post_box', __('Event Espresso Highlights', 'event_espresso'), array( $this, 'espresso_sponsors_post_box'), $this->_wp_page_slug, 'side');
2131
	}
2132
2133
2134
	public function espresso_sponsors_post_box() {
2135
		$templatepath = EE_ADMIN_TEMPLATE . 'admin_general_metabox_contents_espresso_sponsors.template.php';
2136
		EEH_Template::display_template( $templatepath );
2137
	}
2138
2139
2140
2141
	private function _publish_post_box() {
2142
		$meta_box_ref = 'espresso_' . $this->page_slug . '_editor_overview';
2143
2144
		//if there is a array('label' => array('publishbox' => 'some title') ) present in the _page_config array then we'll use that for the metabox label.  Otherwise we'll just use publish (publishbox itself could be an array of labels indexed by routes)
2145
		if ( !empty( $this->_labels['publishbox'] ) ) {
2146
			$box_label = is_array( $this->_labels['publishbox'] ) ? $this->_labels['publishbox'][$this->_req_action] : $this->_labels['publishbox'];
2147
		} else {
2148
			$box_label = __('Publish', 'event_espresso');
2149
		}
2150
2151
		$box_label = apply_filters( 'FHEE__EE_Admin_Page___publish_post_box__box_label', $box_label, $this->_req_action, $this );
2152
2153
		add_meta_box( $meta_box_ref, $box_label, array( $this, 'editor_overview' ), $this->_current_screen->id, 'side', 'high' );
2154
2155
	}
2156
2157
2158
2159
	public function editor_overview() {
2160
		//if we have extra content set let's add it in if not make sure its empty
2161
		$this->_template_args['publish_box_extra_content'] = isset( $this->_template_args['publish_box_extra_content'] ) ? $this->_template_args['publish_box_extra_content'] : '';
2162
		$template_path = EE_ADMIN_TEMPLATE . 'admin_details_publish_metabox.template.php';
2163
		echo EEH_Template::display_template( $template_path, $this->_template_args, TRUE );
2164
	}
2165
2166
2167
	/** end of globally available metaboxes section **/
2168
	/*************************************************/
2169
2170
2171
2172
	/**
2173
	 * Public wrapper for the protected method.  Allows plugins/addons to externally call the
2174
	 * protected method.
2175
	 *
2176
	 * @see $this->_set_publish_post_box_vars for param details
2177
	 * @since 4.6.0
2178
	 */
2179
	public function set_publish_post_box_vars( $name = null, $id = false, $delete = false, $save_close_redirect_URL = null, $both_btns = true ) {
2180
		$this->_set_publish_post_box_vars( $name, $id, $delete, $save_close_redirect_URL, $both_btns );
0 ignored issues
show
Documentation introduced by
$id is of type boolean, but the function expects a false|integer.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
Documentation introduced by
$delete is of type boolean, but the function expects a false|string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
2181
	}
2182
2183
2184
	/**
2185
	 * Sets the _template_args arguments used by the _publish_post_box shortcut
2186
	 *
2187
	 * Note: currently there is no validation for this.  However if you want the delete button, the
2188
	 * save, and save and close buttons to work properly, then you will want to include a
2189
	 * values for the name and id arguments.
2190
	 *
2191
	 * @todo  Add in validation for name/id arguments.
2192
	 *
2193
	 * @param	string	$name		key used for the action ID (i.e. event_id)
2194
	 * @param	int		$id	id attached to the item published
2195
	 * @param	string	$delete	page route callback for the delete action
2196
	 * @param	string	$post_save_redirect_URL	custom URL to redirect to after Save & Close has been completed
0 ignored issues
show
Bug introduced by
There is no parameter named $post_save_redirect_URL. Was it maybe removed?

This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.

Consider the following example. The parameter $italy is not defined by the method finale(...).

/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}

The most likely cause is that the parameter was removed, but the annotation was not.

Loading history...
2197
	 * @param	boolean	$both_btns	whether to display BOTH the "Save & Close" and "Save" buttons or just the Save button
2198
	 */
2199
	protected function _set_publish_post_box_vars( $name = NULL, $id = FALSE, $delete = FALSE, $save_close_redirect_URL = NULL, $both_btns = TRUE ) {
2200
2201
		// if Save & Close, use a custom redirect URL or default to the main page?
2202
		$save_close_redirect_URL = ! empty( $save_close_redirect_URL ) ? $save_close_redirect_URL : $this->_admin_base_url;
2203
		// create the Save & Close and Save buttons
2204
		$this->_set_save_buttons( $both_btns, array(), array(), $save_close_redirect_URL );
2205
		//if we have extra content set let's add it in if not make sure its empty
2206
		$this->_template_args['publish_box_extra_content'] = isset( $this->_template_args['publish_box_extra_content'] ) ? $this->_template_args['publish_box_extra_content'] : '';
2207
2208
2209
		if ( $delete && ! empty( $id )  ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $delete of type false|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For string values, the empty string '' is a special case, in particular the following results might be unexpected:

''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
Loading history...
2210
			$delete = is_bool($delete) ? 'delete' : $delete; //make sure we have a default if just true is sent.
2211
			$delete_link_args = array( $name => $id );
2212
			$delete = $this->get_action_link_or_button( $delete, $delete, $delete_link_args, 'submitdelete deletion');
2213
		}
2214
2215
		$this->_template_args['publish_delete_link'] = !empty( $id ) ? $delete : '';
2216
		if ( ! empty( $name ) && ! empty( $id ) ) {
2217
			$hidden_field_arr[$name] = array(
0 ignored issues
show
Coding Style Comprehensibility introduced by
$hidden_field_arr was never initialized. Although not strictly required by PHP, it is generally a good practice to add $hidden_field_arr = 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...
2218
				'type' => 'hidden',
2219
				'value' => $id
2220
				);
2221
			$hf = $this->_generate_admin_form_fields($hidden_field_arr, 'array');
2222
		} else {
2223
			$hf = '';
2224
		}
2225
		// add hidden field
2226
		$this->_template_args['publish_hidden_fields'] = ! empty( $hf ) ? $hf[$name]['field'] : $hf;
2227
2228
	}
2229
2230
2231
2232
2233
	/**
2234
	 * 		displays an error message to ppl who have javascript disabled
2235
	*		@access 		private
2236
	*		@return 		string
2237
	*/
2238
	private function _display_no_javascript_warning() {
2239
		?>
2240
		<noscript>
2241
			<div id="no-js-message" class="error">
2242
				<p style="font-size:1.3em;">
2243
					<span style="color:red;"><?php _e( 'Warning!', 'event_espresso' ); ?></span>
2244
					<?php _e( 'Javascript is currently turned off for your browser. Javascript must be enabled in order for all of the features on this page to function properly. Please turn your javascript back on.', 'event_espresso' ); ?>
2245
				</p>
2246
			</div>
2247
		</noscript>
2248
		<?php
2249
	}
2250
2251
2252
2253
2254
2255
2256
2257
2258
	/**
2259
	 * 		displays espresso success and/or error notices
2260
	*		@access 		private
2261
	*		@return 		string
2262
	*/
2263
	private function _display_espresso_notices() {
2264
		$notices = $this->_get_transient( TRUE );
2265
		echo stripslashes($notices);
2266
	}
2267
2268
2269
2270
2271
2272
2273
	/**
2274
	*		spinny things pacify the masses
2275
	*		@access private
2276
	*		@return string
2277
	*/
2278
	protected function _add_admin_page_ajax_loading_img() {
2279
		?>
2280
			<div id="espresso-ajax-loading" class="ajax-loading-grey">
2281
				<span class="ee-spinner ee-spin"></span><span class="hidden"><?php _e('loading...', 'event_espresso'); ?></span>
2282
			</div>
2283
		<?php
2284
	}
2285
2286
2287
2288
2289
2290
	/**
2291
	*		add admin page overlay for modal boxes
2292
	*		@access private
2293
	*		@return string
2294
	*/
2295
	protected function _add_admin_page_overlay() {
2296
		?>
2297
		<div id="espresso-admin-page-overlay-dv" class=""></div>
2298
		<?php
2299
	}
2300
2301
2302
2303
2304
2305
2306
	/**
2307
	 * facade for add_meta_box
2308
	 * @param string  $action        where the metabox get's displayed
2309
	 * @param string  $title         Title of Metabox (output in metabox header)
2310
	 * @param string  $callback      If not empty and $create_fun is set to false then we'll use a custom callback instead of the one created in here.
2311
	 * @param array  $callback_args an array of args supplied for the metabox
2312
	 * @param string  $column        what metabox column
2313
	 * @param string  $priority      give this metabox a priority (using accepted priorities for wp meta boxes)
2314
	 * @param boolean $create_func   default is true.  Basically we can say we don't WANT to have the runtime function created but just set our own callback for wp's add_meta_box.
2315
	 */
2316
	public function _add_admin_page_meta_box( $action, $title, $callback, $callback_args, $column = 'normal', $priority = 'high', $create_func = true ) {
2317
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, $callback );
2318
2319
		//if we have empty callback args and we want to automatically create the metabox callback then we need to make sure the callback args are generated.
2320
		if ( empty( $callback_args ) && $create_func ) {
2321
			$callback_args = array(
2322
				'template_path' => $this->_template_path,
2323
				'template_args' => $this->_template_args,
2324
				);
2325
		}
2326
2327
		//if $create_func is true (default) then we automatically create the function for displaying the actual meta box.  If false then we take the $callback reference passed through and use it instead (so callers can define their own callback function/method if they wish)
2328
		$call_back_func = $create_func ? create_function('$post, $metabox', 'do_action( "AHEE_log", __FILE__, __FUNCTION__, ""); echo EEH_Template::display_template( $metabox["args"]["template_path"], $metabox["args"]["template_args"], TRUE );') : $callback;
0 ignored issues
show
Security Best Practice introduced by
The use of create_function is highly discouraged, better use a closure.

create_function can pose a great security vulnerability as it is similar to eval, and could be used for arbitrary code execution. We highly recommend to use a closure instead.

// Instead of
$function = create_function('$a, $b', 'return $a + $b');

// Better use
$function = function($a, $b) { return $a + $b; }
Loading history...
2329
2330
		add_meta_box( str_replace( '_', '-', $action ) . '-mbox', $title, $call_back_func, $this->_wp_page_slug, $column, $priority, $callback_args );
2331
	}
2332
2333
2334
2335
2336
2337
	/**
2338
	 * generates HTML wrapper for and admin details page that contains metaboxes in columns
2339
	 * @return [type] [description]
0 ignored issues
show
Documentation introduced by
The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
2340
	 */
2341
	public function display_admin_page_with_metabox_columns() {
2342
		$this->_template_args['post_body_content'] = $this->_template_args['admin_page_content'];
2343
		$this->_template_args['admin_page_content'] = EEH_Template::display_template( $this->_column_template_path, $this->_template_args, TRUE);
2344
2345
		//the final wrapper
2346
		$this->admin_page_wrapper();
2347
	}
2348
2349
2350
2351
2352
2353
2354
2355
	/**
2356
	*		generates  HTML wrapper for an admin details page
2357
	*		@access public
2358
	*		@return void
2359
	*/
2360
	public function display_admin_page_with_sidebar() {
2361
2362
		$this->_display_admin_page(TRUE);
2363
	}
2364
2365
2366
2367
2368
	/**
2369
	*		generates  HTML wrapper for an admin details page (except no sidebar)
2370
	*		@access public
2371
	*		@return void
2372
	*/
2373
	public function display_admin_page_with_no_sidebar() {
2374
		$this->_display_admin_page();
2375
	}
2376
2377
2378
2379
2380
	/**
2381
	 * generates HTML wrapper for an EE about admin page (no sidebar)
2382
	 * @access public
2383
	 * @return void
2384
	 */
2385
	public function display_about_admin_page() {
2386
		$this->_display_admin_page( FALSE, TRUE );
2387
	}
2388
2389
2390
2391
2392
	/**
2393
	 * display_admin_page
2394
	 * contains the code for actually displaying an admin page
2395
	 *
2396
	 * @access private
2397
	 * @param  boolean $sidebar true with sidebar, false without
2398
	 * @param  boolean $about   use the about admin wrapper instead of the default.
2399
	 * @return html           admin_page
2400
	 */
2401
	private function _display_admin_page($sidebar = false, $about = FALSE) {
2402
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
2403
2404
		//custom remove metaboxes hook to add or remove any metaboxes to/from Admin pages.
2405
		do_action( 'AHEE__EE_Admin_Page___display_admin_page__modify_metaboxes' );
2406
2407
		// set current wp page slug - looks like: event-espresso_page_event_categories
2408
		// keep in mind "event-espresso" COULD be something else if the top level menu label has been translated.
2409
		$this->_template_args['current_page'] = $this->_wp_page_slug;
2410
2411
		$template_path = $sidebar ?  EE_ADMIN_TEMPLATE . 'admin_details_wrapper.template.php' : EE_ADMIN_TEMPLATE . 'admin_details_wrapper_no_sidebar.template.php';
2412
2413
		if ( defined('DOING_AJAX' ) )
2414
			$template_path = EE_ADMIN_TEMPLATE . 'admin_details_wrapper_no_sidebar_ajax.template.php';
2415
2416
		$template_path = !empty($this->_column_template_path) ? $this->_column_template_path : $template_path;
2417
2418
		$this->_template_args['post_body_content'] = isset( $this->_template_args['admin_page_content'] ) ? $this->_template_args['admin_page_content'] : '';
2419
		$this->_template_args['before_admin_page_content'] = isset($this->_template_args['before_admin_page_content']) ? $this->_template_args['before_admin_page_content'] : '';
2420
		$this->_template_args['after_admin_page_content'] = isset($this->_template_args['after_admin_page_content']) ? $this->_template_args['after_admin_page_content'] : '';
2421
		$this->_template_args['admin_page_content'] = EEH_Template::display_template( $template_path, $this->_template_args, TRUE );
2422
2423
2424
		// the final template wrapper
2425
		$this->admin_page_wrapper($about);
2426
	}
2427
2428
2429
2430
2431
2432
	/**
2433
	 * This is used to display caf preview pages.
2434
	 *
2435
	 * @since 4.3.2
2436
	 *
2437
	 * @param string $utm_campaign_source what is the key used for google analytics link
2438
	 * @param bool   $display_sidebar whether to use the sidebar template or the full template for the page.  TRUE = SHOW sidebar, FALSE = no sidebar. Default no sidebar.
2439
	 * @return void
2440
	 */
2441
	public function display_admin_caf_preview_page( $utm_campaign_source = '', $display_sidebar = TRUE ) {
2442
		//let's generate a default preview action button if there isn't one already present.
2443
		$this->_labels['buttons']['buy_now'] = __('Upgrade Now', 'event_espresso');
2444
		$buy_now_url = add_query_arg(
2445
			array(
2446
				'ee_ver' => 'ee4',
2447
				'utm_source' => 'ee4_plugin_admin',
2448
				'utm_medium' => 'link',
2449
				'utm_campaign' => $utm_campaign_source,
2450
				'utm_content' => 'buy_now_button'
2451
			),
2452
		'http://eventespresso.com/pricing/'
2453
		);
2454
		$this->_template_args['preview_action_button'] = ! isset( $this->_template_args['preview_action_button'] ) ? $this->get_action_link_or_button( '', 'buy_now', array(), 'button-primary button-large', $buy_now_url, true ) : $this->_template_args['preview_action_button'];
2455
		$template_path = EE_ADMIN_TEMPLATE . 'admin_caf_full_page_preview.template.php';
2456
		$this->_template_args['admin_page_content'] = EEH_Template::display_template( $template_path, $this->_template_args, TRUE );
2457
		$this->_display_admin_page( $display_sidebar );
2458
	}
2459
2460
2461
	/**
2462
	 * display_admin_list_table_page_with_sidebar
2463
	 * generates HTML wrapper for an admin_page with list_table
2464
	 *
2465
	 * @access public
2466
	 * @return html
2467
	 */
2468
	public function display_admin_list_table_page_with_sidebar() {
2469
		$this->_display_admin_list_table_page(TRUE);
2470
	}
2471
2472
	/**
2473
	 * display_admin_list_table_page_with_no_sidebar
2474
	 * generates HTML wrapper for an admin_page with list_table (but with no sidebar)
2475
	 *
2476
	 * @access public
2477
	 * @return html
2478
	 */
2479
	public function display_admin_list_table_page_with_no_sidebar() {
2480
		$this->_display_admin_list_table_page();
2481
	}
2482
2483
2484
2485
	/**
2486
	 * generates html wrapper for an admin_list_table page
2487
	 * @access private
2488
	 * @param boolean $sidebar whether to display with sidebar or not.
2489
	 * @return html
2490
	 */
2491
	private function _display_admin_list_table_page( $sidebar = false ) {
2492
		//setup search attributes
2493
		$this->_set_search_attributes();
2494
		$this->_template_args['current_page'] = $this->_wp_page_slug;
2495
		$template_path = EE_ADMIN_TEMPLATE . 'admin_list_wrapper.template.php';
2496
2497
		$this->_template_args['table_url'] = defined( 'DOING_AJAX') ? add_query_arg( array( 'noheader' => 'true', 'route' => $this->_req_action), $this->_admin_base_url ) : add_query_arg( array( 'route' => $this->_req_action), $this->_admin_base_url);
2498
		$this->_template_args['list_table'] = $this->_list_table_object;
2499
		$this->_template_args['current_route'] = $this->_req_action;
2500
		$this->_template_args['list_table_class'] = get_class( $this->_list_table_object );
2501
2502
		$ajax_sorting_callback = $this->_list_table_object->get_ajax_sorting_callback();
2503
		if( ! empty( $ajax_sorting_callback )) {
2504
			$sortable_list_table_form_fields = wp_nonce_field( $ajax_sorting_callback . '_nonce', $ajax_sorting_callback . '_nonce', FALSE, FALSE );
2505
//			$reorder_action = 'espresso_' . $ajax_sorting_callback . '_nonce';
2506
//			$sortable_list_table_form_fields = wp_nonce_field( $reorder_action, 'ajax_table_sort_nonce', FALSE, FALSE );
2507
			$sortable_list_table_form_fields .= '<input type="hidden" id="ajax_table_sort_page" name="ajax_table_sort_page" value="' . $this->page_slug .'" />';
2508
			$sortable_list_table_form_fields .= '<input type="hidden" id="ajax_table_sort_action" name="ajax_table_sort_action" value="' . $ajax_sorting_callback . '" />';
2509
		} else {
2510
			$sortable_list_table_form_fields = '';
2511
		}
2512
2513
		$this->_template_args['sortable_list_table_form_fields'] = $sortable_list_table_form_fields;
2514
		$hidden_form_fields = isset( $this->_template_args['list_table_hidden_fields'] ) ? $this->_template_args['list_table_hidden_fields'] : '';
2515
		$nonce_ref = $this->_req_action . '_nonce';
2516
		$hidden_form_fields .= '<input type="hidden" name="' . $nonce_ref . '" value="' . wp_create_nonce( $nonce_ref ) . '">';
2517
		$this->_template_args['list_table_hidden_fields'] = $hidden_form_fields;
2518
2519
		//display message about search results?
2520
		$this->_template_args['before_list_table'] .= apply_filters( 'FHEE__EE_Admin_Page___display_admin_list_table_page__before_list_table__template_arg', !empty( $this->_req_data['s'] ) ? '<p class="ee-search-results">' . sprintf( __('Displaying search results for the search string: <strong><em>%s</em></strong>', 'event_espresso'), trim($this->_req_data['s'], '%') ) . '</p>' : '', $this->page_slug, $this->_req_data, $this->_req_action );
2521
2522
		$this->_template_args['admin_page_content'] = EEH_Template::display_template( $template_path, $this->_template_args, TRUE );
2523
2524
		// the final template wrapper
2525
		if ( $sidebar )
2526
			$this->display_admin_page_with_sidebar();
2527
		else
2528
			$this->display_admin_page_with_no_sidebar();
2529
	}
2530
2531
2532
2533
2534
2535
	/**
2536
	 * This just prepares a legend using the given items and the admin_details_legend.template.php file and returns the html string for the legend.
2537
	 *
2538
	 * $items are expected in an array in the following format:
2539
	 * $legend_items = array(
2540
	 * 		'item_id' => array(
2541
	 * 			'icon' => 'http://url_to_icon_being_described.png',
2542
	 * 			'desc' => __('localized description of item');
2543
	 * 		)
2544
	 * );
2545
	 * @param  array $items  see above for format of array
2546
	 * @return string        html string of legend
2547
	 */
2548
	protected function _display_legend( $items ) {
2549
		$template_args['items'] = apply_filters( 'FHEE__EE_Admin_Page___display_legend__items', (array) $items, $this );
0 ignored issues
show
Coding Style Comprehensibility introduced by
$template_args was never initialized. Although not strictly required by PHP, it is generally a good practice to add $template_args = 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...
2550
		$legend_template = EE_ADMIN_TEMPLATE . 'admin_details_legend.template.php';
2551
		return EEH_Template::display_template($legend_template, $template_args, TRUE);
2552
	}
2553
2554
2555
2556
2557
	/**
2558
	 * this is used whenever we're DOING_AJAX to return a formatted json array that our calling javascript can expect
2559
	 *
2560
	 * @param bool $sticky_notices Used to indicate whether you want to ensure notices are added to a transient instead of displayed.
2561
	 *
2562
	 * The returned json object is created from an array in the following format:
2563
	 * array(
2564
	 * 	'error' => FALSE, //(default FALSE), contains any errors and/or exceptions (exceptions return json early),
2565
	 * 	'success' => FALSE, //(default FALSE) - contains any special success message.
2566
	 * 	'notices' => '', // - contains any EE_Error formatted notices
2567
	 * 	'content' => 'string can be html', //this is a string of formatted content (can be html)
2568
	 * 	'data' => array() //this can be any key/value pairs that a method returns for later json parsing by the js. We're also going to include the template args with every package (so js can pick out any specific template args that might be included in here)
2569
	 * )
2570
	 *
2571
	 * The json object is populated by whatever is set in the $_template_args property.
2572
	 *
2573
	 * @return json object
2574
	 */
2575
	protected function _return_json( $sticky_notices = FALSE ) {
2576
2577
		//make sure any EE_Error notices have been handled.
2578
		$this->_process_notices( array(), true, $sticky_notices );
2579
2580
2581
		$data = isset( $this->_template_args['data'] ) ? $this->_template_args['data'] : array();
2582
		unset($this->_template_args['data']);
2583
		$json = array(
2584
			'error' => isset( $this->_template_args['error'] ) ? $this->_template_args['error'] : FALSE,
2585
			'success' => isset( $this->_template_args['success'] ) ? $this->_template_args['success'] : FALSE,
2586
			'notices' => EE_Error::get_notices(),
2587
			'content' => isset( $this->_template_args['admin_page_content'] ) ? $this->_template_args['admin_page_content'] : '',
2588
			'data' => array_merge( $data, array('template_args' => $this->_template_args ) ),
2589
			'isEEajax' => TRUE //special flag so any ajax.Success methods in js can identify this return package as a EEajax package.
2590
			);
2591
2592
2593
		// make sure there are no php errors or headers_sent.  Then we can set correct json header.
2594
		if ( NULL === error_get_last() || ! headers_sent() )
2595
			header('Content-Type: application/json; charset=UTF-8');
2596
                if( function_exists( 'wp_json_encode' ) ) {
2597
                    echo wp_json_encode( $json );
2598
                } else {
2599
                    echo json_encode( $json );
2600
                }
2601
		exit();
2602
	}
2603
2604
2605
2606
	/**
2607
	 * Simply a wrapper for the protected method so we can call this outside the class (ONLY when doing ajax)
2608
	 * @return json_obj|EE_Error
2609
	 */
2610
	public function return_json() {
2611
		if ( defined('DOING_AJAX') && DOING_AJAX )
2612
			$this->_return_json();
2613
2614
		else {
2615
			throw new EE_Error( sprintf( __('The public %s method can only be called when DOING_AJAX = TRUE', 'event_espresso'), __FUNCTION__ ) );
2616
		}
2617
	}
2618
2619
2620
2621
2622
	/**
2623
	 * This provides a way for child hook classes to send along themselves by reference so methods/properties within them can be accessed by EE_Admin_child pages. This is assigned to the $_hook_obj property.
2624
	 *
2625
	 * @param EE_Admin_Hooks object $hook_obj This will be the object for the EE_Admin_Hooks child
2626
	 *
2627
	 * @access public
2628
	 * @return void
2629
	 */
2630
	public function set_hook_object( EE_Admin_Hooks $hook_obj ) {
2631
		$this->_hook_obj = $hook_obj;
2632
	}
2633
2634
2635
2636
2637
2638
	/**
2639
	*		generates  HTML wrapper with Tabbed nav for an admin page
2640
	*		@access public
2641
	*		@param  boolean $about whether to use the special about page wrapper or default.
2642
	*		@return void
2643
	*/
2644
	public function admin_page_wrapper($about = FALSE) {
2645
2646
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
2647
2648
		$this->_nav_tabs = $this->_get_main_nav_tabs();
2649
2650
		$this->_template_args['nav_tabs'] = $this->_nav_tabs;
2651
		$this->_template_args['admin_page_title'] = $this->_admin_page_title;
2652
2653
		$this->_template_args['before_admin_page_content'] = apply_filters( 'FHEE_before_admin_page_content' . $this->_current_page . $this->_current_view, isset( $this->_template_args['before_admin_page_content'] ) ? $this->_template_args['before_admin_page_content'] : '');
2654
		$this->_template_args['after_admin_page_content'] = apply_filters( 'FHEE_after_admin_page_content' . $this->_current_page . $this->_current_view, isset( $this->_template_args['after_admin_page_content'] ) ? $this->_template_args['after_admin_page_content'] : '');
2655
2656
		$this->_template_args['after_admin_page_content'] .= $this->_set_help_popup_content();
2657
2658
2659
2660
		// load settings page wrapper template
2661
		$template_path = !defined( 'DOING_AJAX' ) ? EE_ADMIN_TEMPLATE . 'admin_wrapper.template.php' : EE_ADMIN_TEMPLATE . 'admin_wrapper_ajax.template.php';
2662
2663
		//about page?
2664
		$template_path = $about ? EE_ADMIN_TEMPLATE . 'about_admin_wrapper.template.php' : $template_path;
2665
2666
2667
		if ( defined( 'DOING_AJAX' ) ) {
2668
			$this->_template_args['admin_page_content'] = EEH_Template::display_template( $template_path, $this->_template_args, TRUE );
2669
2670
			$this->_return_json();
2671
		} else {
2672
			EEH_Template::display_template( $template_path, $this->_template_args );
2673
		}
2674
2675
	}
2676
2677
2678
2679
	/**
2680
	 * This returns the admin_nav tabs html using the configuration in the _nav_tabs property
2681
	 * @return string html
2682
	 */
2683
	protected function _get_main_nav_tabs() {
2684
		//let's generate the html using the EEH_Tabbed_Content helper.  We do this here so that it's possible for child classes to add in nav tabs dynamically at the last minute (rather than setting in the page_routes array)
2685
		EE_Registry::instance()->load_helper( 'Tabbed_Content' );
2686
		return EEH_Tabbed_Content::display_admin_nav_tabs($this->_nav_tabs);
0 ignored issues
show
Bug introduced by
It seems like $this->_nav_tabs can also be of type string; however, EEH_Tabbed_Content::display_admin_nav_tabs() does only seem to accept array, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
2687
	}
2688
2689
2690
2691
2692
2693
2694
2695
2696
	/**
2697
	*		sort nav tabs
2698
	*		@access public
2699
	*		@return void
2700
	*/
2701
	private function _sort_nav_tabs( $a, $b ) {
2702
		if ($a['order'] == $b['order']) {
2703
	        return 0;
2704
	    }
2705
	    return ($a['order'] < $b['order']) ? -1 : 1;
2706
	}
2707
2708
2709
2710
2711
2712
	/**
2713
	 * 	generates HTML for the forms used on admin pages
2714
	 * 	@access protected
2715
	 * 	@param	array $input_vars - array of input field details
2716
	 * 	@param	array $id - used for defining unique identifiers for the form.
2717
	 * 	@param string $generator (options are 'string' or 'array', basically use this to indicate which generator to use)
2718
	 * 	@return string
2719
	 * 	@uses EEH_Form_Fields::get_form_fields (/helper/EEH_Form_Fields.helper.php)
2720
	 * 	@uses EEH_Form_Fields::get_form_fields_array (/helper/EEH_Form_Fields.helper.php)
2721
	 */
2722
	protected function _generate_admin_form_fields( $input_vars = array(), $generator = 'string', $id = FALSE ) {
2723
		EE_Registry::instance()->load_helper( 'Form_Fields' );
2724
		$content = $generator == 'string' ? EEH_Form_Fields::get_form_fields($input_vars, $id) : EEH_Form_Fields::get_form_fields_array($input_vars);
2725
		return $content;
2726
	}
2727
2728
2729
2730
2731
2732
2733
2734
2735
2736
	/**
2737
	 * generates the "Save" and "Save & Close" buttons for edit forms
2738
	 *
2739
	 * @access protected
2740
	 * @param bool $both if true then both buttons will be generated.  If false then just the "Save & Close" button.
2741
	 * @param array $text if included, generator will use the given text for the buttons ( array([0] => 'Save', [1] => 'save & close')
2742
	 * @param array $actions if included allows us to set the actions that each button will carry out (i.e. via the "name" value in the button).  We can also use this to just dump default actions by submitting some other value.
2743
	 * @param bool|string|null $referrer if false then we just do the default action on save and close.  Other wise it will use the $referrer string. IF null, then we don't do ANYTHING on save and close (normal form handling).
2744
	 */
2745
	protected function _set_save_buttons($both = TRUE, $text = array(), $actions = array(), $referrer = NULL ) {
2746
		//make sure $text and $actions are in an array
2747
		$text = (array) $text;
2748
		$actions = (array) $actions;
2749
		$referrer_url = empty($referrer) ? '' : $referrer;
0 ignored issues
show
Unused Code introduced by
$referrer_url 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...
2750
		$referrer_url = !$referrer ? '<input type="hidden" id="save_and_close_referrer" name="save_and_close_referrer" value="' . $_SERVER['REQUEST_URI'] .'" />' : '<input type="hidden" id="save_and_close_referrer" name="save_and_close_referrer" value="' . $referrer .'" />';
2751
2752
		$button_text = !empty($text) ? $text : array( __('Save', 'event_espresso'), __('Save and Close', 'event_espresso') );
2753
		$default_names = array( 'save', 'save_and_close' );
2754
2755
		//add in a hidden index for the current page (so save and close redirects properly)
2756
		$this->_template_args['save_buttons'] = $referrer_url;
2757
2758
		foreach ( $button_text as $key => $button ) {
2759
			$ref = $default_names[$key];
2760
			$id = $this->_current_view . '_' . $ref;
2761
			$name = !empty($actions) ? $actions[$key] : $ref;
2762
			$this->_template_args['save_buttons'] .= '<input type="submit" class="button-primary ' . $ref . '" value="' . $button . '" name="' . $name . '" id="' . $id . '" />';
2763
			if ( !$both ) break;
2764
		}
2765
2766
	}
2767
2768
2769
	/**
2770
	 * Wrapper for the protected function.  Allows plugins/addons to call this to set the form tags.
2771
	 *
2772
	 * @see $this->_set_add_edit_form_tags() for details on params
2773
	 * @since 4.6.0
2774
	 *
2775
	 */
2776
	public function set_add_edit_form_tags( $route = '', $additional_hidden_fields = array() ) {
2777
		$this->_set_add_edit_form_tags( $route, $additional_hidden_fields );
2778
	}
2779
2780
2781
2782
	/**
2783
	 * set form open and close tags on add/edit pages.
2784
	 *
2785
	 * @access protected
2786
	 * @param string $route the route you want the form to direct to
2787
	 * @param array $additional_hidden_fields any additional hidden fields required in the form header
2788
	 * @return void
2789
	 */
2790
	protected function _set_add_edit_form_tags( $route = '', $additional_hidden_fields = array() ) {
2791
2792
		if ( empty( $route )) {
2793
			$user_msg = __('An error occurred. No action was set for this page\'s form.', 'event_espresso');
2794
			$dev_msg = $user_msg . "\n" . sprintf( __('The $route argument is required for the %s->%s method.', 'event_espresso'), __FUNCTION__, __CLASS__ );
2795
			EE_Error::add_error( $user_msg . '||' . $dev_msg, __FILE__, __FUNCTION__, __LINE__ );
2796
		}
2797
		// open form
2798
		$this->_template_args['before_admin_page_content'] = '<form name="form" method="post" action="' . $this->_admin_base_url . '" id="' . $route . '_event_form" >';
2799
		// add nonce
2800
		$nonce = wp_nonce_field( $route . '_nonce', $route . '_nonce', FALSE, FALSE );
2801
//		$nonce = wp_nonce_field( $route . '_nonce', '_wpnonce', FALSE, FALSE );
2802
		$this->_template_args['before_admin_page_content'] .= "\n\t" . $nonce;
2803
		// add REQUIRED form action
2804
		$hidden_fields = array(
2805
				'action' => array( 'type' => 'hidden', 'value' => $route ),
2806
			);
2807
		// merge arrays
2808
		$hidden_fields = is_array( $additional_hidden_fields) ? array_merge( $hidden_fields, $additional_hidden_fields ) : $hidden_fields;
2809
		// generate form fields
2810
		$form_fields = $this->_generate_admin_form_fields( $hidden_fields, 'array' );
2811
		// add fields to form
2812
		foreach ( $form_fields as $field_name => $form_field ) {
0 ignored issues
show
Bug introduced by
The expression $form_fields of type string|array is not guaranteed to be traversable. How about adding an additional type check?

There are different options of fixing this problem.

  1. If you want to be on the safe side, you can add an additional type-check:

    $collection = json_decode($data, true);
    if ( ! is_array($collection)) {
        throw new \RuntimeException('$collection must be an array.');
    }
    
    foreach ($collection as $item) { /** ... */ }
    
  2. If you are sure that the expression is traversable, you might want to add a doc comment cast to improve IDE auto-completion and static analysis:

    /** @var array $collection */
    $collection = json_decode($data, true);
    
    foreach ($collection as $item) { /** .. */ }
    
  3. Mark the issue as a false-positive: Just hover the remove button, in the top-right corner of this issue for more options.

Loading history...
2813
			$this->_template_args['before_admin_page_content'] .= "\n\t" . $form_field['field'];
2814
		}
2815
2816
		// close form
2817
		$this->_template_args['after_admin_page_content'] = '</form>';
2818
2819
	}
2820
2821
2822
2823
	/**
2824
	 * Public Wrapper for _redirect_after_action() method since its
2825
	 * discovered it would be useful for external code to have access.
2826
	 *
2827
	 * @see EE_Admin_Page::_redirect_after_action() for params.
2828
	 * @since 4.5.0
2829
	 */
2830
	public function redirect_after_action( $success = FALSE, $what = 'item', $action_desc = 'processed', $query_args = array(), $override_overwrite = FALSE ) {
2831
		$this->_redirect_after_action( $success, $what, $action_desc, $query_args, $override_overwrite );
0 ignored issues
show
Documentation introduced by
$success is of type boolean, but the function expects a integer.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
2832
	}
2833
2834
2835
2836
	/**
2837
	 *    _redirect_after_action
2838
	 *
2839
	 * @param int $success - whether success was for two or more records, or just one, or none
2840
	 *	@param string 	$what 		- what the action was performed on
2841
	 *	@param string 	$action_desc 	- what was done ie: updated, deleted, etc
2842
	 *	@param array 	$query_args		- an array of query_args to be added to the URL to redirect to after the admin action is completed
2843
	 *	@param BOOL     $override_overwrite by default all EE_Error::success messages are overwritten, this allows you to override this so that they show.
2844
	 *	@access protected
2845
	 *	@return void
2846
	 */
2847
	protected function _redirect_after_action( $success = 0, $what = 'item', $action_desc = 'processed', $query_args = array(), $override_overwrite = FALSE ) {
2848
2849
		do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' );
2850
2851
		//class name for actions/filters.
2852
		$classname = get_class($this);
2853
2854
		//set redirect url. Note if there is a "page" index in the $query_args then we go with vanilla admin.php route, otherwise we go with whatever is set as the _admin_base_url
2855
		$redirect_url = isset( $query_args['page'] ) ? admin_url('admin.php') : $this->_admin_base_url;
2856
		$notices = EE_Error::get_notices( FALSE );
2857
2858
		// overwrite default success messages //BUT ONLY if overwrite not overridden
2859
		if ( ! $override_overwrite || ! empty( $notices['errors'] )) {
2860
			EE_Error::overwrite_success();
2861
		}
2862
		// how many records affected ? more than one record ? or just one ?
2863
		if ( $success > 1 && empty( $notices['errors'] )) {
2864
			// set plural msg
2865
			EE_Error::add_success( sprintf( __('The "%s" have been successfully %s.', 'event_espresso'), $what, $action_desc ), __FILE__, __FUNCTION__, __LINE__);
2866
		} else if ( $success == 1 && empty( $notices['errors'] )) {
2867
			// set singular msg
2868
			EE_Error::add_success( sprintf( __('The "%s" has been successfully %s.', 'event_espresso'), $what, $action_desc), __FILE__, __FUNCTION__, __LINE__ );
2869
		}
2870
2871
		// check that $query_args isn't something crazy
2872
		if ( ! is_array( $query_args )) {
2873
			$query_args = array();
2874
		}
2875
2876
		/**
2877
		 * Allow injecting actions before the query_args are modified for possible different
2878
		 * redirections on save and close actions
2879
		 *
2880
		 * @since 4.2.0
2881
		 *
2882
		 * @param array $query_args   The original query_args array coming into the
2883
		 *                          		method.
2884
		 */
2885
		do_action( 'AHEE__' . $classname . '___redirect_after_action__before_redirect_modification_' . $this->_req_action, $query_args );
2886
2887
		//calculate where we're going (if we have a "save and close" button pushed)
2888
		if ( isset($this->_req_data['save_and_close'] ) && isset($this->_req_data['save_and_close_referrer'] ) ) {
2889
			// even though we have the save_and_close referrer, we need to parse the url for the action in order to generate a nonce
2890
			$parsed_url = parse_url( $this->_req_data['save_and_close_referrer'] );
2891
			// regenerate query args array from refferer URL
2892
			parse_str( $parsed_url['query'], $query_args );
2893
			// correct page and action will be in the query args now
2894
			$redirect_url = admin_url( 'admin.php' );
2895
		}
2896
2897
		//merge any default query_args set in _default_route_query_args property
2898
		if ( ! empty( $this->_default_route_query_args ) && ! $this->_is_UI_request ) {
2899
			$args_to_merge = array();
2900
			foreach ( $this->_default_route_query_args as $query_param => $query_value ) {
2901
				//is there a wp_referer array in our _default_route_query_args property?
2902
				if ( $query_param == 'wp_referer'  ) {
2903
					$query_value = (array) $query_value;
2904
					foreach ( $query_value as $reference => $value ) {
2905
						if ( strpos( $reference, 'nonce' ) !== false ) {
2906
							continue;
2907
						}
2908
2909
						//finally we will override any arguments in the referer with
2910
						//what might be set on the _default_route_query_args array.
2911
						if ( isset( $this->_default_route_query_args[$reference] ) ) {
2912
							$args_to_merge[$reference] = urlencode( $this->_default_route_query_args[$reference] );
2913
						} else {
2914
							$args_to_merge[$reference] = urlencode( $value );
2915
						}
2916
					}
2917
					continue;
2918
				}
2919
2920
				$args_to_merge[$query_param] = $query_value;
2921
			}
2922
2923
			//now let's merge these arguments but override with what was specifically sent in to the
2924
			//redirect.
2925
			$query_args = array_merge( $args_to_merge, $query_args );
2926
		}
2927
2928
		$this->_process_notices($query_args);
0 ignored issues
show
Bug introduced by
It seems like $query_args can also be of type null; however, EE_Admin_Page::_process_notices() does only seem to accept array, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
2929
2930
2931
		// generate redirect url
2932
2933
		// if redirecting to anything other than the main page, add a nonce
2934
		if ( isset( $query_args['action'] )) {
2935
			// manually generate wp_nonce and merge that with the query vars becuz the wp_nonce_url function wrecks havoc on some vars
2936
			$query_args['_wpnonce'] = wp_create_nonce( $query_args['action'] . '_nonce' );
2937
		}
2938
2939
		//we're adding some hooks and filters in here for processing any things just before redirects (example: an admin page has done an insert or update and we want to run something after that).
2940
		do_action( 'AHEE_redirect_' . $classname . $this->_req_action, $query_args );
2941
2942
		$redirect_url = apply_filters( 'FHEE_redirect_' . $classname . $this->_req_action, self::add_query_args_and_nonce( $query_args, $redirect_url ), $query_args );
2943
2944
2945
		// check if we're doing ajax.  If we are then lets just return the results and js can handle how it wants.
2946
		if ( defined('DOING_AJAX' ) ) {
2947
			$default_data = array(
2948
				'close' => TRUE,
2949
				'redirect_url' => $redirect_url,
2950
				'where' => 'main',
2951
				'what' => 'append',
2952
				);
2953
2954
			$this->_template_args['success'] = $success;
2955
			$this->_template_args['data'] = !empty($this->_template_args['data']) ? array_merge($default_data, $this->_template_args['data'] ): $default_data;
2956
			$this->_return_json();
2957
		}
2958
2959
		wp_safe_redirect( $redirect_url );
2960
		exit();
2961
	}
2962
2963
2964
2965
2966
	/**
2967
	 * process any notices before redirecting (or returning ajax request)
2968
	 * This method sets the $this->_template_args['notices'] attribute;
2969
	 *
2970
	 * @param  array  $query_args any query args that need to be used for notice transient ('action')
2971
	 * @param bool    $skip_route_verify This is typically used when we are processing notices REALLY early and page_routes haven't been defined yet.
2972
	 * @param bool    $sticky_notices      This is used to flag that regardless of whether this is doing_ajax or not, we still save a transient for the notice.
2973
	 * @return void
2974
	 */
2975
	protected function _process_notices( $query_args = array(), $skip_route_verify = FALSE , $sticky_notices = TRUE ) {
2976
2977
		$this->_template_args['notices'] = EE_Error::get_notices();
2978
2979
		//IF this isn't ajax we need to create a transient for the notices using the route (however, overridden if $sticky_notices == true)
2980
		if ( ! defined( 'DOING_AJAX' ) || $sticky_notices ) {
2981
			$route = isset( $query_args['action'] ) ? $query_args['action'] : 'default';
2982
			$this->_add_transient( $route, $this->_template_args['notices'], TRUE, $skip_route_verify );
0 ignored issues
show
Documentation introduced by
$this->_template_args['notices'] is of type null|string|array<string...ng","errors":"string"}>, but the function expects a object<data>.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
2983
		}
2984
	}
2985
2986
2987
2988
2989
	/**
2990
	 * get_action_link_or_button
2991
	 * returns the button html for adding, editing, or deleting an item (depending on given type)
2992
	 *
2993
	 * @param string $action use this to indicate which action the url is generated with.
2994
	 * @param string $type accepted strings must be defined in the $_labels['button'] array(as the key) property.
2995
	 * @param array $extra_request if the button requires extra params you can include them in $key=>$value pairs.
2996
	 * @param string $class Use this to give the class for the button. Defaults to 'button-primary'
2997
	 * @param string $base_url If this is not provided the _admin_base_url will be used as the default for the button base_url.  Otherwise this value will be used.
2998
	 * @param bool   $exclude_nonce If true then no nonce will be in the generated button link.
2999
	 *
3000
	 * @throws EE_Error
3001
	 *
3002
	 * @return string html for button
3003
	 */
3004
	public function get_action_link_or_button($action, $type = 'add', $extra_request = array(), $class = 'button-primary', $base_url = FALSE, $exclude_nonce = false ) {
3005
		//first let's validate the action (if $base_url is FALSE otherwise validation will happen further along)
3006
		if ( !isset($this->_page_routes[$action]) && !$base_url )
0 ignored issues
show
Bug Best Practice introduced by
The expression $base_url of type false|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === false instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For string values, the empty string '' is a special case, in particular the following results might be unexpected:

''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
Loading history...
3007
			throw new EE_Error( sprintf( __('There is no page route for given action for the button.  This action was given: %s', 'event_espresso'), $action) );
3008
3009
		if ( !isset( $this->_labels['buttons'][$type] ) )
3010
			throw new EE_Error( sprintf( __('There is no label for the given button type (%s). Labels are set in the <code>_page_config</code> property.', 'event_espresso'), $type) );
3011
3012
		//finally check user access for this button.
3013
		$has_access = $this->check_user_access( $action, TRUE );
3014
		if ( ! $has_access ) {
3015
			return '';
3016
		}
3017
3018
		$_base_url = !$base_url ? $this->_admin_base_url : $base_url;
3019
3020
		$query_args = array(
3021
			'action' => $action  );
3022
3023
		//merge extra_request args but make sure our original action takes precedence and doesn't get overwritten.
3024
		if ( !empty($extra_request) )
3025
			$query_args = array_merge( $extra_request, $query_args );
3026
3027
		$url = self::add_query_args_and_nonce( $query_args, $_base_url, false, $exclude_nonce );
3028
3029
		$button = EEH_Template::get_button_or_link( $url, $this->_labels['buttons'][$type], $class );
3030
3031
		return $button;
3032
	}
3033
3034
3035
3036
3037
3038
3039
	/**
3040
	 * _per_page_screen_option
3041
	 * Utility function for adding in a per_page_option in the screen_options_dropdown.
3042
	 * @return void
3043
	 */
3044
	protected function _per_page_screen_option() {
3045
		$option = 'per_page';
3046
		$args = array(
3047
			'label' => $this->_admin_page_title,
3048
			'default' => 10,
3049
			'option' => $this->_current_page . '_' . $this->_current_view . '_per_page'
3050
			);
3051
		//ONLY add the screen option if the user has access to it.
3052
		if ( $this->check_user_access( $this->_current_view, true ) ) {
3053
			add_screen_option( $option, $args );
3054
		}
3055
	}
3056
3057
3058
3059
3060
	/**
3061
	 * set_per_page_screen_option
3062
	 * All this does is make sure that WordPress saves any per_page screen options (if set) for the current page.
3063
	 * we have to do this rather than running inside the 'set-screen-options' hook because it runs earlier than admin_menu.
3064
	 *
3065
	 * @access private
3066
	 * @return void
3067
	 */
3068
	private function _set_per_page_screen_options() {
3069
		if ( isset($_POST['wp_screen_options']) && is_array($_POST['wp_screen_options']) ) {
3070
			check_admin_referer( 'screen-options-nonce', 'screenoptionnonce' );
3071
3072
			if ( !$user = wp_get_current_user() )
3073
			return;
3074
			$option = $_POST['wp_screen_options']['option'];
3075
			$value = $_POST['wp_screen_options']['value'];
3076
3077
			if ( $option != sanitize_key( $option ) )
3078
				return;
3079
3080
			$map_option = $option;
3081
3082
			$option = str_replace('-', '_', $option);
3083
3084
			switch ( $map_option ) {
3085
				case $this->_current_page . '_' .  $this->_current_view . '_per_page':
3086
					$value = (int) $value;
3087
					if ( $value < 1 || $value > 999 )
3088
						return;
3089
					break;
3090
				default:
3091
					$value = apply_filters( 'FHEE__EE_Admin_Page___set_per_page_screen_options__value', false, $option, $value );
3092
					if ( false === $value )
3093
						return;
3094
					break;
3095
			}
3096
3097
			update_user_meta($user->ID, $option, $value);
3098
			wp_safe_redirect( remove_query_arg( array('pagenum', 'apage', 'paged'), wp_get_referer() ) );
3099
			exit;
3100
		}
3101
	}
3102
3103
3104
3105
	/**
3106
	 * This just allows for setting the $_template_args property if it needs to be set outside the object
3107
	 * @param array $data array that will be assigned to template args.
3108
	 */
3109
	public function set_template_args( $data ) {
3110
		$this->_template_args = array_merge( $this->_template_args, (array) $data );
3111
	}
3112
3113
3114
3115
3116
	/**
3117
	 * This makes available the WP transient system for temporarily moving data between routes
3118
	 *
3119
	 * @access protected
3120
	 * @param route $route the route that should receive the transient
3121
	 * @param data $data  the data that gets sent
3122
	 * @param bool $notices If this is for notices then we use this to indicate so, otherwise its just a normal route transient.
3123
	 * @param bool $skip_route_verify Used to indicate we want to skip route verification.  This is usually ONLY used when we are adding a transient before page_routes have been defined.
3124
	 * @return void
3125
	 */
3126
	protected function _add_transient( $route, $data, $notices = FALSE, $skip_route_verify = FALSE ) {
3127
		$user_id = get_current_user_id();
3128
3129
		if ( !$skip_route_verify )
3130
			$this->_verify_route($route);
3131
3132
3133
		//now let's set the string for what kind of transient we're setting
3134
		$transient = $notices ? 'ee_rte_n_tx_' . $route . '_' . $user_id : 'rte_tx_' . $route . '_' . $user_id;
3135
		$data = $notices ? array( 'notices' => $data ) : $data;
3136
		//is there already a transient for this route?  If there is then let's ADD to that transient
3137
		$existing = is_multisite() && is_network_admin() ? get_site_transient( $transient ) : get_transient( $transient );
3138
		if ( $existing ) {
3139
			$data = array_merge( (array) $data, (array) $existing );
3140
		}
3141
3142
		if ( is_multisite() && is_network_admin() ) {
3143
			set_site_transient( $transient, $data, 8 );
3144
		} else {
3145
			set_transient( $transient, $data, 8 );
3146
		}
3147
	}
3148
3149
3150
3151
3152
	/**
3153
	 * this retrieves the temporary transient that has been set for moving data between routes.
3154
	 * @param bool $notices true we get notices transient. False we just return normal route transient
3155
	 * @return mixed data
3156
	 */
3157
	protected function _get_transient( $notices = FALSE, $route = FALSE ) {
3158
		$user_id = get_current_user_id();
3159
		$route = !$route ? $this->_req_action : $route;
3160
		$transient = $notices ? 'ee_rte_n_tx_' . $route . '_' . $user_id : 'rte_tx_' . $route . '_' . $user_id;
3161
		$data = is_multisite() && is_network_admin() ? get_site_transient( $transient ) : get_transient( $transient );
3162
		//delete transient after retrieval (just in case it hasn't expired);
3163
		if ( is_multisite() && is_network_admin() ) {
3164
			delete_site_transient( $transient );
3165
		} else {
3166
			delete_transient( $transient );
3167
		}
3168
		return $notices && isset( $data['notices'] ) ? $data['notices'] : $data;
3169
	}
3170
3171
3172
3173
3174
	/**
3175
	 * The purpose of this method is just to run garbage collection on any EE transients that might have expired but would not be called later.
3176
	 *
3177
	 * This will be assigned to run on a specific EE Admin page. (place the method in the default route callback on the EE_Admin page you want it run.)
3178
	 * @return void
3179
	 */
3180
	protected function _transient_garbage_collection() {
3181
		global $wpdb;
3182
3183
		//retrieve all existing transients
3184
		$query = "SELECT option_name FROM $wpdb->options WHERE option_name LIKE '%rte_tx_%' OR option_name LIKE '%rte_n_tx_%'";
3185
		if ( $results = $wpdb->get_results( $query ) ) {
3186
			foreach ( $results as $result ) {
3187
				$transient = str_replace( '_transient_', '', $result->option_name );
3188
				get_transient( $transient );
3189
				if ( is_multisite() && is_network_admin() ) {
3190
					get_site_transient( $transient );
3191
				}
3192
			}
3193
		}
3194
	}
3195
3196
3197
3198
3199
3200
	/**
3201
	 * get_view
3202
	 *
3203
	 * @access public
3204
	 * @return string content of _view property
3205
	 */
3206
	public function get_view() {
3207
		return $this->_view;
3208
	}
3209
3210
3211
3212
3213
	/**
3214
	 * getter for the protected $_views property
3215
	 *
3216
	 * @return array
3217
	 */
3218
	public function get_views() {
3219
		return $this->_views;
3220
	}
3221
3222
3223
3224
3225
	/**
3226
	 * get_current_page
3227
	 *
3228
	 * @access public
3229
	 * @return string _current_page property value
3230
	 */
3231
	public function get_current_page() {
3232
		return $this->_current_page;
3233
	}
3234
3235
3236
3237
3238
	/**
3239
	 * get_current_view
3240
	 *
3241
	 * @access public
3242
	 * @return string _current_view property value
3243
	 */
3244
	public function get_current_view() {
3245
		return $this->_current_view;
3246
	}
3247
3248
3249
3250
	/**
3251
	 * get_current_screen
3252
	 *
3253
	 * @access public
3254
	 * @return object The current WP_Screen object
3255
	 */
3256
	public function get_current_screen() {
3257
		return $this->_current_screen;
3258
	}
3259
3260
3261
	/**
3262
	 * get_current_page_view_url
3263
	 *
3264
	 * @access public
3265
	 * @return string This returns the url for the current_page_view.
3266
	 */
3267
	public function get_current_page_view_url() {
3268
		return $this->_current_page_view_url;
3269
	}
3270
3271
3272
3273
3274
	/**
3275
	 * just returns the _req_data property
3276
	 * @return array
3277
	 */
3278
	public function get_request_data() {
3279
		return $this->_req_data;
3280
	}
3281
3282
3283
3284
	/**
3285
	 * returns the _req_data protected property
3286
	 * @return string
3287
	 */
3288
	public function get_req_action() {
3289
		return $this->_req_action;
3290
	}
3291
3292
3293
	/**
3294
	 * @return bool  value of $_is_caf property
3295
	 */
3296
	public function is_caf() {
3297
		return $this->_is_caf;
3298
	}
3299
3300
3301
3302
3303
	/**
3304
	 * updates  espresso configuration settings
3305
	 *
3306
	 * @access 	protected
3307
	 * @param string $tab
3308
	 * @param array $data
0 ignored issues
show
Bug introduced by
There is no parameter named $data. Was it maybe removed?

This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.

Consider the following example. The parameter $italy is not defined by the method finale(...).

/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}

The most likely cause is that the parameter was removed, but the annotation was not.

Loading history...
3309
	 * @param string $file	file where error occurred
3310
	 * @param string $func function  where error occurred
3311
	 * @param string $line	line no where error occurred
3312
	 * @return boolean
3313
	 */
3314
	protected function _update_espresso_configuration( $tab, $config, $file = '', $func = '', $line = '' ) {
3315
3316
		//remove any options that are NOT going to be saved with the config settings.
3317
		if ( isset( $config->core->ee_ueip_optin ) ) {
3318
			$config->core->ee_ueip_has_notified = TRUE;
3319
			// TODO: remove the following two lines and make sure values are migrated from 3.1
3320
			update_option( 'ee_ueip_optin', $config->core->ee_ueip_optin);
3321
			update_option( 'ee_ueip_has_notified', TRUE );
3322
		}
3323
		// and save it (note we're also doing the network save here)
3324
		$net_saved = is_main_site() ? EE_Network_Config::instance()->update_config( FALSE, FALSE ) : TRUE;
3325
		$config_saved = EE_Config::instance()->update_espresso_config( FALSE, FALSE );
3326
		if ( $config_saved && $net_saved ) {
3327
			EE_Error::add_success( sprintf( __('"%s" have been successfully updated.', 'event_espresso'), $tab ));
3328
			return TRUE;
3329
		} else {
3330
			EE_Error::add_error( sprintf( __('The "%s" were not updated.', 'event_espresso'), $tab ), $file, $func, $line  );
3331
			return FALSE;
3332
		}
3333
	}
3334
3335
3336
3337
3338
3339
	/**
3340
	 * Returns an array to be used for EE_FOrm_Fields.helper.php's select_input as the $values argument.
3341
	 * @return array
3342
	 */
3343
	public function get_yes_no_values(){
3344
		return $this->_yes_no_values;
3345
	}
3346
3347
3348
3349
	protected function _get_dir() {
3350
		$reflector = new ReflectionClass(get_class($this));
3351
		return dirname($reflector->getFileName());
3352
	}
3353
3354
3355
3356
3357
	/**
3358
	 * A helper for getting a "next link".
3359
	 *
3360
	 * @param string $url   The url to link to
3361
	 * @param string $class The class to use.
3362
	 *
3363
	 * @return string
3364
	 */
3365
	protected function _next_link( $url, $class = 'dashicons dashicons-arrow-right' ) {
3366
		return '<a class="' . $class . '" href="' . $url . '"></a>';
3367
	}
3368
3369
3370
3371
3372
	/**
3373
	 * A helper for getting a "previous link".
3374
	 *
3375
	 * @param string $url   The url to link to
3376
	 * @param string $class The class to use.
3377
	 *
3378
	 * @return string
3379
	 */
3380
	protected function _previous_link( $url, $class = 'dashicons dashicons-arrow-left' ) {
3381
		return '<a class="' . $class . '" href="' . $url . '"></a>';
3382
	}
3383
3384
3385
3386
3387
3388
3389
3390
	//below are some messages related methods that should be available across the EE_Admin system.  Note, these methods are NOT page specific
3391
3392
3393
3394
3395
	/**
3396
	 * This processes an request to resend a registration and assumes we have a _REG_ID for doing so. So if the caller knows that the _REG_ID isn't in the req_data array but CAN obtain it, the caller should ADD the _REG_ID to the _req_data array.
3397
	 * @return bool success/fail
3398
	 */
3399
	protected function _process_resend_registration() {
3400
		$this->_template_args['success'] = EED_Messages::process_resend( $this->_req_data );
3401
		do_action( 'AHEE__EE_Admin_Page___process_resend_registration', $this->_template_args['success'], $this->_req_data );
3402
		return $this->_template_args['success'];
3403
	}
3404
3405
3406
3407
	/**
3408
	 * This automatically processes any payment message notifications when manual payment has been applied.
3409
	 *
3410
	 * @access protected
3411
	 * @param \EE_Payment $payment
3412
	 * @return bool success/fail
3413
	 */
3414
	protected function _process_payment_notification( EE_Payment $payment ) {
3415
		add_filter( 'FHEE__EE_Payment_Processor__process_registration_payments__display_notifications', '__return_true' );
3416
		$success = apply_filters( 'FHEE__EE_Admin_Page___process_admin_payment_notification__success', FALSE, $payment );
3417
		$this->_template_args['success'] = $success;
3418
		return $success;
3419
	}
3420
3421
3422
}
3423
// end of file:  includes/core/admin/EE_Admin_Page.core.php
3424