[ddrumm]

Hi folks
im getting the below error message when i try and display the website, and cant figure out what the problem is
Element BROCHURE is undefined in URL.

Heres the code if it helps

[/b]<CFQUERY NAME="qupdate" DATASOURCE="#request.dsn#">
	SELECT * 
	FROM BROCHURE, BROCHURESIZE
	where BROCHURE_ItemID = #url.brochure#
</CFQUERY>


<form method="post" action = "results.cfm">
<cfif isdefined("url.brochure")>
<input type="hidden" name="BROCHURE_ItemID" value="#url.brochure#">
<input type="hidden" name="BROCHURESIZE_Key" value="#url.brochure#">

</cfif>

<cfif isdefined("form.BROCHURE_ItemID")>
<!--- this is an update --->
<cfquery name="qupdate" datasource="#request.dsn#">
update BROCHURE, BROCHURESIZE
set DIVISIONS_Key = #form.DIVISIONS_Key#,
BROCHURESIZE_Key = '#form.BROCHURESIZE_Key#',
BROCHURE_Title = '#form.BROCHURE_Title#',
BROCHURE_Detail = '#form.BROCHURE_Detail#'
BROCHURESIZE_Name = '#form.BROCHURESIZE_Name#'
where BROCHURE_ItemID = #form.BROCHURE_ItemID#
</cfquery>
<cfelse>
<!--- this is an insert --->
<cfquery name="qupdate" datasource="#request.dsn#">
insert into BROCHURE, BROCHURESIZE
(DIVISIONS_Key,BROCHURE_Size,BROCHURE_Title,BROCHURE_Detail,)
values
(#form.DIVISIONS_Key#,'#form.BROCHURE_Title#','#form.BROCHURE_Detail#','#form.BROCHURESIZE_Name#')
</cfquery>
</cfif>
<cflocation url="index.cfm"> [b]

[CR250]

For starters please use <cfqueryparam /> especially for URL, post parameters but really everywhere.

As for your issue dump url and make sure its appearing correctly.

[xheartonfire43x]

The error is being called from you query most likely which doesn't check whether or now url.brochure is defined. So either put a <cfif isDefined('url.brochure')> or put <cfparam name="url.brochure" default="whatever brochure you want as the default">.

[ddrumm]

xheartonfire43x, on 24 Mar, 2009 - 06:38 AM, said:

The error is being called from you query most likely which doesn't check whether or now url.brochure is defined. So either put a <cfif isDefined('url.brochure')> or put <cfparam name="url.brochure" default="whatever brochure you want as the default">.

Cheers Thanks for that