a-shell icon indicating copy to clipboard operation
a-shell copied to clipboard

a-shell keeps forgetting custom bookmarked locations

Open NightMarekApps opened this issue 1 year ago • 16 comments

hi! since the last update the app has forgotten my bookmarked locations multiple times. i dont know what causes it

NightMarekApps avatar Sep 07 '22 05:09 NightMarekApps

What I've done in the last update is check that bookmarks are still valid at startup, and remove the ones that aren't. This was to avoid the "cd ~bookmark" that was followed by "no such file or directory". It shouldn't remove a bookmark that is still valid.

holzschu avatar Sep 07 '22 06:09 holzschu

i did some testing to see when that happens. it happens everytime i restart my device. bookmarks disappear and i get this jump: [bookmark] not found

NightMarekApps avatar Sep 07 '22 06:09 NightMarekApps

IMG_0103 i restarted the device before using cd command above happens both on ipad and iphone (both beta) since the latest a-shell update

NightMarekApps avatar Sep 07 '22 06:09 NightMarekApps

Yes, restarting the device was making the bookmarks invalid, so that "cd ~bookmarkName" was returning "No such file or directory". You're saying that it was working with jump before, even though it was not working with cd?

holzschu avatar Sep 07 '22 06:09 holzschu

i only ever used jump and it always worked for me. since last update this happens after restart

NightMarekApps avatar Sep 07 '22 06:09 NightMarekApps

So the problem was with cd, not with the bookmarks. Thank you very much. I'm going to change that.

holzschu avatar Sep 07 '22 06:09 holzschu

Thank you. i dont quite understand how cd is the problem here my workflow is this: pick folder, bookmark it once with custom shortname using bookmark command. then always go to that folder using the jump command. now after restart it says jump: shortname not found

im happy to do some further testing if you need

NightMarekApps avatar Sep 07 '22 06:09 NightMarekApps

cd was checking both isReadableFileAtPath and changeCurrentDirectoryPath for each directory, jump was just trying changeCurrentDirectoryPath. It seems that after a reboot, isReadableFileAtPath becomes false for a bookmarked directory, even though changeCurrentDirectoryPath still works. That's weird and unexpected, but I'll adapt the code.

holzschu avatar Sep 07 '22 06:09 holzschu

Thank you. I did some more testing though and after bookmarking locations, i ran showmarks to show my newly created bookmarks, and they are listed there

i then rebooted and ran showmarks again, and these bookmarks are gone. they disappear after restart

NightMarekApps avatar Sep 07 '22 06:09 NightMarekApps

That makes sense: that's what I wrote the code for, to avoid the issue with "cd ~bookmark: permission denied".

holzschu avatar Sep 07 '22 07:09 holzschu

im sorry that is complicated for me, does that mean your fix will fix it? or that it forgets them by design :)

NightMarekApps avatar Sep 07 '22 07:09 NightMarekApps

Yes, the new fix will remove the previous fix, and you will get back the previous behaviour with version 1.9.7. As a bonus, people who use "cd ~bookmark" instead of "jump bookmark" will also enjoy bookmarks that remain valid after a reboot.

holzschu avatar Sep 07 '22 07:09 holzschu

Thank you so much for all this and for quick responses!

NightMarekApps avatar Sep 07 '22 07:09 NightMarekApps

When can we expect a new release with the fix?

AlbertBaubleDeem avatar Apr 20 '23 14:04 AlbertBaubleDeem

I believe that fix was released months ago, around version 1.9.7. Do you still observe a problematic behaviour?

holzschu avatar Apr 20 '23 15:04 holzschu

I just got the behaviour the first time. I'm not aware of a recent update. So could this be due to the latest iOS update?

SirLukeSchande avatar May 05 '23 13:05 SirLukeSchande