search

ilia3101 / MLV-App

All in one MLV processing app.
https://mlv.app/
GNU General Public License v3.0
274 stars 30 forks source link

Deleting active video from session messes up other video properties #233

Closed vastunghia closed 2 years ago

vastunghia commented 2 years ago

To replicate issue:

  1. open new session
  2. import 3 MLV files
  3. double click on 2nd video
  4. set "Cut Out" to 2 (or whatever property to whatever value you like)
  5. right click on active video (2nd video) and select "Delete Selected File from Session"
  6. confirm "Remove"

You will see that the 3rd video, having become the active one, is inheriting 2nd video properties ("Cut Out" = 2).

What's worse: even if you skip step 4 (no edit at all for video 2), you will end up having video 3 with "Cut Out" = full length of video 2 (that could be shorter than full length of video 3).

Very annoying. Took me a long time to understand why some of my exported cDNG's were shorter than due.

Thank you!

Sergio

masc4ii commented 2 years ago

Thank you for pointing this out. I am able to reproduce the bad behavior. Now "just" someone has to fix it... ;)

masc4ii commented 2 years ago

Please test commit aec844ac22ba8c67b9bce7e4226b40de060dd9f8

vastunghia commented 2 years ago

Commit aec844a built and tested, confirm bug solved. Thank you!

masc4ii commented 2 years ago

Thank you!