[Mono-bugs] [Bug 80836][Nor] Changed - TypeDescriptor assumes component always has ITypeDescriptorFilterService

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by gert.driesen at pandora.be.

http://bugzilla.ximian.com/show_bug.cgi?id=80836

--- shadow/80836	2007-02-13 13:08:58.000000000 -0500
+++ shadow/80836.tmp.30162	2007-02-13 18:30:33.000000000 -0500
@@ -1,23 +1,23 @@
 Bug#: 80836
 Product: Mono: Class Libraries
 Version: 1.2
-OS: 
+OS: unknown
 OS Details: Ubuntu Edgy Eft, using Mono SVN
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
-Component: Windows.Forms
+Component: System
 AssignedTo: toshok at ximian.com                            
 ReportedBy: frederik.carlier at ugent.be               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
-Summary: PropertyGrid: NullReferenceException
+Summary: TypeDescriptor assumes component always has ITypeDescriptorFilterService
 
 Please fill in this template when reporting a bug, unless you know what you
 are doing.
 Description of Problem:
 
 Running the attached application with mono gives the following output:
@@ -64,6 +64,21 @@
 Additional Information:
 
 ------- Additional Comments From frederik.carlier at ugent.be  2007-02-13 13:08 -------
 Created an attachment (id=18692)
 Source code required to reproduce the issue
 
+
+------- Additional Comments From gert.driesen at pandora.be  2007-02-13 18:30 -------
+This is actually not a problem in PropertyGrid, but a bug in 
+TypeDescriptor since it always assumes a component's site has a 
+ITypeDescriptorFilterService.
+
+I'll add a small code snippet which demonstrates the problem, and 
+I'll provide a patch which fixed the issue.
+
+I'll hold off on committing the patch until I've had time to create 
+unit tests that cover the fix (but this should be easy).
+
+Freferik, some (friendly) remarks:
+1. Try to create smaller repro's, if possible.
+2. I'll stop by for a beer soon ;-)