update name to reflect logic behind margin
This commit is contained in:
@@ -58,7 +58,7 @@ class Wiaas_Admin_Package_Pricing {
|
||||
$package = wc_get_product( $post->ID );
|
||||
$pricing_rules = Wiaas_Package_Pricing::get_package_prices($package);
|
||||
$commission = Wiaas_Package_Pricing::get_package_pricing_commission($package);
|
||||
$minimum_cost_margin = Wiaas_Package_Pricing::get_package_minimum_cost_margin($package);
|
||||
$max_cost_margin = Wiaas_Package_Pricing::get_package_max_cost_margin($package);
|
||||
|
||||
include 'views/html-package-pricing.php';
|
||||
}
|
||||
@@ -73,7 +73,7 @@ class Wiaas_Admin_Package_Pricing {
|
||||
wc_get_product( $post_id ),
|
||||
$_POST['wiaas_pricing_rules'],
|
||||
$_POST['wiaas_pricing_rules_commision'],
|
||||
$_POST['wiaas_minimum_cost_margin']);
|
||||
$_POST['wiaas_max_cost_margin']);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -148,10 +148,10 @@ if ( ! defined( 'ABSPATH' ) ) {
|
||||
<?php
|
||||
woocommerce_wp_text_input(
|
||||
array(
|
||||
'id' => '_wiaas_minimum_cost_margin',
|
||||
'name' => 'wiaas_minimum_cost_margin',
|
||||
'value' => $minimum_cost_margin,
|
||||
'label' => __( 'Minimum cost margin:', 'wiaas' ),
|
||||
'id' => '_wiaas_max_cost_margin',
|
||||
'name' => 'wiaas_max_cost_margin',
|
||||
'value' => $max_cost_margin,
|
||||
'label' => __( 'Max cost margin:', 'wiaas' ),
|
||||
'type' => 'number',
|
||||
)
|
||||
);
|
||||
|
||||
@@ -101,8 +101,8 @@ class Wiaas_Package_Pricing {
|
||||
return self::_get_package_pricing_commision($package);
|
||||
}
|
||||
|
||||
public static function get_package_minimum_cost_margin($package){
|
||||
return self::_get_package_minimum_cost_margin($package);
|
||||
public static function get_package_max_cost_margin($package){
|
||||
return self::_get_package_max_cost_margin($package);
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -110,11 +110,11 @@ class Wiaas_Package_Pricing {
|
||||
* @param $package
|
||||
* @param $pricing_rules
|
||||
*/
|
||||
public static function set_package_prices($package, $pricing_rules, $commision, $minimum_cost_margin) {
|
||||
public static function set_package_prices($package, $pricing_rules, $commision, $max_cost_margin) {
|
||||
if ( isset( $pricing_rules ) ) {
|
||||
$package->update_meta_data( '_wiaas_pricing_rules', $pricing_rules );
|
||||
$package->update_meta_data('_package_pricing_commision', $commision, true);
|
||||
$package->update_meta_data('_package_minimum_cost_margin', $minimum_cost_margin, true);
|
||||
$package->update_meta_data('_package_max_cost_margin', $max_cost_margin, true);
|
||||
} else {
|
||||
$package->delete_meta_data( '_wiaas_pricing_rules' );
|
||||
}
|
||||
@@ -160,14 +160,14 @@ class Wiaas_Package_Pricing {
|
||||
return (float) $commision;
|
||||
}
|
||||
|
||||
private static function _get_package_minimum_cost_margin($package) {
|
||||
$minimum_cost_margin = $package->get_meta( '_package_minimum_cost_margin', true);
|
||||
private static function _get_package_max_cost_margin($package) {
|
||||
$max_cost_margin = $package->get_meta( '_package_max_cost_margin', true);
|
||||
|
||||
if (!isset($minimum_cost_margin) || $minimum_cost_margin === '') {
|
||||
if (!isset($max_cost_margin) || $max_cost_margin === '') {
|
||||
return 0;
|
||||
}
|
||||
|
||||
return (float) $minimum_cost_margin;
|
||||
return (float) $max_cost_margin;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user