Changeset 7267

Timestamp:

07/11/2013 01:31:42 AM (12 years ago)

Author:

johnjamesjacoby

Message:

In bp-default, swap the 'bp_sidebar_login_form' action order. Also use esc_url() in bp_dtheme_sidebar_login_redirect_to() instead of esc_attr(). Fixes #5080.

Location:

trunk/bp-themes/bp-default

Files:

• functions.php (modified) (1 diff)
• sidebar.php (modified) (1 diff)

trunk/bp-themes/bp-default/functions.php

@@ -720 +720 @@
     $redirect_to = apply_filters( 'bp_no_access_redirect', $redirect_to ); ?>
 
-    <input type="hidden" name="redirect_to" value="<?php echo esc_attr( $redirect_to ); ?>" />
+    <input type="hidden" name="redirect_to" value="<?php echo esc_url( $redirect_to ); ?>" />
 
 <?php

trunk/bp-themes/bp-default/sidebar.php

@@ -50 +50 @@
             <p class="forgetmenot"><label><input name="rememberme" type="checkbox" id="sidebar-rememberme" value="forever" tabindex="99" /> <?php _e( 'Remember Me', 'buddypress' ); ?></label></p>
 
+            <input type="submit" name="wp-submit" id="sidebar-wp-submit" value="<?php _e( 'Log In', 'buddypress' ); ?>" tabindex="100" />
+
             <?php do_action( 'bp_sidebar_login_form' ); ?>
-            <input type="submit" name="wp-submit" id="sidebar-wp-submit" value="<?php _e( 'Log In', 'buddypress' ); ?>" tabindex="100" />
+
             <input type="hidden" name="testcookie" value="1" />
         </form>