numere-org / NumeRe

Framework for numerical computations, data analysis and visualisation
https://www.numere.org
GNU General Public License v3.0
18 stars 6 forks source link

Clicking on the wrong place on the dependency viewer triggers a debug dialog #55

Closed numeredev closed 1 year ago

numeredev commented 1 year ago

DESCRIPTION

Describe the bug If we click on the wrong place in the dependency viewer, then a debug alert dialog is triggered.

To Reproduce Steps to reproduce the behavior:

  1. Open the dependency viewer
  2. Click on something, which is not part of the list
  3. See error

Expected behavior This dialog should not appear.

Screenshots grafik

Desktop (please complete the following information):

Additional context Add any other context about the problem here.

(Do not write below this line)


DEVS' SECTION

ANALYSIS

The problem is in void DependencyDialog::OnItemSelected(wxTreeEvent& event) in dependencydialog.cpp. Here, it isn't checked, whether the selected item is actually valid, i.e. event.GetItem().IsOk(). Add that check and only proceed, if it returns true.

IMPLEMENTATION STEPS

(see also our Wiki for implementation guidelines)

DOCUMENTATION STEPS

(see also our Wiki for further information)

PULL REQUEST