Changeset 12990 for trunk/src/bp-messages/bp-messages-template.php

Timestamp:

07/08/2021 05:23:32 PM (5 years ago)

Author:

dcavins

Message:

Add php fallback for dismissing site-wide notices.

Dismissal of sitewide notices has been handled
solely via JavaScript in the Legacy and Nouveau
template packs. This commit adds a new action
URI at {user_profile}/messages/notices/dismiss
that will dismiss the currently active notice for
the logged-in user.

We also add a new template function to create the
nonced link, bp_get_message_notice_dismiss_link(),
and make use of it in the core template function
bp_message_get_notices() and in the Nouveau
template for site-wide notices. In both cases, button
elements have been converted to anchor elements
linking to the new dismissal URI, so the action can be
completed even if there is a problem with JavaScript.

Fixes #8505.

File:

• trunk/src/bp-messages/bp-messages-template.php (modified) (2 diffs)

trunk/src/bp-messages/bp-messages-template.php

@@ -1312 +1312 @@
 
 /**
+ * Output the URL for dismissing the current notice for the current user.
+ *
+ * @since 9.0.0
+ * @return string URL for dismissing the current notice for the current user.
+ */
+function bp_message_notice_dismiss_link() {
+        echo esc_url( bp_get_message_notice_dismiss_link() );
+}
+        /**
+         * Get the URL for dismissing the current notice for the current user.
+         *
+         * @since 9.0.0
+         * @return string URL for dismissing the current notice for the current user.
+         */
+        function bp_get_message_notice_dismiss_link() {
+
+                $link = wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . bp_get_messages_slug() . '/notices/dismiss/' ), 'messages_dismiss_notice' );
+
+                /**
+                 * Filters the URL for dismissing the current notice for the current user.
+                 *
+                 * @since 9.0.0
+                 *
+                 * @param string $link URL for dismissing the current notice.
+                 */
+                return apply_filters( 'bp_get_message_notice_dismiss_link', $link );
+        }
+
+/**
  * Output the messages component slug.
  *
@@ -1360 +1389 @@
                         <div id="message" class="info notice" rel="n-<?php echo esc_attr( $notice->id ); ?>">
                                 <strong><?php bp_message_notice_subject( $notice ); ?></strong>
-                                <button type="button" id="close-notice" class="bp-tooltip" data-bp-tooltip="<?php esc_attr_e( 'Dismiss this notice', 'buddypress' ) ?>"><span class="bp-screen-reader-text"><?php _e( 'Dismiss this notice', 'buddypress' ) ?></span> <span aria-hidden="true">&Chi;</span></button>
+                                <a href="<?php bp_message_notice_dismiss_link(); ?>" id="close-notice" class="bp-tooltip button" data-bp-tooltip="<?php esc_attr_e( 'Dismiss this notice', 'buddypress' ) ?>"><span class="bp-screen-reader-text"><?php _e( 'Dismiss this notice', 'buddypress' ) ?></span> <span aria-hidden="true">&Chi;</span></a>
                                 <?php bp_message_notice_text( $notice ); ?>
                                 <?php wp_nonce_field( 'bp_messages_close_notice', 'close-notice-nonce' ); ?>