You keep getting it wrong. The key request.IN.FIELDNAMES must always exist, just like form.fieldnames. So it must be declared specifically.

"Do you intend to support other databases at some point?" Now that is a fair question. At this time, the tool uses the ANSI SQL Standard Information_Schema views for meta-data. (Information_Schema is a non-conforming extension to the standard) If the database supports that part of the standard, and supports stored procedures, I would consider the database. Since the code is open source, you are free to build an information_schema.cfc for another database.

"And yet you get upset when someone suggests improvements?" The tool has had many improves thanks to suggestions by others. If you have something constructive to offer put it on cfopen.

You see Sean you cannot wear 2 hats as you claim. The Macromedia hat is to big. When you make flaming attacks on someone's products, you are the cause of restraint of trade and that reflects on Macromedia. So I ask you again:

You said, " if someone is looking to learn *good* OO practices, CFSQLTool
look."  So if CFQLTool outputs getter and setter will you retract this
statement?



At 10:19 PM 11/8/2005, you wrote:
On 11/8/05, Joseph Flanigan <[EMAIL PROTECTED]> wrote:
> 1. The tool clearly states MSSQL 2000. The tool does not make any claims to
> support other databases.

Do you intend to support other databases at some point?

>  4. Request.IN.FIELDNAMES must be set to a value before calling
> structKeyList because: "A list of keys; if structure does not exist,
> ColdFusion throws an exception"

Earlier in that function you do this:

        if(not isDefined("request.IN")) {request.IN = structNew();}

That ensures that request.IN has a struct value so the assignment to
request.IN.FIELDNAMES is not required, as I said.

> 5. "Joseph's whole point is about best practices and arguing that CFSQLTool > is a good example to follow." That is not the whole point. The point is that
> CFSQLTool is a new design tool for creating CRUD queries. I published the
> entire code on CFOpen so people can choose to improve.

And yet you get upset when someone suggests improvements?

> It is correct that I do not agree what some of the things you preach are
> best practices, certainly not always good architecture. I have said before
> you are a good with programing tricks.

We certainly do disagree on a lot of things :)

>  I will ask this again.
>
>  You said, " if someone is looking to learn *good* OO practices, CFSQLTool
> is definitely not the place to
>  look."  So if CFQLTool outputs getter and setter will you retract this
> statement?

Sorry, but it would take a lot more than that small change to change
my opinion of CFSQLTool.
--
Sean A Corfield -- http://corfield.org/
Got frameworks?

"If you're not annoying somebody, you're not really alive."
-- Margaret Atwood


----------------------------------------------------------
You are subscribed to cfcdev. To unsubscribe, send an email to [email protected] with the words 'unsubscribe cfcdev' as the subject of the email.

CFCDev is run by CFCZone (www.cfczone.org) and supported by CFXHosting (www.cfxhosting.com).

An archive of the CFCDev list is available at www.mail-archive.com/[email protected]

-----------------------------------------------------------------------
http://www.switch-box.org/CFSQLTool/Download/

Switch_box                      MediaFirm, Inc.
www.Switch-box.org              Loveland, CO  USA



----------------------------------------------------------
You are subscribed to cfcdev. To unsubscribe, send an email to 
[email protected] with the words 'unsubscribe cfcdev' as the subject of the 
email.

CFCDev is run by CFCZone (www.cfczone.org) and supported by CFXHosting 
(www.cfxhosting.com).

An archive of the CFCDev list is available at 
www.mail-archive.com/[email protected]


Reply via email to