update name to reflect logic behind margin

This commit is contained in:
Bilal Catic
2018-10-07 14:09:11 +02:00
parent ea63870704
commit ac9c89e8eb
3 changed files with 14 additions and 14 deletions

View File

@@ -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']);
}
}

View File

@@ -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',
)
);

View File

@@ -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;
}
}