Fix the "Move rows failed" error message when dragging a root task in the process view

dlr-gtlab / gtlab-core

GTlab Core Framework

https://www.gtlab.de

Other

7 stars 2 forks source link

Fix the "Move rows failed" error message when dragging a root task in the process view #1173

Closed mariusalexander closed 6 months ago

mariusalexander commented 6 months ago

Summary

An error (fatal) message "Move rows failed" is triggered when dragging a root task within the process dock

Expected Behaviour

The error message should neither be a "Fatal" message and it should only trigger if something unexpected happend.

Reproduction steps

1. Select a root task
2. Drag it to an empty space in the process view
3. Oberserve the error message (see below)

Screenshots

grafik

Logs

No response

GTlab environment

GTlab Version: 2.0.6-dev

OS

Windows

jensschmeink commented 6 months ago

@mariusalexander with the latest changes of #565 the error does not appear anymore. But if the task is dropped at another position nothing happens and the order of the tasks stays the same.

Do you know if it was moved in the old version?

mariusalexander commented 6 months ago

Hm you are right this no longer happens, at least I can reproduce it easily. Still the output is very verbose when performing this action. And the "Fatal" message should at least be refactored to an error, with a more helpful error message.