DO NOT REPLY [Bug 24133] New: - Macro in velstruts example 3 (velocity) is missing ()

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24133>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24133

Macro in velstruts example 3 (velocity) is missing ()

           Summary: Macro in velstruts example 3 (velocity) is missing ()
           Product: Velocity
           Version: 1.3.1
          Platform: PC
               URL: n/a
        OS/Version: Windows XP
            Status: NEW
          Severity: Minor
          Priority: Other
         Component: Tools
        AssignedTo: velocity-dev@jakarta.apache.org
        ReportedBy: jamesor@telus.net


The errorMarkup macro in index.vm in the velocity version of the logon example 
(3) is missing () on the $errors.exist

eg:
        #macro (errorMarkup)
            #if ($errors.exist )
                <ul>
                #foreach ($e in $errors.all )
                    $e
                #end
                </ul>
            #end
        #end                    
        #errorMarkup()

should be:

        #macro (errorMarkup)
            #if ($errors.exist() )
                <ul>
                #foreach ($e in $errors.all )
                    $e
                #end
                </ul>
            #end
        #end                    
        #errorMarkup()

The reason this does not pop up in the example is that the global macro 
defined in VM_global_library.vm is correct, and it overrides the local macro.
This frustrated me for several hours... good thing it's a quickfix.

---------------------------------------------------------------------
To unsubscribe, e-mail: velocity-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: velocity-dev-help@jakarta.apache.org