Changeset 12756 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-author-notice.php

Timestamp:

07/24/2023 08:44:49 AM (20 months ago)

Author:

dd32

Message:

Plugin Directory: Author Notice: Style ordered lists and pre tags better. Add a set() method for other code to set the notice.

See #6880.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-author-notice.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-author-notice.php

@@ -106 +106 @@
     static function save_post( $post ) {
         $post          = get_post( $post );
-        $post_meta     = get_post_meta( $post->ID, self::POST_META_KEY, true );
-        $author_notice = $post_meta ?: [ 'type' => '', 'html' => '' ];
 
         if (
@@ -115 +113 @@
             current_user_can( 'plugin_admin_edit', $post->ID )
         ) {
-            $new_author_notice         = wp_unslash( $_REQUEST['author_notice'] );
-            $new_author_notice['type'] = sanitize_key( $new_author_notice['type'] );
-            $new_author_notice['html'] = wp_kses( trim( $new_author_notice['html'] ), self::ALLOWED_TAGS );
+            $new_author_notice = wp_unslash( $_REQUEST['author_notice'] );
+            self::set( $post, $new_author_notice['html'], $new_author_notice['type'] );
+        }
+    }
 
-            // Check it's not empty with tags removed.
-            if (
-                $new_author_notice['html'] &&
-                ! trim( wp_strip_all_tags( $new_author_notice['html'] ) )
-            ) {
-                $new_author_notice['html'] = '';
-            }
+    /**
+     * Set the Author notice.
+     */
+    static function set( $post, $notice, $type = 'error' ) {
+        $post_meta         = get_post_meta( $post->ID, self::POST_META_KEY, true );
+        $author_notice     = $post_meta ?: [ 'type' => '', 'html' => '' ];
+        $new_author_notice = [
+            'type' => sanitize_key( $type ),
+            'html' => wp_kses( trim( $notice ), self::ALLOWED_TAGS ),
+        ];
 
-            // Default or no text, remove the notice.
-            if ( empty( $new_author_notice['html'] ) || self::DEFAULT_TEXT === $new_author_notice['html'] ) {
-                $new_author_notice['type'] = '';
-                $new_author_notice['html'] = '';
-            }
+        // Check it's not empty with tags removed.
+        if (
+            $new_author_notice['html'] &&
+            ! trim( wp_strip_all_tags( $new_author_notice['html'] ) )
+        ) {
+            $new_author_notice['html'] = '';
+        }
 
-            // Remove it.
-            if ( $post_meta && ! $new_author_notice['type'] && ! $new_author_notice['html'] ) {
-                delete_post_meta( $post->ID, self::POST_META_KEY );
-                Tools::audit_log( 'Author notice removed.' );
-                return;
-            }
+        // Default or no text, remove the notice.
+        if ( empty( $new_author_notice['html'] ) || self::DEFAULT_TEXT === $new_author_notice['html'] ) {
+            $new_author_notice['type'] = '';
+            $new_author_notice['html'] = '';
+        }
 
-            // Value changed?
-            if (
-                $author_notice['html'] != $new_author_notice['html'] ||
-                $author_notice['type'] != $new_author_notice['type']
-            ) {
-                $author_notice = [
-                    'type' => $new_author_notice['type'],
-                    'html' => $new_author_notice['html'],
-                    'when' => time(),
-                    'who'  => get_current_user_id(),
-                ];
+        // Remove it.
+        if ( $post_meta && ! $new_author_notice['type'] && ! $new_author_notice['html'] ) {
+            delete_post_meta( $post->ID, self::POST_META_KEY );
+            Tools::audit_log( 'Author notice removed.' );
+            return;
+        }
 
-                update_post_meta( $post->ID, self::POST_META_KEY, $author_notice );
+        // Value changed?
+        if (
+            $author_notice['html'] != $new_author_notice['html'] ||
+            $author_notice['type'] != $new_author_notice['type']
+        ) {
+            $author_notice = [
+                'type' => $new_author_notice['type'],
+                'html' => $new_author_notice['html'],
+                'when' => time(),
+                'who'  => get_current_user_id(),
+            ];
 
-                $type_text = $author_notice['type'] ?: __( 'Do not show', 'wporg-plugins' );
-                Tools::audit_log(
-                    sprintf(
-                        'Author notice set to: [%s] %s',
-                        $type_text,
-                        wp_strip_all_tags( $author_notice['html'] )
-                    )
-                );
-            }
+            update_post_meta( $post->ID, self::POST_META_KEY, $author_notice );
+
+            $type_text = $author_notice['type'] ?: __( 'Do not show', 'wporg-plugins' );
+            Tools::audit_log(
+                sprintf(
+                    'Author notice set to: [%s] %s',
+                    $type_text,
+                    wp_strip_all_tags( $author_notice['html'] )
+                )
+            );
         }
     }