EDDiscovery / EDDiscovery

Captains log and 3d star map for Elite Dangerous
Apache License 2.0
769 stars 171 forks source link

Crash on filtering events #3317

Closed Herkfixer closed 2 years ago

Herkfixer commented 2 years ago

Description

Click on "None" for filters and crsahed.

Additional Information

Exception Details:

EDDiscovery v15.1.2.0 There was an unhandled UI exception.

==== BEGIN ====
System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  at System.Collections.ArrayList.get_Item(Int32 index)
  at System.Windows.Forms.DataGridViewRowCollection.SharedRow(Int32 rowIndex)
  at System.Windows.Forms.DataGridViewRowCollection.get_Item(Int32 index)
  at EDDiscovery.UserControls.UserControlTravelGrid.<>c__DisplayClass37_0.<HistoryChanged>b__0() in C:\Code\EDDiscovery\EDDiscovery\UserControls\History\UserControlTravelGrid.cs:line 350
  at System.Windows.Forms.Timer.OnTick(EventArgs e)
  at System.Windows.Forms.Timer.TimerNativeWindow.WndProc(Message& m)
  at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
===== END =====
robbyxp1 commented 2 years ago

Have outlining on, and then go to None.

image

Reproduced.. i'll look into from wed on. Pretty easy fix.

thanks for the report.

robbyxp1 commented 2 years ago

Ok, found time, fixed on master and Dev. Thanks

Herkfixer commented 2 years ago

Wow, you work fast. Thanks.

On Sun, Sep 11, 2022, 3:48 AM Rob @.***> wrote:

Ok, found time, fixed on master and Dev. Thanks

— Reply to this email directly, view it on GitHub https://github.com/EDDiscovery/EDDiscovery/issues/3317#issuecomment-1242909707, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD5LQN6R7LMVRUULZRE7SQTV5WFF5ANCNFSM6AAAAAAQJRSEDA . You are receiving this because you authored the thread.Message ID: @.***>