Changeset 13618 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/readme/class-validator.php

Timestamp:

04/30/2024 04:41:42 AM (14 months ago)

Author:

dd32

Message:

Plugin Directory: Readme: Validate that the license field contains a likely valid license.

This adds to the existing "no license" with two new ones:

• Warning: License appears to be not-GPL-compatible, invalid
• Note: We could not parse the license specified, unknown

Fixes #1944.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/readme/class-validator.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/readme/class-validator.php

@@ -145 +145 @@
         }
 
-        if ( empty( $readme->license ) ) {
+        if ( ! empty( $readme->warnings['license_missing'] ) ) {
             $warnings['license_missing'] = true;
+        } elseif ( ! empty( $readme->warnings['invalid_license'] ) ) {
+            $errors['invalid_license'] = $readme->warnings['invalid_license'];
+        } elseif ( ! empty( $readme->warnings['unknown_license'] ) ) {
+            $notes['unknown_license'] = $readme->warnings['unknown_license'];
         }
 
@@ -398 +402 @@
             case 'donate_link_missing':
                 return __( 'No donate link was found', 'wporg-plugins' );
+
             case 'license_missing':
                 return sprintf(
-                    /* translators: 1: 'License' */
-                    __( 'The %1$s field is missing or invalid. A GPLv2 or later compatible license should be specified.', 'wporg-plugins' ),
-                    '<code>License</code>'
+                    /* translators: 1: 'License', 2: Link to a compatible licenses page. */
+                    __( 'The %1$s field is missing. <a href="%2$s">A GPLv2 or later compatible license</a> should be specified.', 'wporg-plugins' ),
+                    '<code>License</code>',
+                    'https://www.gnu.org/licenses/license-list.en.html'
+                );
+
+            case 'invalid_license':
+                return sprintf(
+                    /* translators: 1: 'License', 2: Link to a compatible licenses page. */
+                    __( 'The %1$s field appears to be invalid. <a href="%2$s">A GPLv2 or later compatible license</a> should be specified.', 'wporg-plugins' ),
+                    '<code>License</code>',
+                    'https://www.gnu.org/licenses/license-list.en.html'
+                );
+
+            case 'unknown_license':
+                return sprintf(
+                    /* translators: 1: 'License', 2: Link to a compatible licenses page. */
+                    __( 'The %1$s field could not be validated. <a href="%2$s">A GPLv2 or later compatible license</a> should be specified. The specified license may be compatible.', 'wporg-plugins' ),
+                    '<code>License</code>',
+                    'https://www.gnu.org/licenses/license-list.en.html'
                 );