Ticket #351 (closed defect: fixed)

Opened 2 years ago

Last modified 2 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 2 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 2 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 2 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.