← All changes
|
src/Packetery/Module/Checkout/ShippingRateFactory.php
+1
-29
trunk
→
2.1
View file →
| @@ -7,9 +7,8 @@ | ||
| 7 | 7 | use Packetery\Core\Entity; |
| 8 | 8 | use Packetery\Module\Carrier; |
| 9 | 9 | use Packetery\Module\Carrier\CarDeliveryConfig; |
| 10 | 10 | use Packetery\Module\Carrier\CarrierOptionsFactory; |
| 11 | -use Packetery\Module\DiagnosticsLogger\DiagnosticsLogger; | |
| 12 | 11 | use Packetery\Module\Exception\ProductNotFoundException; |
| 13 | 12 | use Packetery\Module\Framework\WcAdapter; |
| 14 | 13 | use Packetery\Module\Framework\WpAdapter; |
| 15 | 14 | use Packetery\Module\Options\OptionsProvider; |
| @@ -62,13 +61,8 @@ | ||
| 62 | 61 | * @var OptionsProvider |
| 63 | 62 | */ |
| 64 | 63 | private $optionsProvider; |
| 65 | 64 | |
| 66 | - /** | |
| 67 | - * @var DiagnosticsLogger | |
| 68 | - */ | |
| 69 | - private $diagnosticsLogger; | |
| 70 | - | |
| 71 | 65 | public function __construct( |
| 72 | 66 | WpAdapter $wpAdapter, |
| 73 | 67 | WcAdapter $wcAdapter, |
| 74 | 68 | CheckoutService $checkoutService, |
| @@ -76,10 +70,9 @@ | ||
| 76 | 70 | CartService $cartService, |
| 77 | 71 | CarrierOptionsFactory $carrierOptionsFactory, |
| 78 | 72 | CarDeliveryConfig $carDeliveryConfig, |
| 79 | 73 | RateCalculator $rateCalculator, |
| 80 | - OptionsProvider $optionsProvider, | |
| 81 | - DiagnosticsLogger $diagnosticsLogger | |
| 74 | + OptionsProvider $optionsProvider | |
| 82 | 75 | ) { |
| 83 | 76 | $this->wpAdapter = $wpAdapter; |
| 84 | 77 | $this->wcAdapter = $wcAdapter; |
| 85 | 78 | $this->checkoutService = $checkoutService; |
| @@ -88,9 +81,8 @@ | ||
| 88 | 81 | $this->carrierOptionsFactory = $carrierOptionsFactory; |
| 89 | 82 | $this->carDeliveryConfig = $carDeliveryConfig; |
| 90 | 83 | $this->rateCalculator = $rateCalculator; |
| 91 | 84 | $this->optionsProvider = $optionsProvider; |
| 92 | - $this->diagnosticsLogger = $diagnosticsLogger; | |
| 93 | 85 | } |
| 94 | 86 | |
| 95 | 87 | /** |
| 96 | 88 | * Prepare shipping rates based on cart properties. |
| @@ -103,17 +95,8 @@ | ||
| 103 | 95 | * @throws ProductNotFoundException Product not found. |
| 104 | 96 | */ |
| 105 | 97 | public function createShippingRates( ?array $allowedCarrierNames, string $methodId, int $instanceId ): array { |
| 106 | 98 | $customerCountry = $this->checkoutService->getCustomerCountry(); |
| 107 | - $this->diagnosticsLogger->log( | |
| 108 | - 'createShippingRates parameters', | |
| 109 | - [ | |
| 110 | - 'customerCountry' => $customerCountry, | |
| 111 | - 'allowedCarrierNames' => $allowedCarrierNames, | |
| 112 | - 'methodId' => $methodId, | |
| 113 | - 'instanceId' => $instanceId, | |
| 114 | - ] | |
| 115 | - ); | |
| 116 | 99 | if ( $customerCountry === null ) { |
| 117 | 100 | return []; |
| 118 | 101 | } |
| 119 | 102 | |
| @@ -141,19 +124,8 @@ | ||
| 141 | 124 | $carrier, |
| 142 | 125 | $allowedCarrierNames, |
| 143 | 126 | $optionId, |
| 144 | 127 | $rateId |
| 145 | - ); | |
| 146 | - | |
| 147 | - $this->diagnosticsLogger->log( | |
| 148 | - 'Shipping rate created or is null', | |
| 149 | - [ | |
| 150 | - 'shippingRate' => $shippingRate, | |
| 151 | - 'carrier' => $carrier, | |
| 152 | - 'allowedCarrierNames' => $allowedCarrierNames, | |
| 153 | - 'rateId' => $rateId, | |
| 154 | - 'optionId' => $optionId, | |
| 155 | - ] | |
| 156 | 128 | ); |
| 157 | 129 | |
| 158 | 130 | if ( $shippingRate !== null ) { |
| 159 | 131 | $customRates[ $rateId ] = $shippingRate; |