windrobin / winforms-geplugin-control-library

Automatically exported from code.google.com/p/winforms-geplugin-control-library
GNU General Public License v3.0
0 stars 1 forks source link

Constructor of GEEventArgs doesn't parse EventId #89

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
In the code

        public GEEventArgs(string message, string data)
            : this(message)
        {
            EventId id = EventId.None;
            if (Enum.TryParse<EventId>(data, out id))
            {
                this.EventId = id;
            }

            this.Data = data;
        }

method Enum.TryParse<EventId>(data, out id) according to msdn 
http://msdn.microsoft.com/en-us/library/dd783499 is case-sensitive, "data" is 
always lower case, so EventId is always None. It should be changed to 
Enum.TryParse<EventId>(data, true, out id).

Original issue reported on code.google.com by yuri.zab...@gmail.com on 21 Aug 2012 at 3:55

GoogleCodeExporter commented 9 years ago
Good catch, I think the latest source actually has the correct code but I will 
make sure in the next commit.
Thanks

Original comment by fraser.c...@gmail.com on 25 Aug 2012 at 2:43

GoogleCodeExporter commented 9 years ago
Fixed in the latest commit

Original comment by fraser.c...@gmail.com on 4 Sep 2012 at 12:09