You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2022/05/28 15:50:39 UTC

[GitHub] [maven-doxia-sitetools] michael-o commented on a diff in pull request #42: [DOXIASITETOOLS-230] Upgrade to Velocity Engine 2.3 and Velocity Tool…

michael-o commented on code in PR #42:
URL: https://github.com/apache/maven-doxia-sitetools/pull/42#discussion_r884149934


##########
doxia-site-renderer/src/site/apt/index.apt.vm:
##########
@@ -76,62 +76,62 @@ Doxia Sitetools - Site Renderer
 
   There are also some tools for general use:
 
-#set( $plexus = "http://codehaus-plexus.github.io/plexus-utils/apidocs/org/codehaus/plexus" )
+#set( $plexus = "https://codehaus-plexus.github.io/plexus-utils/apidocs/org/codehaus/plexus/util" )
 *---------------------------------+------------------------------------------------------+-------------------------------+
 || Variable                       || Type                                                || Description                 ||
 *---------------------------------+------------------------------------------------------+-------------------------------+
-| <<<PathTool>>>                  | {{{$plexus/util/PathTool.html}<<<PathTool>>>}}       |                               |
+| <<<PathTool>>>                  | {{{$plexus/PathTool.html}<<<PathTool>>>}}            |                               |
 *---------------------------------+------------------------------------------------------+-------------------------------+
-| <<<StringUtils>>>               | {{{$plexus/util/StringUtils.html}<<<StringUtils>>>}} |                               |
+| <<<StringUtils>>>               | {{{$plexus/StringUtils.html}<<<StringUtils>>>}}      |                               |
 *---------------------------------+------------------------------------------------------+-------------------------------+
-| <<<plexus>>>                    | {{{http://git.eclipse.org/c/sisu/org.eclipse.sisu.plexus.git/tree/org.eclipse.sisu.plexus/src/org/codehaus/plexus/PlexusContainer.java}<<<PlexusContainer>>>}} |  |
+| <<<plexus>>>                    | {{{https://github.com/eclipse/sisu.plexus/blob/master/org.eclipse.sisu.plexus/src/org/codehaus/plexus/PlexusContainer.java}<<<PlexusContainer>>>}} |  |
 *---------------------------------+------------------------------------------------------+-------------------------------+
 
-  Additionally, there are {{{http://velocity.apache.org/tools/releases/2.0/generic.html} Velocity Generic Tools}} populated
+  Additionally, there are {{{https://velocity.apache.org/tools/${velocityToolsVersion}/tools-summary.html} Velocity Generic Tools}} populated
   with the site locale, the decoration model's date format, and site renderer's resource bundle:
 
-#set( $generic = "http://velocity.apache.org/tools/releases/2.0/javadoc/org/apache/velocity/tools/generic" )
-*------------------+----------------------------------------------------------+-------------------------------+
-|| Variable        || Type                                                    || Description                 ||
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<alternator>>> | {{{$generic/AlternatorTool.html}AlternatorTool}}         | <<Deprecated>>: use CSS3 nth-child(even/odd) selectors or #if($foreach.index % 2). For creating alternators to easily alternate over a set of values.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<class>>>      | {{{$generic/ClassTool.html}ClassTool}}                   | For simplifying reflective lookup of information about classes and their fields, methods and constructors.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<context>>>    | {{{$generic/ContextTool.html}ContextTool}}               | For convenient access to context data and metadata.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<convert>>>    | {{{$generic/ConversionTool.html}ConversionTool}}         | <<Deprecated>>: use NumberTool for numbers formatting/parsing, DateTool for date/time formatting/parsing, or CollectionTool for toStrings(). For converting String values to richer object Types.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<date>>>       | {{{$generic/ComparisonDateTool.html}ComparisonDateTool}} | For manipulating, formatting, and comparing dates.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<display>>>    | {{{$generic/DisplayTool.html}DisplayTool}}               | For controlling display of references (e.g., truncating values, "pretty printing" lists, and displaying alternates when a reference is null).
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<esc>>>        | {{{$generic/EscapeTool.html}EscapeTool}}                 | For common escaping needs in Velocity templates (e.g. escaping html, xml, javascript etc.).
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<field>>>      | {{{$generic/FieldTool.html}FieldTool}}                   | For (easy) access to static fields in a class, such as string constants.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<link>>>       | {{{$generic/LinkTool.html}LinkTool}}                     | For creating and manipulating URIs and URLs. The API for this tool is designed to closely resemble that of the VelocityView tool of the same name.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<loop>>>       | {{{$generic/LoopTool.html}LoopTool}}                     | A convenience tool to use with \#foreach loops. It wraps a list with a custom iterator to provide greater control, allowing loops to end early, skip ahead and more.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<math>>>       | {{{$generic/MathTool.html}MathTool}}                     | For performing math functions.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<number>>>     | {{{$generic/NumberTool.html}NumberTool}}                 | For formatting and converting numbers.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<render>>>     | {{{$generic/RenderTool.html}RenderTool}}                 | To evaluate and render arbitrary strings of VTL, including recursive rendering.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<text>>>       | {{{$generic/ResourceTool.html}ResourceTool}}             | For simplified access to resource bundles for internationalization or other dynamic content needs.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<sorter>>>     | {{{$generic/SortTool.html}SortTool}}                     | <<Deprecated>>: use CollectionTool sort methods. Used to sort collections (or arrays, iterators, etc) on any arbitary set of properties exposed by the objects contained within the collection.
-*------------------+----------------------------------------------------------+-------------------------------+
-| <<<xml>>>        | {{{$generic/XmlTool.html}XmlTool}}                       | For reading/navigating XML files. This uses dom4j under the covers and provides complete XPath support.
-*------------------+----------------------------------------------------------+-------------------------------+
+#set( $generic = "https://velocity.apache.org/tools/3.1/tools-summary.html" )

Review Comment:
   Should use the placeholder, no?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org