Hi Rafa & John:

I try again according your advise.


Regards

fanyun

-----------------------------------------------------------------

? patchfile.diff.gz
? patchfile.diff.gz1
? sql/ExecuteOdbcTurbineSchema.java
? sql/Odbc_id_table.sql
? sql/Odbc_users_roles_permissions.sql
? sql/Turbine-AccessY2K.mdb
Index: java/org/apache/turbine/om/peer/BasePeer.java
===================================================================
RCS file: 
/products/cvs/turbine/turbine/src/java/org/apache/turbine/om/peer/BasePeer.java,v
retrieving revision 1.4
diff -u -r1.4 BasePeer.java
--- java/org/apache/turbine/om/peer/BasePeer.java 2000/08/17 05:06:23 1.4
+++ java/org/apache/turbine/om/peer/BasePeer.java 2000/08/18 12:56:47
@@ -77,8 +77,7 @@
  * Peer classes are responsible for isolating all of the database
  * access for a specific business object.  They execute all of the SQL
  * against the database.  Over time this class has grown to include
- * utility methods which ease execution of cross-database database queries 
- * and the implementation of concrete Peers.
+ * utility methods which
  *
  * @author Frank Y. Kim
  * @author John D. McNally
@@ -708,7 +707,16 @@
         {
             String columnName = select.get(i);
             selectClause.add(columnName);
-            fromClause.add(columnName.substring(0,columnName.indexOf('.') ));
+   
+            int braceplace = columnName.indexOf('(');
+            if(braceplace == -1)
+            {
+                fromClause.add(columnName.substring(0,columnName.indexOf('.') ));
+            }
+            else
+            {
+                
+fromClause.add(columnName.substring(braceplace+1,columnName.indexOf('.') ));
+            }
         }
 
         Enumeration e = criteria.keys();



----- Original Message ----- 
From: "Rafal Krzewski" <[EMAIL PROTECTED]>
To: "Turbine" <[EMAIL PROTECTED]>
Sent: Tuesday, August 22, 2000 7:24 PM
Subject: Re: Change to BasePeer


> fanyun wrote:
> > I do not know weather you have received my patch. I try again here.  My god,
> > I do the change in 2 hours but take me 2 weeks to send the diff file.
> 
> This is unfortunate, but you are almost there... Please note that the second
> line in the quotation below got wraped in your mail program. This is easy
> to catch here, but can easily render long patches unusable. 
> I use Netscape Messenger as my mail client. Under Prefercnces -> Mail & Newsgroups
> -> Messages -> MessageWraping there is an option labeled 'Wrap outgoing plain
> text messages at [   ] messages' Find a similar setting in your mail client
> and set it to some large value (like 255).
> 
> 
> > +            {
> > +
> > fromClause.add(columnName.substring(0,columnName.indexOf('.') ));
> > +            }
> 
> 
> Rafal
> 
> 
> ------------------------------------------------------------
> To subscribe:        [EMAIL PROTECTED]
> To unsubscribe:      [EMAIL PROTECTED]
> Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
> Problems?:           [EMAIL PROTECTED]
> 




------------------------------------------------------------
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