OpenChemistry / avogadroapp

Avogadro is an advanced molecular editor designed for cross-platform use in computational chemistry, molecular modeling, bioinformatics, materials science, and related areas.
https://two.avogadro.cc/
BSD 3-Clause "New" or "Revised" License
179 stars 69 forks source link

[Bug] :If no location is selected while saving a file the app crashes #479

Closed secretkontributer closed 4 months ago

secretkontributer commented 4 months ago

Avogadro version: (please complete the following information from the About box):

Desktop version: (please complete the following information):

Describe the bug App carshes when you dont select the location in which you want to save the file .

To Reproduce Steps to reproduce the behavior:

  1. Go to save menu option .(You can recreate this bug with save as also) Screenshot from 2024-02-25 12-46-37

  2. Click on Save or Save As option . A window will pop up for selecting the location.

screenshot 3.Now dont specify any location just exit from the screen shown in above picture.(Basically press the cancel button on the top left corner) Screenshot from 2024-02-25 12-47-25

  1. The App will crash .
  2. Console output is given in the picture attached below. Screenshot from 2024-02-25 12-48-42 Expected behavior App should not crash

Additional context Assert statement in the file location given in the console leads to this line. Screenshot from 2024-02-25 12-52-52

avo-bot commented 4 months ago

This issue has been mentioned on Avogadro Discussion. There might be relevant details there:

https://discuss.avogadro.cc/t/if-not-selected-any-location-while-saving-the-file-the-app-carshes/5323/1

welcome[bot] commented 4 months ago

Thanks for opening your first issue here! Please try to include example files and screenshots if possible. If you're looking for support, please post on our forum: https://discuss.avogadro.cc/