eventespresso / event-espresso-core

caffeinated/admin/extend/registration_form/Extend_Registration_Form_Questions_Admin_List_Table.class.php

Indentation +97 added lines, -97 removed lines

@@ -16,113 +16,113 @@
 class Extend_Registration_Form_Questions_Admin_List_Table extends Registration_Form_Questions_Admin_List_Table
 {
 
-    public function __construct($admin_page)
-    {
-        parent::__construct($admin_page);
-    }
+	public function __construct($admin_page)
+	{
+		parent::__construct($admin_page);
+	}
 
-    public function column_display_text(EE_Question $item)
-    {
-        $system_question = $item->is_system_question();
-        $actions = array();
+	public function column_display_text(EE_Question $item)
+	{
+		$system_question = $item->is_system_question();
+		$actions = array();
 
-        if (! defined('REG_ADMIN_URL')) {
-            define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
-        }
+		if (! defined('REG_ADMIN_URL')) {
+			define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
+		}
 
-        $edit_query_args = array(
-            'action' => 'edit_question',
-            'QST_ID' => $item->ID(),
-        );
+		$edit_query_args = array(
+			'action' => 'edit_question',
+			'QST_ID' => $item->ID(),
+		);
 
-        $trash_query_args = array(
-            'action' => 'trash_question',
-            'QST_ID' => $item->ID(),
-        );
+		$trash_query_args = array(
+			'action' => 'trash_question',
+			'QST_ID' => $item->ID(),
+		);
 
-        $restore_query_args = array(
-            'action' => 'restore_question',
-            'QST_ID' => $item->ID(),
-        );
+		$restore_query_args = array(
+			'action' => 'restore_question',
+			'QST_ID' => $item->ID(),
+		);
 
-        $delete_query_args = array(
-            'action' => 'delete_questions',
-            'QST_ID' => $item->ID(),
-        );
+		$delete_query_args = array(
+			'action' => 'delete_questions',
+			'QST_ID' => $item->ID(),
+		);
 
-        $duplicate_query_args = array(
-            'action' => 'duplicate_question',
-            'QST_ID' => $item->ID(),
-        );
+		$duplicate_query_args = array(
+			'action' => 'duplicate_question',
+			'QST_ID' => $item->ID(),
+		);
 
-        $edit_link = EE_Admin_Page::add_query_args_and_nonce($edit_query_args, EE_FORMS_ADMIN_URL);
-        $trash_link = EE_Admin_Page::add_query_args_and_nonce($trash_query_args, EE_FORMS_ADMIN_URL);
-        $restore_link = EE_Admin_Page::add_query_args_and_nonce($restore_query_args, EE_FORMS_ADMIN_URL);
-        $delete_link = EE_Admin_Page::add_query_args_and_nonce($delete_query_args, EE_FORMS_ADMIN_URL);
-        $duplicate_link = EE_Admin_Page::add_query_args_and_nonce($duplicate_query_args, EE_FORMS_ADMIN_URL);
+		$edit_link = EE_Admin_Page::add_query_args_and_nonce($edit_query_args, EE_FORMS_ADMIN_URL);
+		$trash_link = EE_Admin_Page::add_query_args_and_nonce($trash_query_args, EE_FORMS_ADMIN_URL);
+		$restore_link = EE_Admin_Page::add_query_args_and_nonce($restore_query_args, EE_FORMS_ADMIN_URL);
+		$delete_link = EE_Admin_Page::add_query_args_and_nonce($delete_query_args, EE_FORMS_ADMIN_URL);
+		$duplicate_link = EE_Admin_Page::add_query_args_and_nonce($duplicate_query_args, EE_FORMS_ADMIN_URL);
 
-        if (EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_question',
-            'espresso_registration_form_edit_question',
-            $item->ID()
-        )) {
-            $actions = array(
-                'edit' => '<a href="' . $edit_link . '" title="'
-                          . __('Edit Question', 'event_espresso') . '">'
-                          . __('Edit', 'event_espresso') . '</a>',
-            );
-        }
+		if (EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_question',
+			'espresso_registration_form_edit_question',
+			$item->ID()
+		)) {
+			$actions = array(
+				'edit' => '<a href="' . $edit_link . '" title="'
+						  . __('Edit Question', 'event_espresso') . '">'
+						  . __('Edit', 'event_espresso') . '</a>',
+			);
+		}
 
-        if (! $system_question
-            && $this->_view != 'trash'
-            && EE_Registry::instance()->CAP->current_user_can(
-                'ee_delete_question',
-                'espresso_registration_form_trash_question',
-                $item->ID()
-            )) {
-                $actions['delete'] = '<a href="' . $trash_link . '" title="'
-                                     . __('Trash Question', 'event_espresso') . '">'
-                                     . __('Trash', 'event_espresso') . '</a>';
-        }
+		if (! $system_question
+			&& $this->_view != 'trash'
+			&& EE_Registry::instance()->CAP->current_user_can(
+				'ee_delete_question',
+				'espresso_registration_form_trash_question',
+				$item->ID()
+			)) {
+				$actions['delete'] = '<a href="' . $trash_link . '" title="'
+									 . __('Trash Question', 'event_espresso') . '">'
+									 . __('Trash', 'event_espresso') . '</a>';
+		}
 
-        if ($this->_view == 'trash') {
-            if (EE_Registry::instance()->CAP->current_user_can(
-                'ee_delete_question',
-                'espresso_registration_form_restore_question',
-                $item->ID()
-            )) {
-                $actions['restore'] = '<a href="' . $restore_link . '" title="'
-                                      . __('Restore Question', 'event_espresso') . '">'
-                                      . __('Restore', 'event_espresso') . '</a>';
-            }
-            if ($item->count_related('Answer') === 0
-                && EE_Registry::instance()->CAP->current_user_can(
-                    'ee_delete_question',
-                    'espresso_registration_form_delete_questions',
-                    $item->ID()
-                )) {
-                    $actions['delete'] = '<a href="' . $delete_link . '" title="'
-                                         . __('Delete Question Permanently', 'event_espresso') . '">'
-                                         . __('Delete Permanently', 'event_espresso') . '</a>';
-            }
-        }
-        if (EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_questions',
-            'espresso_registration_form_edit_question'
-        )) {
-            $actions['duplicate'] = '<a href="' . $duplicate_link . '" title="'
-                                    . __('Duplicate Question', 'event_espresso') . '">'
-                                    . __('Duplicate', 'event_espresso') . '</a>';
-        }
+		if ($this->_view == 'trash') {
+			if (EE_Registry::instance()->CAP->current_user_can(
+				'ee_delete_question',
+				'espresso_registration_form_restore_question',
+				$item->ID()
+			)) {
+				$actions['restore'] = '<a href="' . $restore_link . '" title="'
+									  . __('Restore Question', 'event_espresso') . '">'
+									  . __('Restore', 'event_espresso') . '</a>';
+			}
+			if ($item->count_related('Answer') === 0
+				&& EE_Registry::instance()->CAP->current_user_can(
+					'ee_delete_question',
+					'espresso_registration_form_delete_questions',
+					$item->ID()
+				)) {
+					$actions['delete'] = '<a href="' . $delete_link . '" title="'
+										 . __('Delete Question Permanently', 'event_espresso') . '">'
+										 . __('Delete Permanently', 'event_espresso') . '</a>';
+			}
+		}
+		if (EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_questions',
+			'espresso_registration_form_edit_question'
+		)) {
+			$actions['duplicate'] = '<a href="' . $duplicate_link . '" title="'
+									. __('Duplicate Question', 'event_espresso') . '">'
+									. __('Duplicate', 'event_espresso') . '</a>';
+		}
 
-        $content = EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_question',
-            'espresso_registration_form_edit_question',
-            $item->ID()
-        )
-            ? '<strong><a class="row-title" href="' . $edit_link . '">' . $item->display_text() . '</a></strong>'
-            : $item->display_text();
-        $content .= $this->row_actions($actions);
-        return $content;
-    }
+		$content = EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_question',
+			'espresso_registration_form_edit_question',
+			$item->ID()
+		)
+			? '<strong><a class="row-title" href="' . $edit_link . '">' . $item->display_text() . '</a></strong>'
+			: $item->display_text();
+		$content .= $this->row_actions($actions);
+		return $content;
+	}
 }

Spacing +18 added lines, -18 removed lines

@@ -26,7 +26,7 @@ 
         $system_question = $item->is_system_question();
         $actions = array();
 
-        if (! defined('REG_ADMIN_URL')) {
+        if ( ! defined('REG_ADMIN_URL')) {
             define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
         }
 
@@ -67,22 +67,22 @@ 
             $item->ID()
         )) {
             $actions = array(
-                'edit' => '<a href="' . $edit_link . '" title="'
-                          . __('Edit Question', 'event_espresso') . '">'
-                          . __('Edit', 'event_espresso') . '</a>',
+                'edit' => '<a href="'.$edit_link.'" title="'
+                          . __('Edit Question', 'event_espresso').'">'
+                          . __('Edit', 'event_espresso').'</a>',
             );
         }
 
-        if (! $system_question
+        if ( ! $system_question
             && $this->_view != 'trash'
             && EE_Registry::instance()->CAP->current_user_can(
                 'ee_delete_question',
                 'espresso_registration_form_trash_question',
                 $item->ID()
             )) {
-                $actions['delete'] = '<a href="' . $trash_link . '" title="'
-                                     . __('Trash Question', 'event_espresso') . '">'
-                                     . __('Trash', 'event_espresso') . '</a>';
+                $actions['delete'] = '<a href="'.$trash_link.'" title="'
+                                     . __('Trash Question', 'event_espresso').'">'
+                                     . __('Trash', 'event_espresso').'</a>';
         }
 
         if ($this->_view == 'trash') {
@@ -91,9 +91,9 @@ 
                 'espresso_registration_form_restore_question',
                 $item->ID()
             )) {
-                $actions['restore'] = '<a href="' . $restore_link . '" title="'
-                                      . __('Restore Question', 'event_espresso') . '">'
-                                      . __('Restore', 'event_espresso') . '</a>';
+                $actions['restore'] = '<a href="'.$restore_link.'" title="'
+                                      . __('Restore Question', 'event_espresso').'">'
+                                      . __('Restore', 'event_espresso').'</a>';
             }
             if ($item->count_related('Answer') === 0
                 && EE_Registry::instance()->CAP->current_user_can(
@@ -101,18 +101,18 @@ 
                     'espresso_registration_form_delete_questions',
                     $item->ID()
                 )) {
-                    $actions['delete'] = '<a href="' . $delete_link . '" title="'
-                                         . __('Delete Question Permanently', 'event_espresso') . '">'
-                                         . __('Delete Permanently', 'event_espresso') . '</a>';
+                    $actions['delete'] = '<a href="'.$delete_link.'" title="'
+                                         . __('Delete Question Permanently', 'event_espresso').'">'
+                                         . __('Delete Permanently', 'event_espresso').'</a>';
             }
         }
         if (EE_Registry::instance()->CAP->current_user_can(
             'ee_edit_questions',
             'espresso_registration_form_edit_question'
         )) {
-            $actions['duplicate'] = '<a href="' . $duplicate_link . '" title="'
-                                    . __('Duplicate Question', 'event_espresso') . '">'
-                                    . __('Duplicate', 'event_espresso') . '</a>';
+            $actions['duplicate'] = '<a href="'.$duplicate_link.'" title="'
+                                    . __('Duplicate Question', 'event_espresso').'">'
+                                    . __('Duplicate', 'event_espresso').'</a>';
         }
 
         $content = EE_Registry::instance()->CAP->current_user_can(
@@ -120,7 +120,7 @@ 
             'espresso_registration_form_edit_question',
             $item->ID()
         )
-            ? '<strong><a class="row-title" href="' . $edit_link . '">' . $item->display_text() . '</a></strong>'
+            ? '<strong><a class="row-title" href="'.$edit_link.'">'.$item->display_text().'</a></strong>'
             : $item->display_text();
         $content .= $this->row_actions($actions);
         return $content;

caffeinated/admin/extend/registration_form/Registration_Form_Question_Groups_Admin_List_Table.class.php

Indentation +227 added lines, -227 removed lines

@@ -30,231 +30,231 @@
 {
 
 
-    public function __construct($admin_page)
-    {
-        parent::__construct($admin_page);
-    }
-
-
-    protected function _setup_data()
-    {
-        $this->_data = $this->_view != 'trash'
-            ? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, false)
-            : $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, false);
-        $this->_all_data_count = $this->_view != 'trash'
-            ? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, true)
-            : $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, true);
-    }
-
-
-    protected function _set_properties()
-    {
-        $this->_wp_list_args = array(
-            'singular' => __('question group', 'event_espresso'),
-            'plural'   => __('question groups', 'event_espresso'),
-            'ajax'     => true, // for now,
-            'screen'   => $this->_admin_page->get_current_screen()->id,
-        );
-
-        $this->_columns = array(
-            'cb'              => '<input type="checkbox" />',
-            'id'              => __('ID', 'event_espresso'),
-            'name'            => __('Group Name', 'event_espresso'),
-            'description'     => __('Description', 'event_espresso'),
-            'show_group_name' => __('Show Name', 'event_espresso'),
-            'show_group_desc' => __('Show Desc', 'event_espresso'),
-        );
-
-        $this->_sortable_columns = array(
-            'id'   => array('QSG_ID' => false),
-            'name' => array('QSG_name' => false),
-        );
-
-        $this->_hidden_columns = array(
-            'id',
-        );
-
-        $this->_ajax_sorting_callback = 'update_question_group_order';
-    }
-
-
-    // not needed
-    protected function _get_table_filters()
-    {
-        return array();
-    }
-
-
-    protected function _add_view_counts()
-    {
-        $this->_views['all']['count'] = $this->_admin_page->get_question_groups(
-            $this->_per_page,
-            $this->_current_page,
-            true
-        );
-        if (EE_Registry::instance()->CAP->current_user_can(
-            'ee_delete_question_groups',
-            'espresso_registration_form_trash_question_group'
-        )) {
-            $this->_views['trash']['count'] = $this->_admin_page->get_trashed_question_groups(
-                $this->_per_page,
-                $this->_current_page,
-                true
-            );
-        }
-    }
-
-
-    public function column_cb($item)
-    {
-        $system_group = $item->get('QSG_system');
-        $has_questions_with_answers = $item->has_questions_with_answers();
-        $lock_icon = $system_group === 0 && $this->_view == 'trash' && $has_questions_with_answers
-            ? 'ee-lock-icon ee-alternate-color'
-            : 'ee-lock-icon ee-system-lock';
-        return $system_group > 0
-               || ($system_group === 0
-                    && $this->_view == 'trash'
-                    && $has_questions_with_answers
-                )
-               || ! EE_Registry::instance()->CAP->current_user_can(
-                   'ee_delete_question_groups',
-                   'espresso_registration_form_trash_question_groups',
-                   $item->ID()
-               )
-            ? '<span class="' . $lock_icon . '"></span>'
-              . sprintf(
-                  '<input type="hidden" name="hdnchk[%1$d]" value="%1$d" />',
-                  $item->ID()
-              )
-            : sprintf(
-                '<input type="checkbox" id="QSG_ID[%1$d]" name="checkbox[%1$d]" value="%1$d" />',
-                $item->ID()
-            );
-    }
-
-
-    public function column_id(EE_Question_Group $item)
-    {
-        $content = $item->ID();
-        $content .= '  <span class="show-on-mobile-view-only">' . $item->name() . '</span>';
-        return $content;
-    }
-
-
-    public function column_name(EE_Question_Group $item)
-    {
-        $actions = array();
-
-        // return $item->name();
-        if (! defined('REG_ADMIN_URL')) {
-            define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
-        }
-
-        $edit_query_args = array(
-            'action' => 'edit_question_group',
-            'QSG_ID' => $item->ID(),
-        );
-
-        $trash_query_args = array(
-            'action' => 'trash_question_group',
-            'QSG_ID' => $item->ID(),
-        );
-
-        $restore_query_args = array(
-            'action' => 'restore_question_group',
-            'QSG_ID' => $item->ID(),
-        );
-
-        $delete_query_args = array(
-            'action' => 'delete_question_group',
-            'QSG_ID' => $item->ID(),
-        );
-
-
-        $edit_link = EE_Admin_Page::add_query_args_and_nonce($edit_query_args, EE_FORMS_ADMIN_URL);
-        $trash_link = EE_Admin_Page::add_query_args_and_nonce($trash_query_args, EE_FORMS_ADMIN_URL);
-        $restore_link = EE_Admin_Page::add_query_args_and_nonce($restore_query_args, EE_FORMS_ADMIN_URL);
-        $delete_link = EE_Admin_Page::add_query_args_and_nonce($delete_query_args, EE_FORMS_ADMIN_URL);
-
-        if (EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_question_group',
-            'espresso_registration_form_edit_question_group',
-            $item->ID()
-        )) {
-            $actions = array(
-                'edit' => '<a href="' . $edit_link . '" title="'
-                          . esc_attr__('Edit Question Group', 'event_espresso') . '">'
-                          . __('Edit', 'event_espresso') . '</a>',
-            );
-        }
-        if ($item->get('QSG_system') < 1
-            && $this->_view != 'trash'
-            && EE_Registry::instance()->CAP->current_user_can(
-                'ee_delete_question_group',
-                'espresso_registration_form_trash_question_group',
-                $item->ID()
-            )) {
-            $actions['delete'] = '<a href="' . $trash_link . '" title="'
-                                 . esc_attr__('Delete Question Group', 'event_espresso') . '">'
-                                 . __('Trash', 'event_espresso') . '</a>';
-        }
-
-        if ($this->_view == 'trash') {
-            if (EE_Registry::instance()->CAP->current_user_can(
-                'ee_delete_question_group',
-                'espresso_registration_form_restore_question_group',
-                $item->ID()
-            )) {
-                $actions['restore'] = '<a href="' . $restore_link . '" title="'
-                                      . esc_attr__('Restore Question Group', 'event_espresso') . '">'
-                                      . __('Restore', 'event_espresso') . '</a>';
-            }
-
-            if (! $item->has_questions_with_answers()
-                && EE_Registry::instance()->CAP->current_user_can(
-                    'ee_delete_question_group',
-                    'espresso_registration_form_delete_question_group',
-                    $item->ID()
-                )) {
-                    $actions['delete'] = '<a href="' . $delete_link . '" title="'
-                                         . esc_attr__('Delete Question Group Permanently', 'event_espresso') . '">'
-                                         . __('Delete Permanently', 'event_espresso') . '</a>';
-            }
-        }
-
-        $content = EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_question_group',
-            'espresso_registration_form_edit_question_group',
-            $item->ID()
-        )
-            ? '<strong><a class="row-title" href="' . $edit_link . '">' . $item->name() . '</a></strong>'
-            : $item->name();
-        $content .= $this->row_actions($actions);
-        return $content;
-    }
-
-
-    public function column_identifier(EE_Question_Group $item)
-    {
-        return $item->identifier();
-    }
-
-
-    public function column_description(EE_Question_Group $item)
-    {
-        return $item->desc();
-    }
-
-
-    public function column_show_group_name(EE_Question_Group $item)
-    {
-        return $this->_yes_no[ $item->show_group_name() ];
-    }
-
-
-    public function column_show_group_desc(EE_Question_Group $item)
-    {
-        return $this->_yes_no[ $item->show_group_desc() ];
-    }
+	public function __construct($admin_page)
+	{
+		parent::__construct($admin_page);
+	}
+
+
+	protected function _setup_data()
+	{
+		$this->_data = $this->_view != 'trash'
+			? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, false)
+			: $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, false);
+		$this->_all_data_count = $this->_view != 'trash'
+			? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, true)
+			: $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, true);
+	}
+
+
+	protected function _set_properties()
+	{
+		$this->_wp_list_args = array(
+			'singular' => __('question group', 'event_espresso'),
+			'plural'   => __('question groups', 'event_espresso'),
+			'ajax'     => true, // for now,
+			'screen'   => $this->_admin_page->get_current_screen()->id,
+		);
+
+		$this->_columns = array(
+			'cb'              => '<input type="checkbox" />',
+			'id'              => __('ID', 'event_espresso'),
+			'name'            => __('Group Name', 'event_espresso'),
+			'description'     => __('Description', 'event_espresso'),
+			'show_group_name' => __('Show Name', 'event_espresso'),
+			'show_group_desc' => __('Show Desc', 'event_espresso'),
+		);
+
+		$this->_sortable_columns = array(
+			'id'   => array('QSG_ID' => false),
+			'name' => array('QSG_name' => false),
+		);
+
+		$this->_hidden_columns = array(
+			'id',
+		);
+
+		$this->_ajax_sorting_callback = 'update_question_group_order';
+	}
+
+
+	// not needed
+	protected function _get_table_filters()
+	{
+		return array();
+	}
+
+
+	protected function _add_view_counts()
+	{
+		$this->_views['all']['count'] = $this->_admin_page->get_question_groups(
+			$this->_per_page,
+			$this->_current_page,
+			true
+		);
+		if (EE_Registry::instance()->CAP->current_user_can(
+			'ee_delete_question_groups',
+			'espresso_registration_form_trash_question_group'
+		)) {
+			$this->_views['trash']['count'] = $this->_admin_page->get_trashed_question_groups(
+				$this->_per_page,
+				$this->_current_page,
+				true
+			);
+		}
+	}
+
+
+	public function column_cb($item)
+	{
+		$system_group = $item->get('QSG_system');
+		$has_questions_with_answers = $item->has_questions_with_answers();
+		$lock_icon = $system_group === 0 && $this->_view == 'trash' && $has_questions_with_answers
+			? 'ee-lock-icon ee-alternate-color'
+			: 'ee-lock-icon ee-system-lock';
+		return $system_group > 0
+			   || ($system_group === 0
+					&& $this->_view == 'trash'
+					&& $has_questions_with_answers
+				)
+			   || ! EE_Registry::instance()->CAP->current_user_can(
+				   'ee_delete_question_groups',
+				   'espresso_registration_form_trash_question_groups',
+				   $item->ID()
+			   )
+			? '<span class="' . $lock_icon . '"></span>'
+			  . sprintf(
+				  '<input type="hidden" name="hdnchk[%1$d]" value="%1$d" />',
+				  $item->ID()
+			  )
+			: sprintf(
+				'<input type="checkbox" id="QSG_ID[%1$d]" name="checkbox[%1$d]" value="%1$d" />',
+				$item->ID()
+			);
+	}
+
+
+	public function column_id(EE_Question_Group $item)
+	{
+		$content = $item->ID();
+		$content .= '  <span class="show-on-mobile-view-only">' . $item->name() . '</span>';
+		return $content;
+	}
+
+
+	public function column_name(EE_Question_Group $item)
+	{
+		$actions = array();
+
+		// return $item->name();
+		if (! defined('REG_ADMIN_URL')) {
+			define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
+		}
+
+		$edit_query_args = array(
+			'action' => 'edit_question_group',
+			'QSG_ID' => $item->ID(),
+		);
+
+		$trash_query_args = array(
+			'action' => 'trash_question_group',
+			'QSG_ID' => $item->ID(),
+		);
+
+		$restore_query_args = array(
+			'action' => 'restore_question_group',
+			'QSG_ID' => $item->ID(),
+		);
+
+		$delete_query_args = array(
+			'action' => 'delete_question_group',
+			'QSG_ID' => $item->ID(),
+		);
+
+
+		$edit_link = EE_Admin_Page::add_query_args_and_nonce($edit_query_args, EE_FORMS_ADMIN_URL);
+		$trash_link = EE_Admin_Page::add_query_args_and_nonce($trash_query_args, EE_FORMS_ADMIN_URL);
+		$restore_link = EE_Admin_Page::add_query_args_and_nonce($restore_query_args, EE_FORMS_ADMIN_URL);
+		$delete_link = EE_Admin_Page::add_query_args_and_nonce($delete_query_args, EE_FORMS_ADMIN_URL);
+
+		if (EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_question_group',
+			'espresso_registration_form_edit_question_group',
+			$item->ID()
+		)) {
+			$actions = array(
+				'edit' => '<a href="' . $edit_link . '" title="'
+						  . esc_attr__('Edit Question Group', 'event_espresso') . '">'
+						  . __('Edit', 'event_espresso') . '</a>',
+			);
+		}
+		if ($item->get('QSG_system') < 1
+			&& $this->_view != 'trash'
+			&& EE_Registry::instance()->CAP->current_user_can(
+				'ee_delete_question_group',
+				'espresso_registration_form_trash_question_group',
+				$item->ID()
+			)) {
+			$actions['delete'] = '<a href="' . $trash_link . '" title="'
+								 . esc_attr__('Delete Question Group', 'event_espresso') . '">'
+								 . __('Trash', 'event_espresso') . '</a>';
+		}
+
+		if ($this->_view == 'trash') {
+			if (EE_Registry::instance()->CAP->current_user_can(
+				'ee_delete_question_group',
+				'espresso_registration_form_restore_question_group',
+				$item->ID()
+			)) {
+				$actions['restore'] = '<a href="' . $restore_link . '" title="'
+									  . esc_attr__('Restore Question Group', 'event_espresso') . '">'
+									  . __('Restore', 'event_espresso') . '</a>';
+			}
+
+			if (! $item->has_questions_with_answers()
+				&& EE_Registry::instance()->CAP->current_user_can(
+					'ee_delete_question_group',
+					'espresso_registration_form_delete_question_group',
+					$item->ID()
+				)) {
+					$actions['delete'] = '<a href="' . $delete_link . '" title="'
+										 . esc_attr__('Delete Question Group Permanently', 'event_espresso') . '">'
+										 . __('Delete Permanently', 'event_espresso') . '</a>';
+			}
+		}
+
+		$content = EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_question_group',
+			'espresso_registration_form_edit_question_group',
+			$item->ID()
+		)
+			? '<strong><a class="row-title" href="' . $edit_link . '">' . $item->name() . '</a></strong>'
+			: $item->name();
+		$content .= $this->row_actions($actions);
+		return $content;
+	}
+
+
+	public function column_identifier(EE_Question_Group $item)
+	{
+		return $item->identifier();
+	}
+
+
+	public function column_description(EE_Question_Group $item)
+	{
+		return $item->desc();
+	}
+
+
+	public function column_show_group_name(EE_Question_Group $item)
+	{
+		return $this->_yes_no[ $item->show_group_name() ];
+	}
+
+
+	public function column_show_group_desc(EE_Question_Group $item)
+	{
+		return $this->_yes_no[ $item->show_group_desc() ];
+	}
 }

Spacing +19 added lines, -19 removed lines

@@ -122,7 +122,7 @@ 
                    'espresso_registration_form_trash_question_groups',
                    $item->ID()
                )
-            ? '<span class="' . $lock_icon . '"></span>'
+            ? '<span class="'.$lock_icon.'"></span>'
               . sprintf(
                   '<input type="hidden" name="hdnchk[%1$d]" value="%1$d" />',
                   $item->ID()
@@ -137,7 +137,7 @@ 
     public function column_id(EE_Question_Group $item)
     {
         $content = $item->ID();
-        $content .= '  <span class="show-on-mobile-view-only">' . $item->name() . '</span>';
+        $content .= '  <span class="show-on-mobile-view-only">'.$item->name().'</span>';
         return $content;
     }
 
@@ -147,7 +147,7 @@ 
         $actions = array();
 
         // return $item->name();
-        if (! defined('REG_ADMIN_URL')) {
+        if ( ! defined('REG_ADMIN_URL')) {
             define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
         }
 
@@ -183,9 +183,9 @@ 
             $item->ID()
         )) {
             $actions = array(
-                'edit' => '<a href="' . $edit_link . '" title="'
-                          . esc_attr__('Edit Question Group', 'event_espresso') . '">'
-                          . __('Edit', 'event_espresso') . '</a>',
+                'edit' => '<a href="'.$edit_link.'" title="'
+                          . esc_attr__('Edit Question Group', 'event_espresso').'">'
+                          . __('Edit', 'event_espresso').'</a>',
             );
         }
         if ($item->get('QSG_system') < 1
@@ -195,9 +195,9 @@ 
                 'espresso_registration_form_trash_question_group',
                 $item->ID()
             )) {
-            $actions['delete'] = '<a href="' . $trash_link . '" title="'
-                                 . esc_attr__('Delete Question Group', 'event_espresso') . '">'
-                                 . __('Trash', 'event_espresso') . '</a>';
+            $actions['delete'] = '<a href="'.$trash_link.'" title="'
+                                 . esc_attr__('Delete Question Group', 'event_espresso').'">'
+                                 . __('Trash', 'event_espresso').'</a>';
         }
 
         if ($this->_view == 'trash') {
@@ -206,20 +206,20 @@ 
                 'espresso_registration_form_restore_question_group',
                 $item->ID()
             )) {
-                $actions['restore'] = '<a href="' . $restore_link . '" title="'
-                                      . esc_attr__('Restore Question Group', 'event_espresso') . '">'
-                                      . __('Restore', 'event_espresso') . '</a>';
+                $actions['restore'] = '<a href="'.$restore_link.'" title="'
+                                      . esc_attr__('Restore Question Group', 'event_espresso').'">'
+                                      . __('Restore', 'event_espresso').'</a>';
             }
 
-            if (! $item->has_questions_with_answers()
+            if ( ! $item->has_questions_with_answers()
                 && EE_Registry::instance()->CAP->current_user_can(
                     'ee_delete_question_group',
                     'espresso_registration_form_delete_question_group',
                     $item->ID()
                 )) {
-                    $actions['delete'] = '<a href="' . $delete_link . '" title="'
-                                         . esc_attr__('Delete Question Group Permanently', 'event_espresso') . '">'
-                                         . __('Delete Permanently', 'event_espresso') . '</a>';
+                    $actions['delete'] = '<a href="'.$delete_link.'" title="'
+                                         . esc_attr__('Delete Question Group Permanently', 'event_espresso').'">'
+                                         . __('Delete Permanently', 'event_espresso').'</a>';
             }
         }
 
@@ -228,7 +228,7 @@ 
             'espresso_registration_form_edit_question_group',
             $item->ID()
         )
-            ? '<strong><a class="row-title" href="' . $edit_link . '">' . $item->name() . '</a></strong>'
+            ? '<strong><a class="row-title" href="'.$edit_link.'">'.$item->name().'</a></strong>'
             : $item->name();
         $content .= $this->row_actions($actions);
         return $content;
@@ -249,12 +249,12 @@ 
 
     public function column_show_group_name(EE_Question_Group $item)
     {
-        return $this->_yes_no[ $item->show_group_name() ];
+        return $this->_yes_no[$item->show_group_name()];
     }
 
 
     public function column_show_group_desc(EE_Question_Group $item)
     {
-        return $this->_yes_no[ $item->show_group_desc() ];
+        return $this->_yes_no[$item->show_group_desc()];
     }
 }

caffeinated/admin/extend/registration_form/help_tours/Registration_Form_Questions_Overview_Help_Tour.class.php

Indentation +139 added lines, -139 removed lines

@@ -28,155 +28,155 @@
 class Registration_Form_Questions_Overview_Help_Tour extends EE_Help_Tour
 {
 
-    protected function _set_tour_properties()
-    {
-        $this->_label = __('Questions Overview Tour', 'event_espresso');
-        $this->_slug = $this->_is_caf ? 'questions-overview-caf-joyride' : 'questions-overview-joyride';
-    }
+	protected function _set_tour_properties()
+	{
+		$this->_label = __('Questions Overview Tour', 'event_espresso');
+		$this->_slug = $this->_is_caf ? 'questions-overview-caf-joyride' : 'questions-overview-joyride';
+	}
 
-    protected function _set_tour_stops()
-    {
-        $this->_stops = array(
-            10  => array(
-                'content' => $this->_start(),
-            ),
-            30  => array(
-                'id'      => 'display_text',
-                'content' => $this->_display_text_stop(),
-                'options' => array(
-                    'tipLocation'    => 'top',
-                    'tipAdjustmentX' => -5,
-                    'tipAdjustmentY' => -25,
-                ),
-            ),
-            40  => array(
-                'id'      => 'admin_label',
-                'content' => $this->_admin_label_stop(),
-                'options' => array(
-                    'tipLocation'    => 'top',
-                    'tipAdjustmentX' => 20,
-                    'tipAdjustmentY' => -25,
-                ),
-            ),
-            50  => array(
-                'id'      => 'type',
-                'content' => $this->_type_stop(),
-                'options' => array(
-                    'tipLocation'    => 'top',
-                    'tipAdjustmentX' => -5,
-                    'tipAdjustmentY' => -25,
-                ),
-            ),
-            60  => array(
-                'id'      => 'values',
-                'content' => $this->_values_stop(),
-                'options' => array(
-                    'tipLocation'    => 'top',
-                    'tipAdjustmentX' => -5,
-                    'tipAdjustmentY' => -25,
-                ),
-            ),
-            70  => array(
-                'id'      => 'required',
-                'content' => $this->_required_stop(),
-                'options' => array(
-                    'tipLocation'    => 'top',
-                    'tipAdjustmentY' => -20,
-                    'tipAdjustmentX' => -15,
-                ),
-            ),
-            80  => array(
-                'class'   => 'bulkactions',
-                'content' => $this->_bulk_actions_stop(),
-                'options' => array(
-                    'tipLocation'    => 'top',
-                    'tipAdjustmentY' => -30,
-                    'tipAdjustmentX' => -15,
-                ),
-            ),
-            90  => array(
-                'id'      => 'event-espresso_page_espresso_registration_form-search-input',
-                'content' => $this->_search_stop(),
-                'options' => array(
-                    'tipLocation'    => 'left',
-                    'tipAdjustmentY' => -50,
-                    'tipAdjustmentX' => -15,
-                ),
-            ),
-            100 => array(
-                'id'      => 'add-new-question',
-                'content' => $this->_add_new_question_stop(),
-                'options' => array(
-                    'tipLocation'    => 'right',
-                    'tipAdjustmentY' => -50,
-                    'tipAdjustmentX' => 15,
-                ),
-            ),
-        );
-    }
+	protected function _set_tour_stops()
+	{
+		$this->_stops = array(
+			10  => array(
+				'content' => $this->_start(),
+			),
+			30  => array(
+				'id'      => 'display_text',
+				'content' => $this->_display_text_stop(),
+				'options' => array(
+					'tipLocation'    => 'top',
+					'tipAdjustmentX' => -5,
+					'tipAdjustmentY' => -25,
+				),
+			),
+			40  => array(
+				'id'      => 'admin_label',
+				'content' => $this->_admin_label_stop(),
+				'options' => array(
+					'tipLocation'    => 'top',
+					'tipAdjustmentX' => 20,
+					'tipAdjustmentY' => -25,
+				),
+			),
+			50  => array(
+				'id'      => 'type',
+				'content' => $this->_type_stop(),
+				'options' => array(
+					'tipLocation'    => 'top',
+					'tipAdjustmentX' => -5,
+					'tipAdjustmentY' => -25,
+				),
+			),
+			60  => array(
+				'id'      => 'values',
+				'content' => $this->_values_stop(),
+				'options' => array(
+					'tipLocation'    => 'top',
+					'tipAdjustmentX' => -5,
+					'tipAdjustmentY' => -25,
+				),
+			),
+			70  => array(
+				'id'      => 'required',
+				'content' => $this->_required_stop(),
+				'options' => array(
+					'tipLocation'    => 'top',
+					'tipAdjustmentY' => -20,
+					'tipAdjustmentX' => -15,
+				),
+			),
+			80  => array(
+				'class'   => 'bulkactions',
+				'content' => $this->_bulk_actions_stop(),
+				'options' => array(
+					'tipLocation'    => 'top',
+					'tipAdjustmentY' => -30,
+					'tipAdjustmentX' => -15,
+				),
+			),
+			90  => array(
+				'id'      => 'event-espresso_page_espresso_registration_form-search-input',
+				'content' => $this->_search_stop(),
+				'options' => array(
+					'tipLocation'    => 'left',
+					'tipAdjustmentY' => -50,
+					'tipAdjustmentX' => -15,
+				),
+			),
+			100 => array(
+				'id'      => 'add-new-question',
+				'content' => $this->_add_new_question_stop(),
+				'options' => array(
+					'tipLocation'    => 'right',
+					'tipAdjustmentY' => -50,
+					'tipAdjustmentX' => 15,
+				),
+			),
+		);
+	}
 
 
-    protected function _start()
-    {
-        $content = '<h3>' . __('Questions Overview', 'event_espresso') . '</h3>';
-        $content .= '<p>'
-                    . __(
-                        'This tour of the Questions Overview page will go over different areas of the screen to help you understand what they are used for.',
-                        'event_espresso'
-                    ) . '</p>';
+	protected function _start()
+	{
+		$content = '<h3>' . __('Questions Overview', 'event_espresso') . '</h3>';
+		$content .= '<p>'
+					. __(
+						'This tour of the Questions Overview page will go over different areas of the screen to help you understand what they are used for.',
+						'event_espresso'
+					) . '</p>';
 
-        return $content;
-    }
+		return $content;
+	}
 
-    protected function _display_text_stop()
-    {
-        return '<p>' . __('View available questions.', 'event_espresso') . '</p>';
-    }
+	protected function _display_text_stop()
+	{
+		return '<p>' . __('View available questions.', 'event_espresso') . '</p>';
+	}
 
-    protected function _admin_label_stop()
-    {
-        return '<p>' . __('View the admin label for your questions.', 'event_espresso') . '</p>';
-    }
+	protected function _admin_label_stop()
+	{
+		return '<p>' . __('View the admin label for your questions.', 'event_espresso') . '</p>';
+	}
 
-    protected function _type_stop()
-    {
-        return '<p>'
-               . __(
-                   'View the type of question. Available options are Text, Textarea, Checkboxes, Radio Buttons, Dropdown, State/Province Dropdown, Country Dropdown, and Date Picker.',
-                   'event_espresso'
-               ) . '</p>';
-    }
+	protected function _type_stop()
+	{
+		return '<p>'
+			   . __(
+				   'View the type of question. Available options are Text, Textarea, Checkboxes, Radio Buttons, Dropdown, State/Province Dropdown, Country Dropdown, and Date Picker.',
+				   'event_espresso'
+			   ) . '</p>';
+	}
 
-    protected function _values_stop()
-    {
-        return '<p>'
-               . __(
-                   'View stored values for checkboxes, radio buttons, and select boxes.',
-                   'event_espresso'
-               ) . '</p>';
-    }
+	protected function _values_stop()
+	{
+		return '<p>'
+			   . __(
+				   'View stored values for checkboxes, radio buttons, and select boxes.',
+				   'event_espresso'
+			   ) . '</p>';
+	}
 
-    protected function _required_stop()
-    {
-        return '<p>' . __('View if a question is required.', 'event_espresso') . '</p>';
-    }
+	protected function _required_stop()
+	{
+		return '<p>' . __('View if a question is required.', 'event_espresso') . '</p>';
+	}
 
-    protected function _bulk_actions_stop()
-    {
-        return '<p>' . __('Perform bulk actions to multiple questions.', 'event_espresso') . '</p>';
-    }
+	protected function _bulk_actions_stop()
+	{
+		return '<p>' . __('Perform bulk actions to multiple questions.', 'event_espresso') . '</p>';
+	}
 
-    protected function _search_stop()
-    {
-        return '<p>'
-               . __(
-                   'Search through questions. The following sources will be searched: Name of Question (display text).',
-                   'event_espresso'
-               ) . '</p>';
-    }
+	protected function _search_stop()
+	{
+		return '<p>'
+			   . __(
+				   'Search through questions. The following sources will be searched: Name of Question (display text).',
+				   'event_espresso'
+			   ) . '</p>';
+	}
 
-    protected function _add_new_question_stop()
-    {
-        return '<p>' . __('Click here to add a new question.', 'event_espresso') . '</p>';
-    }
+	protected function _add_new_question_stop()
+	{
+		return '<p>' . __('Click here to add a new question.', 'event_espresso') . '</p>';
+	}
 }

Spacing +10 added lines, -10 removed lines

@@ -118,24 +118,24 @@ 
 
     protected function _start()
     {
-        $content = '<h3>' . __('Questions Overview', 'event_espresso') . '</h3>';
+        $content = '<h3>'.__('Questions Overview', 'event_espresso').'</h3>';
         $content .= '<p>'
                     . __(
                         'This tour of the Questions Overview page will go over different areas of the screen to help you understand what they are used for.',
                         'event_espresso'
-                    ) . '</p>';
+                    ).'</p>';
 
         return $content;
     }
 
     protected function _display_text_stop()
     {
-        return '<p>' . __('View available questions.', 'event_espresso') . '</p>';
+        return '<p>'.__('View available questions.', 'event_espresso').'</p>';
     }
 
     protected function _admin_label_stop()
     {
-        return '<p>' . __('View the admin label for your questions.', 'event_espresso') . '</p>';
+        return '<p>'.__('View the admin label for your questions.', 'event_espresso').'</p>';
     }
 
     protected function _type_stop()
@@ -144,7 +144,7 @@ 
                . __(
                    'View the type of question. Available options are Text, Textarea, Checkboxes, Radio Buttons, Dropdown, State/Province Dropdown, Country Dropdown, and Date Picker.',
                    'event_espresso'
-               ) . '</p>';
+               ).'</p>';
     }
 
     protected function _values_stop()
@@ -153,17 +153,17 @@ 
                . __(
                    'View stored values for checkboxes, radio buttons, and select boxes.',
                    'event_espresso'
-               ) . '</p>';
+               ).'</p>';
     }
 
     protected function _required_stop()
     {
-        return '<p>' . __('View if a question is required.', 'event_espresso') . '</p>';
+        return '<p>'.__('View if a question is required.', 'event_espresso').'</p>';
     }
 
     protected function _bulk_actions_stop()
     {
-        return '<p>' . __('Perform bulk actions to multiple questions.', 'event_espresso') . '</p>';
+        return '<p>'.__('Perform bulk actions to multiple questions.', 'event_espresso').'</p>';
     }
 
     protected function _search_stop()
@@ -172,11 +172,11 @@ 
                . __(
                    'Search through questions. The following sources will be searched: Name of Question (display text).',
                    'event_espresso'
-               ) . '</p>';
+               ).'</p>';
     }
 
     protected function _add_new_question_stop()
     {
-        return '<p>' . __('Click here to add a new question.', 'event_espresso') . '</p>';
+        return '<p>'.__('Click here to add a new question.', 'event_espresso').'</p>';
     }
 }

caffeinated/admin/extend/events/Tickets_List_Table.class.php

Indentation +147 added lines, -147 removed lines

@@ -17,151 +17,151 @@
 {
 
 
-    protected function _setup_data()
-    {
-        $trashed = $this->_admin_page->get_view() == 'trashed' ? true : false;
-        $this->_data = $this->_admin_page->get_default_tickets($this->_per_page, false, $trashed);
-        $this->_all_data_count = $this->_admin_page->get_default_tickets($this->_per_page, true, false);
-        $this->_trashed_count = $this->_admin_page->get_default_tickets($this->_per_page, true, true);
-    }
-
-
-    protected function _set_properties()
-    {
-        $this->_wp_list_args = array(
-            'singular' => __('ticket', 'event_espresso'),
-            'plural'   => __('tickets', 'event_espresso'),
-            'ajax'     => true,
-            'screen'   => $this->_admin_page->get_current_screen()->id,
-        );
-
-        $this->_columns = array(
-            'cb'              => '<input type="checkbox" />', // Render a checkbox instead of text
-            'TKT_name'        => __('Name', 'event_espresso'),
-            'TKT_description' => __('Description', 'event_espresso'),
-            'TKT_qty'         => __('Quantity', 'event_espresso'),
-            'TKT_uses'        => __('Datetimes', 'event_espresso'),
-            'TKT_min'         => __('Minimum', 'event_espresso'),
-            'TKT_max'         => __('Maximum', 'event_espresso'),
-            'TKT_price'       => __('Price', 'event_espresso'),
-            'TKT_taxable'     => __('Taxable', 'event_espresso'),
-        );
-
-        $this->_sortable_columns = array(
-            // TRUE means its already sorted
-            'TKT_name'        => array('TKT_name' => true),
-            'TKT_description' => array('TKT_description' => false),
-            'TKT_qty'         => array('TKT_qty' => false),
-            'TKT_uses'        => array('TKT_uses' => false),
-            'TKT_min'         => array('TKT_min' => false),
-            'TKT_max'         => array('TKT_max' => false),
-            'TKT_price'       => array('TKT_price' => false),
-        );
-
-        $this->_hidden_columns = array();
-    }
-
-
-    protected function _get_table_filters()
-    {
-    }
-
-
-    protected function _add_view_counts()
-    {
-        $this->_views['all']['count'] = $this->_all_data_count;
-        $this->_views['trashed']['count'] = $this->_trashed_count;
-    }
-
-
-    public function column_cb($item)
-    {
-        return $item->ID() === 1
-            ? '<span class="ee-lock-icon"></span>'
-            : sprintf(
-                '<input type="checkbox" name="checkbox[%1$s]" value="%1$s" />',
-                $item->ID()
-            );
-    }
-
-
-    public function column_TKT_name($item)
-    {
-        // build row actions
-        $actions = array();
-
-        // trash links
-        if ($item->ID() !== 1) {
-            if ($this->_view == 'all') {
-                $trash_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
-                    'action' => 'trash_ticket',
-                    'TKT_ID' => $item->ID(),
-                ), EVENTS_ADMIN_URL);
-                $actions['trash'] = '<a href="' . $trash_lnk_url . '" title="'
-                                    . esc_attr__('Move Ticket to trash', 'event_espresso') . '">'
-                                    . __('Trash', 'event_espresso') . '</a>';
-            } else {
-                // restore price link
-                $restore_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
-                    'action' => 'restore_ticket',
-                    'TKT_ID' => $item->ID(),
-                ), EVENTS_ADMIN_URL);
-                $actions['restore'] = '<a href="' . $restore_lnk_url . '" title="'
-                                      . esc_attr__('Restore Ticket', 'event_espresso') . '">'
-                                      . __('Restore', 'event_espresso') . '</a>';
-                // delete price link
-                $delete_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
-                    'action' => 'delete_ticket',
-                    'TKT_ID' => $item->ID(),
-                ), EVENTS_ADMIN_URL);
-                $actions['delete'] = '<a href="' . $delete_lnk_url . '" title="'
-                                     . esc_attr__('Delete Ticket Permanently', 'event_espresso') . '">'
-                                     . __('Delete Permanently', 'event_espresso') . '</a>';
-            }
-        }
-
-        return $item->get('TKT_name') . $this->row_actions($actions);
-    }
-
-
-    public function column_TKT_description($item)
-    {
-        return $item->get('TKT_description');
-    }
-
-
-    public function column_TKT_qty($item)
-    {
-        return $item->get_pretty('TKT_qty', 'text');
-    }
-
-
-    public function column_TKT_uses($item)
-    {
-        return $item->get_pretty('TKT_uses', 'text');
-    }
-
-
-    public function column_TKT_min($item)
-    {
-        return $item->get('TKT_min');
-    }
-
-
-    public function column_TKT_max($item)
-    {
-        return $item->get_pretty('TKT_max', 'text');
-    }
-
-
-    public function column_TKT_price($item)
-    {
-        return EEH_Template::format_currency($item->get('TKT_price'));
-    }
-
-
-    public function column_TKT_taxable($item)
-    {
-        return $item->get('TKT_taxable') ? __('Yes', 'event_espresso') : __('No', 'event_espresso');
-    }
+	protected function _setup_data()
+	{
+		$trashed = $this->_admin_page->get_view() == 'trashed' ? true : false;
+		$this->_data = $this->_admin_page->get_default_tickets($this->_per_page, false, $trashed);
+		$this->_all_data_count = $this->_admin_page->get_default_tickets($this->_per_page, true, false);
+		$this->_trashed_count = $this->_admin_page->get_default_tickets($this->_per_page, true, true);
+	}
+
+
+	protected function _set_properties()
+	{
+		$this->_wp_list_args = array(
+			'singular' => __('ticket', 'event_espresso'),
+			'plural'   => __('tickets', 'event_espresso'),
+			'ajax'     => true,
+			'screen'   => $this->_admin_page->get_current_screen()->id,
+		);
+
+		$this->_columns = array(
+			'cb'              => '<input type="checkbox" />', // Render a checkbox instead of text
+			'TKT_name'        => __('Name', 'event_espresso'),
+			'TKT_description' => __('Description', 'event_espresso'),
+			'TKT_qty'         => __('Quantity', 'event_espresso'),
+			'TKT_uses'        => __('Datetimes', 'event_espresso'),
+			'TKT_min'         => __('Minimum', 'event_espresso'),
+			'TKT_max'         => __('Maximum', 'event_espresso'),
+			'TKT_price'       => __('Price', 'event_espresso'),
+			'TKT_taxable'     => __('Taxable', 'event_espresso'),
+		);
+
+		$this->_sortable_columns = array(
+			// TRUE means its already sorted
+			'TKT_name'        => array('TKT_name' => true),
+			'TKT_description' => array('TKT_description' => false),
+			'TKT_qty'         => array('TKT_qty' => false),
+			'TKT_uses'        => array('TKT_uses' => false),
+			'TKT_min'         => array('TKT_min' => false),
+			'TKT_max'         => array('TKT_max' => false),
+			'TKT_price'       => array('TKT_price' => false),
+		);
+
+		$this->_hidden_columns = array();
+	}
+
+
+	protected function _get_table_filters()
+	{
+	}
+
+
+	protected function _add_view_counts()
+	{
+		$this->_views['all']['count'] = $this->_all_data_count;
+		$this->_views['trashed']['count'] = $this->_trashed_count;
+	}
+
+
+	public function column_cb($item)
+	{
+		return $item->ID() === 1
+			? '<span class="ee-lock-icon"></span>'
+			: sprintf(
+				'<input type="checkbox" name="checkbox[%1$s]" value="%1$s" />',
+				$item->ID()
+			);
+	}
+
+
+	public function column_TKT_name($item)
+	{
+		// build row actions
+		$actions = array();
+
+		// trash links
+		if ($item->ID() !== 1) {
+			if ($this->_view == 'all') {
+				$trash_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
+					'action' => 'trash_ticket',
+					'TKT_ID' => $item->ID(),
+				), EVENTS_ADMIN_URL);
+				$actions['trash'] = '<a href="' . $trash_lnk_url . '" title="'
+									. esc_attr__('Move Ticket to trash', 'event_espresso') . '">'
+									. __('Trash', 'event_espresso') . '</a>';
+			} else {
+				// restore price link
+				$restore_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
+					'action' => 'restore_ticket',
+					'TKT_ID' => $item->ID(),
+				), EVENTS_ADMIN_URL);
+				$actions['restore'] = '<a href="' . $restore_lnk_url . '" title="'
+									  . esc_attr__('Restore Ticket', 'event_espresso') . '">'
+									  . __('Restore', 'event_espresso') . '</a>';
+				// delete price link
+				$delete_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
+					'action' => 'delete_ticket',
+					'TKT_ID' => $item->ID(),
+				), EVENTS_ADMIN_URL);
+				$actions['delete'] = '<a href="' . $delete_lnk_url . '" title="'
+									 . esc_attr__('Delete Ticket Permanently', 'event_espresso') . '">'
+									 . __('Delete Permanently', 'event_espresso') . '</a>';
+			}
+		}
+
+		return $item->get('TKT_name') . $this->row_actions($actions);
+	}
+
+
+	public function column_TKT_description($item)
+	{
+		return $item->get('TKT_description');
+	}
+
+
+	public function column_TKT_qty($item)
+	{
+		return $item->get_pretty('TKT_qty', 'text');
+	}
+
+
+	public function column_TKT_uses($item)
+	{
+		return $item->get_pretty('TKT_uses', 'text');
+	}
+
+
+	public function column_TKT_min($item)
+	{
+		return $item->get('TKT_min');
+	}
+
+
+	public function column_TKT_max($item)
+	{
+		return $item->get_pretty('TKT_max', 'text');
+	}
+
+
+	public function column_TKT_price($item)
+	{
+		return EEH_Template::format_currency($item->get('TKT_price'));
+	}
+
+
+	public function column_TKT_taxable($item)
+	{
+		return $item->get('TKT_taxable') ? __('Yes', 'event_espresso') : __('No', 'event_espresso');
+	}
 }

Spacing +10 added lines, -10 removed lines

@@ -97,30 +97,30 @@
                     'action' => 'trash_ticket',
                     'TKT_ID' => $item->ID(),
                 ), EVENTS_ADMIN_URL);
-                $actions['trash'] = '<a href="' . $trash_lnk_url . '" title="'
-                                    . esc_attr__('Move Ticket to trash', 'event_espresso') . '">'
-                                    . __('Trash', 'event_espresso') . '</a>';
+                $actions['trash'] = '<a href="'.$trash_lnk_url.'" title="'
+                                    . esc_attr__('Move Ticket to trash', 'event_espresso').'">'
+                                    . __('Trash', 'event_espresso').'</a>';
             } else {
                 // restore price link
                 $restore_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
                     'action' => 'restore_ticket',
                     'TKT_ID' => $item->ID(),
                 ), EVENTS_ADMIN_URL);
-                $actions['restore'] = '<a href="' . $restore_lnk_url . '" title="'
-                                      . esc_attr__('Restore Ticket', 'event_espresso') . '">'
-                                      . __('Restore', 'event_espresso') . '</a>';
+                $actions['restore'] = '<a href="'.$restore_lnk_url.'" title="'
+                                      . esc_attr__('Restore Ticket', 'event_espresso').'">'
+                                      . __('Restore', 'event_espresso').'</a>';
                 // delete price link
                 $delete_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
                     'action' => 'delete_ticket',
                     'TKT_ID' => $item->ID(),
                 ), EVENTS_ADMIN_URL);
-                $actions['delete'] = '<a href="' . $delete_lnk_url . '" title="'
-                                     . esc_attr__('Delete Ticket Permanently', 'event_espresso') . '">'
-                                     . __('Delete Permanently', 'event_espresso') . '</a>';
+                $actions['delete'] = '<a href="'.$delete_lnk_url.'" title="'
+                                     . esc_attr__('Delete Ticket Permanently', 'event_espresso').'">'
+                                     . __('Delete Permanently', 'event_espresso').'</a>';
             }
         }
 
-        return $item->get('TKT_name') . $this->row_actions($actions);
+        return $item->get('TKT_name').$this->row_actions($actions);
     }
 
 

caffeinated/admin/extend/events/Extend_Events_Admin_List_Table.class.php

Indentation +10 added lines, -10 removed lines

@@ -12,15 +12,15 @@
 class Extend_Events_Admin_List_Table extends Events_Admin_List_Table
 {
 
-    protected function _column_name_action_setup(EE_Event $item)
-    {
-        $export_query_args = array(
-            'action' => 'export_events',
-            'EVT_ID' => $item->ID(),
-        );
-        $export_event_link = EE_Admin_Page::add_query_args_and_nonce($export_query_args, EVENTS_ADMIN_URL);
+	protected function _column_name_action_setup(EE_Event $item)
+	{
+		$export_query_args = array(
+			'action' => 'export_events',
+			'EVT_ID' => $item->ID(),
+		);
+		$export_event_link = EE_Admin_Page::add_query_args_and_nonce($export_query_args, EVENTS_ADMIN_URL);
 
-        $actions = parent::_column_name_action_setup($item);
-        return $actions;
-    }
+		$actions = parent::_column_name_action_setup($item);
+		return $actions;
+	}
 }

caffeinated/admin/extend/registrations/Extend_EE_Attendee_Contact_List_Table.class.php

Indentation +13 added lines, -13 removed lines

@@ -15,17 +15,17 @@
 class Extend_EE_Attendee_Contact_List_Table extends EE_Attendee_Contact_List_Table
 {
 
-    protected function _set_properties()
-    {
-        parent::_set_properties();
-        $this->_bottom_buttons = array(
-            'contact_list_report' => array(
-                'route'         => 'contact_list_report',
-                'extra_request' =>
-                    array(
-                        'return_url' => urlencode("//{$_SERVER['HTTP_HOST']}{$_SERVER['REQUEST_URI']}"),
-                    ),
-            ),
-        );
-    }
+	protected function _set_properties()
+	{
+		parent::_set_properties();
+		$this->_bottom_buttons = array(
+			'contact_list_report' => array(
+				'route'         => 'contact_list_report',
+				'extra_request' =>
+					array(
+						'return_url' => urlencode("//{$_SERVER['HTTP_HOST']}{$_SERVER['REQUEST_URI']}"),
+					),
+			),
+		);
+	}
 }

caffeinated/admin/extend/registrations/Extend_EE_Registrations_List_Table.class.php

Indentation +108 added lines, -108 removed lines

@@ -13,115 +13,115 @@
 class Extend_EE_Registrations_List_Table extends EE_Registrations_List_Table
 {
 
-    /**
-     * @param EE_Registration $item
-     * @return string
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    public function column__REG_date(EE_Registration $item)
-    {
-        $date_linked = parent::column__REG_date($item);
-        $actions = array();
-        // Build row actions
-        $check_in_url = EE_Admin_Page::add_query_args_and_nonce(array(
-            'action'   => 'event_registrations',
-            'event_id' => $item->event_ID(),
-        ), REG_ADMIN_URL);
-        $actions['check_in'] = EE_Registry::instance()->CAP->current_user_can(
-            'ee_read_registration',
-            'espresso_registrations_registration_checkins',
-            $item->ID()
-        ) && EE_Registry::instance()->CAP->current_user_can(
-            'ee_read_checkins',
-            'espresso_registrations_registration_checkins'
-        )
-            ? '<a href="' . $check_in_url . '"'
-              . ' title="' . esc_attr__(
-                  'The Check-In List allows you to easily toggle check-in status for this event',
-                  'event_espresso'
-              )
-              . '">' . esc_html__('View Check-ins', 'event_espresso') . '</a>'
-            : esc_html__('View Check-ins', 'event_espresso');
+	/**
+	 * @param EE_Registration $item
+	 * @return string
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	public function column__REG_date(EE_Registration $item)
+	{
+		$date_linked = parent::column__REG_date($item);
+		$actions = array();
+		// Build row actions
+		$check_in_url = EE_Admin_Page::add_query_args_and_nonce(array(
+			'action'   => 'event_registrations',
+			'event_id' => $item->event_ID(),
+		), REG_ADMIN_URL);
+		$actions['check_in'] = EE_Registry::instance()->CAP->current_user_can(
+			'ee_read_registration',
+			'espresso_registrations_registration_checkins',
+			$item->ID()
+		) && EE_Registry::instance()->CAP->current_user_can(
+			'ee_read_checkins',
+			'espresso_registrations_registration_checkins'
+		)
+			? '<a href="' . $check_in_url . '"'
+			  . ' title="' . esc_attr__(
+				  'The Check-In List allows you to easily toggle check-in status for this event',
+				  'event_espresso'
+			  )
+			  . '">' . esc_html__('View Check-ins', 'event_espresso') . '</a>'
+			: esc_html__('View Check-ins', 'event_espresso');
 
-        return sprintf('%1$s %2$s', $date_linked, $this->row_actions($actions));
-    }
+		return sprintf('%1$s %2$s', $date_linked, $this->row_actions($actions));
+	}
 
 
-    /**
-     *        column_default
-     *
-     * @param \EE_Registration $item
-     * @return string
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws ReflectionException
-     */
-    public function column_DTT_EVT_start(EE_Registration $item)
-    {
-        $remove_defaults = array('default_where_conditions' => 'none');
-        $ticket = $item->ticket();
-        $datetimes = $ticket instanceof EE_Ticket ? $ticket->datetimes($remove_defaults) : array();
-        $EVT_ID = $item->event_ID();
-        $datetimes_for_display = array();
-        foreach ($datetimes as $datetime) {
-            $datetime_string = '';
-            if (EE_Registry::instance()->CAP->current_user_can(
-                'ee_read_checkin',
-                'espresso_registrations_registration_checkins',
-                $item->ID()
-            )) {
-                // open "a" tag and "href"
-                $datetime_string .= '<a href="';
-                // checkin URL
-                $datetime_string .= EE_Admin_Page::add_query_args_and_nonce(
-                    array(
-                        'action'   => 'event_registrations',
-                        'event_id' => $EVT_ID,
-                        'DTT_ID'   => $datetime->ID(),
-                    ),
-                    REG_ADMIN_URL
-                );
-                // close "href"
-                $datetime_string .= '"';
-                // open "title" tag
-                $datetime_string .= ' title="';
-                // link title text
-                $datetime_string .= esc_attr__('View Checkins for this Event', 'event_espresso');
-                // close "title" tag and end of "a" tag opening
-                $datetime_string .= '">';
-                // link text
-                $datetime_string .= $datetime->get_i18n_datetime('DTT_EVT_start');
-                // close "a" tag
-                $datetime_string .= '</a>';
-            } else {
-                $datetime_string .= $datetime->get_i18n_datetime('DTT_EVT_start');
-            }
-            // add a "View Registrations" link that filters list by event AND datetime
-            $datetime_string .= $this->row_actions(
-                array(
-                    'event_datetime_filter' => '<a href="' . EE_Admin_Page::add_query_args_and_nonce(
-                        array('event_id' => $EVT_ID, 'datetime_id' => $datetime->ID()),
-                        REG_ADMIN_URL
-                    )
-                                               . '" title="' . sprintf(
-                                                   esc_attr__(
-                                                       'Filter this list to only show registrations for this datetime %s',
-                                                       'event_espresso'
-                                                   ),
-                                                   $datetime->name()
-                                               ) . '">'
-                                               . esc_html__('View Registrations', 'event_espresso')
-                                               . '</a>',
-                )
-            );
-            $datetimes_for_display[] = $datetime_string;
-        }
-        return $this->generateDisplayForDateTimes($datetimes_for_display);
-    }
+	/**
+	 *        column_default
+	 *
+	 * @param \EE_Registration $item
+	 * @return string
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws ReflectionException
+	 */
+	public function column_DTT_EVT_start(EE_Registration $item)
+	{
+		$remove_defaults = array('default_where_conditions' => 'none');
+		$ticket = $item->ticket();
+		$datetimes = $ticket instanceof EE_Ticket ? $ticket->datetimes($remove_defaults) : array();
+		$EVT_ID = $item->event_ID();
+		$datetimes_for_display = array();
+		foreach ($datetimes as $datetime) {
+			$datetime_string = '';
+			if (EE_Registry::instance()->CAP->current_user_can(
+				'ee_read_checkin',
+				'espresso_registrations_registration_checkins',
+				$item->ID()
+			)) {
+				// open "a" tag and "href"
+				$datetime_string .= '<a href="';
+				// checkin URL
+				$datetime_string .= EE_Admin_Page::add_query_args_and_nonce(
+					array(
+						'action'   => 'event_registrations',
+						'event_id' => $EVT_ID,
+						'DTT_ID'   => $datetime->ID(),
+					),
+					REG_ADMIN_URL
+				);
+				// close "href"
+				$datetime_string .= '"';
+				// open "title" tag
+				$datetime_string .= ' title="';
+				// link title text
+				$datetime_string .= esc_attr__('View Checkins for this Event', 'event_espresso');
+				// close "title" tag and end of "a" tag opening
+				$datetime_string .= '">';
+				// link text
+				$datetime_string .= $datetime->get_i18n_datetime('DTT_EVT_start');
+				// close "a" tag
+				$datetime_string .= '</a>';
+			} else {
+				$datetime_string .= $datetime->get_i18n_datetime('DTT_EVT_start');
+			}
+			// add a "View Registrations" link that filters list by event AND datetime
+			$datetime_string .= $this->row_actions(
+				array(
+					'event_datetime_filter' => '<a href="' . EE_Admin_Page::add_query_args_and_nonce(
+						array('event_id' => $EVT_ID, 'datetime_id' => $datetime->ID()),
+						REG_ADMIN_URL
+					)
+											   . '" title="' . sprintf(
+												   esc_attr__(
+													   'Filter this list to only show registrations for this datetime %s',
+													   'event_espresso'
+												   ),
+												   $datetime->name()
+											   ) . '">'
+											   . esc_html__('View Registrations', 'event_espresso')
+											   . '</a>',
+				)
+			);
+			$datetimes_for_display[] = $datetime_string;
+		}
+		return $this->generateDisplayForDateTimes($datetimes_for_display);
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -39,12 +39,12 @@ 
             'ee_read_checkins',
             'espresso_registrations_registration_checkins'
         )
-            ? '<a href="' . $check_in_url . '"'
-              . ' title="' . esc_attr__(
+            ? '<a href="'.$check_in_url.'"'
+              . ' title="'.esc_attr__(
                   'The Check-In List allows you to easily toggle check-in status for this event',
                   'event_espresso'
               )
-              . '">' . esc_html__('View Check-ins', 'event_espresso') . '</a>'
+              . '">'.esc_html__('View Check-ins', 'event_espresso').'</a>'
             : esc_html__('View Check-ins', 'event_espresso');
 
         return sprintf('%1$s %2$s', $date_linked, $this->row_actions($actions));
@@ -105,17 +105,17 @@ 
             // add a "View Registrations" link that filters list by event AND datetime
             $datetime_string .= $this->row_actions(
                 array(
-                    'event_datetime_filter' => '<a href="' . EE_Admin_Page::add_query_args_and_nonce(
+                    'event_datetime_filter' => '<a href="'.EE_Admin_Page::add_query_args_and_nonce(
                         array('event_id' => $EVT_ID, 'datetime_id' => $datetime->ID()),
                         REG_ADMIN_URL
                     )
-                                               . '" title="' . sprintf(
+                                               . '" title="'.sprintf(
                                                    esc_attr__(
                                                        'Filter this list to only show registrations for this datetime %s',
                                                        'event_espresso'
                                                    ),
                                                    $datetime->name()
-                                               ) . '">'
+                                               ).'">'
                                                . esc_html__('View Registrations', 'event_espresso')
                                                . '</a>',
                 )

caffeinated/admin/extend/messages/Custom_Messages_Template_List_Table.class.php

Indentation +225 added lines, -225 removed lines

@@ -12,252 +12,252 @@
 class Custom_Messages_Template_List_Table extends Messages_Template_List_Table
 {
 
-    /**
-     * Setup initial data.
-     */
-    protected function _setup_data()
-    {
-        $this->_data = $this->get_admin_page()->get_message_templates(
-            $this->_per_page,
-            $this->_view,
-            false,
-            false,
-            false
-        );
-        $this->_all_data_count = $this->get_admin_page()->get_message_templates(
-            $this->_per_page,
-            $this->_view,
-            true,
-            true,
-            false
-        );
-    }
+	/**
+	 * Setup initial data.
+	 */
+	protected function _setup_data()
+	{
+		$this->_data = $this->get_admin_page()->get_message_templates(
+			$this->_per_page,
+			$this->_view,
+			false,
+			false,
+			false
+		);
+		$this->_all_data_count = $this->get_admin_page()->get_message_templates(
+			$this->_per_page,
+			$this->_view,
+			true,
+			true,
+			false
+		);
+	}
 
 
-    /**
-     * Set initial properties
-     */
-    protected function _set_properties()
-    {
-        parent::_set_properties();
-        $this->_wp_list_args = array(
-            'singular' => esc_html__('Message Template Group', 'event_espresso'),
-            'plural'   => esc_html__('Message Template', 'event_espresso'),
-            'ajax'     => true, // for now,
-            'screen'   => $this->get_admin_page()->get_current_screen()->id,
-        );
+	/**
+	 * Set initial properties
+	 */
+	protected function _set_properties()
+	{
+		parent::_set_properties();
+		$this->_wp_list_args = array(
+			'singular' => esc_html__('Message Template Group', 'event_espresso'),
+			'plural'   => esc_html__('Message Template', 'event_espresso'),
+			'ajax'     => true, // for now,
+			'screen'   => $this->get_admin_page()->get_current_screen()->id,
+		);
 
-        $this->_columns = array_merge(
-            array(
-                'cb'   => '<input type="checkbox" />',
-                'name' => esc_html__('Template Name', 'event_espresso'),
-            ),
-            $this->_columns,
-            array(
-                'events'  => esc_html__('Events', 'event_espresso'),
-                'actions' => '',
-            )
-        );
-    }
+		$this->_columns = array_merge(
+			array(
+				'cb'   => '<input type="checkbox" />',
+				'name' => esc_html__('Template Name', 'event_espresso'),
+			),
+			$this->_columns,
+			array(
+				'events'  => esc_html__('Events', 'event_espresso'),
+				'actions' => '',
+			)
+		);
+	}
 
 
-    /**
-     * Custom message for when there are no items found.
-     *
-     * @since 4.3.0
-     */
-    public function no_items()
-    {
-        if ($this->_view !== 'trashed') {
-            printf(
-                esc_html__(
-                    '%sNo Custom Templates found.%s To create your first custom message template, go to the "Default Message Templates" tab and click the "Create Custom" button next to the template you want to use as a base for the new one.',
-                    'event_espresso'
-                ),
-                '<strong>',
-                '</strong>'
-            );
-        } else {
-            parent::no_items();
-        }
-    }
+	/**
+	 * Custom message for when there are no items found.
+	 *
+	 * @since 4.3.0
+	 */
+	public function no_items()
+	{
+		if ($this->_view !== 'trashed') {
+			printf(
+				esc_html__(
+					'%sNo Custom Templates found.%s To create your first custom message template, go to the "Default Message Templates" tab and click the "Create Custom" button next to the template you want to use as a base for the new one.',
+					'event_espresso'
+				),
+				'<strong>',
+				'</strong>'
+			);
+		} else {
+			parent::no_items();
+		}
+	}
 
 
-    /**
-     * @param EE_Message_Template_Group $item
-     * @return string
-     */
-    public function column_cb($item)
-    {
-        return sprintf('<input type="checkbox" name="checkbox[%s]" value="1" />', $item->GRP_ID());
-    }
+	/**
+	 * @param EE_Message_Template_Group $item
+	 * @return string
+	 */
+	public function column_cb($item)
+	{
+		return sprintf('<input type="checkbox" name="checkbox[%s]" value="1" />', $item->GRP_ID());
+	}
 
 
-    /**
-     * @param EE_Message_Template_Group $item
-     * @return string
-     */
-    public function column_name($item)
-    {
-        return '<p>' . $item->name() . '</p>';
-    }
+	/**
+	 * @param EE_Message_Template_Group $item
+	 * @return string
+	 */
+	public function column_name($item)
+	{
+		return '<p>' . $item->name() . '</p>';
+	}
 
 
-    /**
-     * @param EE_Message_Template_Group $item
-     * @return string
-     */
-    public function column_actions($item)
-    {
-        if (EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_messages',
-            'espresso_messages_add_new_message_template'
-        )) {
-            $create_args = array(
-                'GRP_ID'       => $item->ID(),
-                'messenger'    => $item->messenger(),
-                'message_type' => $item->message_type(),
-                'action'       => 'add_new_message_template',
-            );
-            $create_link = EE_Admin_Page::add_query_args_and_nonce($create_args, EE_MSG_ADMIN_URL);
-            return sprintf(
-                '<p><a href="%s" class="button button-small">%s</a></p>',
-                $create_link,
-                esc_html__('Create Custom', 'event_espresso')
-            );
-        }
-        return '';
-    }
+	/**
+	 * @param EE_Message_Template_Group $item
+	 * @return string
+	 */
+	public function column_actions($item)
+	{
+		if (EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_messages',
+			'espresso_messages_add_new_message_template'
+		)) {
+			$create_args = array(
+				'GRP_ID'       => $item->ID(),
+				'messenger'    => $item->messenger(),
+				'message_type' => $item->message_type(),
+				'action'       => 'add_new_message_template',
+			);
+			$create_link = EE_Admin_Page::add_query_args_and_nonce($create_args, EE_MSG_ADMIN_URL);
+			return sprintf(
+				'<p><a href="%s" class="button button-small">%s</a></p>',
+				$create_link,
+				esc_html__('Create Custom', 'event_espresso')
+			);
+		}
+		return '';
+	}
 
-    /**
-     * Set the view counts on the _views property
-     */
-    protected function _add_view_counts()
-    {
-        foreach ($this->_views as $view => $args) {
-            $this->_views[ $view ]['count'] = $this->get_admin_page()->get_message_templates(
-                $this->_per_page,
-                $view,
-                true,
-                true,
-                false
-            );
-        }
-    }
+	/**
+	 * Set the view counts on the _views property
+	 */
+	protected function _add_view_counts()
+	{
+		foreach ($this->_views as $view => $args) {
+			$this->_views[ $view ]['count'] = $this->get_admin_page()->get_message_templates(
+				$this->_per_page,
+				$view,
+				true,
+				true,
+				false
+			);
+		}
+	}
 
 
-    /**
-     * column_events
-     * This provides a count of events using this custom template
-     *
-     * @param  EE_Message_Template_Group $item message_template group data
-     * @return string column output
-     */
-    public function column_events($item)
-    {
-        return $item->count_events();
-    }
+	/**
+	 * column_events
+	 * This provides a count of events using this custom template
+	 *
+	 * @param  EE_Message_Template_Group $item message_template group data
+	 * @return string column output
+	 */
+	public function column_events($item)
+	{
+		return $item->count_events();
+	}
 
 
-    /**
-     * Add additional actions for custom message template list view.
-     *
-     * @param EE_Message_Template_Group $item
-     * @return array
-     * @throws EE_Error
-     */
-    protected function _get_actions_for_messenger_column(EE_Message_Template_Group $item)
-    {
-        $actions = parent::_get_actions_for_messenger_column($item);
+	/**
+	 * Add additional actions for custom message template list view.
+	 *
+	 * @param EE_Message_Template_Group $item
+	 * @return array
+	 * @throws EE_Error
+	 */
+	protected function _get_actions_for_messenger_column(EE_Message_Template_Group $item)
+	{
+		$actions = parent::_get_actions_for_messenger_column($item);
 
-        // add additional actions for trash/restore etc.
-        $trash_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
-            'action'   => 'trash_message_template',
-            'id'       => $item->GRP_ID(),
-            'noheader' => true,
-        ), EE_MSG_ADMIN_URL);
-        // restore link
-        $restore_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
-            'action'   => 'restore_message_template',
-            'id'       => $item->GRP_ID(),
-            'noheader' => true,
-        ), EE_MSG_ADMIN_URL);
-        // delete price link
-        $delete_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
-            'action'   => 'delete_message_template',
-            'id'       => $item->GRP_ID(),
-            'noheader' => true,
-        ), EE_MSG_ADMIN_URL);
+		// add additional actions for trash/restore etc.
+		$trash_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
+			'action'   => 'trash_message_template',
+			'id'       => $item->GRP_ID(),
+			'noheader' => true,
+		), EE_MSG_ADMIN_URL);
+		// restore link
+		$restore_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
+			'action'   => 'restore_message_template',
+			'id'       => $item->GRP_ID(),
+			'noheader' => true,
+		), EE_MSG_ADMIN_URL);
+		// delete price link
+		$delete_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array(
+			'action'   => 'delete_message_template',
+			'id'       => $item->GRP_ID(),
+			'noheader' => true,
+		), EE_MSG_ADMIN_URL);
 
-        if (! $item->get('MTP_deleted')
-            && EE_Registry::instance()->CAP->current_user_can(
-                'ee_delete_message',
-                'espresso_messages_trash_message_template',
-                $item->ID()
-            )
-        ) {
-            $actions['trash'] = '<a href="'
-                                . $trash_lnk_url
-                                . '" title="'
-                                . esc_attr__('Move Template Group to Trash', 'event_espresso')
-                                . '">'
-                                . esc_html__('Move to Trash', 'event_espresso')
-                                . '</a>';
-        } else {
-            if (EE_Registry::instance()->CAP->current_user_can(
-                'ee_delete_message',
-                'espresso_messages_restore_message_template',
-                $item->ID()
-            )) {
-                $actions['restore'] = '<a href="'
-                                      . $restore_lnk_url
-                                      . '" title="'
-                                      . esc_attr__('Restore Message Template', 'event_espresso')
-                                      . '">'
-                                      . esc_html__('Restore', 'event_espresso') . '</a>';
-            }
+		if (! $item->get('MTP_deleted')
+			&& EE_Registry::instance()->CAP->current_user_can(
+				'ee_delete_message',
+				'espresso_messages_trash_message_template',
+				$item->ID()
+			)
+		) {
+			$actions['trash'] = '<a href="'
+								. $trash_lnk_url
+								. '" title="'
+								. esc_attr__('Move Template Group to Trash', 'event_espresso')
+								. '">'
+								. esc_html__('Move to Trash', 'event_espresso')
+								. '</a>';
+		} else {
+			if (EE_Registry::instance()->CAP->current_user_can(
+				'ee_delete_message',
+				'espresso_messages_restore_message_template',
+				$item->ID()
+			)) {
+				$actions['restore'] = '<a href="'
+									  . $restore_lnk_url
+									  . '" title="'
+									  . esc_attr__('Restore Message Template', 'event_espresso')
+									  . '">'
+									  . esc_html__('Restore', 'event_espresso') . '</a>';
+			}
 
-            if ($this->_view === 'trashed'
-                && EE_Registry::instance()->CAP->current_user_can(
-                    'ee_delete_message',
-                    'espresso_messages_delete_message_template',
-                    $item->ID()
-                )) {
-                $actions['delete'] = '<a href="'
-                                     . $delete_lnk_url
-                                     . '" title="'
-                                     . esc_attr__('Delete Template Group Permanently', 'event_espresso')
-                                     . '">'
-                                     . esc_html__('Delete Permanently', 'event_espresso')
-                                     . '</a>';
-            }
-        }
-        return $actions;
-    }
+			if ($this->_view === 'trashed'
+				&& EE_Registry::instance()->CAP->current_user_can(
+					'ee_delete_message',
+					'espresso_messages_delete_message_template',
+					$item->ID()
+				)) {
+				$actions['delete'] = '<a href="'
+									 . $delete_lnk_url
+									 . '" title="'
+									 . esc_attr__('Delete Template Group Permanently', 'event_espresso')
+									 . '">'
+									 . esc_html__('Delete Permanently', 'event_espresso')
+									 . '</a>';
+			}
+		}
+		return $actions;
+	}
 
 
-    /**
-     * Generate dropdown filter select input for messengers
-     *
-     * @param bool $global
-     * @return string
-     * @throws EE_Error
-     */
-    protected function _get_messengers_dropdown_filter($global = true)
-    {
-        return parent::_get_messengers_dropdown_filter(false);
-    }
+	/**
+	 * Generate dropdown filter select input for messengers
+	 *
+	 * @param bool $global
+	 * @return string
+	 * @throws EE_Error
+	 */
+	protected function _get_messengers_dropdown_filter($global = true)
+	{
+		return parent::_get_messengers_dropdown_filter(false);
+	}
 
 
-    /**
-     * Generate dropdown filter select input for message types
-     *
-     * @param bool $global
-     * @return string
-     * @throws EE_Error
-     */
-    protected function _get_message_types_dropdown_filter($global = true)
-    {
-        return parent::_get_message_types_dropdown_filter(false);
-    }
+	/**
+	 * Generate dropdown filter select input for message types
+	 *
+	 * @param bool $global
+	 * @return string
+	 * @throws EE_Error
+	 */
+	protected function _get_message_types_dropdown_filter($global = true)
+	{
+		return parent::_get_message_types_dropdown_filter(false);
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -99,7 +99,7 @@ 
      */
     public function column_name($item)
     {
-        return '<p>' . $item->name() . '</p>';
+        return '<p>'.$item->name().'</p>';
     }
 
 
@@ -135,7 +135,7 @@ 
     protected function _add_view_counts()
     {
         foreach ($this->_views as $view => $args) {
-            $this->_views[ $view ]['count'] = $this->get_admin_page()->get_message_templates(
+            $this->_views[$view]['count'] = $this->get_admin_page()->get_message_templates(
                 $this->_per_page,
                 $view,
                 true,
@@ -189,7 +189,7 @@ 
             'noheader' => true,
         ), EE_MSG_ADMIN_URL);
 
-        if (! $item->get('MTP_deleted')
+        if ( ! $item->get('MTP_deleted')
             && EE_Registry::instance()->CAP->current_user_can(
                 'ee_delete_message',
                 'espresso_messages_trash_message_template',
@@ -214,7 +214,7 @@ 
                                       . '" title="'
                                       . esc_attr__('Restore Message Template', 'event_espresso')
                                       . '">'
-                                      . esc_html__('Restore', 'event_espresso') . '</a>';
+                                      . esc_html__('Restore', 'event_espresso').'</a>';
             }
 
             if ($this->_view === 'trashed'

caffeinated/admin/extend/messages/espresso_events_Messages_Hooks_Extend.class.php

Indentation +267 added lines, -267 removed lines

@@ -12,276 +12,276 @@
  */
 class espresso_events_Messages_Hooks_Extend extends espresso_events_Messages_Hooks
 {
-    /**
-     * espresso_events_Messages_Hooks_Extend constructor.
-     *
-     * @param \EE_Admin_Page $admin_page
-     */
-    public function __construct(EE_Admin_Page $admin_page)
-    {
-        /**
-         * Add cap restriction ... metaboxes should not show if user does not have the ability to edit_custom_messages
-         */
-        if (! EE_Registry::instance()->CAP->current_user_can(
-            'ee_edit_messages',
-            'messages_events_editor_metabox'
-        )) {
-            return;
-        }
-        add_filter(
-            'FHEE__Events_Admin_Page___insert_update_cpt_item__event_update_callbacks',
-            array($this, 'caf_updates'),
-            10
-        );
-        add_action(
-            'AHEE__Extend_Events_Admin_Page___duplicate_event__after',
-            array($this, 'duplicate_custom_message_settings'),
-            10,
-            2
-        );
-        parent::__construct($admin_page);
-    }
-
-
-    /**
-     * extending the properties set in espresso_events_Messages_Hooks
-     *
-     * @access protected
-     * @return void
-     */
-    protected function _extend_properties()
-    {
-        define('EE_MSGS_EXTEND_ASSETS_URL', EE_CORE_CAF_ADMIN_EXTEND_URL . 'messages/assets/');
-        $this->_ajax_func = array(
-            'ee_msgs_create_new_custom' => 'create_new_custom',
-        );
-        $this->_metaboxes = array(
-            0 => array(
-                'page_route' => array('edit', 'create_new'),
-                'func'       => 'messages_metabox',
-                'label'      => esc_html__('Notifications', 'event_espresso'),
-                'priority'   => 'high',
-            ),
-        );
-
-        // see explanation for layout in EE_Admin_Hooks
-        $this->_scripts_styles = array(
-            'registers' => array(
-                'events_msg_admin'     => array(
-                    'url'     => EE_MSGS_EXTEND_ASSETS_URL . 'events_messages_admin.js',
-                    'depends' => array('ee-dialog', 'ee-parse-uri', 'ee-serialize-full-array'),
-                ),
-                'events_msg_admin_css' => array(
-                    'url'  => EE_MSGS_EXTEND_ASSETS_URL . 'ee_msg_events_admin.css',
-                    'type' => 'css',
-                ),
-            ),
-            'enqueues'  => array(
-                'events_msg_admin'     => array('edit', 'create_new'),
-                'events_msg_admin_css' => array('edit', 'create_new'),
-            ),
-        );
-    }
-
-
-    public function caf_updates($update_callbacks)
-    {
-        $update_callbacks[] = array($this, 'attach_evt_message_templates');
-        return $update_callbacks;
-    }
-
-
-    /**
-     * Handles attaching Message Templates to the Event on save.
-     *
-     * @param  EE_Event $event EE event object
-     * @param  array    $data  The request data from the form
-     * @return bool success or fail
-     * @throws EE_Error
-     */
-    public function attach_evt_message_templates($event, $data)
-    {
-        // first we remove all existing relations on the Event for message types.
-        $event->_remove_relations('Message_Template_Group');
-        // now let's just loop through the selected templates and add relations!
-        if (isset($data['event_message_templates_relation'])) {
-            foreach ($data['event_message_templates_relation'] as $grp_ID) {
-                $event->_add_relation_to($grp_ID, 'Message_Template_Group');
-            }
-        }
-        // now save
-        return $event->save();
-    }
-
-
-    /**
-     * @param $event
-     * @param $callback_args
-     * @return string
-     * @throws \EE_Error
-     */
-    public function messages_metabox($event, $callback_args)
-    {
-        // let's get the active messengers (b/c messenger objects have the active message templates)
-        // convert 'evt_id' to 'EVT_ID'
-        $this->_req_data['EVT_ID'] = isset($this->_req_data['EVT_ID']) ? $this->_req_data['EVT_ID'] : null;
-        $this->_req_data['EVT_ID'] = isset($this->_req_data['post']) && empty($this->_req_data['EVT_ID'])
-            ? $this->_req_data['post']
-            : $this->_req_data['EVT_ID'];
-
-        $this->_req_data['EVT_ID'] = empty($this->_req_data['EVT_ID']) && isset($this->_req_data['evt_id'])
-            ? $this->_req_data['evt_id']
-            : $this->_req_data['EVT_ID'];
-        /** @type EE_Message_Resource_Manager $message_resource_manager */
-        $message_resource_manager = EE_Registry::instance()->load_lib('Message_Resource_Manager');
-        $active_messengers = $message_resource_manager->active_messengers();
-        $tabs = array();
-
-        // empty messengers?
-        // Note message types will always have at least one available because every messenger has a default message type
-        // associated with it (payment) if no other message types are selected.
-        if (empty($active_messengers)) {
-            $msg_activate_url = EE_Admin_Page::add_query_args_and_nonce(
-                array('action' => 'settings'),
-                EE_MSG_ADMIN_URL
-            );
-            $error_msg = sprintf(
-                esc_html__(
-                    'There are no active messengers. So no notifications will go out for %1$sany%2$s events.  You will want to %3$sActivate a Messenger%4$s.',
-                    'event_espresso'
-                ),
-                '<strong>',
-                '</strong>',
-                '<a href="' . $msg_activate_url . '">',
-                '</a>'
-            );
-            $error_content = '<div class="error"><p>' . $error_msg . '</p></div>';
-            $internal_content = '<div id="messages-error"><p>' . $error_msg . '</p></div>';
-
-            echo $error_content;
-            echo $internal_content;
-            return '';
-        }
-
-        $event_id = isset($this->_req_data['EVT_ID']) ? $this->_req_data['EVT_ID'] : null;
-        // get content for active messengers
-        foreach ($active_messengers as $name => $messenger) {
-            // first check if there are any active message types for this messenger.
-            $active_mts = $message_resource_manager->get_active_message_types_for_messenger($name);
-            if (empty($active_mts)) {
-                continue;
-            }
-
-            $tab_content = $messenger->get_messenger_admin_page_content(
-                'events',
-                'edit',
-                array('event' => $event_id)
-            );
-
-            if (! empty($tab_content)) {
-                $tabs[ $name ] = $tab_content;
-            }
-        }
-
-
-        // we want this to be tabbed content so let's use the EEH_Tabbed_Content::display helper.
-        $tabbed_content = EEH_Tabbed_Content::display($tabs);
-        if ($tabbed_content instanceof WP_Error) {
-            $tabbed_content = $tabbed_content->get_error_message();
-        }
-
-        $notices = '
+	/**
+	 * espresso_events_Messages_Hooks_Extend constructor.
+	 *
+	 * @param \EE_Admin_Page $admin_page
+	 */
+	public function __construct(EE_Admin_Page $admin_page)
+	{
+		/**
+		 * Add cap restriction ... metaboxes should not show if user does not have the ability to edit_custom_messages
+		 */
+		if (! EE_Registry::instance()->CAP->current_user_can(
+			'ee_edit_messages',
+			'messages_events_editor_metabox'
+		)) {
+			return;
+		}
+		add_filter(
+			'FHEE__Events_Admin_Page___insert_update_cpt_item__event_update_callbacks',
+			array($this, 'caf_updates'),
+			10
+		);
+		add_action(
+			'AHEE__Extend_Events_Admin_Page___duplicate_event__after',
+			array($this, 'duplicate_custom_message_settings'),
+			10,
+			2
+		);
+		parent::__construct($admin_page);
+	}
+
+
+	/**
+	 * extending the properties set in espresso_events_Messages_Hooks
+	 *
+	 * @access protected
+	 * @return void
+	 */
+	protected function _extend_properties()
+	{
+		define('EE_MSGS_EXTEND_ASSETS_URL', EE_CORE_CAF_ADMIN_EXTEND_URL . 'messages/assets/');
+		$this->_ajax_func = array(
+			'ee_msgs_create_new_custom' => 'create_new_custom',
+		);
+		$this->_metaboxes = array(
+			0 => array(
+				'page_route' => array('edit', 'create_new'),
+				'func'       => 'messages_metabox',
+				'label'      => esc_html__('Notifications', 'event_espresso'),
+				'priority'   => 'high',
+			),
+		);
+
+		// see explanation for layout in EE_Admin_Hooks
+		$this->_scripts_styles = array(
+			'registers' => array(
+				'events_msg_admin'     => array(
+					'url'     => EE_MSGS_EXTEND_ASSETS_URL . 'events_messages_admin.js',
+					'depends' => array('ee-dialog', 'ee-parse-uri', 'ee-serialize-full-array'),
+				),
+				'events_msg_admin_css' => array(
+					'url'  => EE_MSGS_EXTEND_ASSETS_URL . 'ee_msg_events_admin.css',
+					'type' => 'css',
+				),
+			),
+			'enqueues'  => array(
+				'events_msg_admin'     => array('edit', 'create_new'),
+				'events_msg_admin_css' => array('edit', 'create_new'),
+			),
+		);
+	}
+
+
+	public function caf_updates($update_callbacks)
+	{
+		$update_callbacks[] = array($this, 'attach_evt_message_templates');
+		return $update_callbacks;
+	}
+
+
+	/**
+	 * Handles attaching Message Templates to the Event on save.
+	 *
+	 * @param  EE_Event $event EE event object
+	 * @param  array    $data  The request data from the form
+	 * @return bool success or fail
+	 * @throws EE_Error
+	 */
+	public function attach_evt_message_templates($event, $data)
+	{
+		// first we remove all existing relations on the Event for message types.
+		$event->_remove_relations('Message_Template_Group');
+		// now let's just loop through the selected templates and add relations!
+		if (isset($data['event_message_templates_relation'])) {
+			foreach ($data['event_message_templates_relation'] as $grp_ID) {
+				$event->_add_relation_to($grp_ID, 'Message_Template_Group');
+			}
+		}
+		// now save
+		return $event->save();
+	}
+
+
+	/**
+	 * @param $event
+	 * @param $callback_args
+	 * @return string
+	 * @throws \EE_Error
+	 */
+	public function messages_metabox($event, $callback_args)
+	{
+		// let's get the active messengers (b/c messenger objects have the active message templates)
+		// convert 'evt_id' to 'EVT_ID'
+		$this->_req_data['EVT_ID'] = isset($this->_req_data['EVT_ID']) ? $this->_req_data['EVT_ID'] : null;
+		$this->_req_data['EVT_ID'] = isset($this->_req_data['post']) && empty($this->_req_data['EVT_ID'])
+			? $this->_req_data['post']
+			: $this->_req_data['EVT_ID'];
+
+		$this->_req_data['EVT_ID'] = empty($this->_req_data['EVT_ID']) && isset($this->_req_data['evt_id'])
+			? $this->_req_data['evt_id']
+			: $this->_req_data['EVT_ID'];
+		/** @type EE_Message_Resource_Manager $message_resource_manager */
+		$message_resource_manager = EE_Registry::instance()->load_lib('Message_Resource_Manager');
+		$active_messengers = $message_resource_manager->active_messengers();
+		$tabs = array();
+
+		// empty messengers?
+		// Note message types will always have at least one available because every messenger has a default message type
+		// associated with it (payment) if no other message types are selected.
+		if (empty($active_messengers)) {
+			$msg_activate_url = EE_Admin_Page::add_query_args_and_nonce(
+				array('action' => 'settings'),
+				EE_MSG_ADMIN_URL
+			);
+			$error_msg = sprintf(
+				esc_html__(
+					'There are no active messengers. So no notifications will go out for %1$sany%2$s events.  You will want to %3$sActivate a Messenger%4$s.',
+					'event_espresso'
+				),
+				'<strong>',
+				'</strong>',
+				'<a href="' . $msg_activate_url . '">',
+				'</a>'
+			);
+			$error_content = '<div class="error"><p>' . $error_msg . '</p></div>';
+			$internal_content = '<div id="messages-error"><p>' . $error_msg . '</p></div>';
+
+			echo $error_content;
+			echo $internal_content;
+			return '';
+		}
+
+		$event_id = isset($this->_req_data['EVT_ID']) ? $this->_req_data['EVT_ID'] : null;
+		// get content for active messengers
+		foreach ($active_messengers as $name => $messenger) {
+			// first check if there are any active message types for this messenger.
+			$active_mts = $message_resource_manager->get_active_message_types_for_messenger($name);
+			if (empty($active_mts)) {
+				continue;
+			}
+
+			$tab_content = $messenger->get_messenger_admin_page_content(
+				'events',
+				'edit',
+				array('event' => $event_id)
+			);
+
+			if (! empty($tab_content)) {
+				$tabs[ $name ] = $tab_content;
+			}
+		}
+
+
+		// we want this to be tabbed content so let's use the EEH_Tabbed_Content::display helper.
+		$tabbed_content = EEH_Tabbed_Content::display($tabs);
+		if ($tabbed_content instanceof WP_Error) {
+			$tabbed_content = $tabbed_content->get_error_message();
+		}
+
+		$notices = '
 	<div id="espresso-ajax-loading" class="ajax-loader-grey">
 		<span class="ee-spinner ee-spin"></span><span class="hidden">'
-                   . esc_html__('loading...', 'event_espresso')
-                   . '</span>
+				   . esc_html__('loading...', 'event_espresso')
+				   . '</span>
 	</div>
 	<div class="ee-notices"></div>';
 
-        if (defined('DOING_AJAX')) {
-            return $tabbed_content;
-        }
-
-        do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__before_content');
-        echo $notices . '<div class="messages-tabs-content">' . $tabbed_content . '</div>';
-        do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__after_content');
-    }
-
-
-    /**
-     * Ajax callback for ee_msgs_create_new_custom ajax request.
-     * Takes incoming GRP_ID and name and description values from ajax request
-     * to create a new custom template based off of the incoming GRP_ID.
-     *
-     * @access public
-     * @return string either an html string will be returned or a success message
-     * @throws EE_Error
-     */
-    public function create_new_custom()
-    {
-        if (! EE_Registry::instance()->CAP->current_user_can('ee_edit_messages', 'create_new_custom_ajax')) {
-            wp_die(__('You don\'t have privileges to do this action', 'event_espresso'));
-        }
-
-        // let's clean up the _POST global a bit for downstream usage of name and description.
-        $_POST['templateName'] = ! empty($this->_req_data['custom_template_args']['MTP_name'])
-            ? $this->_req_data['custom_template_args']['MTP_name']
-            : '';
-        $_POST['templateDescription'] = ! empty($this->_req_data['custom_template_args']['MTP_description'])
-            ? $this->_req_data['custom_template_args']['MTP_description']
-            : '';
-
-
-        // set EE_Admin_Page object (see method details in EE_Admin_Hooks parent
-        $this->_set_page_object();
-
-        // is this a template switch if so EE_Admin_Page child needs this object
-        $this->_page_object->set_hook_object($this);
-
-        $this->_page_object->add_message_template(
-            $this->_req_data['messageType'],
-            $this->_req_data['messenger'],
-            $this->_req_data['group_ID']
-        );
-    }
-
-
-    public function create_new_admin_footer()
-    {
-        $this->edit_admin_footer();
-    }
-
-
-    /**
-     * This is the dynamic method for this class
-     * that will end up hooking into the 'admin_footer' hook on the 'edit_event' route in the events page.
-     *
-     * @return string
-     * @throws DomainException
-     */
-    public function edit_admin_footer()
-    {
-        EEH_Template::display_template(
-            EE_CORE_CAF_ADMIN_EXTEND . 'messages/templates/create_custom_template_form.template.php'
-        );
-    }
-
-
-    /**
-     * Callback for AHEE__Extend_Events_Admin_Page___duplicate_event__after hook used to ensure new events duplicate
-     * the assigned custom message templates.
-     *
-     * @param EE_Event $new_event
-     * @param EE_Event $original_event
-     * @throws EE_Error
-     */
-    public function duplicate_custom_message_settings(EE_Event $new_event, EE_Event $original_event)
-    {
-        $message_template_groups = $original_event->get_many_related('Message_Template_Group');
-        foreach ($message_template_groups as $message_template_group) {
-            $new_event->_add_relation_to($message_template_group, 'Message_Template_Group');
-        }
-        // save new event
-        $new_event->save();
-    }
+		if (defined('DOING_AJAX')) {
+			return $tabbed_content;
+		}
+
+		do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__before_content');
+		echo $notices . '<div class="messages-tabs-content">' . $tabbed_content . '</div>';
+		do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__after_content');
+	}
+
+
+	/**
+	 * Ajax callback for ee_msgs_create_new_custom ajax request.
+	 * Takes incoming GRP_ID and name and description values from ajax request
+	 * to create a new custom template based off of the incoming GRP_ID.
+	 *
+	 * @access public
+	 * @return string either an html string will be returned or a success message
+	 * @throws EE_Error
+	 */
+	public function create_new_custom()
+	{
+		if (! EE_Registry::instance()->CAP->current_user_can('ee_edit_messages', 'create_new_custom_ajax')) {
+			wp_die(__('You don\'t have privileges to do this action', 'event_espresso'));
+		}
+
+		// let's clean up the _POST global a bit for downstream usage of name and description.
+		$_POST['templateName'] = ! empty($this->_req_data['custom_template_args']['MTP_name'])
+			? $this->_req_data['custom_template_args']['MTP_name']
+			: '';
+		$_POST['templateDescription'] = ! empty($this->_req_data['custom_template_args']['MTP_description'])
+			? $this->_req_data['custom_template_args']['MTP_description']
+			: '';
+
+
+		// set EE_Admin_Page object (see method details in EE_Admin_Hooks parent
+		$this->_set_page_object();
+
+		// is this a template switch if so EE_Admin_Page child needs this object
+		$this->_page_object->set_hook_object($this);
+
+		$this->_page_object->add_message_template(
+			$this->_req_data['messageType'],
+			$this->_req_data['messenger'],
+			$this->_req_data['group_ID']
+		);
+	}
+
+
+	public function create_new_admin_footer()
+	{
+		$this->edit_admin_footer();
+	}
+
+
+	/**
+	 * This is the dynamic method for this class
+	 * that will end up hooking into the 'admin_footer' hook on the 'edit_event' route in the events page.
+	 *
+	 * @return string
+	 * @throws DomainException
+	 */
+	public function edit_admin_footer()
+	{
+		EEH_Template::display_template(
+			EE_CORE_CAF_ADMIN_EXTEND . 'messages/templates/create_custom_template_form.template.php'
+		);
+	}
+
+
+	/**
+	 * Callback for AHEE__Extend_Events_Admin_Page___duplicate_event__after hook used to ensure new events duplicate
+	 * the assigned custom message templates.
+	 *
+	 * @param EE_Event $new_event
+	 * @param EE_Event $original_event
+	 * @throws EE_Error
+	 */
+	public function duplicate_custom_message_settings(EE_Event $new_event, EE_Event $original_event)
+	{
+		$message_template_groups = $original_event->get_many_related('Message_Template_Group');
+		foreach ($message_template_groups as $message_template_group) {
+			$new_event->_add_relation_to($message_template_group, 'Message_Template_Group');
+		}
+		// save new event
+		$new_event->save();
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -22,7 +22,7 @@ 
         /**
          * Add cap restriction ... metaboxes should not show if user does not have the ability to edit_custom_messages
          */
-        if (! EE_Registry::instance()->CAP->current_user_can(
+        if ( ! EE_Registry::instance()->CAP->current_user_can(
             'ee_edit_messages',
             'messages_events_editor_metabox'
         )) {
@@ -51,7 +51,7 @@ 
      */
     protected function _extend_properties()
     {
-        define('EE_MSGS_EXTEND_ASSETS_URL', EE_CORE_CAF_ADMIN_EXTEND_URL . 'messages/assets/');
+        define('EE_MSGS_EXTEND_ASSETS_URL', EE_CORE_CAF_ADMIN_EXTEND_URL.'messages/assets/');
         $this->_ajax_func = array(
             'ee_msgs_create_new_custom' => 'create_new_custom',
         );
@@ -68,11 +68,11 @@ 
         $this->_scripts_styles = array(
             'registers' => array(
                 'events_msg_admin'     => array(
-                    'url'     => EE_MSGS_EXTEND_ASSETS_URL . 'events_messages_admin.js',
+                    'url'     => EE_MSGS_EXTEND_ASSETS_URL.'events_messages_admin.js',
                     'depends' => array('ee-dialog', 'ee-parse-uri', 'ee-serialize-full-array'),
                 ),
                 'events_msg_admin_css' => array(
-                    'url'  => EE_MSGS_EXTEND_ASSETS_URL . 'ee_msg_events_admin.css',
+                    'url'  => EE_MSGS_EXTEND_ASSETS_URL.'ee_msg_events_admin.css',
                     'type' => 'css',
                 ),
             ),
@@ -152,11 +152,11 @@ 
                 ),
                 '<strong>',
                 '</strong>',
-                '<a href="' . $msg_activate_url . '">',
+                '<a href="'.$msg_activate_url.'">',
                 '</a>'
             );
-            $error_content = '<div class="error"><p>' . $error_msg . '</p></div>';
-            $internal_content = '<div id="messages-error"><p>' . $error_msg . '</p></div>';
+            $error_content = '<div class="error"><p>'.$error_msg.'</p></div>';
+            $internal_content = '<div id="messages-error"><p>'.$error_msg.'</p></div>';
 
             echo $error_content;
             echo $internal_content;
@@ -178,8 +178,8 @@ 
                 array('event' => $event_id)
             );
 
-            if (! empty($tab_content)) {
-                $tabs[ $name ] = $tab_content;
+            if ( ! empty($tab_content)) {
+                $tabs[$name] = $tab_content;
             }
         }
 
@@ -203,7 +203,7 @@ 
         }
 
         do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__before_content');
-        echo $notices . '<div class="messages-tabs-content">' . $tabbed_content . '</div>';
+        echo $notices.'<div class="messages-tabs-content">'.$tabbed_content.'</div>';
         do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__after_content');
     }
 
@@ -219,7 +219,7 @@ 
      */
     public function create_new_custom()
     {
-        if (! EE_Registry::instance()->CAP->current_user_can('ee_edit_messages', 'create_new_custom_ajax')) {
+        if ( ! EE_Registry::instance()->CAP->current_user_can('ee_edit_messages', 'create_new_custom_ajax')) {
             wp_die(__('You don\'t have privileges to do this action', 'event_espresso'));
         }
 
@@ -262,7 +262,7 @@ 
     public function edit_admin_footer()
     {
         EEH_Template::display_template(
-            EE_CORE_CAF_ADMIN_EXTEND . 'messages/templates/create_custom_template_form.template.php'
+            EE_CORE_CAF_ADMIN_EXTEND.'messages/templates/create_custom_template_form.template.php'
         );
     }