Merge pull request #4947 from BFTrick/shipping-php-standards
Use WordPress Brace Code Style in Flat Rate Shipping Class #2
This commit is contained in:
commit
6d7e7370c6
|
@ -177,23 +177,25 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
|
||||
$shipping_total = $this->order_shipping( $package );
|
||||
|
||||
if ( ! is_null( $shipping_total ) || $cost_per_order > 0 )
|
||||
if ( ! is_null( $shipping_total ) || $cost_per_order > 0 ) {
|
||||
$rate = array(
|
||||
'id' => $this->id,
|
||||
'label' => $this->title,
|
||||
'cost' => $shipping_total + $cost_per_order,
|
||||
);
|
||||
}
|
||||
|
||||
} elseif ( $this->type == 'class' ) {
|
||||
|
||||
$shipping_total = $this->class_shipping( $package );
|
||||
|
||||
if ( ! is_null( $shipping_total ) || $cost_per_order > 0 )
|
||||
if ( ! is_null( $shipping_total ) || $cost_per_order > 0 ) {
|
||||
$rate = array(
|
||||
'id' => $this->id,
|
||||
'label' => $this->title,
|
||||
'cost' => $shipping_total + $cost_per_order,
|
||||
);
|
||||
}
|
||||
|
||||
} elseif ( $this->type == 'item' ) {
|
||||
|
||||
|
@ -201,8 +203,9 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
|
||||
if ( ! is_null( $costs ) || $cost_per_order > 0 ) {
|
||||
|
||||
if ( ! is_array( $costs ) )
|
||||
if ( ! is_array( $costs ) ) {
|
||||
$costs = array();
|
||||
}
|
||||
|
||||
$costs['order'] = $cost_per_order;
|
||||
|
||||
|
@ -216,25 +219,29 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
}
|
||||
}
|
||||
|
||||
if ( isset( $rate ) )
|
||||
if ( isset( $rate ) ) {
|
||||
$this->add_rate( $rate );
|
||||
}
|
||||
|
||||
// Add any extra rates
|
||||
if ( sizeof( $this->options ) > 0) {
|
||||
|
||||
if ( ! isset( $rate ) )
|
||||
if ( ! isset( $rate ) ) {
|
||||
$rate = array(
|
||||
'id' => $this->id,
|
||||
'label' => $this->title,
|
||||
'cost' => 0,
|
||||
);
|
||||
}
|
||||
|
||||
// Get item qty
|
||||
$total_quantity = 0;
|
||||
|
||||
foreach ( $package['contents'] as $item_id => $values )
|
||||
if ( $values['quantity'] > 0 && $values['data']->needs_shipping() )
|
||||
foreach ( $package['contents'] as $item_id => $values ) {
|
||||
if ( $values['quantity'] > 0 && $values['data']->needs_shipping() ) {
|
||||
$total_quantity += $values['quantity'];
|
||||
}
|
||||
}
|
||||
|
||||
// Loop options
|
||||
foreach ( $this->options as $option ) {
|
||||
|
@ -406,8 +413,9 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
$found_shipping_classes_values = array();
|
||||
|
||||
foreach ( $found_shipping_classes as $shipping_class => $products ) {
|
||||
if ( ! isset( $found_shipping_classes_values[ $shipping_class ] ) )
|
||||
if ( ! isset( $found_shipping_classes_values[ $shipping_class ] ) ) {
|
||||
$found_shipping_classes_values[ $shipping_class ] = 0;
|
||||
}
|
||||
|
||||
foreach ( $products as $product ) {
|
||||
$found_shipping_classes_values[ $shipping_class ] += $product['data']->get_price() * $product['quantity'];
|
||||
|
@ -430,11 +438,12 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
}
|
||||
|
||||
// Total
|
||||
if ( $matched )
|
||||
if ( $matched ) {
|
||||
return $cost + $fee;
|
||||
else
|
||||
} else {
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
|
@ -473,11 +482,12 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
}
|
||||
}
|
||||
|
||||
if ( $matched )
|
||||
if ( $matched ) {
|
||||
return $costs;
|
||||
else
|
||||
} else {
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Finds and returns shipping classes and the products with said class.
|
||||
|
@ -494,8 +504,9 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
foreach ( $package['contents'] as $item_id => $values ) {
|
||||
if ( $values['data']->needs_shipping() ) {
|
||||
$found_class = $values['data']->get_shipping_class();
|
||||
if ( ! isset( $found_shipping_classes[ $found_class ] ) )
|
||||
if ( ! isset( $found_shipping_classes[ $found_class ] ) ) {
|
||||
$found_shipping_classes[ $found_class ] = array();
|
||||
}
|
||||
|
||||
$found_shipping_classes[ $found_class ][ $item_id ] = $values;
|
||||
}
|
||||
|
@ -654,10 +665,11 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
|
||||
$flat_rate_cost[ $i ] = wc_format_decimal( $flat_rate_cost[$i] );
|
||||
|
||||
if ( ! strstr( $flat_rate_fee[$i], '%' ) )
|
||||
if ( ! strstr( $flat_rate_fee[$i], '%' ) ) {
|
||||
$flat_rate_fee[ $i ] = wc_format_decimal( $flat_rate_fee[$i] );
|
||||
else
|
||||
} else {
|
||||
$flat_rate_fee[ $i ] = wc_clean( $flat_rate_fee[$i] );
|
||||
}
|
||||
|
||||
// Add to flat rates array
|
||||
$flat_rates[ sanitize_title($flat_rate_class[$i]) ] = array(
|
||||
|
@ -682,10 +694,11 @@ class WC_Shipping_Flat_Rate extends WC_Shipping_Method {
|
|||
function save_default_costs( $fields ) {
|
||||
$default_cost = ( $_POST['default_cost'] === '' ) ? '' : wc_format_decimal( $_POST['default_cost'] );
|
||||
|
||||
if ( ! strstr( $_POST['default_fee'], '%' ) )
|
||||
if ( ! strstr( $_POST['default_fee'], '%' ) ) {
|
||||
$default_fee = ( $_POST['default_fee'] === '' ) ? '' : wc_format_decimal( $_POST['default_fee'] );
|
||||
else
|
||||
} else {
|
||||
$default_fee = wc_clean( $_POST['default_fee'] );
|
||||
}
|
||||
|
||||
$fields['cost'] = $default_cost;
|
||||
$fields['fee'] = $default_fee;
|
||||
|
|
Loading…
Reference in New Issue