Occtax-mobile pouvoir spécifier une url absolue pour l'apk

[joelclems]

Version
2.13

Description du bug
Si l'on défini une url absolue pour la config mobile en base, les settings renvoyés sont un dictionnaire vide

Comportement attendu
Pouvoir définir soit une url aboslue soit une url relative

Comment reproduire
dans geonature admin mettre Chemin relatif de l'APK à null et donner une valeur à URL externe de l'APK

Piste

Dans l'api le chemin du fichier de config est défini par rapport au fichier apk relatif (il doit se trouver dans le meme répertoire).
Il faudrait pouvoir mettre ce chemin en dur (/mobiles/occtax/settings.json) et le rendre indépendant de l'apk

GeoNature/backend/geonature/core/gn_commons/routes.py

Lines 205 to 222 in a978d6a

	for app in q.all():
	app_dict = app.as_dict(exclude=["relative_path_apk"])
	app_dict["settings"] = {}
	#  if local
	if app.relative_path_apk:
	relative_apk_path = Path("mobile", app.relative_path_apk)
	app_dict["url_apk"] = url_for("media", filename=str(relative_apk_path), _external=True)
	relative_settings_path = relative_apk_path.parent / "settings.json"
	app_dict["url_settings"] = url_for(
	"media", filename=relative_settings_path, _external=True
	)
	settings_file = Path(current_app.config["MEDIA_FOLDER"]) / relative_settings_path
	with settings_file.open() as f:
	app_dict["settings"] = json.load(f)
	mobile_apps.append(app_dict)
	if len(mobile_apps) == 1:
	return mobile_apps[0]
	return mobile_apps

[camillemonchicourt]

C'était géré initialement.
A voir pourquoi et quand ça a sauté.

[camillemonchicourt]

OK je comprends, si l'APK est ailleurs, donc sur un chemin absolu, il ne trouve plus le fichier settings.json.

Testé sur la démo, l'API renvoie alors :

{"app_code": "OCCTAX", "id_mobile_app": 3, "package": "fr.geonature.occtax2", "settings": {}, "url_apk": "https://github.com/PnX-SI/gn_mobile_occtax/releases/download/2.6.1/occtax-2.6.1-generic-release.apk", "url_settings": null, "version_code": "3187"}

[camillemonchicourt]

Corrigé dans la 2.13.2