Trying to upgrade from revision 3399 to 4864 :)

This is the place to report bugs and get support. When posting in this forum, please always provide as much detail as possible.

Please do not report problems with a custom build or custom code in this forum. If you are producing your own build from the source code and have problems or questions, ask in the developer forum, do not report it as a bug.

This is the place to report bugs and get support

When posting in this forum, please try to provide as many relevant details as possible. Particularly the following:

What operating system were you running when the bug appeared?
What database platform is your site using?
What version of mojoPortal are you running?
What version of .NET do you use?
What steps are necessary to reproduce the issue? Compare expected results vs actual results.

This thread is closed to new posts. You must sign in to post in the forums.

3/13/2009 8:57:49 AM

Alexander V. Yushchenko

Total Posts 488

View Posts

Re: Trying to upgrade from revision 3399 to 4864 :)

Joe,

no, this problem is under IIS.

May be, I was too short in the problem description, let me describe it in a more detailed way.

Actually, there are 2 independent problems.

1. The first one is with UrlRewriter. Actually it must not do anything for requests into /Data folder, but it does. The solution you suggest is for this problem, but I suggest to replace the criteria with || (app.Request.Path.StartsWith("/Data/", StringComparison.InvariantCultureIgnoreCase)).

2. The second one is with CSSHandler, but actually with CacheHelper.GetSiteSettingsFromCache(). When request is to the /Data/ subfolder, it does not recognize folder-based subsites, thinking it's always the root site.

This problem actually does not depend on the first one: now SiteSettings are requested from CacheHelper.GetSiteSettingsFromCache() and stored in HttpContext by UrlRewriter, but if it doesn't CSSHandler will do absolutely the same.

As far as I can see, there are 2 possible solutions for the second problem: either correct CacheHelper.GetSiteSettingsFromCache() so that it can recognize the site in this case as well or modify CSSHandler so that it does not use SiteSettings. I suggest the last variant - SiteSetting are really not required there, path to the skin can be got from the request path.

3/13/2009 9:09:34 AM

Joe Audette

Total Posts 18439

View Posts

Re: Trying to upgrade from revision 3399 to 4864 :)

Hi Alexander,

I think I understand the problem now. I'm looking into it. The siteSettings are determined by the url of therequest but since the current url of csshandler does not include the folder for the folder site it gets sitesettings for the first site.

I'll post again after I have it solved.

Best,

Joe

3/13/2009 10:15:27 AM

Joe Audette

Total Posts 18439

View Posts

Re: Trying to upgrade from revision 3399 to 4864 :)

Hi Alexander,

Can you do svnupdate and rebuild and let me know if its solved.

Thanks,

Joe

3/13/2009 10:46:50 AM

Alexander V. Yushchenko

Total Posts 488

View Posts

Re: Trying to upgrade from revision 3399 to 4864 :)

Joe,

It works, thanks.

P.S. Why such a strange solution for this handler, by the way? What for does it need so many parameters? As I can see, it can potentially be called without any parameters as the path to the skin requested is alredy in the request path...

3/13/2009 12:57:44 PM

Joe Audette

Total Posts 18439

View Posts

Re: Trying to upgrade from revision 3399 to 4864 :)

Yeah its true, combining css was a tough challenge and I tried several approaches for adding files and some of those params were for a different approach than the one I ended up with in the end. I've cleaned that up this afternoon and removed the parts I'm not using to reduce a little complexity.

Best,

Joe

You must sign in to post in the forums. This thread is closed to new posts.