| @@ -7,13 +7,16 @@ | ||
| 7 | 7 | * @author Tobias Bäthge |
| 8 | 8 | * @since 1.0.0 |
| 9 | 9 | */ |
| 10 | 10 | |
| 11 | +declare(strict_types=1); | |
| 12 | + | |
| 11 | 13 | // Prohibit direct script loading. |
| 12 | 14 | defined( 'ABSPATH' ) || die( 'No direct script access allowed!' ); |
| 13 | 15 | |
| 14 | 16 | /** |
| 15 | 17 | * TablePress Base View class |
| 18 | + * | |
| 16 | 19 | * @package TablePress |
| 17 | 20 | * @subpackage Views |
| 18 | 21 | * @author Tobias Bäthge |
| 19 | 22 | * @since 1.0.0 |
| @@ -23,51 +26,41 @@ | ||
| 23 | 26 | /** |
| 24 | 27 | * Data for the view. |
| 25 | 28 | * |
| 26 | 29 | * @since 1.0.0 |
| 27 | - * @var array | |
| 30 | + * @var array<string, mixed> | |
| 28 | 31 | */ |
| 29 | - protected $data = array(); | |
| 32 | + protected array $data = array(); | |
| 30 | 33 | |
| 31 | 34 | /** |
| 32 | 35 | * Number of screen columns for post boxes. |
| 33 | 36 | * |
| 34 | 37 | * @since 1.0.0 |
| 35 | - * @var int | |
| 36 | 38 | */ |
| 37 | - protected $screen_columns = 0; | |
| 39 | + protected int $screen_columns = 0; | |
| 38 | 40 | |
| 39 | 41 | /** |
| 40 | 42 | * User action for this screen. |
| 41 | 43 | * |
| 42 | 44 | * @since 1.0.0 |
| 43 | - * @var string | |
| 44 | 45 | */ |
| 45 | - protected $action = ''; | |
| 46 | + protected string $action = ''; | |
| 46 | 47 | |
| 47 | 48 | /** |
| 48 | - * Instance of the Admin Page Helper Class, with necessary functions. | |
| 49 | - * | |
| 50 | - * @since 1.0.0 | |
| 51 | - * @var TablePress_Admin_Page | |
| 52 | - */ | |
| 53 | - protected $admin_page; | |
| 54 | - | |
| 55 | - /** | |
| 56 | 49 | * List of text boxes (similar to post boxes, but just with text and without extra functionality). |
| 57 | 50 | * |
| 58 | 51 | * @since 1.0.0 |
| 59 | - * @var array | |
| 52 | + * @var array<string, array<string, array<string, mixed>>> | |
| 60 | 53 | */ |
| 61 | - protected $textboxes = array(); | |
| 54 | + protected array $textboxes = array(); | |
| 62 | 55 | |
| 63 | 56 | /** |
| 64 | 57 | * List of messages that are to be displayed as boxes below the page title. |
| 65 | 58 | * |
| 66 | 59 | * @since 1.0.0 |
| 67 | - * @var array | |
| 60 | + * @var string[] | |
| 68 | 61 | */ |
| 69 | - protected $header_messages = array(); | |
| 62 | + protected array $header_messages = array(); | |
| 70 | 63 | |
| 71 | 64 | /** |
| 72 | 65 | * Whether there are post boxes registered for this screen, |
| 73 | 66 | * is automatically set to true, when a meta box is added. |
| @@ -72,22 +65,21 @@ | ||
| 72 | 65 | * Whether there are post boxes registered for this screen, |
| 73 | 66 | * is automatically set to true, when a meta box is added. |
| 74 | 67 | * |
| 75 | 68 | * @since 1.0.0 |
| 76 | - * @var bool | |
| 77 | 69 | */ |
| 78 | - protected $has_meta_boxes = false; | |
| 70 | + protected bool $has_meta_boxes = false; | |
| 79 | 71 | |
| 80 | 72 | /** |
| 81 | 73 | * List of WP feature pointers for this view. |
| 82 | 74 | * |
| 83 | 75 | * @since 1.0.0 |
| 84 | - * @var array | |
| 76 | + * @var string[] | |
| 85 | 77 | */ |
| 86 | - protected $wp_pointers = array(); | |
| 78 | + protected array $wp_pointers = array(); | |
| 87 | 79 | |
| 88 | 80 | /** |
| 89 | - * Initialize the View class, by setting the correct screen columns and adding help texts. | |
| 81 | + * Initializes the View class, by setting the correct screen columns and adding help texts. | |
| 90 | 82 | * |
| 91 | 83 | * @since 1.0.0 |
| 92 | 84 | */ |
| 93 | 85 | public function __construct() { |
| @@ -92,77 +84,92 @@ | ||
| 92 | 84 | */ |
| 93 | 85 | public function __construct() { |
| 94 | 86 | $screen = get_current_screen(); |
| 95 | 87 | if ( 0 !== $this->screen_columns ) { |
| 96 | - $screen->add_option( 'layout_columns', array( 'max' => $this->screen_columns ) ); | |
| 88 | + $screen->add_option( 'layout_columns', array( 'max' => $this->screen_columns ) ); // @phpstan-ignore method.nonObject | |
| 97 | 89 | } |
| 98 | 90 | // Enable two column layout. |
| 99 | - add_filter( "get_user_option_screen_layout_{$screen->id}", array( $this, 'set_current_screen_layout_columns' ) ); | |
| 91 | + add_filter( "get_user_option_screen_layout_{$screen->id}", array( $this, 'set_current_screen_layout_columns' ) ); // @phpstan-ignore property.nonObject | |
| 100 | 92 | |
| 101 | - $screen->add_help_tab( array( | |
| 93 | + /* translators: %1$s: URL to TablePress website, %2$s: URL to WordPress Plugin Directory */ | |
| 94 | + $common_content = '<p>' . sprintf( __( 'More information about TablePress can be found on the <a href="%1$s">plugin website</a> or on its page in the <a href="%2$s">WordPress Plugin Directory</a>.', 'tablepress' ), 'https://tablepress.org/', 'https://wordpress.org/plugins/tablepress/' ) . '</p>'; | |
| 95 | + /* translators: %s: URL to Documentation page */ | |
| 96 | + $common_content .= '<p>' . sprintf( __( 'For technical information, please see the <a href="%s">Documentation</a>.', 'tablepress' ), 'https://tablepress.org/documentation/' ) . ' '; | |
| 97 | + /* translators: %s: URL to FAQ page */ | |
| 98 | + $common_content .= sprintf( __( 'Common questions are answered in the <a href="%s">FAQ</a>.', 'tablepress' ), 'https://tablepress.org/faq/' ) . '</p>'; | |
| 99 | + | |
| 100 | + if ( tb_tp_fs()->is_free_plan() ) { | |
| 101 | + $common_content .= '<p>' | |
| 102 | + . sprintf( __( '<a href="%1$s">Support</a> is provided through the <a href="%2$s">WordPress Support Forums</a>.', 'tablepress' ), 'https://tablepress.org/support/', 'https://wordpress.org/support/plugin/tablepress/' ) | |
| 103 | + . ' ' | |
| 104 | + . sprintf( __( 'Before asking for support, please carefully read the <a href="%s">Frequently Asked Questions</a>, where you will find answers to the most common questions, and search through the forums.', 'tablepress' ), 'https://tablepress.org/faq/' ) | |
| 105 | + . '</p>'; | |
| 106 | + $common_content .= '<p><strong>' . sprintf( __( 'More great features for you and your site’s visitors and priority email support are available with a Premium license plan of TablePress. <a href="%s">Go check them out!</a>', 'tablepress' ), 'https://tablepress.org/premium/?utm_source=plugin&utm_medium=textlink&utm_content=help-tab' ) . '</strong></p>'; | |
| 107 | + } | |
| 108 | + | |
| 109 | + $screen->add_help_tab( array( // @phpstan-ignore method.nonObject | |
| 102 | 110 | 'id' => 'tablepress-help', // This should be unique for the screen. |
| 103 | 111 | 'title' => __( 'TablePress Help', 'tablepress' ), |
| 104 | - 'content' => '<p>' . $this->help_tab_content() . '</p>' | |
| 105 | - . '<p>' . sprintf( __( 'More information about TablePress can be found on the <a href="%1$s">plugin website</a> or on its page in the <a href="%2$s">WordPress Plugin Directory</a>.', 'tablepress' ), 'https://tablepress.org/', 'https://wordpress.org/plugins/tablepress/' ) . ' ' | |
| 106 | - . sprintf( __( 'For technical information, please see the <a href="%s">Documentation</a>.', 'tablepress' ), 'https://tablepress.org/documentation/' ) . ' ' | |
| 107 | - . sprintf( __( '<a href="%1$s">Support</a> is provided through the <a href="%2$s">WordPress Support Forums</a>.', 'tablepress' ), 'https://tablepress.org/support/', 'https://wordpress.org/tags/tablepress' ) . ' ' | |
| 108 | - . sprintf( __( 'Before asking for support, please carefully read the <a href="%s">Frequently Asked Questions</a>, where you will find answers to the most common questions, and search through the forums.', 'tablepress' ), 'https://tablepress.org/faq/' ) . '<br />' | |
| 109 | - . sprintf( __( 'If you like the plugin, <a href="%1$s"><strong>a donation</strong></a> is recommended.', 'tablepress' ), 'https://tablepress.org/donate/' ) . '</p>', | |
| 112 | + 'content' => '<p>' . $this->help_tab_content() . '</p>' . $common_content, | |
| 110 | 113 | ) ); |
| 111 | - // "sidebar" in the help tab. | |
| 112 | - $screen->set_help_sidebar( '<p><strong>' . __( 'For more information:', 'tablepress' ) . '</strong></p><p><a href="https://tablepress.org/">TablePress Website</a></p><p><a href="https://tablepress.org/faq/">TablePress FAQ</a></p><p><a href="https://tablepress.org/documentation/">TablePress Documentation</a></p><p><a href="https://tablepress.org/support/">TablePress Support</a></p>' ); | |
| 114 | + // "Sidebar" in the help tab. | |
| 115 | + $screen->set_help_sidebar( // @phpstan-ignore method.nonObject | |
| 116 | + '<p><strong>' . __( 'For more information:', 'tablepress' ) . '</strong></p>' | |
| 117 | + . '<p><a href="https://tablepress.org/">TablePress Website</a></p>' | |
| 118 | + . '<p><a href="https://tablepress.org/faq/">TablePress FAQ</a></p>' | |
| 119 | + . '<p><a href="https://tablepress.org/documentation/">TablePress Documentation</a></p>' | |
| 120 | + . '<p><a href="https://tablepress.org/support/">TablePress Support</a></p>' | |
| 121 | + ); | |
| 113 | 122 | } |
| 114 | 123 | |
| 115 | 124 | /** |
| 116 | - * Change the value of the user option "screen_layout_{$screen->id}" through a filter. | |
| 125 | + * Changes the value of the user option "screen_layout_{$screen->id}" through a filter. | |
| 117 | 126 | * |
| 118 | 127 | * @since 1.0.0 |
| 119 | 128 | * |
| 120 | - * @param int|bool Current value of the user option. | |
| 129 | + * @param string|int|false $result Current value of the user option. | |
| 121 | 130 | * @return int New value for the user option. |
| 122 | 131 | */ |
| 123 | - public function set_current_screen_layout_columns( $result ) { | |
| 132 | + public function set_current_screen_layout_columns( /* string|int|false */ $result ): int { | |
| 124 | 133 | if ( false === $result ) { |
| 125 | 134 | // The user option does not yet exist. |
| 126 | - $result = $this->screen_columns; | |
| 127 | - } elseif ( $result > $this->screen_columns ) { | |
| 135 | + return $this->screen_columns; | |
| 136 | + } | |
| 137 | + | |
| 138 | + $result = (int) $result; | |
| 139 | + if ( $result > $this->screen_columns ) { | |
| 128 | 140 | // The value of the user option is bigger than what is possible on this screen (e.g. because the number of columns was reduced in an update). |
| 129 | 141 | $result = $this->screen_columns; |
| 130 | 142 | } |
| 143 | + | |
| 131 | 144 | return $result; |
| 132 | 145 | } |
| 133 | 146 | |
| 134 | 147 | /** |
| 135 | - * Set up the view with data and do things that are necessary for all views. | |
| 148 | + * Sets up the view with data and do things that are necessary for all views. | |
| 136 | 149 | * |
| 137 | 150 | * @since 1.0.0 |
| 138 | 151 | * |
| 139 | - * @param string $action Action for this view. | |
| 140 | - * @param array $data Data for this view. | |
| 152 | + * @param string $action Action for this view. | |
| 153 | + * @param array<string, mixed> $data Data for this view. | |
| 141 | 154 | */ |
| 142 | - public function setup( $action, array $data ) { | |
| 155 | + public function setup( /* string */ $action, array $data ) /* : void */ { | |
| 156 | + // Don't use type hints (except array $data) in method declaration, as the method is extended in some TablePress Extensions which are no longer updated. | |
| 157 | + | |
| 143 | 158 | $this->action = $action; |
| 144 | 159 | $this->data = $data; |
| 145 | 160 | |
| 146 | 161 | // Set page title. |
| 147 | - $GLOBALS['title'] = sprintf( __( '%1$s ‹ %2$s', 'tablepress' ), $this->data['view_actions'][ $this->action ]['page_title'], 'TablePress' ); | |
| 162 | + $GLOBALS['title'] = sprintf( __( '%1$s ‹ %2$s', 'tablepress' ), $this->data['view_actions'][ $this->action ]['page_title'], 'TablePress' ); // phpcs:ignore WordPress.WP.GlobalVariablesOverride.Prohibited | |
| 148 | 163 | |
| 149 | - // Admin page helpers, like script/style loading, could be moved to view. | |
| 150 | - $this->admin_page = TablePress::load_class( 'TablePress_Admin_Page', 'class-admin-page-helper.php', 'classes' ); | |
| 151 | - $this->admin_page->enqueue_style( 'common' ); | |
| 164 | + add_filter( 'admin_footer_text', array( $this, 'add_admin_footer_text' ) ); | |
| 165 | + | |
| 166 | + TablePress::enqueue_style( 'common', array( 'wp-components' ) ); | |
| 152 | 167 | // RTL styles for the admin interface. |
| 153 | 168 | if ( is_rtl() ) { |
| 154 | - $this->admin_page->enqueue_style( 'common-rtl', array( 'tablepress-common' ) ); | |
| 169 | + TablePress::enqueue_style( 'common-rtl', array( 'tablepress-common' ) ); | |
| 155 | 170 | } |
| 156 | - $this->admin_page->enqueue_script( 'common', array( 'jquery', 'postbox' ), array( | |
| 157 | - 'common' => array( | |
| 158 | - 'ays_delete_single_table' => _n( 'Do you really want to delete this table?', 'Do you really want to delete these tables?', 1, 'tablepress' ), | |
| 159 | - 'ays_delete_multiple_tables' => _n( 'Do you really want to delete this table?', 'Do you really want to delete these tables?', 2, 'tablepress' ), | |
| 160 | - ) | |
| 161 | - ) ); | |
| 162 | 171 | |
| 163 | - $this->admin_page->add_admin_footer_text(); | |
| 164 | - | |
| 165 | 172 | // Initialize WP feature pointers for TablePress. |
| 166 | 173 | $this->_init_wp_pointers(); |
| 167 | 174 | |
| 168 | 175 | // Necessary fields for all views. |
| @@ -171,54 +178,60 @@ | ||
| 171 | 178 | $this->add_text_box( 'action_field', array( $this, 'action_field' ), 'header', false ); |
| 172 | 179 | } |
| 173 | 180 | |
| 174 | 181 | /** |
| 175 | - * Register a header message for the view. | |
| 182 | + * Registers a header message for the view. | |
| 176 | 183 | * |
| 177 | 184 | * @since 1.0.0 |
| 178 | 185 | * |
| 179 | - * @param string $text Text for the header message. | |
| 180 | - * @param string $class Optional. Additional CSS class for the header message. | |
| 181 | - * @param string $title Optional. Text for the header title. | |
| 186 | + * @param string $text Text for the header message. | |
| 187 | + * @param string $css_class Optional. Additional CSS class for the header message. | |
| 188 | + * @param string $title Optional. Text for the header title. | |
| 182 | 189 | */ |
| 183 | - protected function add_header_message( $text, $class = 'notice-success', $title = '' ) { | |
| 184 | - if ( ! stripos( $class, 'not-dismissible' ) ) { | |
| 185 | - $class .= ' is-dismissible'; | |
| 190 | + protected function add_header_message( string $text, string $css_class = 'is-success notice-success', string $title = '' ): void { | |
| 191 | + if ( ! str_contains( $css_class, 'not-dismissible' ) ) { | |
| 192 | + $css_class .= ' is-dismissible'; | |
| 186 | 193 | } |
| 187 | - if ( ! empty( $title ) ) { | |
| 194 | + if ( '' !== $title ) { | |
| 188 | 195 | $title = "<h3>{$title}</h3>"; |
| 189 | 196 | } |
| 190 | - if ( ! empty( $text ) ) { | |
| 197 | + // Wrap the message text in HTML <p> tags if it does not already start with one (potentially with attributes), indicating custom message HTML. | |
| 198 | + if ( '' !== $text && ! str_starts_with( $text, '<p' ) ) { | |
| 191 | 199 | $text = "<p>{$text}</p>"; |
| 192 | 200 | } |
| 193 | - $this->header_messages[] = "<div class=\"notice {$class}\">{$title}{$text}</div>\n"; | |
| 201 | + $this->header_messages[] = "<div class=\"notice components-notice {$css_class}\"><div class=\"components-notice__content\">{$title}{$text}</div></div>\n"; | |
| 194 | 202 | } |
| 195 | 203 | |
| 196 | 204 | /** |
| 197 | - * Process header action messages, i.e. check if a message should be added to the page. | |
| 205 | + * Processes header action messages, i.e. check if a message should be added to the page. | |
| 198 | 206 | * |
| 199 | 207 | * @since 1.0.0 |
| 200 | 208 | * |
| 201 | - * @param array $action_messages Action messages for the screen. | |
| 209 | + * @param array<string, string> $action_messages Action messages for the screen. | |
| 202 | 210 | */ |
| 203 | - protected function process_action_messages( array $action_messages ) { | |
| 211 | + protected function process_action_messages( array $action_messages ): void { | |
| 204 | 212 | if ( $this->data['message'] && isset( $action_messages[ $this->data['message'] ] ) ) { |
| 205 | - $class = ( 'error' === substr( $this->data['message'], 0, 5 ) ) ? 'notice-error' : 'notice-success'; | |
| 206 | - $this->add_header_message( "<strong>{$action_messages[ $this->data['message'] ]}</strong>", $class ); | |
| 213 | + $class = ( str_starts_with( $this->data['message'], 'error' ) ) ? 'is-error notice-error' : 'is-success notice-success'; | |
| 214 | + | |
| 215 | + if ( '' !== $this->data['error_details'] ) { | |
| 216 | + $this->data['error_details'] = '</p><p>' . sprintf( __( 'Error code: %s', 'tablepress' ), '<code>' . esc_html( $this->data['error_details'] ) . '</code>' ); | |
| 217 | + } | |
| 218 | + | |
| 219 | + $this->add_header_message( "<strong>{$action_messages[ $this->data['message'] ]}</strong>{$this->data['error_details']}", $class ); | |
| 207 | 220 | } |
| 208 | 221 | } |
| 209 | 222 | |
| 210 | 223 | /** |
| 211 | - * Register a text box for the view. | |
| 224 | + * Registers a text box for the view. | |
| 212 | 225 | * |
| 213 | 226 | * @since 1.0.0 |
| 214 | 227 | * |
| 215 | 228 | * @param string $id Unique HTML ID for the text box container (only visible with $wrap = true). |
| 216 | - * @param callback $callback Callback that prints the contents of the text box. | |
| 229 | + * @param callable $callback Callback that prints the contents of the text box. | |
| 217 | 230 | * @param string $context Optional. Context/position of the text box (normal, side, additional, header, submit). |
| 218 | - * @param bool $ wrap Whether the content of the text box shall be wrapped in a <div> container. | |
| 231 | + * @param bool $wrap Whether the content of the text box shall be wrapped in a <div> container. | |
| 219 | 232 | */ |
| 220 | - protected function add_text_box( $id, $callback, $context = 'normal', $wrap = false ) { | |
| 233 | + protected function add_text_box( string $id, callable $callback, string $context = 'normal', bool $wrap = false ): void { | |
| 221 | 234 | if ( ! isset( $this->textboxes[ $context ] ) ) { |
| 222 | 235 | $this->textboxes[ $context ] = array(); |
| 223 | 236 | } |
| 224 | 237 | |
| @@ -231,32 +244,32 @@ | ||
| 231 | 244 | ); |
| 232 | 245 | } |
| 233 | 246 | |
| 234 | 247 | /** |
| 235 | - * Register a post meta box for the view, that is drag/droppable with WordPress functionality. | |
| 248 | + * Registers a post meta box for the view, that is drag/droppable with WordPress functionality. | |
| 236 | 249 | * |
| 237 | 250 | * @since 1.0.0 |
| 238 | 251 | * |
| 239 | - * @param string $id Unique ID for the meta box. | |
| 240 | - * @param string $title Title for the meta box. | |
| 241 | - * @param callback $callback Callback that prints the contents of the post meta box. | |
| 242 | - * @param string $context Optional. Context/position of the post meta box (normal, side, additional). | |
| 243 | - * @param string $priority Optional. Order of the post meta box for the $context position (high, default, low). | |
| 244 | - * @param bool $callback_args Optional. Additional data for the callback function (e.g. useful when in different class). | |
| 252 | + * @param string $id Unique ID for the meta box. | |
| 253 | + * @param string $title Title for the meta box. | |
| 254 | + * @param callable $callback Callback that prints the contents of the post meta box. | |
| 255 | + * @param 'normal'|'side'|'additional' $context Optional. Context/position of the post meta box (normal, side, additional). | |
| 256 | + * @param 'core'|'default'|'high'|'low' $priority Optional. Order of the post meta box for the $context position (high, default, low). | |
| 257 | + * @param mixed[]|null $callback_args Optional. Additional data for the callback function (e.g. useful when in different class). | |
| 245 | 258 | */ |
| 246 | - protected function add_meta_box( $id, $title, $callback, $context = 'normal', $priority = 'default', $callback_args = null ) { | |
| 259 | + protected function add_meta_box( string $id, string $title, callable $callback, string $context = 'normal', string $priority = 'default', ?array $callback_args = null ): void { | |
| 247 | 260 | $this->has_meta_boxes = true; |
| 248 | 261 | add_meta_box( "tablepress_{$this->action}-{$id}", $title, $callback, null, $context, $priority, $callback_args ); |
| 249 | 262 | } |
| 250 | 263 | |
| 251 | 264 | /** |
| 252 | - * Render all text boxes for the given context. | |
| 265 | + * Renders all text boxes for the given context. | |
| 253 | 266 | * |
| 254 | 267 | * @since 1.0.0 |
| 255 | 268 | * |
| 256 | 269 | * @param string $context Context (normal, side, additional, header, submit) for which registered text boxes shall be rendered. |
| 257 | 270 | */ |
| 258 | - protected function do_text_boxes( $context ) { | |
| 271 | + protected function do_text_boxes( string $context ): void { | |
| 259 | 272 | if ( empty( $this->textboxes[ $context ] ) ) { |
| 260 | 273 | return; |
| 261 | 274 | } |
| 262 | 275 | |
| @@ -271,32 +284,31 @@ | ||
| 271 | 284 | } |
| 272 | 285 | } |
| 273 | 286 | |
| 274 | 287 | /** |
| 275 | - * Render all post meta boxes for the given context, if there are post meta boxes. | |
| 288 | + * Renders all post meta boxes for the given context, if there are post meta boxes. | |
| 276 | 289 | * |
| 277 | 290 | * @since 1.0.0 |
| 278 | 291 | * |
| 279 | 292 | * @param string $context Context (normal, side, additional) for which registered post meta boxes shall be rendered. |
| 280 | 293 | */ |
| 281 | - protected function do_meta_boxes( $context ) { | |
| 282 | - if ( ! $this->has_meta_boxes ) { | |
| 283 | - return; | |
| 294 | + protected function do_meta_boxes( string $context ): void { | |
| 295 | + if ( $this->has_meta_boxes ) { | |
| 296 | + do_meta_boxes( get_current_screen(), $context, $this->data ); // @phpstan-ignore argument.type | |
| 284 | 297 | } |
| 285 | - do_meta_boxes( null, $context, $this->data ); | |
| 286 | 298 | } |
| 287 | 299 | |
| 288 | 300 | /** |
| 289 | - * Print hidden fields with nonces for post meta box AJAX handling, if there are post meta boxes on the screen. | |
| 301 | + * Prints hidden fields with nonces for post meta box AJAX handling, if there are post meta boxes on the screen. | |
| 290 | 302 | * |
| 291 | 303 | * The check is possible as this function is executed after post meta boxes have to be registered. |
| 292 | 304 | * |
| 293 | 305 | * @since 1.0.0 |
| 294 | 306 | * |
| 295 | - * @param array $data Data for this screen. | |
| 296 | - * @param array $box Information about the text box. | |
| 307 | + * @param array<string, mixed> $data Data for this screen. | |
| 308 | + * @param array<string, mixed> $box Information about the text box. | |
| 297 | 309 | */ |
| 298 | - protected function default_nonce_fields( array $data, array $box ) { | |
| 310 | + protected function default_nonce_fields( array $data, array $box ): void { | |
| 299 | 311 | if ( ! $this->has_meta_boxes ) { |
| 300 | 312 | return; |
| 301 | 313 | } |
| 302 | 314 | wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false ); |
| @@ -305,54 +317,60 @@ | ||
| 305 | 317 | echo "\n"; |
| 306 | 318 | } |
| 307 | 319 | |
| 308 | 320 | /** |
| 309 | - * Print hidden field with a nonce for the screen's action, to be transmitted in HTTP requests. | |
| 321 | + * Prints hidden field with a nonce for the screen's action, to be transmitted in HTTP requests. | |
| 310 | 322 | * |
| 311 | 323 | * @since 1.0.0 |
| 312 | 324 | * |
| 313 | - * @param array $data Data for this screen. | |
| 314 | - * @param array $box Information about the text box. | |
| 325 | + * @param array<string, mixed> $data Data for this screen. | |
| 326 | + * @param array<string, mixed> $box Information about the text box. | |
| 315 | 327 | */ |
| 316 | - protected function action_nonce_field( array $data, array $box ) { | |
| 328 | + protected function action_nonce_field( array $data, array $box ): void { | |
| 317 | 329 | wp_nonce_field( TablePress::nonce( $this->action ) ); |
| 318 | 330 | echo "\n"; |
| 319 | 331 | } |
| 320 | 332 | |
| 321 | 333 | /** |
| 322 | - * Print hidden field with the screen action. | |
| 334 | + * Prints hidden field with the screen action. | |
| 323 | 335 | * |
| 324 | 336 | * @since 1.0.0 |
| 325 | 337 | * |
| 326 | - * @param array $data Data for this screen. | |
| 327 | - * @param array $box Information about the text box. | |
| 338 | + * @param array<string, mixed> $data Data for this screen. | |
| 339 | + * @param array<string, mixed> $box Information about the text box. | |
| 328 | 340 | */ |
| 329 | - protected function action_field( array $data, array $box ) { | |
| 330 | - echo "<input type=\"hidden\" name=\"action\" value=\"tablepress_{$this->action}\" />\n"; | |
| 341 | + protected function action_field( array $data, array $box ): void { | |
| 342 | + echo "<input type=\"hidden\" name=\"action\" value=\"tablepress_{$this->action}\">\n"; | |
| 331 | 343 | } |
| 332 | 344 | |
| 333 | 345 | /** |
| 334 | - * Render the current view. | |
| 346 | + * Renders the current view. | |
| 335 | 347 | * |
| 336 | 348 | * @since 1.0.0 |
| 337 | 349 | */ |
| 338 | - public function render() { | |
| 350 | + public function render(): void { | |
| 339 | 351 | ?> |
| 340 | 352 | <div id="tablepress-page" class="wrap"> |
| 341 | 353 | <?php |
| 342 | 354 | $this->print_nav_tab_menu(); |
| 343 | - // Print all header messages. | |
| 344 | - foreach ( $this->header_messages as $message ) { | |
| 345 | - echo $message; | |
| 346 | - } | |
| 347 | - // "Import" screen has file upload. | |
| 348 | - $enctype = ( 'import' === $this->action ) ? ' enctype="multipart/form-data"' : ''; | |
| 349 | 355 | ?> |
| 350 | - <form action="<?php echo esc_url( admin_url( 'admin-post.php' ) ); ?>" method="post"<?php echo $enctype; ?>> | |
| 356 | + <div id="tablepress-body"> | |
| 357 | + <hr class="wp-header-end"> | |
| 358 | + <script>document.querySelectorAll('.notice:has(~#tablepress-page)').forEach(e=>document.querySelector('.wp-header-end').after(e));</script> | |
| 359 | + <?php | |
| 360 | + // Print all header messages. | |
| 361 | + foreach ( $this->header_messages as $message ) { | |
| 362 | + echo $message; | |
| 363 | + } | |
| 364 | + // "Import" screen has file upload. | |
| 365 | + $enctype = ( 'import' === $this->action ) ? ' enctype="multipart/form-data"' : ''; | |
| 366 | + ?> | |
| 367 | + <form action="<?php echo esc_url( admin_url( 'admin-post.php' ) ); ?>" method="post"<?php echo $enctype; ?> id="tablepress-page-form"> | |
| 351 | 368 | <?php |
| 352 | - $this->do_text_boxes( 'header' ); | |
| 369 | + $this->do_text_boxes( 'header' ); | |
| 370 | + $hide_if_no_js = ( in_array( $this->action, array( 'export', 'import' ), true ) ) ? ' class="hide-if-no-js"' : ''; | |
| 353 | 371 | ?> |
| 354 | - <div id="poststuff"> | |
| 372 | + <div id="poststuff"<?php echo $hide_if_no_js; ?>> | |
| 355 | 373 | <div id="post-body" class="metabox-holder columns-<?php echo ( isset( $GLOBALS['screen_layout_columns'] ) && ( 2 === $GLOBALS['screen_layout_columns'] ) ) ? '2' : '1'; ?>"> |
| 356 | 374 | <div id="postbox-container-2" class="postbox-container"> |
| 357 | 375 | <?php |
| 358 | 376 | $this->do_text_boxes( 'normal' ); |
| @@ -372,24 +390,52 @@ | ||
| 372 | 390 | $this->do_meta_boxes( 'side' ); |
| 373 | 391 | ?> |
| 374 | 392 | </div> |
| 375 | 393 | </div> |
| 376 | - <br class="clear" /> | |
| 394 | + <br class="clear"> | |
| 377 | 395 | </div> |
| 378 | 396 | </form> |
| 379 | 397 | </div> |
| 398 | + </div> | |
| 380 | 399 | <?php |
| 381 | 400 | } |
| 382 | 401 | |
| 383 | 402 | /** |
| 384 | - * Render the navigation menu with links to the possible actions, highlighting the current one. | |
| 403 | + * Renders the navigation menu with links to the possible actions, highlighting the current one. | |
| 385 | 404 | * |
| 386 | 405 | * @since 1.0.0 |
| 387 | 406 | */ |
| 388 | - protected function print_nav_tab_menu() { | |
| 407 | + protected function print_nav_tab_menu(): void { | |
| 408 | + $milestone_period_ended = ( strtotime( '2026-10-10' ) < strtotime( 'today' ) ); | |
| 409 | + $name = __( 'TablePress', 'tablepress' ); | |
| 410 | + $filename = 'admin/img/tablepress.svg'; | |
| 389 | 411 | ?> |
| 390 | - <div id="tablepress-nav" class="nav-tab-wrapper"> | |
| 391 | - <h1 class="wp-heading-inline"><?php _e( 'TablePress', 'tablepress' ); ?></h1> | |
| 412 | + <div id="tablepress-header" class="header"> | |
| 413 | + <h1 class="name"> | |
| 414 | + <img src="<?php echo plugins_url( $filename, TABLEPRESS__FILE__ ); ?>" alt="<?php esc_attr_e( 'TablePress plugin logo', 'tablepress' ); ?>"> | |
| 415 | + <span class="screen-reader-text"><?php echo $name; ?></span> | |
| 416 | + </h1> | |
| 417 | + <?php if ( 'list' !== $this->action || ! $this->data['messages']['plugin_update'] || $milestone_period_ended ) : ?> | |
| 418 | + <?php if ( ! TABLEPRESS_IS_PLAYGROUND_PREVIEW && tb_tp_fs()->is_free_plan() ) : ?> | |
| 419 | + <div class="buttons"> | |
| 420 | + <a href="<?php echo $milestone_period_ended ? esc_url( tb_tp_fs()->pricing_url( WP_FS__PERIOD_ANNUALLY, false ) ) : 'https://tablepress.org/premium/?utm_source=plugin&utm_medium=header&utm_campaign=upgrade-message340&utm_content=button'; ?>" class="tablepress-button"> | |
| 421 | + <span> | |
| 422 | + <?php | |
| 423 | + if ( $milestone_period_ended ) { | |
| 424 | + _e( 'Upgrade to Premium', 'tablepress' ); | |
| 425 | + } else { | |
| 426 | + echo in_array( get_user_locale(), array( 'de_DE', 'de_AT', 'de_CH' ), true ) ? 'Premium: Spare jetzt 25 %' : 'Premium: Get 25% off now'; | |
| 427 | + } | |
| 428 | + ?> | |
| 429 | + </span> | |
| 430 | + <span class="dashicons dashicons-arrow-right-alt"></span> | |
| 431 | + </a> | |
| 432 | + </div> | |
| 433 | + <?php endif; ?> | |
| 434 | + <?php endif; ?> | |
| 435 | + </div> | |
| 436 | + <nav id="tablepress-nav"> | |
| 437 | + <ul class="nav-menu"> | |
| 392 | 438 | <?php |
| 393 | 439 | foreach ( $this->data['view_actions'] as $action => $entry ) { |
| 394 | 440 | if ( '' === $entry['nav_tab_title'] ) { |
| 395 | 441 | continue; |
| @@ -398,50 +444,146 @@ | ||
| 398 | 444 | continue; |
| 399 | 445 | } |
| 400 | 446 | |
| 401 | 447 | $url = esc_url( TablePress::url( array( 'action' => $action ) ) ); |
| 402 | - $active = ( $action === $this->action ) ? ' nav-tab-active' : ''; | |
| 403 | - $separator = ( 'options' === $action ) ? ' nav-tab-separator' : ''; // Make the "Plugin Options" entry a separator, for some spacing. | |
| 404 | - echo "<a class=\"nav-tab{$active}{$separator}\" href=\"{$url}\">{$entry['nav_tab_title']}</a>"; | |
| 448 | + $active = ( $action === $this->action ) ? ' active' : ''; | |
| 449 | + $separator = ( 'export' === $action ) ? ' separator' : ''; // Make the "Export" entry a separator, for some spacing. | |
| 450 | + echo "<li class=\"nav-item\"><a id=\"tablepress-nav-item-{$action}\" class=\"nav-link{$active}{$separator}\" href=\"{$url}\">{$entry['nav_tab_title']}</a></li>"; | |
| 405 | 451 | } |
| 406 | 452 | ?> |
| 407 | - </div><hr class="wp-header-end" /> | |
| 453 | + </ul> | |
| 454 | + </nav> | |
| 408 | 455 | <?php |
| 409 | 456 | } |
| 410 | 457 | |
| 411 | 458 | /** |
| 412 | - * Print a submit button (only done when function is used as a callback for a text box). | |
| 459 | + * Prints a notification about JavaScript not being activated in the browser. | |
| 413 | 460 | * |
| 461 | + * @since 2.0.0 | |
| 462 | + * | |
| 463 | + * @param array<string, mixed> $data Data for this screen. | |
| 464 | + * @param array<string, mixed> $box Information about the text box. | |
| 465 | + */ | |
| 466 | + public function textbox_no_javascript( array $data, array $box ): void { | |
| 467 | + ?> | |
| 468 | + <div class="notice components-notice is-error notice-error hide-if-js"> | |
| 469 | + <div class="components-notice__content"> | |
| 470 | + <h3><em> | |
| 471 | + <?php _e( 'Attention: Unfortunately, there is a problem!', 'tablepress' ); ?> | |
| 472 | + </em></h3> | |
| 473 | + <p style="font-size:14px"> | |
| 474 | + <strong><?php _e( 'This screen requires JavaScript. Please enable JavaScript in your browser settings.', 'tablepress' ); ?></strong><br> | |
| 475 | + <?php _e( 'For help, please follow <a href="https://www.enable-javascript.com/">the instructions on how to enable JavaScript in your browser</a>.', 'tablepress' ); ?> | |
| 476 | + </p> | |
| 477 | + <p> | |
| 478 | + <?php echo '<a href="' . esc_url( TablePress::url( array( 'action' => 'list' ) ) ) . '">' . __( 'Back to the List of Tables', 'tablepress' ) . '</a>'; ?> | |
| 479 | + </p> | |
| 480 | + </div> | |
| 481 | + </div> | |
| 482 | + <?php | |
| 483 | + } | |
| 484 | + | |
| 485 | + /** | |
| 486 | + * Prints a submit button (only done when function is used as a callback for a text box). | |
| 487 | + * | |
| 488 | + * This method is soft-deprecated. It's no longer used in TablePress, but e.g. in the "TablePress Debug Extension". | |
| 489 | + * | |
| 414 | 490 | * @since 1.0.0 |
| 415 | 491 | * |
| 416 | - * @param array $data Data for this screen. | |
| 417 | - * @param array $box Information about the text box. | |
| 492 | + * @param array<string, mixed> $data Data for this screen. | |
| 493 | + * @param array<string, mixed> $box Information about the text box. | |
| 418 | 494 | */ |
| 419 | - protected function textbox_submit_button( array $data, array $box ) { | |
| 420 | - $caption = isset( $data['submit_button_caption'] ) ? $data['submit_button_caption'] : __( 'Save Changes', 'tablepress' ); | |
| 495 | + protected function textbox_submit_button( array $data, array $box ): void { | |
| 421 | 496 | ?> |
| 422 | - <p class="submit"><input type="submit" value="<?php echo esc_attr( $caption ); ?>" class="button button-primary button-large" name="submit" /></p> | |
| 497 | + <p class="submit"><input type="submit" class="components-button is-primary button-save-changes" value="<?php esc_attr_e( 'Save Changes', 'tablepress' ); ?>"></p> | |
| 423 | 498 | <?php |
| 424 | 499 | } |
| 425 | 500 | |
| 426 | 501 | /** |
| 427 | - * Return the content for the help tab for this screen. | |
| 502 | + * Returns a safe JSON representation of a variable for printing inside of JavaScript code. | |
| 428 | 503 | * |
| 504 | + * @since 2.0.0 | |
| 505 | + * | |
| 506 | + * @param mixed $data Variable to convert to JSON. | |
| 507 | + * @return string Safe JSON representation of a variable for printing inside of JavaScript code. | |
| 508 | + */ | |
| 509 | + public function convert_to_json_parse_output( /* string|array|bool|int|float|null */ $data ): string { | |
| 510 | + $json = wp_json_encode( $data, JSON_HEX_TAG | JSON_UNESCAPED_SLASHES ); | |
| 511 | + if ( false === $json ) { | |
| 512 | + // JSON encoding failed, return an error object. Use a prefixed "_error" key to avoid conflicts with intentionally added "error" keys. | |
| 513 | + $json = '{ "_error": "The data could not be encoded to JSON!" }'; | |
| 514 | + } | |
| 515 | + // Print the JSON data inside a `JSON.parse()` call in JS for speed gains, with necessary escaping of `\` and `'`. | |
| 516 | + $json = str_replace( array( '\\', "'" ), array( '\\\\', "\'" ), $json ); | |
| 517 | + return "JSON.parse( '{$json}' )"; | |
| 518 | + } | |
| 519 | + | |
| 520 | + /** | |
| 521 | + * Prints JavaScript variables for the screen. | |
| 522 | + * | |
| 523 | + * @since 3.1.0 | |
| 524 | + * | |
| 525 | + * @param string $variable Name of the JavaScript variable. | |
| 526 | + * @param array<string, mixed> $data Information about the text box. | |
| 527 | + */ | |
| 528 | + protected function print_script_data_json( string $variable, array $data ): void { | |
| 529 | + echo "<script>\n"; | |
| 530 | + echo "window.tp = window.tp || {};\n"; | |
| 531 | + echo "tp.{$variable} = {\n"; | |
| 532 | + foreach ( $data as $key => $value ) { | |
| 533 | + $value = $this->convert_to_json_parse_output( $value ); | |
| 534 | + echo "\t{$key}: {$value},\n"; | |
| 535 | + } | |
| 536 | + echo "};\n"; | |
| 537 | + echo "</script>\n"; | |
| 538 | + } | |
| 539 | + | |
| 540 | + /** | |
| 541 | + * Returns the content for the help tab for this screen. | |
| 542 | + * | |
| 429 | 543 | * Has to be implemented for every view that is visible in the WP Dashboard! |
| 430 | 544 | * |
| 431 | 545 | * @since 1.0.0 |
| 546 | + * | |
| 547 | + * @return string Help tab content for the view. | |
| 432 | 548 | */ |
| 433 | - protected function help_tab_content() { | |
| 549 | + protected function help_tab_content(): string { | |
| 434 | 550 | // Has to be implemented for every view that is visible in the WP Dashboard! |
| 435 | 551 | return ''; |
| 436 | 552 | } |
| 437 | 553 | |
| 438 | 554 | /** |
| 439 | - * Initialize the WP feature pointers for TablePress. | |
| 555 | + * Adds a TablePress "Thank You" message to the admin footer content. | |
| 440 | 556 | * |
| 441 | 557 | * @since 1.0.0 |
| 558 | + * @since 3.3.0 This method was moved from the now-removed `TablePress_Admin_Page` class. | |
| 559 | + * | |
| 560 | + * @param string $content Current admin footer content. | |
| 561 | + * @return string New admin footer content. | |
| 442 | 562 | */ |
| 443 | - protected function _init_wp_pointers() { | |
| 563 | + public function add_admin_footer_text( /* string */ $content ): string { | |
| 564 | + // Don't use a type hint in the method declaration as many WordPress plugins use the `admin_footer_text` filter without returning a string. | |
| 565 | + | |
| 566 | + // Protect against other plugins not returning a string in their filter callbacks. | |
| 567 | + if ( ! is_string( $content ) ) { // @phpstan-ignore function.alreadyNarrowedType (The `is_string()` check is needed as the input is coming from a filter hook.) | |
| 568 | + $content = ''; | |
| 569 | + } | |
| 570 | + | |
| 571 | + /* translators: %s: URL to TablePress website */ | |
| 572 | + $content .= ' • ' . sprintf( __( 'Thank you for using <a href="%s">TablePress</a>.', 'tablepress' ), 'https://tablepress.org/' ); | |
| 573 | + if ( tb_tp_fs()->is_free_plan() ) { | |
| 574 | + /* translators: %s: URL to TablePress premium features */ | |
| 575 | + $content .= ' ' . sprintf( __( 'Take a look at the <a href="%s">Premium features</a>!', 'tablepress' ), 'https://tablepress.org/premium/?utm_source=plugin&utm_medium=textlink&utm_content=admin-footer' ); | |
| 576 | + } | |
| 577 | + return $content; | |
| 578 | + } | |
| 579 | + | |
| 580 | + /** | |
| 581 | + * Initializes the WP feature pointers for TablePress. | |
| 582 | + * | |
| 583 | + * @since 1.0.0 | |
| 584 | + */ | |
| 585 | + protected function _init_wp_pointers(): void { | |
| 444 | 586 | // Check if there are WP pointers for this view. |
| 445 | 587 | if ( empty( $this->wp_pointers ) ) { |
| 446 | 588 | return; |
| 447 | 589 | } |
| @@ -448,21 +590,74 @@ | ||
| 448 | 590 | |
| 449 | 591 | // Get dismissed pointers. |
| 450 | 592 | $dismissed = explode( ',', (string) get_user_meta( get_current_user_id(), 'dismissed_wp_pointers', true ) ); |
| 451 | 593 | |
| 452 | - $got_pointers = false; | |
| 594 | + $pointers_on_page = false; | |
| 453 | 595 | foreach ( array_diff( $this->wp_pointers, $dismissed ) as $pointer ) { |
| 454 | 596 | // Bind pointer print function. |
| 455 | - add_action( "admin_footer-{$GLOBALS['hook_suffix']}", array( $this, 'wp_pointer_' . $pointer ) ); | |
| 456 | - $got_pointers = true; | |
| 597 | + add_action( "admin_footer-{$GLOBALS['hook_suffix']}", array( $this, 'wp_pointer_' . $pointer ) ); // @phpstan-ignore argument.type | |
| 598 | + $pointers_on_page = true; | |
| 457 | 599 | } |
| 458 | 600 | |
| 459 | - if ( ! $got_pointers ) { | |
| 601 | + if ( $pointers_on_page ) { | |
| 602 | + wp_enqueue_style( 'wp-pointer' ); | |
| 603 | + wp_enqueue_script( 'wp-pointer' ); | |
| 604 | + } | |
| 605 | + } | |
| 606 | + | |
| 607 | + /** | |
| 608 | + * Prints the JavaScript code for a WP feature pointer. | |
| 609 | + * | |
| 610 | + * @since 1.0.0 | |
| 611 | + * @since 3.3.0 This method was moved from the now-removed `TablePress_Admin_Page` class. | |
| 612 | + * | |
| 613 | + * @param string $pointer_id The pointer ID. | |
| 614 | + * @param string $selector The HTML elements, on which the pointer should be attached. | |
| 615 | + * @param array<string, mixed> $args Arguments to be passed to the pointer JS (see wp-pointer.js). | |
| 616 | + */ | |
| 617 | + public function print_wp_pointer_js( string $pointer_id, string $selector, array $args ): void { | |
| 618 | + if ( empty( $pointer_id ) || empty( $selector ) || empty( $args['content'] ) ) { | |
| 460 | 619 | return; |
| 461 | 620 | } |
| 462 | 621 | |
| 463 | - // Add pointers script and style to queue. | |
| 464 | - wp_enqueue_style( 'wp-pointer' ); | |
| 465 | - wp_enqueue_script( 'wp-pointer' ); | |
| 622 | + $keyboard_shortcut = ''; | |
| 623 | + if ( 'tp33_edit_quick_navigation' === $pointer_id ) { | |
| 624 | + $keyboard_shortcut = <<<JS | |
| 625 | + content: options.content.replace( /%metaKey%/g, window?.navigator?.platform?.includes( 'Mac' ) ? wp.i18n._x( '⌘', 'keyboard shortcut modifier key on a Mac keyboard', 'tablepress' ) : wp.i18n._x( 'Ctrl+', 'keyboard shortcut modifier key on a non-Mac keyboard', 'tablepress' ) ),\n | |
| 626 | + JS; | |
| 627 | + } | |
| 628 | + | |
| 629 | + /* | |
| 630 | + * Print JS code for the feature pointers, extended with event handling for opened/closed "Screen Options", so that pointers can | |
| 631 | + * be repositioned. 210 ms is slightly slower than jQuery's "fast" value, to allow all elements to reach their original position. | |
| 632 | + */ | |
| 633 | + ?> | |
| 634 | +<script> | |
| 635 | +( ( $ ) => { | |
| 636 | + let options = <?php echo wp_json_encode( $args, JSON_HEX_TAG | JSON_UNESCAPED_SLASHES ); ?>; | |
| 637 | + if ( ! options ) { | |
| 638 | + return; | |
| 639 | + } | |
| 640 | + | |
| 641 | + options = { | |
| 642 | + ...options, | |
| 643 | + <?php echo $keyboard_shortcut; ?> | |
| 644 | + close() { | |
| 645 | + $.post( ajaxurl, { | |
| 646 | + pointer: '<?php echo $pointer_id; ?>', | |
| 647 | + action: 'dismiss-wp-pointer' | |
| 648 | + } ); | |
| 649 | + $( this ).pointer( { 'disabled': true } ); | |
| 650 | + }, | |
| 651 | + }; | |
| 652 | + | |
| 653 | + $( () => setTimeout( () => $( '<?php echo $selector; ?>' ).pointer( options ).pointer( 'open' ), 1 ) ); | |
| 654 | + | |
| 655 | + $( document ).on( 'screen:options:open screen:options:close', () => { | |
| 656 | + setTimeout( () => $( '<?php echo $selector; ?>' ).pointer( 'reposition' ), 210 ); | |
| 657 | + } ); | |
| 658 | +} )( jQuery ); | |
| 659 | +</script> | |
| 660 | + <?php | |
| 466 | 661 | } |
| 467 | 662 | |
| 468 | 663 | } // class TablePress_View |
