Return-Path: 
 <taglibs-dev-return-9040-apmail-jakarta-taglibs-dev-archive=jakarta.apache.org@jakarta.apache.org>
Delivered-To: apmail-jakarta-taglibs-dev-archive@www.apache.org
Received: (qmail 49962 invoked from network); 28 Dec 2006 01:35:59 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 28 Dec 2006 01:35:59 -0000
Received: (qmail 69948 invoked by uid 500); 28 Dec 2006 01:36:05 -0000
Delivered-To: apmail-jakarta-taglibs-dev-archive@jakarta.apache.org
Received: (qmail 69937 invoked by uid 500); 28 Dec 2006 01:36:05 -0000
Mailing-List: contact taglibs-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:taglibs-dev-unsubscribe@jakarta.apache.org>
List-Help: <mailto:taglibs-dev-help@jakarta.apache.org>
List-Post: <mailto:taglibs-dev@jakarta.apache.org>
List-Id: "Tag Libraries Developers List" <taglibs-dev.jakarta.apache.org>
Reply-To: "Tag Libraries Developers List" <taglibs-dev@jakarta.apache.org>
Delivered-To: mailing list taglibs-dev@jakarta.apache.org
Received: (qmail 69926 invoked by uid 99); 28 Dec 2006 01:36:05 -0000
Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133)
    by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 27 Dec 2006 17:36:05 -0800
X-ASF-Spam-Status: No, hits=-9.4 required=10.0
	tests=ALL_TRUSTED,NO_REAL_NAME
X-Spam-Check-By: apache.org
Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4)
    by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 27 Dec 2006 17:35:57 -0800
Received: by brutus.apache.org (Postfix, from userid 33)
	id 90EA67142A1; Wed, 27 Dec 2006 17:35:37 -0800 (PST)
From: bugzilla@apache.org
To: taglibs-dev@jakarta.apache.org
Subject: DO NOT REPLY [Bug 33054]  - sql:query loses query column order
In-Reply-To: <bug-33054-7889@http.issues.apache.org/bugzilla/>
X-Bugzilla-Reason: AssignedTo
Message-Id: <20061228013537.90EA67142A1@brutus.apache.org>
Date: Wed, 27 Dec 2006 17:35:37 -0800 (PST)
X-Virus-Checked: Checked by ClamAV on apache.org

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=33054>.
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=33054





------- Additional Comments From bayard@apache.org  2006-12-27 17:35 -------
So here's a better test:

  <c:forEach var="row" items="${db.rows}">
    <tr>
      <c:forEach var="column" items="${row}">
        <td><c:out value="${column}"/></td>
      </c:forEach>
    </tr>
  </c:forEach>

You have to use rows and not rowIndex. The former uses the TreeMap and the
latter doesn't.

Definitely proves the bug. Things are ordered case insensitively by column name
(as the code is pretty honest about). From the javadoc for getRows:

     * Returns an array of SortedMap objects. The SortedMap
     * object key is the ColumnName and the value is the ColumnValue.
     * SortedMap was created using the CASE_INSENSITIVE_ORDER
     * Comparator so the key is the case insensitive representation
     * of the ColumnName.

There's nothing in the spec, so I think the next step here is to develop a
patch, and then to consider whether a test can easily be created.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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