https://bugzilla.novell.com/show_bug.cgi?id=371859

User [EMAIL PROTECTED] added comment
https://bugzilla.novell.com/show_bug.cgi?id=371859#c2


Atsushi Enomoto <[EMAIL PROTECTED]> changed:

           What    |Removed                                         |Added
----------------------------------------------------------------------------
 Attachment #202646|0                                               |1
        is obsolete|                                                |




--- Comment #2 from Atsushi Enomoto <[EMAIL PROTECTED]>  2008-03-21 13:06:44 
MST ---
Created an attachment (id=203440)
 --> (https://bugzilla.novell.com/attachment.cgi?id=203440)
more proposed fix

Now I found more preferred way to fix this issue.

PageParser.GetCompiledPageInstance() currently ignores inputFile argument when
it uses BuildManager (it does in 2.0) but it should not ignore this argument.
Since It is not limited to SoapDocumentationHandler, this fix should apply to
PageParser. And as a result, existing fake path hack could be removed. (and and
IMO it should; I didn't include such a change in this patch though.)

I find another possible bug in SoapDocumentationHandler.ctor(). It does not
actually premise custom WSDL help page because it always premise path as
equivalent to machine.config (so even if a web app contains <wsdlHelpGenerator
href="blabla.aspx" />, it will result in "no such file" error). In such case
I'd pass null inputFile so that PageParser can map virtual path (to
wsdlHelpGenerator) to physical path.

FYI: "inputFile" can be null in .NET. Try the following code in simple web app
with Default.aspx:

  PageParser.GetCompiledPageInstance("Default.aspx", null, this.Context);


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
mono-bugs maillist  -  mono-bugs@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to