Ticket #351 (closed defect: fixed)

Opened 17 years ago

Last modified 16 years ago

Error with SES URLs after upgrade to RC

Reported by: teamteama Owned by:
Priority: normal Milestone:
Version: 3.0.178 Severity: normal
Keywords: Cc:

Description

I just updated my beta version of MG3 to the RC version and started getting an error while using SES URLS.

Error is:

Element MOCKCGISCOPE.PATH_INFO is undefined in ARGUMENTS. com/model-glue/ModelGlue/gesture/eventrequest/url/SesUrlManager.cfc: line 35

33 : <cfif structKeyExists(arguments, "mockCgiScope")> 34 : <cfset pathArray = listToArray(arguments.mockCgiScope.PATH_INFO, "/") /> 35 : <cfelseif arguments.mockCgiScope.PATH_INFO NEQ arguments.mockCgiScope.SCRIPT_NAME> 36 : <cfset pathArray = listToArray(arguments.mockCgiScope.PATH_INFO, "/") /> 37 : </cfif>

Looks like the elseif statement is wrong. It will always error out if the if statement is false.

Change History

Changed 17 years ago by teamteama

this error also exists in MG3 final. It looks like the code was changed as a part of Ticket #349. But, the patch suggested in that ticket, did not make it into the code. I took the code from that ticket, and it worked in my environment.

<cfif structKeyExists(arguments, "mockCgiScope")>
	<cfset pathArray = listToArray(arguments.<span class="searchword0">mockCgiScope.PATH_INFO</span>, "/") />
<cfelse>
	<cfif cgi.PATH_INFO NEQ cgi.SCRIPT_NAME>
		<cfset pathArray = listToArray(cgi.PATH_INFO, "/") />
	<cfelse>
		<cfset pathArray = arrayNew(1) />
	</cfif>
</cfif>

Changed 16 years ago by DanWilson

I just committed a version of this logic in SVN revision 184.

We'll add this to a release in the upcoming weeks.

DW

Changed 16 years ago by DanWilson

  • status changed from new to closed
  • resolution set to fixed

This fix was rolled and released a few months ago.

I'm closing the ticket.

Note: See TracTickets for help on using tickets.