Re: [Reactor for CF] DAO generation and Oracle 10

[Beth Bowden]

You are safe. The latest revision incorporates both changes.

On 7/25/06, Van Daele Wouter (DBB) <[EMAIL PROTECTED] > wrote:

I confess that I haven't looked at any specific patches - having just returned from a two-week holiday I downloaded revision 330 and hoped everything would work. Should I look further into this or am I reasonably safe now?

 

Wouter

 

---

From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]] On Behalf Of [EMAIL PROTECTED]
Sent: dinsdag 25 juli 2006 17:00

To: reactor@doughughes.net
Subject: RE: [Reactor for CF] DAO generation and Oracle 10

Van,

 

Did you look at the solutions for Ticket #41 that Beth and I submitted(5 minutes apart, she beat me, but just by a hair)?

For me to get that to work I had to change the query qFields in data\oracle\objectDao.cfc:

 

                    CASE

                          WHEN primaryConstraints.column_name IS NULL THEN 'false'

                          ELSE 'true'

                    END                   as primaryKey,

 

And I also had to change this line in the loop:

 

<cfset Field.nullable = iif(qFields.nullable eq "Y", DE("true"), DE("false")) />

 

Maybe that will help?

 

-Nate

---

From: [EMAIL PROTECTED] [mailto: [EMAIL PROTECTED]] On Behalf Of Beth Bowden

Sent: Tuesday, July 25, 2006 6:24 AM

To: reactor@doughughes.net
Subject: Re: [Reactor for CF] DAO generation and Oracle 10

 

Van,

In your categoryMetadata.cfc, is there a line similiar to:

               <cfset variables.metadata.fields[1]["primaryKey"] = "true" />

?

Thanks, Beth

On 7/25/06, Van Daele Wouter (DBB) < [EMAIL PROTECTED]> wrote:

(Can't seem to get logged in to the Reactor Trac site, so I'm logging it here...)

Doug, all,

This morning I downloaded revision 330 of Reactor, cleared the project directory and restarted my app. Unfortunately, I'm still getting badly formed DAO's for all my tables with a (single) primary key. Create and Read are OK; Save, Update and Delete are flawed...

From what I had to do to get my app running, I'ld say that the primary keys aren't detected by Reactor (cf. Trac ticket #41 http://trac.reactorframework.com/reactor/trac.cgi/ticket/41 ). Example: here's the Save function as currently generated:

        <cffunction name="save" access="public" hint="I create or update a CATEGORY record." output="false" returntype="void">

                <cfargument name="to" hint="I am the transfer object for CATEGORY" required="yes" type="reactor.project.qtau.To.CATEGORYTo" />

                <cfset create( arguments.to) />
        </cffunction>

There's no way this code can decide between "create" and "update"...

Similarly, the generated Update and Delete methods end with:

                        ...
                        WHERE
                </cfquery>
        </cffunction>

No comments needed, I suppose. This problem is blocking my progress with an app that has to be finished soon; in the current state of affairs I'll have to code my  DAO's and Gateways by hand - without Reactor...

<no fun>

W.

-------------------------------------- Dexia Bank disclaimer : http://www.dexia.be/maildisclaimer.htm --------------------------------------

-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Reactor for ColdFusion Mailing List
reactor@doughughes.net
Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --

-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Reactor for ColdFusion Mailing List
reactor@doughughes.net
Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --

-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Reactor for ColdFusion Mailing List
reactor@doughughes.net
Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --

-------------------------------------- Dexia Bank disclaimer : http://www.dexia.be/maildisclaimer.htm --------------------------------------

-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Reactor for ColdFusion Mailing List
reactor@doughughes.net
Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --

-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Reactor for ColdFusion Mailing List
reactor@doughughes.net
Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --