diff --git a/src/main/edu/wisc/services/cbs/api/MockService.php b/src/main/edu/wisc/services/cbs/api/MockService.php
index 280cf16ca8efdb290eed3e4f721a4880341c336a..940b6612e4c7a6b0bf7305859387c296e0771d46 100644
--- a/src/main/edu/wisc/services/cbs/api/MockService.php
+++ b/src/main/edu/wisc/services/cbs/api/MockService.php
@@ -1,7 +1,7 @@
<?php
namespace edu\wisc\services\cbs\api;
-use edu\wisc\services\cbs\common\AbstractServiceResponse;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
/**
* A mock implementation of a CBS service.
@@ -12,9 +12,9 @@ interface MockService extends Service
/**
* Creates a mock implementation of a {@link Service}.
*
- * Service will respond with the given {@link AbstractServiceResponse}, or a successful response if one is not
+ * Service will respond with the given {@link ServiceResponseInterface}, or a successful response if one is not
* given.
- * @param AbstractServiceResponse|null $response defaults to successful if null
+ * @param ServiceResponseInterface|null $response defaults to successful if null
*/
- public function __construct(AbstractServiceResponse $response = null);
+ public function __construct(ServiceResponseInterface $response = null);
}
diff --git a/src/main/edu/wisc/services/cbs/common/AbstractServiceResponse.php b/src/main/edu/wisc/services/cbs/common/AbstractServiceResponse.php
index a8d6ae8e671a26579d149c61aba9c7f30e7bde41..603c75ace0d64e513da9f2d190ec06543ccf72e4 100644
--- a/src/main/edu/wisc/services/cbs/common/AbstractServiceResponse.php
+++ b/src/main/edu/wisc/services/cbs/common/AbstractServiceResponse.php
@@ -5,7 +5,7 @@ namespace edu\wisc\services\cbs\common;
/**
* Abstract base class defining common service response functions.
*/
-abstract class AbstractServiceResponse
+abstract class AbstractServiceResponse implements ServiceResponseInterface
{
/** @var boolean */
@@ -28,7 +28,7 @@ abstract class AbstractServiceResponse
/**
* @return bool
*/
- public function isSuccess()
+ public function isSuccess(): bool
{
return $this->success;
}
@@ -36,7 +36,7 @@ abstract class AbstractServiceResponse
/**
* @return string
*/
- public function getMessage()
+ public function getMessage(): string
{
return $this->message;
}
diff --git a/src/main/edu/wisc/services/cbs/common/ServiceResponseInterface.php b/src/main/edu/wisc/services/cbs/common/ServiceResponseInterface.php
new file mode 100644
index 0000000000000000000000000000000000000000..fdc7d529cb191b852e3cd04a2054033b99135f75
--- /dev/null
+++ b/src/main/edu/wisc/services/cbs/common/ServiceResponseInterface.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace edu\wisc\services\cbs\common;
+
+/**
+ */
+interface ServiceResponseInterface
+{
+
+ /**
+ * @return bool
+ */
+ public function isSuccess(): bool;
+
+ /**
+ * @return string
+ */
+ public function getMessage(): string;
+
+}
\ No newline at end of file
diff --git a/src/main/edu/wisc/services/cbs/order/OrderServiceResponse.php b/src/main/edu/wisc/services/cbs/order/OrderServiceResponse.php
index a280870df7e3a3b2b81beb7ffd5eda3e3b6efbb1..9eabfc9287cb04cd9c6bb33184608b4b837a86de 100644
--- a/src/main/edu/wisc/services/cbs/order/OrderServiceResponse.php
+++ b/src/main/edu/wisc/services/cbs/order/OrderServiceResponse.php
@@ -3,10 +3,11 @@
namespace edu\wisc\services\cbs\order;
use edu\wisc\services\cbs\common\AbstractServiceResponse;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
/**
* The status of a call to an {@link OrderService} class.
*/
-class OrderServiceResponse extends AbstractServiceResponse
+class OrderServiceResponse extends AbstractServiceResponse implements ServiceResponseInterface
{
}
diff --git a/src/main/edu/wisc/services/cbs/order/header/MockOrderHeaderService.php b/src/main/edu/wisc/services/cbs/order/header/MockOrderHeaderService.php
index 23193aa96d60c7b351878e4a28d8a86965eeccce..1573fb870fd18b28e2b43e5d0b9729160dcfefd1 100644
--- a/src/main/edu/wisc/services/cbs/order/header/MockOrderHeaderService.php
+++ b/src/main/edu/wisc/services/cbs/order/header/MockOrderHeaderService.php
@@ -3,9 +3,10 @@
namespace edu\wisc\services\cbs\order\header;
use edu\wisc\services\cbs\api\MockService;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
use edu\wisc\services\cbs\order\OrderServiceResponse;
-class MockOrderHeaderService implements OrderHeaderService
+class MockOrderHeaderService implements OrderHeaderService, MockService
{
/** @var OrderServiceResponse */
@@ -14,7 +15,7 @@ class MockOrderHeaderService implements OrderHeaderService
/**
* {@inheritdoc}
*/
- public function __construct(OrderServiceResponse $response = null)
+ public function __construct(ServiceResponseInterface $response = null)
{
if ($response === null) {
$this->response = new OrderServiceResponse(
@@ -29,7 +30,7 @@ class MockOrderHeaderService implements OrderHeaderService
/**
* {@inheritdoc}
*/
- public function createOrderHeader(OrderHeader $header)
+ public function createOrderHeader(OrderHeader $header): OrderServiceResponse
{
return $this->response;
}
diff --git a/src/main/edu/wisc/services/cbs/order/header/OrderHeader.php b/src/main/edu/wisc/services/cbs/order/header/OrderHeader.php
index 1fa773080fd9725d9b227ea505303a5550a2eed9..4bd791dcb8fd8b285169289c4bd97c171ac27352 100644
--- a/src/main/edu/wisc/services/cbs/order/header/OrderHeader.php
+++ b/src/main/edu/wisc/services/cbs/order/header/OrderHeader.php
@@ -11,7 +11,7 @@ use edu\wisc\services\cbs\order\payment\OrderPayment;
class OrderHeader extends MagentoOrder
{
- /** @var string order number from Magento */
+ /** @var float order number from Magento */
private $orderNumber;
/** @var string Unique identifier, format "MAGE" + Magento order number */
@@ -61,18 +61,18 @@ class OrderHeader extends MagentoOrder
private $testMode;
/**
- * @return string
+ * @return float
*/
- public function getOrderNumber()
+ public function getOrderNumber(): float
{
return $this->orderNumber;
}
/**
- * @param string $orderNumber
+ * @param float $orderNumber
* @return OrderHeader
*/
- public function setOrderNumber($orderNumber)
+ public function setOrderNumber(float $orderNumber): OrderHeader
{
$this->orderNumber = $orderNumber;
return $this;
@@ -81,7 +81,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getOrigSystemDocumentRef()
+ public function getOrigSystemDocumentRef(): string
{
return $this->origSystemDocumentRef;
}
@@ -90,7 +90,7 @@ class OrderHeader extends MagentoOrder
* @param string $origSystemDocumentRef
* @return OrderHeader
*/
- public function setOrigSystemDocumentRef($origSystemDocumentRef)
+ public function setOrigSystemDocumentRef(string $origSystemDocumentRef): OrderHeader
{
$this->origSystemDocumentRef = $origSystemDocumentRef;
return $this;
@@ -99,7 +99,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getCustomerType()
+ public function getCustomerType(): string
{
return $this->customerType;
}
@@ -108,7 +108,7 @@ class OrderHeader extends MagentoOrder
* @param string $customerType
* @return OrderHeader
*/
- public function setCustomerType($customerType)
+ public function setCustomerType(string $customerType): OrderHeader
{
$this->customerType = $customerType;
return $this;
@@ -117,7 +117,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getCustomerIdentifier()
+ public function getCustomerIdentifier(): string
{
return $this->customerIdentifier;
}
@@ -126,7 +126,7 @@ class OrderHeader extends MagentoOrder
* @param string $customerIdentifier
* @return OrderHeader
*/
- public function setCustomerIdentifier($customerIdentifier)
+ public function setCustomerIdentifier(string $customerIdentifier): OrderHeader
{
$this->customerIdentifier = $customerIdentifier;
return $this;
@@ -135,7 +135,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getCustomerIdentifierType()
+ public function getCustomerIdentifierType(): string
{
return $this->customerIdentifierType;
}
@@ -144,7 +144,7 @@ class OrderHeader extends MagentoOrder
* @param string $customerIdentifierType
* @return OrderHeader
*/
- public function setCustomerIdentifierType($customerIdentifierType)
+ public function setCustomerIdentifierType(string $customerIdentifierType): OrderHeader
{
$this->customerIdentifierType = $customerIdentifierType;
return $this;
@@ -153,7 +153,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getPurchaserIdentifier()
+ public function getPurchaserIdentifier(): string
{
return $this->purchaserIdentifier;
}
@@ -162,7 +162,7 @@ class OrderHeader extends MagentoOrder
* @param string $purchaserIdentifier
* @return OrderHeader
*/
- public function setPurchaserIdentifier($purchaserIdentifier)
+ public function setPurchaserIdentifier(string $purchaserIdentifier): OrderHeader
{
$this->purchaserIdentifier = $purchaserIdentifier;
return $this;
@@ -171,7 +171,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getPurchaserIdentifierType()
+ public function getPurchaserIdentifierType(): string
{
return $this->purchaserIdentifierType;
}
@@ -180,7 +180,7 @@ class OrderHeader extends MagentoOrder
* @param string $purchaserIdentifierType
* @return OrderHeader
*/
- public function setPurchaserIdentifierType($purchaserIdentifierType)
+ public function setPurchaserIdentifierType(string $purchaserIdentifierType): OrderHeader
{
$this->purchaserIdentifierType = $purchaserIdentifierType;
return $this;
@@ -189,7 +189,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getCustomerReferenceField()
+ public function getCustomerReferenceField(): string
{
return $this->customerReferenceField;
}
@@ -198,7 +198,7 @@ class OrderHeader extends MagentoOrder
* @param string $customerReferenceField
* @return OrderHeader
*/
- public function setCustomerReferenceField($customerReferenceField)
+ public function setCustomerReferenceField(string $customerReferenceField): OrderHeader
{
$this->customerReferenceField = $customerReferenceField;
return $this;
@@ -207,7 +207,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getOrderSource()
+ public function getOrderSource(): string
{
return $this->orderSource;
}
@@ -216,7 +216,7 @@ class OrderHeader extends MagentoOrder
* @param string $orderSource
* @return OrderHeader
*/
- public function setOrderSource($orderSource)
+ public function setOrderSource(string $orderSource): OrderHeader
{
$this->orderSource = $orderSource;
return $this;
@@ -225,7 +225,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getInterfaceTypeFlag()
+ public function getInterfaceTypeFlag(): string
{
return $this->interfaceTypeFlag;
}
@@ -234,7 +234,7 @@ class OrderHeader extends MagentoOrder
* @param string $interfaceTypeFlag
* @return OrderHeader
*/
- public function setInterfaceTypeFlag($interfaceTypeFlag)
+ public function setInterfaceTypeFlag(string $interfaceTypeFlag): OrderHeader
{
$this->interfaceTypeFlag = $interfaceTypeFlag;
return $this;
@@ -243,7 +243,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getDefaultSet()
+ public function getDefaultSet(): string
{
return $this->defaultSet;
}
@@ -252,7 +252,7 @@ class OrderHeader extends MagentoOrder
* @param string $defaultSet
* @return OrderHeader
*/
- public function setDefaultSet($defaultSet)
+ public function setDefaultSet(string $defaultSet): OrderHeader
{
$this->defaultSet = $defaultSet;
return $this;
@@ -261,7 +261,7 @@ class OrderHeader extends MagentoOrder
/**
* @return string
*/
- public function getTestMode()
+ public function getTestMode(): string
{
return $this->testMode;
}
@@ -270,7 +270,7 @@ class OrderHeader extends MagentoOrder
* @param string $testMode
* @return OrderHeader
*/
- public function setTestMode($testMode)
+ public function setTestMode(string $testMode): OrderHeader
{
$this->testMode = $testMode;
return $this;
diff --git a/src/main/edu/wisc/services/cbs/order/header/OrderHeaderInputParametersMapper.php b/src/main/edu/wisc/services/cbs/order/header/OrderHeaderInputParametersMapper.php
index 84d18ad90ae71c4a0aeafed3be6c3979a1357878..4ad8a152750ec1afcc2ce577dc63e986ce8b6348 100644
--- a/src/main/edu/wisc/services/cbs/order/header/OrderHeaderInputParametersMapper.php
+++ b/src/main/edu/wisc/services/cbs/order/header/OrderHeaderInputParametersMapper.php
@@ -18,7 +18,7 @@ class OrderHeaderInputParametersMapper
public static function toInputParameters(OrderHeader $header)
{
return (new InputParameters(
- '',
+ 0.0,
'',
'',
'',
diff --git a/src/main/edu/wisc/services/cbs/order/header/OrderHeaderService.php b/src/main/edu/wisc/services/cbs/order/header/OrderHeaderService.php
index 728c0e0df62596e9ca9d3b24c1f2c5a92d84910f..49a3e43c4b934f883dccb3c334817208d234f04c 100644
--- a/src/main/edu/wisc/services/cbs/order/header/OrderHeaderService.php
+++ b/src/main/edu/wisc/services/cbs/order/header/OrderHeaderService.php
@@ -15,6 +15,6 @@ interface OrderHeaderService extends Service
* @param OrderHeader $header
* @return OrderServiceResponse
*/
- public function createOrderHeader(OrderHeader $header);
+ public function createOrderHeader(OrderHeader $header): OrderServiceResponse;
}
\ No newline at end of file
diff --git a/src/main/edu/wisc/services/cbs/order/header/SoapOrderHeaderService.php b/src/main/edu/wisc/services/cbs/order/header/SoapOrderHeaderService.php
index 589bc0fd6c71591476bc912ec927fe6962c91873..4adab40bff25019db2506d3df8925680d8a8f719 100644
--- a/src/main/edu/wisc/services/cbs/order/header/SoapOrderHeaderService.php
+++ b/src/main/edu/wisc/services/cbs/order/header/SoapOrderHeaderService.php
@@ -52,7 +52,7 @@ class SoapOrderHeaderService implements OrderHeaderService, SoapService
/**
* {@inheritdoc}
*/
- public function createOrderHeader(OrderHeader $header)
+ public function createOrderHeader(OrderHeader $header): OrderServiceResponse
{
/** @var OutputParameters */
$outputParameters = $this->soapClient->HEADER_CREATE(
diff --git a/src/main/edu/wisc/services/cbs/order/interfacing/MockOrderInterfacingService.php b/src/main/edu/wisc/services/cbs/order/interfacing/MockOrderInterfacingService.php
index 240853796120a764f4fc4f48f367b834c49bd684..017ae868f1aef7428153c0fa36dabf7b34f7c9a3 100644
--- a/src/main/edu/wisc/services/cbs/order/interfacing/MockOrderInterfacingService.php
+++ b/src/main/edu/wisc/services/cbs/order/interfacing/MockOrderInterfacingService.php
@@ -3,19 +3,19 @@
namespace edu\wisc\services\cbs\order\interfacing;
use edu\wisc\services\cbs\api\MockService;
-use edu\wisc\services\cbs\common\AbstractServiceResponse;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
use edu\wisc\services\cbs\order\OrderServiceResponse;
class MockOrderInterfacingService implements OrderInterfacingService, MockService
{
- /** @var AbstractServiceResponse */
+ /** @var OrderServiceResponse */
private $response;
/**
* {@inheritdoc}
*/
- public function __construct(AbstractServiceResponse $response = null)
+ public function __construct(ServiceResponseInterface $response = null)
{
if ($response === null) {
$this->response = new OrderServiceResponse(
diff --git a/src/main/edu/wisc/services/cbs/order/line/MockOrderLineService.php b/src/main/edu/wisc/services/cbs/order/line/MockOrderLineService.php
index 69f3d29a1b43fdab5b2c0b48439addb3d1683138..44ed0491db9c3d9845ec6a548877a6e109f163e2 100644
--- a/src/main/edu/wisc/services/cbs/order/line/MockOrderLineService.php
+++ b/src/main/edu/wisc/services/cbs/order/line/MockOrderLineService.php
@@ -2,7 +2,7 @@
namespace edu\wisc\services\cbs\order\line;
use edu\wisc\services\cbs\api\MockService;
-use edu\wisc\services\cbs\common\AbstractServiceResponse;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
use edu\wisc\services\cbs\order\OrderServiceResponse;
class MockOrderLineService implements OrderLineService, MockService
@@ -12,7 +12,7 @@ class MockOrderLineService implements OrderLineService, MockService
private $response;
/** @inheritdoc */
- public function __construct(AbstractServiceResponse $response = null)
+ public function __construct(ServiceResponseInterface $response = null)
{
$this->response = $response;
if ($response === null) {
diff --git a/src/main/edu/wisc/services/cbs/price/MockPriceService.php b/src/main/edu/wisc/services/cbs/price/MockPriceService.php
index 56a57c01d3f22c9cf95288fdcb2e595dee8d20ef..20478153d4f8c5e7f1a031436180acd73ca7d7df 100644
--- a/src/main/edu/wisc/services/cbs/price/MockPriceService.php
+++ b/src/main/edu/wisc/services/cbs/price/MockPriceService.php
@@ -4,6 +4,8 @@ namespace edu\wisc\services\cbs\price;
use edu\wisc\services\cbs\api\MockService;
use edu\wisc\services\cbs\common\AbstractServiceResponse;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
+use edu\wisc\services\cbs\order\OrderServiceResponse;
use Money\Money;
class MockPriceService implements PriceService, MockService
@@ -15,7 +17,7 @@ class MockPriceService implements PriceService, MockService
/**
* @inheritdoc
*/
- public function __construct(AbstractServiceResponse $response = null)
+ public function __construct(ServiceResponseInterface $response = null)
{
if ($response === null) {
$this->response = new PriceServiceResponse(true, 'Mock CBS Product Price Response');
diff --git a/src/main/edu/wisc/services/cbs/product/MockProductService.php b/src/main/edu/wisc/services/cbs/product/MockProductService.php
index b1f2ca8a22fb38f8782b607b80e6762f3dcd2a06..07b43ac9634accaba3be5f5ef745330a327c4cfd 100755
--- a/src/main/edu/wisc/services/cbs/product/MockProductService.php
+++ b/src/main/edu/wisc/services/cbs/product/MockProductService.php
@@ -3,7 +3,7 @@
namespace edu\wisc\services\cbs\product;
use edu\wisc\services\cbs\api\MockService;
-use edu\wisc\services\cbs\common\AbstractServiceResponse;
+use edu\wisc\services\cbs\common\ServiceResponseInterface;
class MockProductService implements ProductService, MockService
{
@@ -14,7 +14,7 @@ class MockProductService implements ProductService, MockService
/**
* @inheritdoc
*/
- public function __construct(AbstractServiceResponse $response = null)
+ public function __construct(ServiceResponseInterface $response = null)
{
if (null === $response) {
$this->response = new ProductServiceResponse(true, 'Mock CBS Response');