Saving files in custom data dir not working

[bqcrabtree]

Before submitting an issue, please check the following

• I have searched for similar issues (both open and closed) and cannot find a duplicate
• I agree to follow the Code of Conduct

Describe the bug

On the newest Saber version, custom data dirs selected via the UI (in this case a Syncthing directory) do not allow notes to be saved, and thereby the editor cannot be exited without closing the app manually.

To reproduce

1. Be on the newest (v0.21.4) version of Saber on Android
2. Set a custom data directory via the Settings UI
3. Create a note and attempt to save it by either backing out or by clicking the save icon at the top left

Expected behavior

The file to save and the editor to close as normal

Saber version

v0.21.4

Device

• Device: Samsung SM-X510
• OS: Android 14 (OneUI 6.0)

Anything else?

No response

[Appel-flappen]

Can reproduce on pixel 6 with grapheneos.
Notably I cannot find any option to enable storage scopes for the Saber app, which would normally be required by any other app using shared storage space. seems like it might not be requesting permissions for storage?

[Luciogi]

I guess that newly created directory has permission issue

[deepu2]

I wanted to use Synching so I changed the directory and lost all my files (I didn't have much but still..). I can't create new notes too.

[faisal-kursheedali]

Getting this error message in saving the note in the custom directory

Exception has occurred. PathAccessException (PathAccessException: Cannot open file, path = '/storage/emulated/0/Eg/24-05-25 Untitled.sbn2' (OS Error: Operation not permitted, errno = 1))


This exception is thrown from the file: lib/pages/editor/editor.dart at line number 905 when running the app in the debug mode

[Iey4iej3]

Getting this error message in saving the note in the custom directory

Exception has occurred. PathAccessException (PathAccessException: Cannot open file, path = '/storage/emulated/0/Eg/24-05-25 Untitled.sbn2' (OS Error: Operation not permitted, errno = 1)) This exception is thrown from the file: lib/pages/editor/editor.dart at line number 905 when running the app in the debug mode

I think that this should be due to the permission of access of storage. I look at the list of permissions of Saber, and it does not contain such a permission, and thus this error is expected.

[aqazzu]

I also have an issue with scoped permessions. Saber doesn't ask for the permission to access the custom folder on my galaxy tab S6 lite. I also wanted to use this approach to locally sync using Syncthing (sadly this manouver made me lose some notes I havent exported yet).

I'm asking about this bug's priority on the list of things to be done, I'd imagine solving this is important :)

[Iey4iej3]

I'm asking about this bug's priority on the list of things to be done, I'd imagine solving this is important :)

Simply click 👍 in the original post to raise the issue in #1196.

[ComputerElite]

I've forked saber and added the MANAGE_EXTERNAL_STORAGE permission. After enabling it the issues went away so I do support the idea that it has something to do with permissions. I'm now able to sync everything via syncthing

[bbenjadol]

I've forked saber and added the MANAGE_EXTERNAL_STORAGE permission. After enabling it the issues went away so I do support the idea that it has something to do with permissions. I'm now able to sync everything via syncthing

Are you able to release your compiled apk? I keep on facing build issues when trying to compile

[ComputerElite]

Will do that later in like 4h

[ComputerElite]

You can find a compiled version of my fork here https://github.com/ComputerElite/saber/releases/latest

[Iey4iej3]

You can find a compiled version of my fork here https://github.com/ComputerElite/saber/releases/latest

Could you please make this a pull request for Saber? Thanks.

[ComputerElite]

Could you please make this a pull request for Saber? Thanks.

Somebody already made a pull request with my commit (#1442 ). However I'll add a pop up informing you to enable the permission when setting the directory before putting you in the settings and then make a own one.

[ComputerElite]

Made a pr #1448

[TillF]

Made a pr #1448

Solved #1242 for me. Thanks!