Fix remaining PHPStan warnings

slub · Apr 18, 2024 · 52ce198 · 52ce198
1 parent 5bd5fbe
commit 52ce198
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 7 deletions.
diff --git a/.github/phpstan.neon b/.github/phpstan.neon
@@ -2,6 +2,7 @@ parameters:
     ignoreErrors:
         - '#PHPDoc tag @var for property Slub\\Mods\\Element\\Xml\\Element::\$values with type array\|false is not subtype of native type array\.#'
         - '#PHPDoc tag @return with type array\|false is not subtype of native type array\.#'
+        - '#Property Slub\\Mods\\Element\\Specific\\OriginInfo\\Issuance::\$allowedValues is never read, only written\.#'
     level: 5
     paths:
         - ../src/

diff --git a/src/Mods/Element/Specific/Location/HoldingSimple/CopyInformation.php b/src/Mods/Element/Specific/Location/HoldingSimple/CopyInformation.php
@@ -55,7 +55,12 @@ public function __construct(\SimpleXMLElement $xml)
      */
     public function getForm(string $query = ''): ?Form
     {
-        return $this->getLanguageElement('./mods:form' . $query);
+        $xpath = './mods:form' . $query;
+        $element = new Element($this->xml, $xpath);
+        if ($element->exists()) {
+            return new Form($element->getValues()[0]);
+        }
+        return null;
     }
 
     /**

diff --git a/src/Mods/Element/Specific/Part/Detail.php b/src/Mods/Element/Specific/Part/Detail.php
@@ -72,7 +72,7 @@ public function getLevel(): int
      *
      * @param string $query The XPath query for metadata search
      *
-     * @return LanguageElement
+     * @return LanguageElement[]
      */
     public function getNumbers(string $query = ''): array
     {
@@ -87,7 +87,7 @@ public function getNumbers(string $query = ''): array
      *
      * @param string $query The XPath query for metadata search
      *
-     * @return LanguageElement
+     * @return LanguageElement[]
      */
     public function getCaptions(string $query = ''): array
     {
@@ -102,7 +102,7 @@ public function getCaptions(string $query = ''): array
      *
      * @param string $query The XPath query for metadata search
      *
-     * @return LanguageElement
+     * @return LanguageElement[]
      */
     public function getTitles(string $query = ''): array
     {

diff --git a/src/Mods/Element/Subject.php b/src/Mods/Element/Subject.php
@@ -114,7 +114,7 @@ public function getTemporals(string $query = ''): array
         $element = new Element($this->xml, $xpath);
         if ($element->exists()) {
             foreach ($element->getValues() as $value) {
-                $temporals[] = new AuthorityLanguageElement($value);
+                $temporals[] = new AuthorityDateLanguageElement($value);
             }
         }
         return $temporals;

diff --git a/src/Mods/Element/TitleInfo.php b/src/Mods/Element/TitleInfo.php
@@ -134,9 +134,9 @@ public function getSubTitle(string $query = ''): ?LanguageElement
      *
      * @param string $query The XPath query for metadata search
      *
-     * @return LanguageElement
+     * @return ?LanguageElement
      */
-    public function getPartNumber(string $query = ''): LanguageElement
+    public function getPartNumber(string $query = ''): ?LanguageElement
     {
         $xpath = './mods:partNumber' . $query;
         $element = new Element($this->xml, $xpath);