Author: ate
Date: Tue Feb 20 15:23:50 2007
New Revision: 509805

URL: http://svn.apache.org/viewvc?view=rev&rev=509805
Log:
JS2-462: Need to clean up Velocity logs, too many warnings and some errors even
Fixing: 
  ERROR velocity - VM #includeNestedLinksWithIconNavigation: error : too few 
arguments to macro. Wanted 2 got 0
  
Velocity doesn't properly parse recursive macro calls.
Internally it seems to "call" embedded macro references without real parameters 
when loading/parsing the macro, leading to the above error message.
By providing a "forward" definition of such a macro first (one without 
parameters), this parsing error can be suppressed.

Modified:
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/blueocean/decorator-macros.vm
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/jetspeed/decorator-macros.vm
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/decorator-macros.vm
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/xhtml-basic/decorator-macros.vm
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/sunflower/decorator-macros.vm
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/thesolution/decorator-macros.vm
    
portals/jetspeed-2/trunk/src/webapp/decorations/layout/tigris/decorator-macros.vm

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/blueocean/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/blueocean/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/blueocean/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/blueocean/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -125,6 +125,12 @@
       </div>
 #end
 
+#* dummy "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro
+   to prevent "ERROR velocity - VM #includeNestedLinksWithIconNavigation: 
error : too few arguments to macro. Wanted 2 got 0".
+   Seemingly the Velocity macro parser is doing a lookup call during parsing 
for the self-calling includeNestedLinksWithIconNavigation macro
+*#
+#macro (includeNestedLinksWithIconNavigation) #end
+
 #* include nested menu links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_menu $_orientation)
   #set($menuTitle = $_menu.getTitle($preferedLocale))

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/jetspeed/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/jetspeed/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/jetspeed/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/jetspeed/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -1,3 +1,9 @@
+#* dummy "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro
+   to prevent "ERROR velocity - VM #includeNestedLinksWithIconNavigation: 
error : too few arguments to macro. Wanted 2 got 0".
+   Seemingly the Velocity macro parser is doing a lookup call during parsing 
for the self-calling includeNestedLinksWithIconNavigation macro
+*#
+#macro (includeNestedLinksWithIconNavigation) #end
+
 #* include potentially nested links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_nodeSet $_nodePrefix)
   <div>

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -7,6 +7,12 @@
   #set($BOTTOM_TO_TOP = "BottomToTop")
 #end
 
+#* dummy "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro
+   to prevent "ERROR velocity - VM #includeNestedLinksWithIconNavigation: 
error : too few arguments to macro. Wanted 2 got 0".
+   Seemingly the Velocity macro parser is doing a lookup call during parsing 
for the self-calling includeNestedLinksWithIconNavigation macro
+*#
+#macro (includeNestedLinksWithIconNavigation) #end
+
 #* include potentially nested links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_nodeSet $_nodePrefix 
$_orientation)
   <div>

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/xhtml-basic/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/xhtml-basic/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/xhtml-basic/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/simple/xhtml-basic/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -100,6 +100,8 @@
 </div>
 #end
 
+#* "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro *#
+#macro (includeNestedLinksWithIconNavigation) #end
 #* include nested menu links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_menu $_orientation)
 #set($menuTitle = $_menu.getTitle($preferedLocale))

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/sunflower/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/sunflower/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/sunflower/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/sunflower/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -119,6 +119,12 @@
       </div>
 #end
 
+#* dummy "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro
+   to prevent "ERROR velocity - VM #includeNestedLinksWithIconNavigation: 
error : too few arguments to macro. Wanted 2 got 0".
+   Seemingly the Velocity macro parser is doing a lookup call during parsing 
for the self-calling includeNestedLinksWithIconNavigation macro
+*#
+#macro (includeNestedLinksWithIconNavigation) #end
+
 #* include nested menu links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_menu $_orientation)
   #set($menuTitle = $_menu.getTitle($preferedLocale))

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/thesolution/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/thesolution/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/thesolution/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/thesolution/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -119,6 +119,12 @@
       </div>
 #end
 
+#* dummy "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro
+   to prevent "ERROR velocity - VM #includeNestedLinksWithIconNavigation: 
error : too few arguments to macro. Wanted 2 got 0".
+   Seemingly the Velocity macro parser is doing a lookup call during parsing 
for the self-calling includeNestedLinksWithIconNavigation macro
+*#
+#macro (includeNestedLinksWithIconNavigation) #end
+
 #* include nested menu links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_menu $_orientation)
   #set($menuTitle = $_menu.getTitle($preferedLocale))

Modified: 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/tigris/decorator-macros.vm
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/src/webapp/decorations/layout/tigris/decorator-macros.vm?view=diff&rev=509805&r1=509804&r2=509805
==============================================================================
--- 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/tigris/decorator-macros.vm
 (original)
+++ 
portals/jetspeed-2/trunk/src/webapp/decorations/layout/tigris/decorator-macros.vm
 Tue Feb 20 15:23:50 2007
@@ -125,6 +125,12 @@
       </div>
 #end
 
+#* dummy "forward" reference definition of the real 
includeNestedLinksWithIconNavigation($_menu $_orientation) macro
+   to prevent "ERROR velocity - VM #includeNestedLinksWithIconNavigation: 
error : too few arguments to macro. Wanted 2 got 0".
+   Seemingly the Velocity macro parser is doing a lookup call during parsing 
for the self-calling includeNestedLinksWithIconNavigation macro
+*#
+#macro (includeNestedLinksWithIconNavigation) #end
+
 #* include nested menu links navigation component *#
 #macro (includeNestedLinksWithIconNavigation $_menu $_orientation)
   #set($menuTitle = $_menu.getTitle($preferedLocale))



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to