Merge pull request nemiah#496 from ampaze/better-schema-check

nemiah · web-flow · commit 297a99951095 · 2025-09-10T21:02:30.000+02:00
Use the UnterstuetzteSEPADatenformateTrait in all Actions to avoid xml namespace missmatches
diff --git a/lib/Fhp/Action/SendSEPADirectDebit.php b/lib/Fhp/Action/SendSEPADirectDebit.php
@@ -138,14 +138,14 @@ protected function createRequest(BPD $bpd, ?UPD $upd)
 
         if ($hidxes->getVersion() === 2) {
             /** @var HIDMESv2|HIDSESv2 $hidxes */
-            $supportedPainNamespaces = $hidxes->getParameter()->unterstuetzteSEPADatenformate;
+            $supportedPainNamespaces = $hidxes->getParameter()->getUnterstuetzteSEPADatenformate();
         }
 
         // If there are no SEPA formats available in the HIDXES Parameters, we look to the general formats
         if (!is_array($supportedPainNamespaces) || count($supportedPainNamespaces) === 0) {
             /** @var HISPAS $hispas */
             $hispas = $bpd->requireLatestSupportedParameters('HISPAS');
-            $supportedPainNamespaces = $hispas->getParameter()->getUnterstuetzteSepaDatenformate();
+            $supportedPainNamespaces = $hispas->getParameter()->getUnterstuetzteSEPADatenformate();
         }
 
         // Sometimes the Bank reports supported schemas with a "_GBIC_X" postfix.
diff --git a/lib/Fhp/Action/SendSEPARealtimeTransfer.php b/lib/Fhp/Action/SendSEPARealtimeTransfer.php
@@ -58,13 +58,13 @@ protected function createRequest(BPD $bpd, ?UPD $upd)
         /** @var HIIPZSv1|HIIPZSv2 $hiipzs */
         $hiipzs = $bpd->requireLatestSupportedParameters('HIIPZS');
 
-        $supportedSchemas = $hiipzs->parameter->unterstuetzteSEPADatenformate;
+        $supportedSchemas = $hiipzs->parameter->getUnterstuetzteSEPADatenformate();
 
         // If there are no SEPA formats available in the HIIPZS Parameters, we look to the general formats
         if (is_null($supportedSchemas)) {
             /** @var HISPAS $hispas */
             $hispas = $bpd->requireLatestSupportedParameters('HISPAS');
-            $supportedSchemas = $hispas->getParameter()->getUnterstuetzteSepaDatenformate();
+            $supportedSchemas = $hispas->getParameter()->getUnterstuetzteSEPADatenformate();
         }
 
         // Sometimes the Bank reports supported schemas with a "_GBIC_X" postfix.
diff --git a/lib/Fhp/Action/SendSEPATransfer.php b/lib/Fhp/Action/SendSEPATransfer.php
@@ -87,8 +87,8 @@ protected function createRequest(BPD $bpd, ?UPD $upd)
         }
 
         /** @var HISPAS $hispas */
-        $parameters = $bpd->requireLatestSupportedParameters('HISPAS');
-        $supportedSchemas = $parameters->getParameter()->getUnterstuetzteSepaDatenformate();
+        $hispas = $bpd->requireLatestSupportedParameters('HISPAS');
+        $supportedSchemas = $hispas->getParameter()->getUnterstuetzteSEPADatenformate();
 
         // Sometimes the Bank reports supported schemas with a "_GBIC_X" postfix.
         // GIBC_X stands for German Banking Industry Committee and a version counter.
diff --git a/lib/Fhp/Segment/BME/ParameterTerminierteSEPAFirmenSammellastschriftEinreichenV2.php b/lib/Fhp/Segment/BME/ParameterTerminierteSEPAFirmenSammellastschriftEinreichenV2.php
@@ -3,9 +3,13 @@
 namespace Fhp\Segment\BME;
 
 use Fhp\Segment\BSE\ParameterTerminierteSEPAFirmenLastschriftEinreichenV2;
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
-class ParameterTerminierteSEPAFirmenSammellastschriftEinreichenV2 extends ParameterTerminierteSEPAFirmenLastschriftEinreichenV2
+class ParameterTerminierteSEPAFirmenSammellastschriftEinreichenV2 extends ParameterTerminierteSEPAFirmenLastschriftEinreichenV2 implements UnterstuetzteSEPADatenformate
 {
+    use UnterstuetzteSEPADatenformateTrait;
+
     public int $maximaleAnzahlDirectDebitTransferTransactionInformation;
     public bool $summenfeldBenoetigt;
     public bool $einzelbuchungErlaubt;
diff --git a/lib/Fhp/Segment/BSE/ParameterTerminierteSEPAFirmenEinzellastschriftEinreichenV2.php b/lib/Fhp/Segment/BSE/ParameterTerminierteSEPAFirmenEinzellastschriftEinreichenV2.php
@@ -2,8 +2,13 @@
 
 namespace Fhp\Segment\BSE;
 
-class ParameterTerminierteSEPAFirmenEinzellastschriftEinreichenV2 extends ParameterTerminierteSEPAFirmenLastschriftEinreichenV2
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
+
+class ParameterTerminierteSEPAFirmenEinzellastschriftEinreichenV2 extends ParameterTerminierteSEPAFirmenLastschriftEinreichenV2 implements UnterstuetzteSEPADatenformate
 {
+    use UnterstuetzteSEPADatenformateTrait;
+
     /** Max Length: 4096 */
     public ?string $zulaessigePurposecodes = null;
 
diff --git a/lib/Fhp/Segment/DME/ParameterTerminierteSEPASammellastschriftEinreichenV2.php b/lib/Fhp/Segment/DME/ParameterTerminierteSEPASammellastschriftEinreichenV2.php
@@ -3,9 +3,13 @@
 namespace Fhp\Segment\DME;
 
 use Fhp\Segment\DSE\ParameterTerminierteSEPALastschriftEinreichenV2;
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
-class ParameterTerminierteSEPASammellastschriftEinreichenV2 extends ParameterTerminierteSEPALastschriftEinreichenV2
+class ParameterTerminierteSEPASammellastschriftEinreichenV2 extends ParameterTerminierteSEPALastschriftEinreichenV2 implements UnterstuetzteSEPADatenformate
 {
+    use UnterstuetzteSEPADatenformateTrait;
+
     public int $maximaleAnzahlDirectDebitTransferTransactionInformation;
     public bool $summenfeldBenoetigt;
     public bool $einzelbuchungErlaubt;
diff --git a/lib/Fhp/Segment/DSE/HIDSESv1.php b/lib/Fhp/Segment/DSE/HIDSESv1.php
@@ -15,7 +15,7 @@ class HIDSESv1 extends BaseGeschaeftsvorfallparameter implements HIDXES
 {
     public ParameterTerminierteSEPAEinzellastschriftEinreichenV1 $parameter;
 
-    public function getParameter(): SEPADirectDebitMinimalLeadTimeProvider
+    public function getParameter(): ParameterTerminierteSEPAEinzellastschriftEinreichenV1
     {
         return $this->parameter;
     }
diff --git a/lib/Fhp/Segment/DSE/HIDSESv2.php b/lib/Fhp/Segment/DSE/HIDSESv2.php
@@ -15,7 +15,7 @@ class HIDSESv2 extends BaseGeschaeftsvorfallparameter implements HIDXES
 {
     public ParameterTerminierteSEPAEinzellastschriftEinreichenV2 $parameter;
 
-    public function getParameter(): SEPADirectDebitMinimalLeadTimeProvider
+    public function getParameter(): ParameterTerminierteSEPAEinzellastschriftEinreichenV2
     {
         return $this->parameter;
     }
diff --git a/lib/Fhp/Segment/DSE/ParameterTerminierteSEPAEinzellastschriftEinreichenV2.php b/lib/Fhp/Segment/DSE/ParameterTerminierteSEPAEinzellastschriftEinreichenV2.php
@@ -2,8 +2,13 @@
 
 namespace Fhp\Segment\DSE;
 
-class ParameterTerminierteSEPAEinzellastschriftEinreichenV2 extends ParameterTerminierteSEPALastschriftEinreichenV2
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
+
+class ParameterTerminierteSEPAEinzellastschriftEinreichenV2 extends ParameterTerminierteSEPALastschriftEinreichenV2 implements UnterstuetzteSEPADatenformate
 {
+    use UnterstuetzteSEPADatenformateTrait;
+
     /** Max Length: 4096 */
     public ?string $zulaessigePurposecodes = null;
 
diff --git a/lib/Fhp/Segment/IPZ/ParameterSEPAInstantPaymentZahlungV1.php b/lib/Fhp/Segment/IPZ/ParameterSEPAInstantPaymentZahlungV1.php
@@ -3,15 +3,19 @@
 namespace Fhp\Segment\IPZ;
 
 use Fhp\Segment\BaseDeg;
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
 /**
  * Parameter SEPA-Instant Payment Zahlung (Version 1)
  *
  * @link https://www.hbci-zka.de/dokumente/spezifikation_deutsch/fintsv3/FinTS_3.0_Messages_Geschaeftsvorfaelle_2022-04-15_final_version.pdf
  * Section D
  */
-class ParameterSEPAInstantPaymentZahlungV1 extends BaseDeg
+class ParameterSEPAInstantPaymentZahlungV1 extends BaseDeg implements UnterstuetzteSEPADatenformate
 {
+    use UnterstuetzteSEPADatenformateTrait;
+
     /** Max Length: 4096 */
     public ?string $zulaessigePurposecodes = null;
 
diff --git a/lib/Fhp/Segment/IPZ/ParameterSEPAInstantPaymentZahlungV2.php b/lib/Fhp/Segment/IPZ/ParameterSEPAInstantPaymentZahlungV2.php
@@ -3,15 +3,19 @@
 namespace Fhp\lib\Fhp\Segment\IPZ;
 
 use Fhp\Segment\BaseDeg;
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
 /**
  * Parameter SEPA-Instant Payment Zahlung (Version 2)
  *
  * @link https://www.hbci-zka.de/dokumente/spezifikation_deutsch/fintsv3/FinTS_3.0_Messages_Geschaeftsvorfaelle_2022-04-15_final_version.pdf
  * Section D
  */
-class ParameterSEPAInstantPaymentZahlungV2 extends BaseDeg
+class ParameterSEPAInstantPaymentZahlungV2 extends BaseDeg implements UnterstuetzteSEPADatenformate
 {
+    use UnterstuetzteSEPADatenformateTrait;
+
     public bool $umwandlungNachSEPAUeberweisungZulaessigErlaubt;
 
     /** Max Length: 4096 */
diff --git a/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordern.php b/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordern.php
@@ -3,11 +3,12 @@
 
 namespace Fhp\Segment\SPA;
 
+use Fhp\Segment\UnterstuetzteSEPADatenformate;
+
 /**
  * Data Element Group: Parameter SEPA-Kontoverbindung anfordern
  */
-interface ParameterSepaKontoverbindungAnfordern
+interface ParameterSepaKontoverbindungAnfordern extends UnterstuetzteSEPADatenformate
 {
-    /** @return string[] */
-    public function getUnterstuetzteSepaDatenformate(): array;
+
 }
diff --git a/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordernV1.php b/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordernV1.php
@@ -4,6 +4,7 @@
 namespace Fhp\Segment\SPA;
 
 use Fhp\Segment\BaseDeg;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
 /**
  * Data Element Group: Parameter SEPA-Kontoverbindung anfordern (Version 1)
@@ -13,7 +14,7 @@
  */
 class ParameterSepaKontoverbindungAnfordernV1 extends BaseDeg implements ParameterSepaKontoverbindungAnfordern
 {
-    use GetUnterstuetzteSepaDatenformateTrait;
+    use UnterstuetzteSEPADatenformateTrait;
 
     public bool $einzelkontenabrufErlaubt;
     public bool $nationaleKontoverbindungErlaubt;
diff --git a/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordernV2.php b/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordernV2.php
@@ -4,6 +4,7 @@
 namespace Fhp\Segment\SPA;
 
 use Fhp\Segment\BaseDeg;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
 /**
  * Data Element Group: Parameter SEPA-Kontoverbindung anfordern (Version 2)
@@ -13,7 +14,7 @@
  */
 class ParameterSepaKontoverbindungAnfordernV2 extends BaseDeg implements ParameterSepaKontoverbindungAnfordern
 {
-    use GetUnterstuetzteSepaDatenformateTrait;
+    use UnterstuetzteSEPADatenformateTrait;
 
     public bool $einzelkontenabrufErlaubt;
     public bool $nationaleKontoverbindungErlaubt;
diff --git a/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordernV3.php b/lib/Fhp/Segment/SPA/ParameterSepaKontoverbindungAnfordernV3.php
@@ -4,6 +4,7 @@
 namespace Fhp\Segment\SPA;
 
 use Fhp\Segment\BaseDeg;
+use Fhp\Segment\UnterstuetzteSEPADatenformateTrait;
 
 /**
  * Data Element Group: Parameter SEPA-Kontoverbindung anfordern (Version 3)
@@ -13,7 +14,7 @@
  */
 class ParameterSepaKontoverbindungAnfordernV3 extends BaseDeg implements ParameterSepaKontoverbindungAnfordern
 {
-    use GetUnterstuetzteSepaDatenformateTrait;
+    use UnterstuetzteSEPADatenformateTrait;
 
     public bool $einzelkontenabrufErlaubt;
     public bool $nationaleKontoverbindungErlaubt;
diff --git a/lib/Fhp/Segment/UnterstuetzteSEPADatenformate.php b/lib/Fhp/Segment/UnterstuetzteSEPADatenformate.php
@@ -0,0 +1,10 @@
+<?php
+/** @noinspection PhpUnused */
+
+namespace Fhp\Segment;
+
+interface UnterstuetzteSEPADatenformate
+{
+    /** @return string[] */
+    public function getUnterstuetzteSEPADatenformate(): array;
+}
diff --git a/lib/Fhp/Segment/UnterstuetzteSEPADatenformateTrait.php b/lib/Fhp/Segment/UnterstuetzteSEPADatenformateTrait.php
@@ -1,15 +1,15 @@
 <?php
 
-namespace Fhp\Segment\SPA;
+namespace Fhp\Segment;
 
 /**
  * @link https://www.hbci-zka.de/dokumente/spezifikation_deutsch/fintsv3/FinTS_3.0_Messages_Geschaeftsvorfaelle_2015-08-07_final_version.pdf
  * Section: D ("Unterstützte SEPA-Datenformate")
  */
-trait GetUnterstuetzteSepaDatenformateTrait
+trait UnterstuetzteSEPADatenformateTrait
 {
     /** @return string[] */
-    public function getUnterstuetzteSepaDatenformate(): array
+    public function getUnterstuetzteSEPADatenformate(): array
     {
         // Something like "sepade.pain.00x.001.0y.xsd" is allowed here, which is not a valid SEPA XML URN / Namespace
         return array_map(function ($sepaUrn) {
@@ -18,6 +18,6 @@ public function getUnterstuetzteSepaDatenformate(): array
                 'sepade:' => 'urn:iso:std:iso:20022:tech:xsd:',
                 '.xsd' => '',
             ]);
-        }, $this->unterstuetzteSepaDatenformate);
+        }, $this->unterstuetzteSepaDatenformate ?? $this->unterstuetzteSEPADatenformate ?? []);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -138,14 +138,14 @@ protected function createRequest(BPD $bpd, ?UPD $upd)`
`138`	`138`
`139`	`139`	`if ($hidxes->getVersion() === 2) {`
`140`	`140`	`/** @var HIDMESv2\|HIDSESv2 $hidxes */`
`141`		`- $supportedPainNamespaces = $hidxes->getParameter()->unterstuetzteSEPADatenformate;`
	`141`	`+ $supportedPainNamespaces = $hidxes->getParameter()->getUnterstuetzteSEPADatenformate();`
`142`	`142`	`}`
`143`	`143`
`144`	`144`	`// If there are no SEPA formats available in the HIDXES Parameters, we look to the general formats`
`145`	`145`	`if (!is_array($supportedPainNamespaces) \|\| count($supportedPainNamespaces) === 0) {`
`146`	`146`	`/** @var HISPAS $hispas */`
`147`	`147`	`$hispas = $bpd->requireLatestSupportedParameters('HISPAS');`
`148`		`- $supportedPainNamespaces = $hispas->getParameter()->getUnterstuetzteSepaDatenformate();`
	`148`	`+ $supportedPainNamespaces = $hispas->getParameter()->getUnterstuetzteSEPADatenformate();`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`// Sometimes the Bank reports supported schemas with a "_GBIC_X" postfix.`
Original file line number	Diff line number	Diff line change
`@@ -87,8 +87,8 @@ protected function createRequest(BPD $bpd, ?UPD $upd)`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`/** @var HISPAS $hispas */`
`90`		`- $parameters = $bpd->requireLatestSupportedParameters('HISPAS');`
`91`		`- $supportedSchemas = $parameters->getParameter()->getUnterstuetzteSepaDatenformate();`
	`90`	`+ $hispas = $bpd->requireLatestSupportedParameters('HISPAS');`
	`91`	`+ $supportedSchemas = $hispas->getParameter()->getUnterstuetzteSEPADatenformate();`
`92`	`92`
`93`	`93`	`// Sometimes the Bank reports supported schemas with a "_GBIC_X" postfix.`
`94`	`94`	`// GIBC_X stands for German Banking Industry Committee and a version counter.`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ class HIDSESv1 extends BaseGeschaeftsvorfallparameter implements HIDXES`
`15`	`15`	`{`
`16`	`16`	`public ParameterTerminierteSEPAEinzellastschriftEinreichenV1 $parameter;`
`17`	`17`
`18`		`- public function getParameter(): SEPADirectDebitMinimalLeadTimeProvider`
	`18`	`+ public function getParameter(): ParameterTerminierteSEPAEinzellastschriftEinreichenV1`
`19`	`19`	`{`
`20`	`20`	`return $this->parameter;`
`21`	`21`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ class HIDSESv2 extends BaseGeschaeftsvorfallparameter implements HIDXES`
`15`	`15`	`{`
`16`	`16`	`public ParameterTerminierteSEPAEinzellastschriftEinreichenV2 $parameter;`
`17`	`17`
`18`		`- public function getParameter(): SEPADirectDebitMinimalLeadTimeProvider`
	`18`	`+ public function getParameter(): ParameterTerminierteSEPAEinzellastschriftEinreichenV2`
`19`	`19`	`{`
`20`	`20`	`return $this->parameter;`
`21`	`21`	`}`