Why won't this delete???

[weswhite7]

My problem is that this code was previously used, but it does not work now. I
think it may be because I have so many url variables. Any ideas? It's just
supposed to delete the record thats output in that row based on the id.

<cfif isdefined("url.action") and isdefined("url.season") and
isdefined("url.page") and isdefined("url.id")>
<cfset display = "1">
<cfif url.action eq "<cfoutput>#userid#</cfoutput>" and url.season eq
"<cfoutput>#season#</cfoutput>" and url.page eq "del" and isdefined("url.id")>
<cfquery name="delete" username="sla" password="slaera" datasource="sla">
delete FROM login_adm WHERE id = '#url.id#'
</cfquery>
<script>
window.location="yahoo.com";
</script>
</cfif>
</cfif>

<table width="100%" border="1" cellpadding="2" cellspacing="2">
<tr>
<td colspan="5" class="leaguename">Sub-Admin Login Modifications </td>
</tr>
<tr class="coolblue">
<td width="26%"><div align="center">Personal Name </div></td>
<td width="26%"><div align="center">Username</div></td>
<td width="26%"><div align="center">Password</div></td>
<td width="10%"><div align="center">Edit</div></td>
<td width="10%"><div align="center">Delete</div></td>
</tr>
<cfoutput query="checklogin">
<tr class="maintext">
<td width="26%"><div align="center">#name#</div></td>
<td width="26%"><div align="center">#login#</div></td>
<td width="26%"><div align="center">#login_passwd#</div></td>
<td width="10%"><div align="center"><a
href="sub-admin-add.cfm?action=#url.action#&season=#url.season#&ed itid=#login_id
#">[Edit]</a></div></td>
<td width="10%"><div align="center"><a
href="sub-admin.cfm?action=#url.action#&season=#url.season#& page=del&id=#login_i
d#">[Delete]</a></div></td>
</tr>
</cfoutput>

[Fernis]

You must not use <cfoutput> in situations like this. And you shouldn't use #
signs either, since you're not trying to _output_ anything. If your #userid#
was "7" for example, you're comparing if url.action equals
"<cfoutput>7</cfoutput>", not "7".

Change

<cfif url.action eq "<cfoutput>#userid#</cfoutput>" and url.season eq
"<cfoutput>#season#</cfoutput>" and url.page eq "del" and isdefined("url.id")>

to:

<cfif url.action EQ userid AND url.season EQ season AND url.page EQ "del" AND
isDefined("url.id")>

(just capitalized operators for clarity)

You're not trying to _output_ anything on the screen. So do not use <cfoutput>
or #'s, because you don't need them.

[weswhite7]

works great! thank you