Merge pull request #522 from Philipp91/ensuredone

Add missing BaseAction::ensureDone() calls to SEPA transfer samples

Author: nemiah

Samples/directDebit_Sephpa.php

@@ -49,3 +49,6 @@
 if ($sendSEPADirectDebit->needsTan()) {
     handleStrongAuthentication($sendSEPADirectDebit); // See login.php for the implementation.
 }
+
+// Debit requests don't produce any result we could receive through a getter, but we still need to make sure it's done.
+$sendSEPADirectDebit->ensureDone();

Samples/directDebit_phpSepaXml.php

@@ -65,3 +65,6 @@
 if ($sendSEPADirectDebit->needsTan()) {
     handleStrongAuthentication($sendSEPADirectDebit); // See login.php for the implementation.
 }
+
+// Debit requests don't produce any result we could receive through a getter, but we still need to make sure it's done.
+$sendSEPADirectDebit->ensureDone();

Samples/transfer.php

@@ -21,6 +21,15 @@
 /** @var \Fhp\FinTs $fints */
 $fints = require_once 'login.php';
 
+// Just pick the first account, for demonstration purposes. You could also have the user choose, or have SEPAAccount
+// hard-coded and not call getSEPAAccounts() at all.
+$getSepaAccounts = \Fhp\Action\GetSEPAAccounts::create();
+$fints->execute($getSepaAccounts);
+if ($getSepaAccounts->needsTan()) {
+    handleStrongAuthentication($getSepaAccounts); // See login.php for the implementation.
+}
+$oneAccount = $getSepaAccounts->getAccounts()[0];
+
 $dt = new \DateTime();
 $dt->add(new \DateInterval('P1D'));
 
@@ -52,3 +61,6 @@
 if ($sendSEPATransfer->needsTan()) {
     handleStrongAuthentication($sendSEPATransfer); // See login.php for the implementation.
 }
+
+// SEPA transfers don't produce any result we could receive through a getter, but we still need to make sure it's done.
+$sendSEPATransfer->ensureDone();