fix(PnX-SI#264): name permissions from applicationId #265
Conversation
sgrimault
changed the title
occtax/src/main/AndroidManifest.xml
Outdated
| @@ -78,8 +78,8 @@ | |||
| android:name="fr.geonature.commons.data.MainContentProvider" | |||
| android:authorities="${applicationId}.provider" | |||
| android:exported="true" | |||
| android:readPermission="@string/permission_read" | |||
| android:writePermission="@string/permission_write" /> | |||
| android:readPermission="${applicationId}.permission.READ" | |||
There was a problem hiding this comment.
${applicationId}.provider.permission.READ
occtax/src/main/AndroidManifest.xml
Outdated
| android:readPermission="@string/permission_read" | ||
| android:writePermission="@string/permission_write" /> | ||
| android:readPermission="${applicationId}.permission.READ" | ||
| android:writePermission="${applicationId}.permission.WRITE" /> |
There was a problem hiding this comment.
${applicationId}.provider.permission.WRITE
There was a problem hiding this comment.
J'ai fait la modif, curieusement ça n'empêchait pas l'appli de fonctionner 🤔 du moins en apparence
There was a problem hiding this comment.
Il faut que la définition des permissions suivent la déclaration coté ContentProvider. Après comme il est directement embarqué dans l'application, ça ne doit pas changer grand chose... mais j'ai un doute :).
Avant, le ContentProvider faisait partie d'un module à part entière embarqué au sein d'une application dédiée à la synchronisation des données entre GeoNature et l'application mobile.
L'idée était de mutualiser cette partie là si plusieurs applications mobile souhaitent utiliser les données de GeoNature sans embarquer toute la mécanique de synchronisation (d'où cette histoire de permissions pour pouvoir y accéder). Tout ce processus à été simplifié et l'application "Occtax" gère toute seule ses données.
Le ContentProvider fait donc partie du "legacy" sur l'accès aux données (il est encore utilisé sur la partie des observateurs et des taxons), mais à terme, ce serait bien de s'en débarrasser totalement au profit d'une approche plus "conventionnelle", via Room (ce qui est déjà le cas pour beaucoup de choses comme la nomenclature ou les champs additionnels).
This pull request changes the way read and write permissions are named, in order to use the
applicationIddefined for the app inocctax/build.gradlefile. This makes more convenient for developpers to create an application that can be installed and used alongside other Occtax applications.This PR works together with the PR#47 in gn_mobile_core.