Posts: 222
Threads: 44
Joined: Jan 2014
Reputation:
0
This seems like quite a serious bug:
I spent about 20 minutes making some significant annotations to page 2 of one song. I then wanted to share only that page with band members as PDF, and the only way I could see to selectively export one page as a PDF was to temporarily change the Page Order field from "1-2" to "2", do the export, then change it back. (I'll start a separate feature request thread for a better way to do that.)
Unfortunately this caused a complete loss of all the annotations I had just made. I imagine they are maybe still buried in some sqlite table, but possibly with the wrong page reference, therefore invisible and inaccessible. Or maybe something else caused it - I'm just wildly guessing. Even after restoring the page order to both pages, they did not reappear, so presumably I'll have to make them all over again.
Posts: 394
Threads: 28
Joined: May 2019
Reputation:
12
Samsung Galaxy Tab S7 FE Android 12
Samsung Note Pro 12.2 LineageOS 14.1
Huawei Media Pad M3 lite Android 7
Posts: 13,366
Threads: 302
Joined: Apr 2012
Reputation:
235
02-24-2020, 09:32 AM
(This post was last modified: 02-24-2020, 09:36 AM by Zubersoft.)
Changing the page order is going to delete annotations - there is no way to avoid that. If you remove pages from a song, all references to things on those pages are gone. It doesn't make sense to preserve data for pages you've removed. What you should have done is copy the song (long press the song, copy song on the library screen) and then modified the copy. The reason you can't just change the page order like that is that annotations aren't saved in the file itself - they are saved only in the database. So if you remove page references, everything related to those pages are removed from the database.
The annotations redesign is (tentatively) adding a feature that will automatically convert between PDF annotations and MobileSheets annotations. This will allow some modifications to existing PDF annotations, and will allow MobileSheets annotations to be saved in the PDF instead of in the database. This feature will limit what settings are supported though, as the PDF spec doesn't allow some things that MobileSheets supports.
Mike
McAroni - the bug fix for the forum post you linked to is to ensure annotations on pages are correct after changing the page order. I have a fix in place for that. Removing pages in the page order, saving those changes, and then adding the pages back is not going to bring back the annotations. That is not something I am going to support.
Posts: 13,366
Threads: 302
Joined: Apr 2012
Reputation:
235
Then yes, the changes I've made should fix what you are seeing. That's basically just a bug due to code I added to handle the scenario where you remove a file and add back a new version of it (I didn't want users to lose their annotations when they do that).
Mike
Posts: 222
Threads: 44
Joined: Jan 2014
Reputation:
0
OK great - but this bugfix is not in the Play store yet, right? According to my Play app I already have the latest version of MSP and I still see this issue.
Posts: 13,366
Threads: 302
Joined: Apr 2012
Reputation:
235
Correct, I haven't released the update yet as I'm still addressing emails from users. I'm trying to work through all the recent bug reports before releasing the update so I can focus back on the annotations redesign after.
Mike