Skip to:
Content

BuddyPress.org

Changeset 10858


Ignore:
Timestamp:
06/01/2016 09:00:21 PM (9 years ago)
Author:
r-a-y
Message:

Activity: Introduce error handling to the BP_Activity_Activity class.

To support frontend user messages for activity failures, we need the
ability to add errors.

This commit:

  • Introduces the $errors object and $error_type property to the BP_Activity_Activity class.
  • Adds an 'error_type' parameter to all applicable, activity functions.
  • Modifies bp-legacy to output an error message for activity failures.

See #6719.

Location:
trunk/src
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-activity/bp-activity-functions.php

    r10853 r10858  
    17831783 *
    17841784 * @since 1.1.0
     1785 * @since 2.6.0 Added 'error_type' parameter to $args.
    17851786 *
    17861787 * @param array|string $args {
    … …  
    18141815 *                                       Default: false.
    18151816 *     @type bool     $is_spam           Should the item be marked as spam? Default: false.
     1817 *     @type string   $error_type        Optional. Error type. Either 'bool' or 'wp_error'. Default: 'bool'.
    18161818 * }
    18171819 * @return int|bool The ID of the activity on success. False on error.
    … …  
    18321834        'hide_sitewide'     => false,                  // Should this be hidden on the sitewide activity stream?
    18331835        'is_spam'           => false,                  // Is this activity item to be marked as spam?
     1836        'error_type'        => 'bool'
    18341837    ), 'activity_add' );
    18351838
    … …  
    18551858    $activity->hide_sitewide     = $r['hide_sitewide'];
    18561859    $activity->is_spam           = $r['is_spam'];
     1860    $activity->error_type        = $r['error_type'];
    18571861    $activity->action            = ! empty( $r['action'] )
    1858                                         ? $r['action']
    1859                                         : bp_activity_generate_action_string( $activity );
    1860 
    1861     if ( ! $activity->save() ) {
     1862                        ? $r['action']
     1863                        : bp_activity_generate_action_string( $activity );
     1864
     1865    $save = $activity->save();
     1866
     1867    if ( 'wp_error' === $r['error_type'] && is_wp_error( $save ) ) {
     1868        return $save;
     1869    } elseif ('bool' === $r['error_type'] && false === $save ) {
    18621870        return false;
    18631871    }
    … …  
    18991907
    19001908    $r = wp_parse_args( $args, array(
    1901         'content' => false,
    1902         'user_id' => bp_loggedin_user_id()
     1909        'content'    => false,
     1910        'user_id'    => bp_loggedin_user_id(),
     1911        'error_type' => 'bool',
    19031912    ) );
    19041913
    … …  
    19401949        'component'    => buddypress()->activity->id,
    19411950        'type'         => 'activity_update',
     1951        'error_type'   => $r['error_type']
    19421952    ) );
     1953
     1954    if ( is_wp_error( $activity_id ) ) {
     1955        return $activity_id;
     1956    }
    19431957
    19441958    /**
    … …  
    24922506 * @since 2.5.0 Add a new possible parameter $skip_notification for the array of arguments.
    24932507 *              Add the $primary_link parameter for the array of arguments.
     2508 * @since 2.6.0 Added 'error_type' parameter to $args.
    24942509 *
    24952510 * @param array|string $args {
    … …  
    25082523 *     @type bool   $skip_notification Optional. false to send a comment notification, false otherwise.
    25092524 *                                     Defaults to false.
     2525 *     @type string $error_type        Optional. Error type. Either 'bool' or 'wp_error'. Default: 'bool'.
    25102526 * }
    25112527 * @return int|bool The ID of the comment on success, otherwise false.
    25122528 */
    25132529function bp_activity_new_comment( $args = '' ) {
    2514     $bp       = buddypress();
    2515     $errors   = new WP_Error();
    2516     $feedback = __( 'There was an error posting your reply. Please try again.', 'buddypress' );
    2517 
    2518     if ( empty( $bp->activity->errors ) ) {
    2519         $bp->activity->errors = array();
    2520     }
     2530    $bp = buddypress();
    25212531
    25222532    $r = wp_parse_args( $args, array(
    … …  
    25282538        'primary_link'      => '',
    25292539        'skip_notification' => false,
     2540        'error_type'        => 'bool'
    25302541    ) );
     2542
     2543    // Error type is boolean; need to initialize some variables for backpat.
     2544    if ( 'bool' === $r['error_type'] ) {
     2545        if ( empty( $bp->activity->errors ) ) {
     2546            $bp->activity->errors = array();
     2547        }
     2548    }
     2549
     2550    // Default error message.
     2551    $feedback = __( 'There was an error posting your reply. Please try again.', 'buddypress' );
    25312552
    25322553    // Bail if missing necessary data.
    25332554    if ( empty( $r['content'] ) || empty( $r['user_id'] ) || empty( $r['activity_id'] ) ) {
    2534         $errors->add( 'missing_data', $feedback );
    2535         $bp->activity->errors['new_comment'] = $errors;
    2536 
    2537         return false;
     2555        $error = new WP_Error( 'missing_data', $feedback );
     2556
     2557        if ( 'wp_error' === $r['error_type'] ) {
     2558            return $error;
     2559
     2560        // Backpat.
     2561        } else {
     2562            $bp->activity->errors['new_comment'] = $error;
     2563            return false;
     2564        }
    25382565    }
    25392566
    … …  
    25502577    // Bail if the parent activity does not exist.
    25512578    if ( empty( $activity->date_recorded ) ) {
    2552         $errors->add( 'missing_activity', __( 'Sorry, the item you are replying to no longer exists.', 'buddypress' ) );
    2553         $bp->activity->errors['new_comment'] = $errors;
    2554 
    2555         return false;
     2579        $error = new WP_Error( 'missing_activity', __( 'The item you were replying to no longer exists.', 'buddypress' ) );
     2580
     2581        if ( 'wp_error' === $r['error_type'] ) {
     2582            return $error;
     2583
     2584        // Backpat.
     2585        } else {
     2586            $bp->activity->errors['new_comment'] = $error;
     2587            return false;
     2588        }
     2589
    25562590    }
    25572591
    … …  
    25782612        'item_id'           => $activity_id,
    25792613        'secondary_item_id' => $r['parent_id'],
    2580         'hide_sitewide'     => $is_hidden
     2614        'hide_sitewide'     => $is_hidden,
     2615        'error_type'        => $r['error_type']
    25812616    ) );
     2617
     2618    // Return WP Error.
     2619    if ( is_wp_error( $comment_id ) && 'wp_error' === $r['error_type'] ) {
     2620        return $comment_id;
     2621    }
    25822622
    25832623    // Comment caches are stored only with the top-level item.
    … …  
    26202660
    26212661    if ( empty( $comment_id ) ) {
    2622         $errors->add( 'comment_failed', $feedback );
    2623         $bp->activity->errors['new_comment'] = $errors;
     2662        $error = new WP_Error( 'comment_failed', $feedback );
     2663
     2664        if ( 'wp_error' === $r['error_type'] ) {
     2665            return $error;
     2666
     2667        // Backpat.
     2668        } else {
     2669            $bp->activity->errors['new_comment'] = $error;
     2670        }
    26242671    }
    26252672

  • trunk/src/bp-activity/classes/class-bp-activity-activity.php

    r10853 r10858  
    135135
    136136    /**
     137     * Error holder.
     138     *
     139     * @since 2.6.0
     140     *
     141     * @var WP_Error
     142     */
     143    public $errors;
     144
     145    /**
     146     * Error type to return. Either 'bool' or 'wp_error'.
     147     *
     148     * @since 2.6.0
     149     *
     150     * @var string
     151     */
     152    public $error_type = 'bool';
     153
     154    /**
    137155     * Constructor method.
    138156     *
    … …  
    142160     */
    143161    public function __construct( $id = false ) {
     162        // Instantiate errors object.
     163        $this->errors = new WP_Error;
     164
    144165        if ( !empty( $id ) ) {
    145166            $this->id = (int) $id;
    … …  
    236257        do_action_ref_array( 'bp_activity_before_save', array( &$this ) );
    237258
     259        if ( 'wp_error' === $this->error_type && $this->errors->get_error_code() ) {
     260            return $this->errors;
     261        }
     262
    238263        if ( empty( $this->component ) || empty( $this->type ) ) {
    239             return false;
     264            if ( 'bool' === $this->error_type ) {
     265                return false;
     266            } else {
     267                if ( empty( $this->component ) ) {
     268                    $this->errors->add( 'bp_activity_missing_component' );
     269                } else {
     270                    $this->errors->add( 'bp_activity_missing_type' );
     271                }
     272
     273                return $this->errors;
     274            }
    240275        }
    241276

  • trunk/src/bp-groups/bp-groups-activity.php

    r10454 r10858  
    380380        'secondary_item_id' => false,
    381381        'recorded_time'     => bp_core_current_time(),
    382         'hide_sitewide'     => $hide_sitewide
     382        'hide_sitewide'     => $hide_sitewide,
     383        'error_type'        => 'bool'
    383384    ) );
    384385

  • trunk/src/bp-groups/bp-groups-functions.php

    r10819 r10858  
    11501150 *
    11511151 * @since 1.2.0
     1152 * @since 2.6.0 Added 'error_type' parameter to $args.
    11521153 *
    11531154 * @param array|string $args {
    … …  
    11691170
    11701171    $defaults = array(
    1171         'content'  => false,
    1172         'user_id'  => bp_loggedin_user_id(),
    1173         'group_id' => 0
     1172        'content'    => false,
     1173        'user_id'    => bp_loggedin_user_id(),
     1174        'group_id'   => 0,
     1175        'error_type' => 'bool'
    11741176    );
    11751177
    … …  
    12121214
    12131215    $activity_id = groups_record_activity( array(
    1214         'user_id' => $user_id,
    1215         'action'  => $action,
    1216         'content' => $content_filtered,
    1217         'type'    => 'activity_update',
    1218         'item_id' => $group_id
     1216        'user_id'    => $user_id,
     1217        'action'     => $action,
     1218        'content'    => $content_filtered,
     1219        'type'       => 'activity_update',
     1220        'item_id'    => $group_id,
     1221        'error_type' => $error_type
    12191222    ) );
    12201223

  • trunk/src/bp-templates/bp-legacy/buddypress-functions.php

    r10825 r10858  
    920920
    921921    if ( ! $object && bp_is_active( 'activity' ) ) {
    922         $activity_id = bp_activity_post_update( array( 'content' => $_POST['content'] ) );
     922        $activity_id = bp_activity_post_update( array( 'content' => $_POST['content'], 'error_type' => 'wp_error' ) );
    923923
    924924    } elseif ( 'groups' === $object ) {
    925925        if ( $item_id && bp_is_active( 'groups' ) )
    926             $activity_id = groups_post_update( array( 'content' => $_POST['content'], 'group_id' => $item_id ) );
     926            $activity_id = groups_post_update( array( 'content' => $_POST['content'], 'group_id' => $item_id, 'error_type' => 'wp_error' ) );
    927927
    928928    } else {
    … …  
    932932    }
    933933
    934     if ( empty( $activity_id ) )
     934    if ( false === $activity_id ) {
    935935        exit( '-1<div id="message" class="error bp-ajax-message"><p>' . __( 'There was a problem posting your update. Please try again.', 'buddypress' ) . '</p></div>' );
     936    } elseif ( is_wp_error( $activity_id ) && $activity_id->get_error_code() ) {
     937        exit( '-1<div id="message" class="error bp-ajax-message"><p>' . $activity_id->get_error_message() . '</p></div>' );
     938    }
    936939
    937940    $last_recorded = ! empty( $_POST['since'] ) ? date( 'Y-m-d H:i:s', intval( $_POST['since'] ) ) : 0;
    … …  
    9981001        'content'     => $_POST['content'],
    9991002        'parent_id'   => $_POST['comment_id'],
     1003        'error_type'  => 'wp_error'
    10001004    ) );
    10011005
    1002     if ( ! $comment_id ) {
    1003         if ( ! empty( $bp->activity->errors['new_comment'] ) && is_wp_error( $bp->activity->errors['new_comment'] ) ) {
    1004             $feedback = $bp->activity->errors['new_comment']->get_error_message();
    1005             unset( $bp->activity->errors['new_comment'] );
    1006         }
    1007 
    1008         exit( '-1<div id="message" class="error bp-ajax-message"><p>' . esc_html( $feedback ) . '</p></div>' );
     1006    if ( is_wp_error( $comment_id ) ) {
     1007        exit( '-1<div id="message" class="error bp-ajax-message"><p>' . esc_html( $comment_id->get_error_message() ) . '</p></div>' );
    10091008    }
    10101009
Note: See TracChangeset for help on using the changeset viewer.