Changeset 14712

Timestamp:

03/13/2026 05:27:24 PM (6 weeks ago)

Author:

obenland

Message:

WP.org Abilities: Fix PHPCS issues across all plugin files

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities

Files:

• class-autoloader.php (modified) (1 diff)
• plugins/plugin-directory/resources/class-plugin-check-guide.php (modified) (1 diff)
• plugins/plugin-directory/resources/class-reserved-slugs.php (modified) (1 diff)
• plugins/plugin-directory/tools/class-get-plugin-status.php (modified) (5 diffs)
• plugins/plugin-directory/tools/class-validate-readme.php (modified) (1 diff)
• wporg-abilities.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities/class-autoloader.php

@@ -58 +58 @@
      * Loads a class if it starts with `$this->prefix`.
      *
-     * @param string $class The class to be loaded.
+     * @param string $class_name The class to be loaded.
      */
-    public function load( $class ) {
-        if ( strpos( $class, $this->prefix . self::NS_SEPARATOR ) !== 0 ) {
+    public function load( $class_name ) {
+        if ( strpos( $class_name, $this->prefix . self::NS_SEPARATOR ) !== 0 ) {
             return;
         }
 
         // Strip prefix from the start (ala PSR-4).
-        $class = substr( $class, $this->prefix_length + 1 );
+        $class = substr( $class_name, $this->prefix_length + 1 );
         $class = strtolower( $class );
         $file  = '';

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities/plugins/plugin-directory/resources/class-plugin-check-guide.php

@@ -108 +108 @@
 - A clean PCP report speeds up the review process significantly
 - More info: https://wordpress.org/plugins/plugin-check/
-MD,
+MD
+                ,
                 'mimeType' => 'text/markdown',
             ),

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities/plugins/plugin-directory/resources/class-reserved-slugs.php

@@ -5 +5 @@
  * @package WordPressdotorg\Abilities\Plugins\Plugin_Directory\Resources
  */
+
+// phpcs:disable WordPress.WP.CapitalPDangit.MisspelledInText -- Lowercase "wordpress" is intentional (slugs).
 
 declare( strict_types = 1 );

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities/plugins/plugin-directory/tools/class-get-plugin-status.php

@@ -35 +35 @@
                     'type'       => 'object',
                     'properties' => array(
-                        'plugin_slug'    => array(
+                        'plugin_slug'  => array(
                             'type'        => 'string',
                             'description' => 'The plugin slug as assigned during submission.',
@@ -109 +109 @@
                     'mcp'         => array( 'type' => 'tool' ),
                     'annotations' => array(
-                        'readonly'     => true,
-                        'idempotent'   => true,
-                        'destructive'  => false,
+                        'readonly'    => true,
+                        'idempotent'  => true,
+                        'destructive' => false,
                     ),
                 ),
@@ -167 +167 @@
      */
     private static function get_plugin_post( string $slug ): ?\WP_Post {
-        $posts = get_posts( array(
-            'post_type'   => 'plugin',
-            'name'        => $slug,
-            'post_status' => 'any',
-            'author'      => get_current_user_id(),
-            'numberposts' => 1,
-        ) );
+        $posts = get_posts(
+            array(
+                'post_type'   => 'plugin',
+                'name'        => $slug,
+                'post_status' => 'any',
+                'author'      => get_current_user_id(),
+                'numberposts' => 1,
+            )
+        );
 
         return $posts[0] ?? null;
@@ -213 +215 @@
             $reason = get_post_meta( $post->ID, '_rejection_reason', true );
             if ( $reason ) {
-                $reasons      = class_exists( Template::class ) ? Template::get_rejection_reasons() : array();
+                $reasons                  = class_exists( Template::class ) ? Template::get_rejection_reasons() : array();
                 $data['rejection_reason'] = $reasons[ $reason ] ?? $reason;
             }
@@ -292 +294 @@
                 $feedback[] = array(
                     'from' => 'customer' === $thread->type ? 'author' : 'reviewer',
+                    // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase -- HelpScout API property.
                     'date' => $thread->createdAt ?? '',
                     'body' => $body,

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities/plugins/plugin-directory/tools/class-validate-readme.php

@@ -68 +68 @@
                     'mcp'         => array( 'type' => 'tool' ),
                     'annotations' => array(
-                        'readonly'     => true,
-                        'idempotent'   => true,
-                        'destructive'  => false,
+                        'readonly'    => true,
+                        'idempotent'  => true,
+                        'destructive' => false,
                     ),
                 ),

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-abilities/wporg-abilities.php

@@ -38 +38 @@
 McpAdapter::instance();
 
-$action = defined( 'WP_CLI' ) && constant( 'WP_CLI' ) ? 'init' : 'rest_api_init';
+$hook = defined( 'WP_CLI' ) && constant( 'WP_CLI' ) ? 'init' : 'rest_api_init';
 
-add_action( $action, array( Registrar::class, 'init' ) );
+add_action( $hook, array( Registrar::class, 'init' ) );
 add_action( 'mcp_adapter_init', array( MCP_Server::class, 'register' ) );
 add_filter( 'mcp_adapter_create_default_server', '__return_false' );