Revert "refactor API interfaces #10"

This reverts commit 3c18ea1.

Author: gregor-j

src/Api/IArray.php

@@ -10,14 +10,13 @@
 /**
  * Interface IArray
  *
- * API arrays extend the logic of elements to contain member elements. They are
- * the basis for structs and tables.
+ * API extend the logic of values to contain member elements.
  *
  * @package phpsap\interfaces\Api
  * @author  Gregor J.
  * @license MIT
  */
-interface IArray extends IElement
+interface IArray extends IValue
 {
     /**
      * JSON configuration key for members array.
@@ -26,12 +25,12 @@ interface IArray extends IElement
 
     /**
      * Cast a given output value to the implemented value.
-     * @param array<int|string, mixed> $value The output array to typecast.
-     * @return array<int|string, mixed>
+     * @param array $value The output array to typecast.
+     * @return array
      * @throws IArrayElementMissingException
      * @throws IInvalidArgumentException
      */
-    public function cast(array $value): array;
+    public function cast($value): array;
 
     /**
      * Return an array of member elements.

src/Api/IElement.php

@@ -12,9 +12,8 @@
 /**
  * Interface IElement
  *
- * API elements build the basis for API values, structs and tables. They have a
- * name, a type, a direction and an optional flag. However, they don't contain
- * any values.
+ * API elements are struct or table members and have no direction or optional flag
+ * of their own.
  *
  * @package phpsap\interfaces\Api
  * @author  Gregor J.
@@ -23,34 +22,60 @@
 interface IElement extends IJsonSerializable
 {
     /**
-     * JSON configuration key for type value.
+     * API element that casts to PHP string.
      */
-    public const JSON_TYPE = 'type';
+    public const TYPE_STRING = 'string';
 
     /**
-     * JSON configuration key for name value.
+     * API element that casts to PHP int.
      */
-    public const JSON_NAME = 'name';
+    public const TYPE_INTEGER = 'int';
+
+    /**
+     * API element that casts to PHP bool.
+     */
+    public const TYPE_BOOLEAN = 'bool';
 
     /**
-     * API input element.
+     * API element that casts to PHP float.
      */
-    public const DIRECTION_INPUT = 'input';
+    public const TYPE_FLOAT = 'float';
 
     /**
-     * API output element.
+     * API element that casts to a hexadecimal encoded binary to a binary.
+     * (direction: output)
      */
-    public const DIRECTION_OUTPUT = 'output';
+    public const TYPE_HEXBIN = 'hexbin';
 
     /**
-     * JSON configuration key for direction value.
+     * API date element that casts to a DateTime object.
      */
-    public const JSON_DIRECTION = 'direction';
+    public const TYPE_DATE = 'date';
 
     /**
-     * JSON configuration key for is optional flag.
+     * API time element that casts to a DateTime object.
      */
-    public const JSON_OPTIONAL = 'optional';
+    public const TYPE_TIME = 'time';
+
+    /**
+     * API virtual timestamp element (e.g. string) that casts to a DateTime object.
+     */
+    public const TYPE_TIMESTAMP = 'timestamp';
+
+    /**
+     * API virtual calendar week element (e.g. string) that casts to a DateTime object.
+     */
+    public const TYPE_WEEK = 'week';
+
+    /**
+     * JSON configuration key for type value.
+     */
+    public const JSON_TYPE = 'type';
+
+    /**
+     * JSON configuration key for name value.
+     */
+    public const JSON_NAME = 'name';
 
     /**
      * The PHP type of the element.
@@ -65,15 +90,10 @@ public function getType(): string;
     public function getName(): string;
 
     /**
-     * Get the direction of the parameter.
-     * interface.
-     * @return string
-     */
-    public function getDirection(): string;
-
-    /**
-     * Is the element optional?
-     * @return bool
+     * Cast a given output value to the implemented value.
+     * @param bool|int|float|string $value The output to typecast.
+     * @return bool|int|float|string|SapDateTime|SapDateInterval
+     * @throws IInvalidArgumentException
      */
-    public function isOptional(): bool;
+    public function cast($value);
 }

src/Api/IStruct.php

@@ -7,7 +7,7 @@
 /**
  * Interface IStruct
  *
- * API structs extend the logic of arrays to contain an associative array.
+ * API extend the logic of arrays to contain member elements.
  *
  * @package phpsap\interfaces\Api
  * @author  Gregor J.

src/Api/ITable.php

@@ -7,8 +7,7 @@
 /**
  * Interface ITable
  *
- * API tables extend the logic of arrays to contain an array of associative
- * arrays.
+ * API extend the logic of arrays but contains rows of member elements.
  *
  * @package phpsap\interfaces\Api
  * @author  Gregor J.

src/Api/IValue.php

@@ -4,16 +4,11 @@
 
 namespace phpsap\interfaces\Api;
 
-use phpsap\DateTime\SapDateInterval;
-use phpsap\DateTime\SapDateTime;
-use phpsap\interfaces\exceptions\IInvalidArgumentException;
-
 /**
  * Interface IValue
  *
- * API values extend the logic of an element. They describe primitive values,
- * like integers, booleans, strings, etc., but also more complex values, like
- * date and time, or intervals.
+ * API values extend the logic of an element but have a direction (input or output)
+ * and an optional flag, unlike elements.
  *
  * @package phpsap\interfaces\Api
  * @author  Gregor J.
@@ -22,56 +17,35 @@
 interface IValue extends IElement
 {
     /**
-     * API element that casts to PHP string.
-     */
-    public const TYPE_STRING = 'string';
-
-    /**
-     * API element that casts to PHP int.
-     */
-    public const TYPE_INTEGER = 'int';
-
-    /**
-     * API element that casts to PHP bool.
-     */
-    public const TYPE_BOOLEAN = 'bool';
-
-    /**
-     * API element that casts to PHP float.
-     */
-    public const TYPE_FLOAT = 'float';
-
-    /**
-     * API element that casts to a hexadecimal encoded binary to a binary.
-     * (direction: output)
+     * API input element.
      */
-    public const TYPE_HEXBIN = 'hexbin';
+    public const DIRECTION_INPUT = 'input';
 
     /**
-     * API date element that casts to a DateTime object.
+     * API output element.
      */
-    public const TYPE_DATE = 'date';
+    public const DIRECTION_OUTPUT = 'output';
 
     /**
-     * API time element that casts to a DateTime object.
+     * JSON configuration key for direction value.
      */
-    public const TYPE_TIME = 'time';
+    public const JSON_DIRECTION = 'direction';
 
     /**
-     * API virtual timestamp element (e.g. string) that casts to a DateTime object.
+     * JSON configuration key for is optional flag.
      */
-    public const TYPE_TIMESTAMP = 'timestamp';
+    public const JSON_OPTIONAL = 'optional';
 
     /**
-     * API virtual calendar week element (e.g. string) that casts to a DateTime object.
+     * Get the direction of the parameter.
+     * interface.
+     * @return string
      */
-    public const TYPE_WEEK = 'week';
+    public function getDirection(): string;
 
     /**
-     * Cast a given output value to the implemented value.
-     * @param bool|int|float|string $value The output to typecast.
-     * @return bool|int|float|string|SapDateTime|SapDateInterval
-     * @throws IInvalidArgumentException
+     * Is the element optional?
+     * @return bool
      */
-    public function cast(bool|int|float|string $value): bool|int|float|string|SapDateTime|SapDateInterval;
+    public function isOptional(): bool;
 }