Allow custom schemas

ch2877 · Mar 25, 2018 · e0cadcf · e0cadcf
1 parent 3eefd9f
commit e0cadcf
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 10 deletions.
diff --git a/src/Z38/SwissPayment/Message/AbstractMessage.php b/src/Z38/SwissPayment/Message/AbstractMessage.php
@@ -9,8 +9,6 @@
  */
 abstract class AbstractMessage implements MessageInterface
 {
-    const SCHEMA_LOCATION = 'http://www.six-interbank-clearing.com/de/%s';
-
     /**
      * Builds the DOM of the actual message
      *
@@ -27,6 +25,13 @@ abstract protected function buildDom(\DOMDocument $doc);
      */
     abstract protected function getSchemaName();
 
+    /**
+     * Gets the location of the schema
+     *
+     * @return string|null The location or null
+     */
+    abstract protected function getSchemaLocation();
+
     /**
      * Builds a DOM document of the message
      *
@@ -35,13 +40,15 @@ abstract protected function getSchemaName();
     public function asDom()
     {
         $schema = $this->getSchemaName();
-        $ns = sprintf(self::SCHEMA_LOCATION, $schema);
+        $location = $this->getSchemaLocation();
 
         $doc = new \DOMDocument('1.0', 'UTF-8');
         $root = $doc->createElement('Document');
-        $root->setAttribute('xmlns', $ns);
-        $root->setAttribute('xmlns:xsi', 'http://www.w3.org/2001/XMLSchema-instance');
-        $root->setAttribute('xsi:schemaLocation', sprintf('%s %s', $ns, $schema));
+        $root->setAttribute('xmlns', $schema);
+        if ($location !== null) {
+            $root->setAttribute('xmlns:xsi', 'http://www.w3.org/2001/XMLSchema-instance');
+            $root->setAttribute('xsi:schemaLocation', $schema.' '.$location);
+        }
         $root->appendChild($this->buildDom($doc));
         $doc->appendChild($root);
 

diff --git a/src/Z38/SwissPayment/Message/CustomerCreditTransfer.php b/src/Z38/SwissPayment/Message/CustomerCreditTransfer.php
@@ -89,6 +89,14 @@ public function getPaymentCount()
      * {@inheritdoc}
      */
     protected function getSchemaName()
+    {
+        return 'http://www.six-interbank-clearing.com/de/pain.001.001.03.ch.02.xsd';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    protected function getSchemaLocation()
     {
         return 'pain.001.001.03.ch.02.xsd';
     }

diff --git a/tests/Z38/SwissPayment/Tests/Message/CustomerCreditTransferTest.php b/tests/Z38/SwissPayment/Tests/Message/CustomerCreditTransferTest.php
@@ -27,8 +27,8 @@
 
 class CustomerCreditTransferTest extends TestCase
 {
-    const SCHEMA = 'pain.001.001.03.ch.02.xsd';
-    const NS_URI_ROOT = 'http://www.six-interbank-clearing.com/de/';
+    const SCHEMA = 'http://www.six-interbank-clearing.com/de/pain.001.001.03.ch.02.xsd';
+    const SCHEMA_PATH = 'pain.001.001.03.ch.02.xsd';
 
     protected function buildMessage()
     {
@@ -194,7 +194,7 @@ public function testGroupHeader()
         $doc = new \DOMDocument();
         $doc->loadXML($xml);
         $xpath = new \DOMXPath($doc);
-        $xpath->registerNamespace('pain001', self::NS_URI_ROOT.self::SCHEMA);
+        $xpath->registerNamespace('pain001', self::SCHEMA);
 
         $nbOfTxs = $xpath->evaluate('string(//pain001:GrpHdr/pain001:NbOfTxs)');
         $this->assertEquals('12', $nbOfTxs);
@@ -206,7 +206,7 @@ public function testGroupHeader()
     public function testSchemaValidation()
     {
         $xml = $this->buildMessage()->asXml();
-        $schemaPath = __DIR__.'/../../../../'.self::SCHEMA;
+        $schemaPath = __DIR__.'/../../../../'.self::SCHEMA_PATH;
 
         $doc = new \DOMDocument();
         $doc->loadXML($xml);