MobileSheets Forums
Minor bug deleting song that is duplicated in setlist - Printable Version

+- MobileSheets Forums (https://www.zubersoft.com/mobilesheets/forum)
+-- Forum: Support (https://www.zubersoft.com/mobilesheets/forum/forum-1.html)
+--- Forum: MobileSheets (Android) (https://www.zubersoft.com/mobilesheets/forum/forum-23.html)
+--- Thread: Minor bug deleting song that is duplicated in setlist (/thread-4390.html)



Minor bug deleting song that is duplicated in setlist - Geoff Bacon - 02-18-2018

Hi Mike

I had a song mentioned twice in a setlist
When I deleted the song from the "Songs" tab, MSP only removed one of the entries from the setlist (leaving an entry with no associated file)

Created a test Setlist and added a song twice; again it only removed one entry when the song was deleted.

Looks like you are not processing the rest of a setlist after you have found a match.

Cheers
Geoff


RE: Minor bug deleting song that is duplicated in setlist - Geoff Bacon - 02-18-2018

Just noticed that the song that was probably at the bottom of the setlist no longer points to the correct file.
Haven't tested further; it could have been like this for prior to deleting "the song + remove of the bad entry".

Geoff


RE: Minor bug deleting song that is duplicated in setlist - Zubersoft - 02-18-2018

Thanks - I'm surprised this has never come up before. I just added a fix that will be included with the next update this weekend.

Mike


RE: Minor bug deleting song that is duplicated in setlist - Geoff Bacon - 02-18-2018

Glad to have been of help

Geoff


RE: Minor bug deleting song that is duplicated in setlist - Geoff Bacon - 02-27-2018

Mike

While you have fixed the app to handle duplicates in a setlist, the problem still exists in the companion.

Unlikely to cause anyone any issues but, if you are like me, you won't like having bugs in your work.

Geoff


RE: Minor bug deleting song that is duplicated in setlist - Zubersoft - 02-28-2018

Thanks Geoff - I forgot to check the code in the companion application. It likely has the same exact issue in the code.

Mike