[Mono-bugs] [Bug 398274] New: TableLayoutSettingsTypeConverter fails reading (known) bad data

bugzilla_noreply at novell.com bugzilla_noreply at novell.com
Sat Jun 7 08:05:35 EDT 2008


           Summary: TableLayoutSettingsTypeConverter fails reading (known)
                    bad data
           Product: Mono: Class Libraries
           Version: 1.9.0
          Platform: Other
        OS/Version: Other
            Status: ASSIGNED
          Severity: Normal
          Priority: P5 - None
         Component: Windows.Forms
        AssignedTo: jpobst at novell.com
        ReportedBy: andyhume32 at yahoo.co.uk
         QAContact: mono-bugs at lists.ximian.com
          Found By: ---

Created an attachment (id=220872)
 --> (https://bugzilla.novell.com/attachment.cgi?id=220872)
Test cases patch

Additional unit tests attached.  I'll leave the fix to you Jonathan. :-)

Two problems in TableLayoutSettingsTypeConverter (other keywords
TableLayoutSettings, TableLayoutPanel):

1. ConvertTo writes: "Columms"  !! two 'm's !!

2. MSFT (in the past?) generated bad content when the current culture had comma
as its decimal point character, e.g. writing:
instead of:

(and 271059)
also one which notes the fix _didn't_ to fix the output:

Mono crashes on reading the odd content; MSFT (with SP1?) doesn't (I haven't
looked what objects it returns however).

I attach a patch containing additional tests to test such things.  Also
includes a tiny fix to the currently NotWorking test -- it just needed the XML
elements reordered to work (and uncover the "Columms" bug)!  Very simple tests

The ConvertFrom (i.e. input tests) don't check the result (yet? -- does Equals
work??), just that no exception occurs.  Testing of various "Control" elements
is missing too, etc etc.

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

More information about the mono-bugs mailing list