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

The new taxonomy loading system doesn't work for flavors #2345

Open
teolemon opened this issue Mar 3, 2019 · 4 comments
Open

The new taxonomy loading system doesn't work for flavors #2345

teolemon opened this issue Mar 3, 2019 · 4 comments
Labels
Open Beauty Facts Issues relating to Open Beauty Facts Open Pet Food Facts Issues relating to Open Pet Food Facts Open Products Facts Issues relating to Open Products Facts taxonomies

Comments

@teolemon
Copy link
Member

teolemon commented Mar 3, 2019

The new taxonomy loading system doesn't work for flavors

@teolemon teolemon added bug Produces incorrect or unexpected result or behaviour. Open Beauty Facts Issues relating to Open Beauty Facts taxonomies labels Mar 3, 2019
@teolemon
Copy link
Member Author

teolemon commented Mar 3, 2019

introduced by this PR
#2122

@dobriseb
Copy link
Contributor

dobriseb commented Mar 4, 2019

In fact LoadTaxonomiesService is only used by SplashPresenter.java, and this is after a test in line 37 : if (BuildConfig.FLAVOR.equals("off")) {...
Is there differents list of files to be load in OBF, OFF, OPF and OPFF ?
eg : OFF needs to load labels.json, tags.json, allergens.json, countries.json, additives.json, categories.json and soon ingredients.json.
Does OBF needs the same ?
Does this files all exists on server side for OBF ?
Same questions for OPF and OPPF.

@dobriseb
Copy link
Contributor

dobriseb commented Mar 4, 2019

Well, for what I've found :
OFF has labels.json, tags.json, allergens.json, countries.json, additives.json, categories.json and ingredients.json
OBF has labels.json, tags.json, countries.json, additives.json, categories.json and ingredients.json (same except allergens.json)
OPF only has tags.json and categories.json
OPFF only has tags.json and categories.json
Can someone confirm this ?

dobriseb added a commit to dobriseb/openfoodfacts-androidapp that referenced this issue Mar 6, 2019
@dobriseb dobriseb mentioned this issue Mar 6, 2019
7 tasks
@dobriseb
Copy link
Contributor

dobriseb commented Mar 6, 2019

Here is my proposition of correction coupled with ingredients.json import.
#2367

@teolemon teolemon added Open Pet Food Facts Issues relating to Open Pet Food Facts Open Products Facts Issues relating to Open Products Facts labels Dec 26, 2019
@teolemon teolemon removed the bug Produces incorrect or unexpected result or behaviour. label Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Open Beauty Facts Issues relating to Open Beauty Facts Open Pet Food Facts Issues relating to Open Pet Food Facts Open Products Facts Issues relating to Open Products Facts taxonomies
Projects
Status: No status
Development

No branches or pull requests

2 participants