Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

automatic backup doesn't work #144

Closed
7 tasks done
bingoxo opened this issue Jul 22, 2024 · 7 comments
Closed
7 tasks done

automatic backup doesn't work #144

bingoxo opened this issue Jul 22, 2024 · 7 comments
Labels
duplicate This issue or pull request already exists

Comments

@bingoxo
Copy link

bingoxo commented Jul 22, 2024

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.0.1

Affected Android/Custom ROM version

Android 14

Affected device model

Pixel 6a

How did you install the app?

Google Play Store

Steps to reproduce the bug

enable automatic backup

Expected behavior

automatic backup to work

Actual behavior

nothing happens

Screenshots/Screen recordings

No response

Additional information

i also can't choose an existing folder and have to create new one

@bingoxo bingoxo added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Jul 22, 2024
@Aga-C
Copy link
Member

Aga-C commented Jul 22, 2024

  1. What's the path to that existing directory?
  2. Do you have this problem with all existing directories?
  3. By "doesn't work" you also mean, that even if you created a new directory, it also haven't created backups?
  4. Do you have a stock Android 14 or a custom ROM?

@Aga-C Aga-C added question Further information is requested waiting for author If the author does not respond, the issue will be closed. Otherwise, the label will be removed. labels Jul 22, 2024
@bingoxo
Copy link
Author

bingoxo commented Jul 22, 2024

1- Internal/Download/"whatever folder i already have"

2-yes

3-yes

4- custom rom/grapheneos

@github-actions github-actions bot removed the waiting for author If the author does not respond, the issue will be closed. Otherwise, the label will be removed. label Jul 22, 2024
@Aga-C Aga-C added device/software specific Issues that only happen on some devices or with some specific hardware/software and removed question Further information is requested labels Jul 22, 2024
@Switch123456789
Copy link

I noticed the same, I picked a folder in Android/media.
btw, how is it supposed to work? Does it make backups on every change?

Fairphone 5 / iodeOS / Android 14 / f-droid / v1.1

@Aga-C
Copy link
Member

Aga-C commented Aug 28, 2024

@Switch123456789 It runs once a day.

@jfsanchez
Copy link

jfsanchez commented Sep 21, 2024

Same problem here. I've tried this feature in Graphene OS and in Stock ROM in Google Pixel phones and Lineage OS 20 (Android 13). This problem seems only present in Graphene OS (Android 14). Fossify contacts version: 1.0.1 from fdroid. Same problem present in Simple Mobile Tools Contact Pro from the Play Store.

The automatic backup feature won't let you select any folder except the "Internal" because Graphene OS will enforce the file permissions (NONE). In my opinion, the intent to access a folder/file should be made first and then use that same folder given by the user.

screen

This app has no file access permissions and as such you can only see/browse the Internal Storage (in Graphene OS) and then the app will only let you select that same internal folder for permissions. Moreover, it will prevent you from using any character such as ../ or / as a workaround in the filename AND will prevent you from manually editing the folder.

I don't understand why this is implemented in two steps: First you select the folder, then you give permissions by selecting the same folder in an intent and you can't change that folder. I asume is something related to older android versions, however you could just update with the folder you are offered by the system/user in the second step.

I propose two different solutions (I am no android developer, so I could be wrong):

  1. Just use the file intent in a single step (if it has no drawbacks with older android versions that you want to support).
  2. Let the app update the folder to save the file if the users selects another one when grating permissions. As an option you could show a warning.

Affected versions so far:

  • Fairphone 5 / iodeOS / Android 14 / f-droid / v1.1 (reported by: @Switch123456789)
  • Graphene OS / Android 14 / f-droid/ v1.0.1 and Play Store (also affect SMT Contacts Pro from Play Store). All versions since it seems a problem with a common lib (more information on next comment).

@Aga-C Aga-C removed the needs triage Issue is not yet ready for PR authors to take up label Sep 21, 2024
@jfsanchez
Copy link

I've been doing some research and I think the problem is in the common library, so it may affect the rest of the products. It seems that it calls a dialog implemented in a common lib for directory/file selection. As I've said, I am no expert in android and just was looking around. I would recommend having a look at it since it could also solve other problems.

@Aga-C
Copy link
Member

Aga-C commented Oct 9, 2024

Closing in favor of general bug report for all apps: FossifyOrg/General-Discussion#211.

@Aga-C Aga-C closed this as not planned Won't fix, can't repro, duplicate, stale Oct 9, 2024
@Aga-C Aga-C added duplicate This issue or pull request already exists and removed device/software specific Issues that only happen on some devices or with some specific hardware/software bug Something is not working labels Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants