Changeset 11328

Timestamp:

11/16/2021 06:51:33 AM (4 years ago)

Author:

dd32

Message:

PHPUnit Test Reporter: Use a custom post listing table, to avoid OOM errors due to evergrowing parent dropdown list.

See ​https://github.com/WordPress/phpunit-test-reporter/issues/100

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter

Files:

• class-posts-list-table.php (added)
• phpunit-test-reporter.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/phpunit-test-reporter.php

@@ -31 +31 @@
 add_action( 'rest_api_init', array( 'PTR\RestAPI', 'register_routes' ) );
 
+add_action( 'load-edit.php', 'ptr_load_edit_php' );
+
+/**
+ * Override the post type list table.
+ *
+ * The Results post type Quick Edit 'Page Parent' dropdown is tens of thousands of items long,
+ * and causes PHP OOM errors.
+ * This replaces it with a variant that doesn't support inline editing.. through a very non-conventional method.
+ */
+function ptr_load_edit_php() {
+    if ( ! isset( $_GET['post_type'] ) || 'result' != $_GET['post_type'] ) {
+        return;
+    }
+
+    require_once ABSPATH . 'wp-admin/includes/class-wp-posts-list-table.php';
+    require_once __DIR__ . '/src/class-posts-list-table.php';
+
+    add_action( 'parse_request', 'ptr_override_results_list_table' );
+}
+
+/**
+ * Override the edit.php?post_type=results WP_Post_List_Table.
+ *
+ * This is the most ridiculous hack I've hacked, but this totally works.
+ */
+function ptr_override_results_list_table() {
+    global $wp_list_table;
+
+    if (
+        isset( $wp_list_table ) &&
+        'WP_Posts_List_Table' == get_class( $wp_list_table )
+    ) {
+        remove_action( 'parse_request', __FUNCTION__ );
+
+        $wp_list_table = new PTR\Posts_List_Table();
+        // We were within WP_Posts_List_Table::prepare_items() when we overrode it, so we have to query again.
+        $wp_list_table->prepare_items();
+    }
+}
+
 /**
  * Get a rendered template part