Skip to content
This repository has been archived by the owner on Sep 6, 2019. It is now read-only.

Batch fetch fetches system apps #1048

Closed
vanowm opened this issue Jan 1, 2014 · 6 comments
Closed

Batch fetch fetches system apps #1048

vanowm opened this issue Jan 1, 2014 · 6 comments
Labels

Comments

@vanowm
Copy link

vanowm commented Jan 1, 2014

I think it's quiet important bug when you see list of apps filtered the way you like and want fetch permissions for these apps, only to realize that permissions for all other apps, including system getting destroyed in the process and you can't stop it...

@M66B
Copy link
Owner

M66B commented Jan 1, 2014

Duplicate of #1031

Restrictions are only cleared for applications found on the server. Restrictions for system applications can only be fetch individually.

@M66B M66B closed this as completed Jan 1, 2014
@vanowm
Copy link
Author

vanowm commented Jan 1, 2014

Well then it's a bug and not a feature request with a low priority, because system apps got their permissions changed when I did fetch for non-system apps (or so I thought...)

@M66B
Copy link
Owner

M66B commented Jan 1, 2014

System applications are not batch fetched, I have checked:
https://github.com/M66B/XPrivacy/blob/master/src/biz/bokhorst/xprivacy/ActivityShare.java#L528

@vanowm
Copy link
Author

vanowm commented Jan 1, 2014

I cleared all permissions, made sure both user and system apps are cleared, switched to user apps view and selected fetch permissions from menu. The result:
screenshot_2014-01-01-08-41-39
Either I'm doing something wrong or it's a bug?

@vanowm
Copy link
Author

vanowm commented Jan 1, 2014

Then I looked at the line you showed and it's clearly a bug because the statement is true if app not system OR data not empty. It should be "AND"

@M66B
Copy link
Owner

M66B commented Jan 1, 2014

It should be an OR: not system application OR individual fetch.
Nevertheless I will test this later again.

@M66B M66B reopened this Jan 1, 2014
@M66B M66B closed this as completed in 39f9f86 Jan 1, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants