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.

Sign up for GitHub

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

Jump to bottom

fix: Don't pass country code if it's null #324

Merged
merged 11 commits into from Dec 16, 2021
Merged

fix: Don't pass country code if it's null #324

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
2c5999c
Remove obsolete test
Dec 14, 2021
2c3c3dd
Merge branch 'openfoodfacts:master' into mapkp
jasmeet0817 Dec 15, 2021
e5248c6
//
Dec 15, 2021
396bff3
Merge branch 'mapkp' of https://github.com/jasmeet0817/openfoodfacts-…
Dec 15, 2021
3b7370d
//
Dec 15, 2021
8ab427e
Renaming and Fixing
Dec 15, 2021
b8fe85d
Renaming and Fixing
Dec 15, 2021
16c384f
Format
Dec 15, 2021
ba77a1d
Merge branch 'master' of https://github.com/openfoodfacts/openfoodfac…
Dec 16, 2021
423b669
Don't pass country code if it's null
Dec 16, 2021
08ce04e
Format
Dec 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
14 changes: 9 additions & 5 deletions lib/openfoodfacts.dart
View file
Open in desktop
Expand Up @@ -871,14 +871,18 @@ class OpenFoodAPIClient {
) async {
const String KNOWLEDGE_PANELS_FIELD = 'knowledge_panels';

var queryParameters = <String, String>{
'fields': KNOWLEDGE_PANELS_FIELD,
'lc': configuration.language!.code,
};
String? cc = configuration.computeCountryCode();
if (cc != null) {
queryParameters.putIfAbsent('cc', () => cc);
}
var uri = UriHelper.getUri(
path: 'api/v2/product/${configuration.barcode}/',
queryType: queryType,
queryParameters: <String, String>{
'fields': KNOWLEDGE_PANELS_FIELD,
'lc': configuration.language!.code,
'cc': configuration.computeCountryCode()!,
},
queryParameters: queryParameters,
);

try {
Expand Down