You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cocoon.apache.org by bu...@apache.org on 2004/11/02 04:05:19 UTC

DO NOT REPLY [Bug 32011] New: - buildList in AbstractDatabaseAction generates incorrect parameter list

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

http://issues.apache.org/bugzilla/show_bug.cgi?id=32011

buildList in AbstractDatabaseAction generates incorrect parameter list

           Summary: buildList in AbstractDatabaseAction generates incorrect
                    parameter list
           Product: Cocoon 2
           Version: 2.1.5
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: blocks
        AssignedTo: dev@cocoon.apache.org
        ReportedBy: wchao@yahoo.com


The buildList method in AbstractDatabaseAction does not put commas between
parameter names, leading to incorrect SQL queries.

A fix is the following (diff of AbstractDatabaseAction.java):

730a731,733
>                                               if (i < length - 1) {
>                                                       buffer.append(", ");
>                                               }