2

Closed

Pretty large bug in the new drop - Presenters not getting set

description

For anyone using the new drop of the WCSF, there's a pretty large bug that took awhile to diagnose. It will only occur if you have View pages that have QueryString variables on them. If you do, the Presenter will not get set. The reason is because of the way that the WebClientApplication.IsAspxPage determines an Aspx page. I imagine it will get refactored again in the future, but for now, I just changed the return line in that method to:
 
        return request.Contains(Aspx_FileExtension);
 
instead and the code works again. If you don't do this, any pages that have a QueryString variable will not register as ASPX pages and won't get the dependency injection going.
Closed Jan 17, 2007 at 11:45 PM by blainew
Fixed for R1

comments

PauloMorgado wrote Dec 18, 2006 at 2:45 PM

In my opinion, that's not the right solution.

Both your solution and the current implmentation validate /WebUI/Errors/ApplicationError.htm?/WebUI/Default.aspx as an ASPX page.

The right solution would be to replace:

if (WebClientApplication.IsAspxPage(request.Url.ToString()))

with:

if (WebClientApplication.IsAspxPage(request.CurrentExecutionFilePath))