Changeset 7065

Timestamp:

04/09/2018 03:37:38 AM (7 years ago)

Author:

dd32

Message:

Trac: Don't double up the comment deletion buttons.

This needs a follow up once all Tracs are migrated to remove this block of code / the extra template changes.

See #3526.

File:

• sites/trunk/wordpress.org/public_html/style/trac/wp-trac.js (modified) (1 diff)

sites/trunk/wordpress.org/public_html/style/trac/wp-trac.js

@@ -176 +176 @@
             // See https://meta.trac.wordpress.org/changeset/204.
             $('div.change').children('.trac-ticket-buttons').each( function() {
-                var el = $(this);
-                el.children().appendTo( el.prev().children('.trac-ticket-buttons') ).end().end().remove();
+                var el = $(this),
+                    trac_buttons = el.prev().children('.trac-ticket-buttons');
+
+                // Trac 1.2.2 already has a delete button here. TODO See r204 to check if buttons need removing from templates.
+                if ( trac_buttons.find('form input[name="action"][value="delete-comment"]').length ) {
+                    console.log( "wp-trac: .trac-ticket-buttons delete-comment changes not needed on this trac." );
+                    return;
+                }
+
+                el.children().appendTo( trac_buttons ).end().end().remove();
             });
         },