WordPress.org
Get WordPress

Making WordPress.org


Ignore:
Timestamp:
10/17/2018 11:01:41 PM (6 years ago)
Author:
iandunn
Message:

WordCamp Payments: Apply coding standards.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/privacy.php

    r7737 r7738  
    1212add_filter( 'the_posts',                          __NAMESPACE__ . '\hide_others_payment_files', 10, 2 );
    1313add_filter( 'wp_privacy_personal_data_exporters', __NAMESPACE__ . '\register_personal_data_exporters' );
    14 add_filter( 'wp_privacy_personal_data_erasers', __NAMESPACE__ . '\register_personal_data_erasers' );
     14add_filter( 'wp_privacy_personal_data_erasers',   __NAMESPACE__ . '\register_personal_data_erasers'  );
    1515
    1616
    … …  
    9090        'post_type'   => array(
    9191            Reimbursement_Requests\POST_TYPE,
    92             WCP_Payment_Request::POST_TYPE
     92            WCP_Payment_Request::POST_TYPE,
    9393        ),
    9494    ) );
    … …  
    108108     * This is an empty stub, we are not adding an eraser for now, because it contains data which can be used for
    109109     * accounting or reference purpose.
    110      *
    111110     */
    112111    return $erasers;
    … …  
    138137 *
    139138 * @param string $email_address
    140  * @param int $page
     139 * @param int    $page
    141140 *
    142141 * @return array
    143142 */
    144143function vendor_payment_exporter( $email_address, $page ) {
    145 
    146144    $results = array(
    147145        'data' => array(),
    … …  
    149147    );
    150148
    151     $vendor_payment_requests = get_post_wp_query( \WCP_Payment_Request::POST_TYPE, $page, $email_address );
     149    $vendor_payment_requests = get_post_wp_query( WCP_Payment_Request::POST_TYPE, $page, $email_address );
    152150
    153151    if ( empty( $vendor_payment_requests ) ) {
    … …  
    156154
    157155    $data_to_export = array();
     156
    158157    foreach ( $vendor_payment_requests->posts as $post ) {
    159158        $vendor_payment_exp_data = array();
    … …  
    170169
    171170        $vendor_payment_exp_data = array_merge(
    172             $vendor_payment_exp_data, get_meta_details( $meta, \WCP_Payment_Request::POST_TYPE )
     171            $vendor_payment_exp_data, get_meta_details( $meta, WCP_Payment_Request::POST_TYPE )
    173172        );
    174173
    175174        if ( ! empty( $vendor_payment_exp_data ) ) {
    176175            $data_to_export[] = array(
    177                 'group_id'    => \WCP_Payment_Request::POST_TYPE,
     176                'group_id'    => WCP_Payment_Request::POST_TYPE,
    178177                'group_label' => __( 'WordCamp Vendor Payments', 'wordcamporg' ),
    179                 'item_id'     => \WCP_Payment_Request::POST_TYPE . "-{$post->ID}",
     178                'item_id'     => WCP_Payment_Request::POST_TYPE . "-{$post->ID}",
    180179                'data'        => $vendor_payment_exp_data,
    181180            );
    … …  
    193192 *
    194193 * @param string $email_address
    195  * @param int $page
     194 * @param int    $page
    196195 *
    197196 * @return array
    198197 */
    199198function reimbursements_exporter( $email_address, $page ) {
    200 
    201199    $results = array(
    202200        'data' => array(),
    … …  
    211209
    212210    $data_to_export = array();
     211
    213212    foreach ( $reimbursements->posts as $post ) {
    214213        $reimbursement_data_to_export = array();
    … …  
    224223        ];
    225224
    226         // meta fields
     225        // Meta fields.
    227226        $reimbursement_data_to_export = array_merge(
    228227            $reimbursement_data_to_export, get_meta_details( $meta, Reimbursement_Requests\POST_TYPE )
    229228        );
    230 
    231229
    232230        if ( ! empty( $reimbursement_data_to_export ) ) {
    … …  
    252250 * because the vendor contact details could be of an individual (instead of a business), and thus is a potential PII
    253251 *
    254  * @param $query_type string
    255  * @param $page integer
    256  * @param $email_address string Email address of the entity making the request
     252 * @param string  $query_type
     253 * @param integer $page
     254 * @param string  $email_address Email address of the entity making the request.
    257255 *
    258256 * @return null|WP_Query
    259257 */
    260258function get_post_wp_query( $query_type, $page, $email_address ) {
    261 
    262259    $query_args = array(
    263260        'post_type'      => $query_type,
    … …  
    269266
    270267    switch ( $query_type ) {
    271         case Reimbursement_Requests\POST_TYPE :
     268        case Reimbursement_Requests\POST_TYPE:
    272269            $user = get_user_by( 'email', $email_address );
    273270
    … …  
    278275            $query_args = array_merge( $query_args, array( 'post_author' => $user->ID ) );
    279276            break;
    280         case \WCP_Payment_Request::POST_TYPE :
     277
     278        case WCP_Payment_Request::POST_TYPE:
    281279            $query_args['meta_query'] = [
    282280                'relation' => 'AND',
    … …  
    288286            ];
    289287            break;
    290         default :
     288
     289        default:
    291290            return null;
    292291    }
    … …  
    296295
    297296/**
    298  * @param $meta array meta object of post, as retrieved by `get_post_meta( $post->ID )`
    299  * @param $post_type string post_type . could be one of wcb_reimbursement or wcp_payment_request
     297 *
     298 * @param array  $meta      Meta object of post, as retrieved by `get_post_meta( $post->ID )`.
     299 * @param string $post_type Post type. Could be one of `wcb_reimbursement` or `wcp_payment_request`.
    300300 *
    301301 * @return array Details of the reimbursement request
    … …  
    303303function get_meta_details( $meta, $post_type ) {
    304304    $meta_details = array();
     305
    305306    foreach ( get_meta_fields_mapping( $post_type ) as $meta_field => $meta_field_name ) {
    306307        $data = isset( $meta[ $meta_field ] ) ? $meta[ $meta_field ] : null;
     308
    307309        if ( ! empty( $data ) && is_array( $data ) && ! empty( $data[0] ) ) {
    308310            $meta_details[] = [
    … …  
    319321 * Returns array of meta fields and their titles that we want to allow export for.
    320322 *
    321  * @param $post_type string
     323 * @param string $post_type
    322324 *
    323325 * @return array
    … …  
    327329
    328330    if ( Reimbursement_Requests\POST_TYPE === $post_type ) {
    329         $prefix = '_wcbrr_';
     331        $prefix         = '_wcbrr_';
    330332        $mapping_fields = array_merge(
    331333            $mapping_fields,
    … …  
    335337                $prefix . 'payment_method'              => __( 'Payment Method', 'wordcamporg' ),
    336338
    337                 // Payment Method - Direct Deposit
     339                // Payment Method - Direct Deposit.
    338340                $prefix . 'ach_bank_name'               => __( 'Bank Name', 'wordcamporg' ),
    339341                $prefix . 'ach_account_type'            => __( 'Account Type', 'wordcamporg' ),
    … …  
    342344                $prefix . 'ach_account_holder_name'     => __( 'Account Holder Name', 'wordcamporg' ),
    343345
    344                 // Payment Method - Check
     346                // Payment Method - Check.
    345347                $prefix . 'payable_to'                  => __( 'Payable To', 'wordcamporg' ),
    346348                $prefix . 'check_street_address'        => __( 'Street Address', 'wordcamporg' ),
    … …  
    350352                $prefix . 'check_country'               => __( 'Country', 'wordcamporg' ),
    351353
    352                 // Payment Method - Wire
     354                // Payment Method - Wire.
    353355                $prefix . 'bank_name'                   => __( 'Beneficiary’s Bank Name', 'wordcamporg' ),
    354356                $prefix . 'bank_street_address'         => __( 'Beneficiary’s Bank Street Address', 'wordcamporg' ),
    … …  
    360362                $prefix . 'beneficiary_account_number'  => __( 'Beneficiary’s Account Number or IBAN', 'wordcamporg' ),
    361363
    362                 // Intermediary bank details
     364                // Intermediary bank details.
    363365                $prefix . 'interm_bank_name'            => __( 'Intermediary Bank Name', 'wordcamporg' ),
    364366                $prefix . 'interm_bank_street_address'  => __( 'Intermediary Bank Street Address', 'wordcamporg' ),
    … …  
    376378                $prefix . 'beneficiary_zip_code'        => __( 'Beneficiary’s ZIP / Postal Code', 'wordcamporg' ),
    377379                $prefix . 'beneficiary_country_iso3166' => __( 'Beneficiary’s Country', 'wordcamporg' ),
    378 
    379380            )
    380381        );
    381     } elseif ( \WCP_Payment_Request::POST_TYPE === $post_type ) {
    382         $prefix = '_camppayments_';
     382    } elseif ( WCP_Payment_Request::POST_TYPE === $post_type ) {
     383        $prefix         = '_camppayments_';
    383384        $mapping_fields = array_merge(
    384385            $mapping_fields,
    385386            array(
    386                 // Vendor payment fields
     387                // Vendor payment fields.
    387388                $prefix . 'description'            => __( 'Description', 'wordcamporg' ),
    388389                $prefix . 'general_notes'          => __( 'Notes', 'wordcamporg' ),
    … …  
    401402    return $mapping_fields;
    402403}
    403 
Note: See TracChangeset for help on using the changeset viewer.