gabrielbull / php-ups-api

src/Entity/Tradeability/Shipment.php

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace Ups\Entity\Tradeability;
 
-use DomDocument;
-use DomElement;
 use Ups\Entity\FreightCharges;
 use Ups\NodeInterface;
 

Doc Comments +3 added lines, -3 removed lines

@@ -71,9 +71,9 @@ 
     private $tariffCodeAlert;
 
     /**
-     * @param null|DOMDocument $document
+     * @param null|\DOMDocument $document
      *
-     * @return DOMElement
+     * @return \DOMElement
      */
     public function toNode(DOMDocument $document = null)
     {
@@ -303,7 +303,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string|null
      */
     public function getTransactionReferenceId()
     {

src/Locator.php

Doc Comments +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
      * Create the TimeInTransit request.
      *
      * @param LocatorRequest $locatorRequest The request details. Refer to the UPS documentation for available structure
-     * @param $requestOption
+     * @param integer $requestOption
      *
      * @return string
      */

src/TimeInTransit.php

Doc Comments +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
      *
      * @throws Exception
      *
-     * @return TimeInTransitRequest
+     * @return TimeInTransitResponse
      */
     public function getTimeInTransit(TimeInTransitRequest $shipment)
     {
@@ -56,7 +56,7 @@ 
      *
      * @throws Exception
      *
-     * @return TimeInTransitRequest
+     * @return TimeInTransitResponse
      */
     private function sendRequest(TimeInTransitRequest $timeInTransitRequest)
     {
@@ -154,7 +154,7 @@ 
      *
      * @param SimpleXMLElement $response
      *
-     * @return TimeInTransitRequest
+     * @return TimeInTransitResponse
      */
     private function formatResponse(SimpleXMLElement $response)
     {

src/Entity/COD.php

Indentation +5 added lines, -5 removed lines

@@ -30,11 +30,11 @@
         }
     }
 
-     /**
-     * @param null|DOMDocument $document
-     *
-     * @return DOMElement
-     */
+        /**
+         * @param null|DOMDocument $document
+         *
+         * @return DOMElement
+         */
     public function toNode(DOMDocument $document = null)
     {
         if (null === $document) {

src/Tracking.php

Spacing +17 added lines, -17 removed lines

@@ -136,43 +136,43 @@
         $patterns = [
 
             // UPS Mail Innovations tracking numbers
-            '/^MI\d{6}\d{1,22}$/',// MI 000000 00000000+
+            '/^MI\d{6}\d{1,22}$/', // MI 000000 00000000+
 
             // USPS - Certified Mail
-            '/^94071\d{17}$/',    // 9407 1000 0000 0000 0000 00
-            '/^7\d{19}$/',        // 7000 0000 0000 0000 0000
+            '/^94071\d{17}$/', // 9407 1000 0000 0000 0000 00
+            '/^7\d{19}$/', // 7000 0000 0000 0000 0000
 
             // USPS - Collect on Delivery
-            '/^93033\d{17}$/',    // 9303 3000 0000 0000 0000 00
-            '/^M\d{9}$/',         // M000 0000 00
+            '/^93033\d{17}$/', // 9303 3000 0000 0000 0000 00
+            '/^M\d{9}$/', // M000 0000 00
 
             // USPS - Global Express Guaranteed
-            '/^82\d{10}$/',       // 82 000 000 00
+            '/^82\d{10}$/', // 82 000 000 00
 
             // USPS - Priority Mail Express International
-            '/^EC\d{9}US$/',      // EC 000 000 000 US
+            '/^EC\d{9}US$/', // EC 000 000 000 US
 
             // USPS - Priority Mail Express
-            '/^927\d{19}$/',      // 9270 1000 0000 0000 0000 00
-            '/^EA\d{9}US$/',      // EA 000 000 000 US
+            '/^927\d{19}$/', // 9270 1000 0000 0000 0000 00
+            '/^EA\d{9}US$/', // EA 000 000 000 US
 
             // USPS - Priority Mail International
-            '/^CP\d{9}US$/',      // CP 000 000 000 US
+            '/^CP\d{9}US$/', // CP 000 000 000 US
 
             // USPS - Priority Mail
-            '/^92055\d{17}$/',    // 9205 5000 0000 0000 0000 00
-            '/^14\d{18}$/',       // 1400 0000 0000 0000 0000
+            '/^92055\d{17}$/', // 9205 5000 0000 0000 0000 00
+            '/^14\d{18}$/', // 1400 0000 0000 0000 0000
 
             // USPS - Registered Mail
-            '/^92088\d{17}$/',    // 9208 8000 0000 0000 0000 00
-            '/^RA\d{9}US$/',      // RA 000 000 000 US
+            '/^92088\d{17}$/', // 9208 8000 0000 0000 0000 00
+            '/^RA\d{9}US$/', // RA 000 000 000 US
 
             // USPS - Signature Confirmation
-            '/^9202\d{16}US$/',   // 9202 1000 0000 0000 0000 00
-            '/^23\d{16}US$/',     // 2300 0000 0000 0000 0000
+            '/^9202\d{16}US$/', // 9202 1000 0000 0000 0000 00
+            '/^23\d{16}US$/', // 2300 0000 0000 0000 0000
 
             // USPS - Tracking
-            '/^94\d{20}$/',       // 9400 1000 0000 0000 0000 00
+            '/^94\d{20}$/', // 9400 1000 0000 0000 0000 00
             '/^03\d{18}$/'        // 0300 0000 0000 0000 0000
         ];
 

src/Shipping.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      *
      * @throws Exception
      *
-     * @return stdClass
+     * @return \stdClass
      */
     public function confirm(
         $validation,

src/Tradeability.php

Doc Comments +5 added lines, -5 removed lines

@@ -112,14 +112,14 @@
     /**
      * Creates and sends a request for the given data. Most errors are handled in SoapRequest
      *
-     * @param $request
-     * @param $endpoint
-     * @param $operation
-     * @param $wsdl
+     * @param string $request
+     * @param string $endpoint
+     * @param string $operation
+     * @param string $wsdl
      *
      * @throws Exception
      *
-     * @return TimeInTransitRequest
+     * @return \stdClass
      */
     private function sendRequest($request, $endpoint, $operation, $wsdl)
     {

src/Ups.php

Spacing +1 added lines, -1 removed lines

@@ -212,6 +212,6 @@
     {
         $base = ($this->useIntegration ? $this->integrationBaseUrl : $this->productionBaseUrl);
 
-        return $base . $segment;
+        return $base.$segment;
     }
 }

src/Entity/AddressArtifactFormat.php

Spacing +2 added lines, -2 removed lines

@@ -40,9 +40,9 @@
         $node = $document->createElement('AddressArtifactFormat');
 
         for ($i = 1; $i <= 3; $i++) {
-            $line = $this->{'getPoliticalDivision' . $i}();
+            $line = $this->{'getPoliticalDivision'.$i}();
             if ($line) {
-                $node->appendChild($document->createElement('PoliticalDivision' . $i, $line));
+                $node->appendChild($document->createElement('PoliticalDivision'.$i, $line));
             }
         }