gabrielbull / php-ups-api

src/Entity/ItemizedPaymentInformation.php

Doc Comments +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
     }
 
     /**
-     * @return transportationShipmentCharge
+     * @return ShipmentCharge
      */
     public function getTransportationShipmentCharge()
     {
@@ -43,7 +43,7 @@ 
     }
 
     /**
-     * @return dutiesAndTaxesShipmentCharge
+     * @return ShipmentCharge
      */
     public function getDutiesAndTaxesShipmentCharge()
     {

Indentation +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
     public function setShipmentCharge(ShipmentCharge $shipmentCharge)
     {
         if ($shipmentCharge->getType() == ShipmentCharge::SHIPMENT_CHARGE_TYPE_TRANSPORTATION) {
-           $this->transportationShipmentCharge = $shipmentCharge;
+            $this->transportationShipmentCharge = $shipmentCharge;
         } else if ($shipmentCharge->getType() == ShipmentCharge::SHIPMENT_CHARGE_TYPE_DUTIES) {
             $this->dutiesAndTaxesShipmentCharge = $shipmentCharge;
         } else {

src/Entity/ShipmentCharge.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
      * @param BillShipper $billShipper
      * @return ShipmentCharge
      */
-    public function setBillShipper(BillShipper $billShipper= null)
+    public function setBillShipper(BillShipper $billShipper = null)
     {
         $this->billShipper = $billShipper;
 
@@ -104,7 +104,7 @@ 
      * @param BillReceiver $billReceiver
      * @return ShipmentCharge
      */
-    public function setBillReceiver(BillReceiver $billReceiver= null)
+    public function setBillReceiver(BillReceiver $billReceiver = null)
     {
         $this->billReceiver = $billReceiver;
 

src/Shipping.php

Indentation +9 added lines, -9 removed lines

@@ -310,15 +310,15 @@ 
 
             for ($shipmentChargeRec = 1; $shipmentChargeRec <= 2; $shipmentChargeRec++) {
                 if ($shipmentChargeRec == 1) { 
-                   $rec = $shipment->getItemizedPaymentInformation()->getTransportationShipmentCharge(); 
-                   if ($rec == null) continue; 
-                   $node = $paymentNode->appendChild($xml->createElement('ShipmentCharge'));
-                   $node->appendChild($xml->createElement('Type', \Ups\Entity\ShipmentCharge::SHIPMENT_CHARGE_TYPE_TRANSPORTATION));
+                    $rec = $shipment->getItemizedPaymentInformation()->getTransportationShipmentCharge(); 
+                    if ($rec == null) continue; 
+                    $node = $paymentNode->appendChild($xml->createElement('ShipmentCharge'));
+                    $node->appendChild($xml->createElement('Type', \Ups\Entity\ShipmentCharge::SHIPMENT_CHARGE_TYPE_TRANSPORTATION));
                 } else {
-                   $rec = $shipment->getItemizedPaymentInformation()->getDutiesAndTaxesShipmentCharge(); 
-                   if ($rec == null) continue; 
-                   $node = $paymentNode->appendChild($xml->createElement('ShipmentCharge'));
-                   $node->appendChild($xml->createElement('Type', \Ups\Entity\ShipmentCharge::SHIPMENT_CHARGE_TYPE_DUTIES));
+                    $rec = $shipment->getItemizedPaymentInformation()->getDutiesAndTaxesShipmentCharge(); 
+                    if ($rec == null) continue; 
+                    $node = $paymentNode->appendChild($xml->createElement('ShipmentCharge'));
+                    $node->appendChild($xml->createElement('Type', \Ups\Entity\ShipmentCharge::SHIPMENT_CHARGE_TYPE_DUTIES));
                 }
                 
                 if ($rec->getBillShipper()) {
@@ -362,7 +362,7 @@ 
                 }
             }  
             if ($shipment->getItemizedPaymentInformation()->getSplitDutyVATIndicator()) { 
-               $paymentNode->appendChild($xml->createElement('SplitDutyVATIndicator'));
+                $paymentNode->appendChild($xml->createElement('SplitDutyVATIndicator'));
             }
         }
 

Braces +6 added lines, -2 removed lines

@@ -311,12 +311,16 @@
             for ($shipmentChargeRec = 1; $shipmentChargeRec <= 2; $shipmentChargeRec++) {
                 if ($shipmentChargeRec == 1) { 
                    $rec = $shipment->getItemizedPaymentInformation()->getTransportationShipmentCharge(); 
-                   if ($rec == null) continue; 
+                   if ($rec == null) {
+                       continue;
+                   }
                    $node = $paymentNode->appendChild($xml->createElement('ShipmentCharge'));
                    $node->appendChild($xml->createElement('Type', \Ups\Entity\ShipmentCharge::SHIPMENT_CHARGE_TYPE_TRANSPORTATION));
                 } else {
                    $rec = $shipment->getItemizedPaymentInformation()->getDutiesAndTaxesShipmentCharge(); 
-                   if ($rec == null) continue; 
+                   if ($rec == null) {
+                       continue;
+                   }
                    $node = $paymentNode->appendChild($xml->createElement('ShipmentCharge'));
                    $node->appendChild($xml->createElement('Type', \Ups\Entity\ShipmentCharge::SHIPMENT_CHARGE_TYPE_DUTIES));
                 }