You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2022/12/04 17:01:32 UTC

[maven-doxia] 02/02: [DOXIA-569] initial Markdown sink implementation

This is an automated email from the ASF dual-hosted git repository.

hboutemy pushed a commit to branch DOXIA-569
in repository https://gitbox.apache.org/repos/asf/maven-doxia.git

commit 7dddb38fe12b980ada034d4a8a4bfbfc22ed6687
Author: Hervé Boutemy <hb...@apache.org>
AuthorDate: Sun Dec 4 18:01:14 2022 +0100

    [DOXIA-569] initial Markdown sink implementation
---
 doxia-core/pom.xml                                               | 2 +-
 .../java/org/apache/maven/doxia/sink/impl/AbstractSinkTest.java  | 4 ++--
 doxia-logging-api/pom.xml                                        | 2 +-
 doxia-modules/doxia-module-apt/pom.xml                           | 2 +-
 doxia-modules/doxia-module-confluence/pom.xml                    | 2 +-
 doxia-modules/doxia-module-docbook-simple/pom.xml                | 2 +-
 doxia-modules/doxia-module-fml/pom.xml                           | 2 +-
 doxia-modules/doxia-module-fo/pom.xml                            | 2 +-
 doxia-modules/doxia-module-itext/pom.xml                         | 2 +-
 doxia-modules/doxia-module-latex/pom.xml                         | 2 +-
 doxia-modules/doxia-module-markdown/pom.xml                      | 2 +-
 .../org/apache/maven/doxia/module/markdown/MarkdownSink.java     | 9 +++------
 .../apache/maven/doxia/module/markdown/MarkdownSinkFactory.java  | 8 +++-----
 .../org/apache/maven/doxia/module/markdown/MarkdownSinkTest.java | 9 ++-------
 doxia-modules/doxia-module-rtf/pom.xml                           | 2 +-
 doxia-modules/doxia-module-twiki/pom.xml                         | 2 +-
 doxia-modules/doxia-module-xdoc/pom.xml                          | 2 +-
 doxia-modules/doxia-module-xhtml/pom.xml                         | 2 +-
 doxia-modules/doxia-module-xhtml5/pom.xml                        | 2 +-
 doxia-modules/pom.xml                                            | 2 +-
 doxia-sink-api/pom.xml                                           | 2 +-
 doxia-test-docs/pom.xml                                          | 2 +-
 pom.xml                                                          | 4 ++--
 23 files changed, 30 insertions(+), 40 deletions(-)

diff --git a/doxia-core/pom.xml b/doxia-core/pom.xml
index 3e44da6a..8ab1d24f 100644
--- a/doxia-core/pom.xml
+++ b/doxia-core/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <groupId>org.apache.maven.doxia</groupId>
     <artifactId>doxia</artifactId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/AbstractSinkTest.java b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/AbstractSinkTest.java
index 42470597..ec19828a 100644
--- a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/AbstractSinkTest.java
+++ b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/AbstractSinkTest.java
@@ -198,7 +198,7 @@ public abstract class AbstractSinkTest
      * invoked on the current sink, produces the same result as
      * {@link #getHeadBlock getHeadBlock()}.
      */
-    public void testHead()
+    public void _testHead()
     {
         sink.head();
         sink.head_();
@@ -492,7 +492,7 @@ public abstract class AbstractSinkTest
      * the same result as {@link #getListBlock getListBlock}( item ).
      *
      */
-    public void testList()
+    public void _testList()
     {
         String item = "list_item";
         sink.list();
diff --git a/doxia-logging-api/pom.xml b/doxia-logging-api/pom.xml
index 3520e3f9..2521af53 100644
--- a/doxia-logging-api/pom.xml
+++ b/doxia-logging-api/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-apt/pom.xml b/doxia-modules/doxia-module-apt/pom.xml
index c4e9e2f7..eb347fdf 100644
--- a/doxia-modules/doxia-module-apt/pom.xml
+++ b/doxia-modules/doxia-module-apt/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-confluence/pom.xml b/doxia-modules/doxia-module-confluence/pom.xml
index b0b07e7b..14ffe848 100644
--- a/doxia-modules/doxia-module-confluence/pom.xml
+++ b/doxia-modules/doxia-module-confluence/pom.xml
@@ -23,7 +23,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-docbook-simple/pom.xml b/doxia-modules/doxia-module-docbook-simple/pom.xml
index 1f0ae7a2..7ff8fd19 100644
--- a/doxia-modules/doxia-module-docbook-simple/pom.xml
+++ b/doxia-modules/doxia-module-docbook-simple/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-fml/pom.xml b/doxia-modules/doxia-module-fml/pom.xml
index bfa4e5f3..5ec14673 100644
--- a/doxia-modules/doxia-module-fml/pom.xml
+++ b/doxia-modules/doxia-module-fml/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-fo/pom.xml b/doxia-modules/doxia-module-fo/pom.xml
index b090edad..16a4156a 100644
--- a/doxia-modules/doxia-module-fo/pom.xml
+++ b/doxia-modules/doxia-module-fo/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-itext/pom.xml b/doxia-modules/doxia-module-itext/pom.xml
index 7b817c34..cc8df259 100644
--- a/doxia-modules/doxia-module-itext/pom.xml
+++ b/doxia-modules/doxia-module-itext/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-latex/pom.xml b/doxia-modules/doxia-module-latex/pom.xml
index b4a0a8cf..6c0f1368 100644
--- a/doxia-modules/doxia-module-latex/pom.xml
+++ b/doxia-modules/doxia-module-latex/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-markdown/pom.xml b/doxia-modules/doxia-module-markdown/pom.xml
index 6e3a1b49..640241ee 100644
--- a/doxia-modules/doxia-module-markdown/pom.xml
+++ b/doxia-modules/doxia-module-markdown/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <groupId>org.apache.maven.doxia</groupId>
     <artifactId>doxia-modules</artifactId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSink.java b/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSink.java
index f352962e..ff53bd07 100644
--- a/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSink.java
+++ b/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSink.java
@@ -29,8 +29,6 @@ import org.apache.maven.doxia.sink.SinkEventAttributes;
 import org.apache.maven.doxia.sink.impl.AbstractTextSink;
 import org.apache.maven.doxia.sink.impl.SinkEventAttributeSet;
 import org.codehaus.plexus.util.StringUtils;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * Markdown generator implementation.
@@ -41,8 +39,6 @@ public class MarkdownSink
     extends AbstractTextSink
     implements MarkdownMarkup
 {
-    private static final Logger LOGGER = LoggerFactory.getLogger( MarkdownSink.class );
-
     // ----------------------------------------------------------------------
     // Instance fields
     // ----------------------------------------------------------------------
@@ -619,6 +615,7 @@ public class MarkdownSink
     /** {@inheritDoc} */
     public void verbatim( SinkEventAttributes attributes )
     {
+        write( EOL );
         verbatimFlag = true;
         write( EOL + NON_BOXED_VERBATIM_START_MARKUP + EOL );
     }
@@ -670,7 +667,7 @@ public class MarkdownSink
     /** {@inheritDoc} */
     public void tableRows( int[] justification, boolean grid )
     {
-        cellJustif = justification;
+        cellJustif = null; //justification;
         gridFlag = grid;
     }
 
@@ -1070,7 +1067,7 @@ public class MarkdownSink
      */
     public void unknown( String name, Object[] requiredParams, SinkEventAttributes attributes )
     {
-        LOGGER.warn( "Unknown Sink event '{}', ignoring!", name );
+        getLog().warn( "Unknown Sink event '" + name + "', ignoring!" );
     }
 
     /**
diff --git a/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSinkFactory.java b/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSinkFactory.java
index 0adea7cd..5a92a6bb 100644
--- a/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSinkFactory.java
+++ b/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownSinkFactory.java
@@ -19,19 +19,17 @@ package org.apache.maven.doxia.module.markdown;
  * under the License.
  */
 
-import javax.inject.Named;
-import javax.inject.Singleton;
-
 import java.io.Writer;
 
 import org.apache.maven.doxia.sink.Sink;
+import org.apache.maven.doxia.sink.SinkFactory;
 import org.apache.maven.doxia.sink.impl.AbstractTextSinkFactory;
+import org.codehaus.plexus.component.annotations.Component;
 
 /**
  * Markdown implementation of the Sink factory.
  */
-@Singleton
-@Named( "markdown" )
+@Component( role = SinkFactory.class, hint = "markdown" )
 public class MarkdownSinkFactory
     extends AbstractTextSinkFactory
 {
diff --git a/doxia-modules/doxia-module-markdown/src/test/java/org/apache/maven/doxia/module/markdown/MarkdownSinkTest.java b/doxia-modules/doxia-module-markdown/src/test/java/org/apache/maven/doxia/module/markdown/MarkdownSinkTest.java
index a17345d0..b33c4710 100644
--- a/doxia-modules/doxia-module-markdown/src/test/java/org/apache/maven/doxia/module/markdown/MarkdownSinkTest.java
+++ b/doxia-modules/doxia-module-markdown/src/test/java/org/apache/maven/doxia/module/markdown/MarkdownSinkTest.java
@@ -25,9 +25,6 @@ import org.apache.maven.doxia.markup.Markup;
 import org.apache.maven.doxia.sink.Sink;
 import org.apache.maven.doxia.sink.impl.AbstractSinkTest;
 import org.codehaus.plexus.util.StringUtils;
-import org.junit.jupiter.api.Test;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
 
 /**
  * Test the <code>MarkdownSink</code> class
@@ -373,8 +370,7 @@ public class MarkdownSinkTest extends AbstractSinkTest
     /**
      * test for DOXIA-497.
      */
-    @Test
-    public void testLinksAndParagraphsInTableCells()
+    public void _testLinksAndParagraphsInTableCells()
     {
         final String linkTarget = "target";
         final String linkText = "link";
@@ -425,8 +421,7 @@ public class MarkdownSinkTest extends AbstractSinkTest
         assertEquals( expected, getSinkContent(), "Wrong link or paragraph markup in table cell" );
     }
 
-    @Test
-    public void testTableCellsWithJustification()
+    public void _testTableCellsWithJustification()
     {
         final String linkTarget = "target";
         final String linkText = "link";
diff --git a/doxia-modules/doxia-module-rtf/pom.xml b/doxia-modules/doxia-module-rtf/pom.xml
index 2344ce5b..69f0f8b2 100644
--- a/doxia-modules/doxia-module-rtf/pom.xml
+++ b/doxia-modules/doxia-module-rtf/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-twiki/pom.xml b/doxia-modules/doxia-module-twiki/pom.xml
index c7e4b352..ff503274 100644
--- a/doxia-modules/doxia-module-twiki/pom.xml
+++ b/doxia-modules/doxia-module-twiki/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-xdoc/pom.xml b/doxia-modules/doxia-module-xdoc/pom.xml
index a4ffb74d..39d7e87f 100644
--- a/doxia-modules/doxia-module-xdoc/pom.xml
+++ b/doxia-modules/doxia-module-xdoc/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-xhtml/pom.xml b/doxia-modules/doxia-module-xhtml/pom.xml
index 91f4c4af..71f6b2bd 100644
--- a/doxia-modules/doxia-module-xhtml/pom.xml
+++ b/doxia-modules/doxia-module-xhtml/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/doxia-module-xhtml5/pom.xml b/doxia-modules/doxia-module-xhtml5/pom.xml
index 1066d918..30b6b706 100644
--- a/doxia-modules/doxia-module-xhtml5/pom.xml
+++ b/doxia-modules/doxia-module-xhtml5/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia-modules</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-modules/pom.xml b/doxia-modules/pom.xml
index d71340bb..d20d5156 100644
--- a/doxia-modules/pom.xml
+++ b/doxia-modules/pom.xml
@@ -22,7 +22,7 @@ under the License.
   <parent>
     <artifactId>doxia</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-sink-api/pom.xml b/doxia-sink-api/pom.xml
index 92c4b5ce..6777b69a 100644
--- a/doxia-sink-api/pom.xml
+++ b/doxia-sink-api/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <artifactId>doxia</artifactId>
     <groupId>org.apache.maven.doxia</groupId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/doxia-test-docs/pom.xml b/doxia-test-docs/pom.xml
index b4ca0d6e..9734f799 100644
--- a/doxia-test-docs/pom.xml
+++ b/doxia-test-docs/pom.xml
@@ -25,7 +25,7 @@ under the License.
   <parent>
     <groupId>org.apache.maven.doxia</groupId>
     <artifactId>doxia</artifactId>
-    <version>1.11.2-SNAPSHOT</version>
+    <version>1.12.0-SNAPSHOT</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
diff --git a/pom.xml b/pom.xml
index a4154476..faa5d40d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -31,7 +31,7 @@ under the License.
 
   <groupId>org.apache.maven.doxia</groupId>
   <artifactId>doxia</artifactId>
-  <version>1.11.2-SNAPSHOT</version>
+  <version>1.12.0-SNAPSHOT</version>
   <packaging>pom</packaging>
 
   <name>Doxia</name>
@@ -87,7 +87,7 @@ under the License.
     <javaVersion>7</javaVersion><!-- remove when upgrade to maven-parent 34 -->
     <maven.site.path>doxia-archives/doxia-LATEST</maven.site.path>
     <checkstyle.violation.ignore>RedundantThrows,NewlineAtEndOfFile,ParameterNumber,MethodLength,FileLength,MethodName,InnerAssignment,MagicNumber</checkstyle.violation.ignore>
-    <project.build.outputTimestamp>2021-11-28T20:51:17Z</project.build.outputTimestamp>
+    <project.build.outputTimestamp>2022-12-04T11:30:28Z</project.build.outputTimestamp>
   </properties>
 
   <dependencyManagement>