Changeset 5120 for trunk/includes/replies/template.php

Timestamp:

10/04/2013 06:35:56 PM (13 years ago)

Author:

johnjamesjacoby

Message:

Fix bug where nested ternary comparisons in form-anonymous.php were causing duplicated, sometimes conflicting, output.

Introduce convenience template functions for better handling and output of anonymous user form field data. Wrap complex form-anonymous.php template-part access logic in a helper function (similar to other template parts.) Deprecate ambiguous bbp_topic/reply_author() functions, in favor of author_display_name() equivalents.

Fixes #2426.

File:

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

trunk/includes/replies/template.php

@@ -889 +889 @@
 
 /**
+ * Deprecated. Use bbp_reply_author_display_name() instead.
+ *
  * Output the author of the reply
  *
  * @since bbPress (r2667)
+ * @deprecated bbPress (r5119)
  *
  * @param int $reply_id Optional. Reply id
@@ -900 +903 @@
 }
     /**
+     * Deprecated. Use bbp_get_reply_author_display_name() instead.
+     *
      * Return the author of the reply
      *
      * @since bbPress (r2667)
+     * @deprecated bbPress (r5119)
      *
      * @param int $reply_id Optional. Reply id
@@ -1081 +1087 @@
      * @uses bbp_is_reply_anonymous() To check if the reply is by an
      *                                 anonymous user
-     * @uses bbp_get_reply_author() To get the reply author name
      * @uses bbp_get_reply_author_url() To get the reply author url
      * @uses bbp_get_reply_author_avatar() To get the reply author avatar