Demizo / Daily_You

Every day is worth remembering...
GNU General Public License v3.0
127 stars 6 forks source link

Can't access files on system #65

Closed Ranger-NF closed 10 months ago

Ranger-NF commented 11 months ago

Issues:

Device info: Android 8

App Version: 1.7.0 (Downloaded using droid-ify)

Demizo commented 11 months ago

@Ranger-NF Did the app prompt for file/storage permissions and was the permission granted?

Also what are your image/log folders (the folder paths are listed on the settings page)

Ranger-NF commented 11 months ago

@Demizo I have double checked the permission. It has been granted

Log folder stays the same even when I set the folder. Here are the screenshots:

Demizo commented 11 months ago

@Ranger-NF I ran the app in an android 8 emulator and ran into similar issues. I can use the app normally and add photos if the image and log paths are defaulted to the app's internal folders. However, like you I can not change the log or image folders to anything meaningful. This is do to a lack of read/write permissions on android 8. The storage permissions I am requesting aren't enough for old android versions. I am exploring how to best add these permissions without effecting later android versions.

Ranger-NF commented 11 months ago

@Demizo Ok, I have installed V1.7.1 unfortunately the issue is not completely fixed. I can change the path to all directories except Documents and its sub-directories.

ie, When I try to change the path to /0/Documents or /0/Documents/Diary/, the app behaves as mentioned in my earlier comment (path doesn't change for log folder and it changes to / for images)

Its not a big issue for me rn. I'm okay with creating a new folder in /0/ (It seems to work). I will update if there are any other such weird bugs

Demizo commented 11 months ago

@Ranger-NF I will double check if this is true of the documents folder, but I know some directories are protected on android from apps just reading/writing to them without a specific api. Additionally flutter has some limitations when it comes to platform specific apis like android. I may need to implement the file access logic in native android code at some point. I should also have an error message when a folder can't be accessed to make the behavior less confusing.

Demizo commented 10 months ago

@Ranger-NF Let me know if the move to SAF in the latest update fixes this!

Ranger-NF commented 10 months ago

@Demizo It seems to have been fixed on the new version. Good job mate!