search

sdenier / Geco

Lightweight desktop application for management of orienteering events
http://sdenier.github.io/Geco
10 stars 6 forks source link

Changing name of stage in Launch Wizard #44

Open ghost opened 8 years ago

ghost commented 8 years ago

Steps to reproduce:

1) Type something (name1) in name field 2) Go out of the field (allow blur event to happen) 3) Change the name field (to name2) 4) Go out of the field (allow blur event to happen) 5) Path will now be GECOROOT/name1/name2/ 6) Click create 7) Try to add new runner => Get error message ("No course defined in stage")

Everytime the name field blur event happens the path field has name appended

Expected behavior:

1) Type "name1" in name field, and leave field (allow blur to happen) 2) Path becomes GECOROOT/name1/ 3) Change name field to be "name2", and leave field (allow blur to happen) 4) Path becomes GECOROOT/name2/

sdenier commented 8 years ago

Probable cause (to confirm): GECOROOT/name1/name2/ is not created recursively when creating the new stage. Then the save is completely faulty, data are missing, then you get errors such as "No course defined in stage".

Root cause is that the ergonomy of this little launch screen is quite unsatisfactory and the user can fall into this unexpected case. Working on the UX would require quite some work, so probably the easy solution is to create dirs recursively.

ghost commented 8 years ago

Geco should be able to tell which folder it is running from? Rather than appending name2 to the path field, change the path to GECOROOT/name2

sdenier commented 8 years ago

Well, one have to dive into the Java doc to be sure. Hopefully this is platform independent, but I don't much time to work on Geco right now.