Support JATS articles (#30)

* Support JATS articles directly

* Keep it wrapped in a Libero item

* Make a dedicated bundle

* Cross bundle so create a function

* Test

* Avoid setting a fake argument

* Not meant

* Move Libero XML to its own bundle

Author: thewilkybarkid

.docker/api/data/blog-articles/post1/1.xml

@@ -2,9 +2,14 @@
 
 <item xmlns="http://libero.pub">
 
-    <front xml:lang="en">
+    <meta>
 
         <id>post1</id>
+
+    </meta>
+
+    <front xml:lang="en">
+
         <title>Post 1</title>
 
     </front>

.docker/api/data/scholarly-articles/article1/1.xml

@@ -1,12 +1,27 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<item xmlns="http://libero.pub">
+<item xmlns="http://libero.pub" xmlns:jats="http://jats.nlm.nih.gov">
 
-    <front xml:lang="en">
+    <meta>
 
         <id>article1</id>
-        <title>Article 1</title>
 
-    </front>
+    </meta>
+
+    <jats:article>
+
+        <jats:front>
+
+            <jats:article-meta>
+
+                <jats:title-group>
+                    <jats:article-title>Article 1</jats:article-title>
+                </jats:title-group>
+
+            </jats:article-meta>
+
+        </jats:front>
+
+    </jats:article>
 
 </item>

README.md

@@ -16,9 +16,11 @@ To run a website reading from two content services (`blog-articles` and `scholar
     content_page:
         pages:
             blog_article:
+                handler: 'libero'
                 path: '/blog/{id}'
                 service: 'blog-articles'
             scholarly_article:
+                handler: 'libero'
                 path: '/articles/{id}'
                 service: 'scholarly-articles'
     ```

composer.json

@@ -11,7 +11,9 @@
     "autoload-dev": {
         "psr-4": {
             "tests\\Libero\\Browser\\": "tests/",
-            "tests\\Libero\\ContentPageBundle\\": "vendor-extra/ContentPageBundle/tests/"
+            "tests\\Libero\\ContentPageBundle\\": "vendor-extra/ContentPageBundle/tests/",
+            "tests\\Libero\\JatsContentBundle\\": "vendor-extra/JatsContentBundle/tests/",
+            "tests\\Libero\\LiberoContentBundle\\": "vendor-extra/LiberoContentBundle/tests/"
         }
     },
     "require": {
@@ -20,6 +22,8 @@
         "ext-mbstring": "*",
         "csa/guzzle-bundle": "^3.1",
         "libero/content-page-bundle": "^1.0@dev",
+        "libero/jats-content-bundle": "^1.0@dev",
+        "libero/libero-content-bundle": "^1.0@dev",
         "libero/patterns-bundle": "^1.0@dev",
         "symfony/asset": "^4.1",
         "symfony/config": "^4.1",

composer.lock

@@ -5,6 +5,8 @@
 return [
     Csa\Bundle\GuzzleBundle\CsaGuzzleBundle::class => ['all' => true],
     Libero\ContentPageBundle\ContentPageBundle::class => ['all' => true],
+    Libero\JatsContentBundle\JatsContentBundle::class => ['all' => true],
+    Libero\LiberoContentBundle\LiberoContentBundle::class => ['all' => true],
     Libero\LiberoPatternsBundle\LiberoPatternsBundle::class => ['all' => true],
     Symfony\Bundle\FrameworkBundle\FrameworkBundle::class => ['all' => true],
     Symfony\Bundle\MonologBundle\MonologBundle::class => ['all' => true],

config/bundles.php

@@ -1,8 +1,10 @@
 content_page:
     pages:
         blog_article:
+            handler: 'libero'
             path: '/blog/{id}'
             service: 'blog-articles'
         scholarly_article:
+            handler: 'jats'
             path: '/articles/{id}'
             service: 'scholarly-articles'

config/packages/dev/content_page.yaml

@@ -1,8 +1,10 @@
 content_page:
     pages:
         blog_article:
+            handler: 'libero'
             path: '/blog/{id}'
             service: 'blog-articles'
         scholarly_article:
+            handler: 'libero'
             path: '/articles/{id}'
             service: 'scholarly-articles'

config/packages/test/content_page.yaml

@@ -18,12 +18,20 @@
         <testsuite name="ContentPageBundle">
             <directory suffix=".php">vendor-extra/ContentPageBundle/tests</directory>
         </testsuite>
+        <testsuite name="JatsContentBundle">
+            <directory suffix=".php">vendor-extra/JatsContentBundle/tests</directory>
+        </testsuite>
+        <testsuite name="LiberoContentBundle">
+            <directory suffix=".php">vendor-extra/LiberoContentBundle/tests</directory>
+        </testsuite>
     </testsuites>
 
     <filter>
         <whitelist>
             <directory>src</directory>
             <directory>vendor-extra/ContentPageBundle/src</directory>
+            <directory>vendor-extra/JatsContentBundle/src</directory>
+            <directory>vendor-extra/LiberoContentBundle/src</directory>
         </whitelist>
     </filter>
 

phpunit.xml.dist

@@ -47,6 +47,12 @@
     "libero/content-page-bundle": {
         "version": "1.0.x-dev"
     },
+    "libero/jats-content-bundle": {
+        "version": "1.0.x-dev"
+    },
+    "libero/libero-content-bundle": {
+        "version": "1.0.x-dev"
+    },
     "libero/patterns-bundle": {
         "version": "1.0.x-dev"
     },