on 8/24/2000 11:32 AM, "Diethelm Guallar, Gonzalo"
<[EMAIL PROTECTED]> wrote:

> This file
> 
> src/java/org/apache/turbine/admin/screens/admin/user/EditUser.java
> 
> uses straight SQL:
> 
> qds = new QueryDataSet(connection,
> "SELECT * FROM Visitor WHERE VISITORID=" + user_id );
> 
> Is this right? Or should this file use a Peer class for this?
> I found two other files that also use QueryDataSet directly
> (but no SQL):
> 
> src/java/org/apache/turbine/util/db/IDBroker.java
> src/java/org/apache/turbine/util/db/LargeSelect.java
> 
> Does any of this need fixing?

Good catch. That should be fixed.

-jon

-- 
Scarab -
      Java Servlet Based - Open Source
         Bug/Issue Tracking System
        <http://scarab.tigris.org/>



------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to