[Mono-bugs] [Bug 331250] Replace System.Web.Util.UrlUtils by System.Web. VirtualPathUtility

[bugzilla_noreply at novell.com]

https://bugzilla.novell.com/show_bug.cgi?id=331250#c1


Marek Habersack <mhabersack at novell.com> changed:

           What    |Removed                                         |Added
----------------------------------------------------------------------------
             Status|NEW                                             |RESOLVED
         Resolution|                                                |WONTFIX




--- Comment #1 from Marek Habersack <mhabersack at novell.com>  2007-10-16 03:04:49 MST ---
Juraj, on surface it might seem like a good idea, but in reality the two
classes are not compatible enough to make the move. The "old" UrlUtils code
works differently in a few cases (Combine, IsRelativeUrl - the two are used
extensively)  and the code that uses the methods expects different results to
what VirtualPathUtility provides (try, e.g., Combine with application virtual
path set to something different than "/", IsAppRelative has a different set of
checks to IsRelativeUrl etc.).
Just out of curiosity, I replaced the calls to UrlUtils.Combine with
VirtualPathUtility.Combine throughout the Sys.Web code and it resulted in 4
test failures in the 1.1 profile, 253 in the 2.0 profile - and all of the
failures were "valid" (i.e. the "old" UrlUtils behavior matches the .NET one)
Therefore, I will close the bug as WONTFIX.


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.