Changeset 5691 for trunk/src/includes/replies/template.php

Timestamp:

04/20/2015 04:20:34 PM (11 years ago)

Author:

johnjamesjacoby

Message:

All: ensure all URLs are escaped (trunk) and cleanup some phpdoc

Also introduce bbp_dropdown class for dropdowns.

File:

• trunk/src/includes/replies/template.php (modified) (9 diffs)

trunk/src/includes/replies/template.php

@@ -1684 +1684 @@
 
         // Get the reply to use it's ID and post_parent
-        $reply = bbp_get_reply( bbp_get_reply_id( $r['id'] ) );
+        $reply = bbp_get_reply( $r['id'] );
 
         // Bail if no reply or user cannot reply
@@ -1972 +1972 @@
         ), 'get_reply_edit_link' );
 
-        $reply = bbp_get_reply( bbp_get_reply_id( $r['id'] ) );
+        $reply = bbp_get_reply( $r['id'] );
 
         // Bypass check if user has caps
@@ -2024 +2024 @@
         global $wp_rewrite;
 
-        $reply = bbp_get_reply( bbp_get_reply_id( $reply_id ) );
+        $reply = bbp_get_reply( $reply_id );
         if ( empty( $reply ) ) {
             return;
@@ -2099 +2099 @@
         ), 'get_reply_trash_link' );
 
-        $reply = bbp_get_reply( bbp_get_reply_id( $r['id'] ) );
+        $reply = bbp_get_reply( $r['id'] );
 
         if ( empty( $reply ) || ! current_user_can( 'delete_reply', $reply->ID ) ) {
@@ -2170 +2170 @@
         ), 'get_reply_spam_link' );
 
-        $reply = bbp_get_reply( bbp_get_reply_id( $r['id'] ) );
+        $reply = bbp_get_reply( $r['id'] );
 
         if ( empty( $reply ) || ! current_user_can( 'moderate', $reply->ID ) ) {
@@ -2367 +2367 @@
         ), 'get_reply_approve_link' );
 
-        $reply = bbp_get_reply( bbp_get_reply_id( $r['id'] ) );
+        $reply = bbp_get_reply( $r['id'] );
 
         if ( empty( $reply ) || ! current_user_can( 'moderate', $reply->ID ) ) {
@@ -2674 +2674 @@
             'show_none'    => sprintf( esc_attr__( '%1$s - %2$s', 'bbpress' ), $topic_id, bbp_get_topic_title( $topic_id ) ),
             'select_id'    => 'bbp_reply_to',
+            'select_class' => 'bbp_dropdown',
             'exclude'      => $reply_id,
             'selected'     => $reply_to,
@@ -2791 +2792 @@
         // Parse arguments against default values
         $r = bbp_parse_args( $args, array(
-            'select_id' => 'bbp_reply_status',
-            'tab'       => false,
-            'reply_id'  => 0,
-            'selected'  => false
+            'select_id'    => 'bbp_reply_status',
+            'select_class' => 'bbp_dropdown',
+            'tab'          => false,
+            'reply_id'     => 0,
+            'selected'     => false
         ), 'reply_status_dropdown' );
 
@@ -2825 +2827 @@
         ob_start(); ?>
 
-        <select name="<?php echo esc_attr( $r['select_id'] ) ?>" id="<?php echo esc_attr( $r['select_id'] ); ?>_select"<?php echo $tab; ?>>
+        <select name="<?php echo esc_attr( $r['select_id'] ) ?>" id="<?php echo esc_attr( $r['select_id'] ); ?>_select" class="<?php echo esc_attr( $r['select_class'] ); ?>"<?php echo $tab; ?>>
 
             <?php foreach ( bbp_get_reply_statuses( $r['reply_id'] ) as $key => $label ) : ?>