"Diethelm Guallar, Gonzalo" wrote:
> src/java/org/apache/turbine/admin/screens/admin/user/EditUser.java
> uses straight SQL:
> Is this right? Or should this file use a Peer class for this?
It looks like this code was copied from old ECS-based admin application.
It should be fixed to use Peers sometime but it's not critical.
> 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
It's OK for these classes to use Village directly, they are
pretty low-level anyhow.
Rafal
PS. You're doing great Gonzalo!
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?: [EMAIL PROTECTED]
