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

fix: Product name/brands exception #4705

Merged

Conversation

WildOrangutan
Copy link
Contributor

What

Fix NPE when product brands are null.

Screenshot

image

Fixes bug(s)

Fixes #4701

@codecov-commenter
Copy link

Codecov Report

Merging #4705 (c214f36) into develop (25068e5) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           develop   #4705   +/-   ##
=======================================
  Coverage     9.90%   9.90%           
=======================================
  Files          310     310           
  Lines        15808   15807    -1     
=======================================
  Hits          1566    1566           
+ Misses       14242   14241    -1     
Files Coverage Δ
...es/smooth_app/lib/pages/product/edit_ocr_page.dart 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@monsieurtanuki monsieurtanuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @WildOrangutan for this PR, that's good!

@monsieurtanuki monsieurtanuki merged commit 34f3364 into openfoodfacts:develop Oct 8, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

_TypeError: Null check operator used on a null value in _EditOcrPageState.build
3 participants