Fix JATS section levels (#53)

* Fix JATS section levels

* Trigger Travis build

Author: thewilkybarkid

vendor-extra/JatsContentBundle/src/ViewConverter/SectionVisitor.php

@@ -29,7 +29,7 @@ public function __construct(ViewConverter $converter)
 
     protected function doVisit(Element $object, View $view) : View
     {
-        if ($view->hasContext('level')) {
+        if (!$view->hasContext('level')) {
             $view = $view->withContext(['level' => 1]);
         }
 

vendor-extra/JatsContentBundle/tests/ViewConverter/SectionVisitorTest.php

@@ -79,13 +79,12 @@ public function it_sets_the_heading_and_content_arguments(string $xml, array $ex
         $visitor = new SectionVisitor($this->createDumpingConverter());
 
         $element = $this->loadElement($xml);
-        $context = ['level' => 1];
 
-        $view = $visitor->visit($element, new View(null, [], $context));
+        $view = $visitor->visit($element, new View(null));
 
         $this->assertSame('@LiberoPatterns/section.html.twig', $view->getTemplate());
         $this->assertEquals($expectedArguments, $view->getArguments());
-        $this->assertSame($context, $view->getContext());
+        $this->assertSame(['level' => 1], $view->getContext());
     }
 
     public function contentProvider() : iterable
@@ -156,4 +155,51 @@ public function contentProvider() : iterable
             ],
         ];
     }
+
+    /**
+     * @test
+     */
+    public function it_uses_the_current_level() : void
+    {
+        $visitor = new SectionVisitor($this->createDumpingConverter());
+
+        $element = $this->loadElement(
+            <<<XML
+<jats:sec xmlns:jats="http://jats.nlm.nih.gov">
+    <jats:p>foo</jats:p>
+    <jats:p>bar</jats:p>
+</jats:sec>
+XML
+        );
+
+        $context = ['level' => 3];
+
+        $view = $visitor->visit($element, new View(null, [], $context));
+
+        $this->assertSame('@LiberoPatterns/section.html.twig', $view->getTemplate());
+        $this->assertEquals(
+            [
+                'content' => [
+                    new View(
+                        null,
+                        [
+                            'node' => '/jats:sec/jats:p[1]',
+                            'template' => null,
+                            'context' => ['level' => 4],
+                        ]
+                    ),
+                    new View(
+                        null,
+                        [
+                            'node' => '/jats:sec/jats:p[2]',
+                            'template' => null,
+                            'context' => ['level' => 4],
+                        ]
+                    ),
+                ],
+            ],
+            $view->getArguments()
+        );
+        $this->assertSame($context, $view->getContext());
+    }
 }