Merge pull request #4 from NystronSolar/ClientInterfaces

fix: add Numeric String Doc Block Annotaion for Energy Class
NystronSolar · Jun 2, 2023 · fdc547f · fdc547f
2 parents 5c9739b + 85a8758
commit fdc547f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/Entity/Energy.php b/src/Entity/Energy.php
@@ -4,12 +4,20 @@
 
 use NystronSolar\GrowattApi\Client\ApiClientInterface;
 
+/**
+ * @property numeric-string $generation
+ *
+ * @method numeric-string getGeneration()
+ */
 class Energy
 {
     private \DateTimeInterface $date;
 
     private string $generation;
 
+    /**
+     * @param numeric-string $generation
+     */
     public function __construct(string $date, string $generation)
     {
         $this->generation = $generation;

diff --git a/src/Response/ApiPlantHistoryResponse.php b/src/Response/ApiPlantHistoryResponse.php
@@ -69,12 +69,12 @@ public static function generate(ResponseInterface $httpResponse): ApiPlantHistor
             $generation = $rawEnergy->energy;
             if (
                 !is_string($date)
-                || !is_string($generation)
+                || !is_numeric($generation)
             ) {
                 return false;
             }
 
-            $energy = new Energy($date, $generation);
+            $energy = new Energy($date, (string) $generation);
 
             $energies[] = $energy;
         }