Fix TypeError in clean_nutrition function

[zackbcom]

Fixes #1033
Using this recipe: https://www.marthastewart.com/336792/breaded-chicken-breasts

Changes

• If sodiumContent is None, it blows up when trying to check if m or g are in the string.
  • Fixed by checking that it's a string before checking if it has substrings.
• Flake8 and black incompatability
  • fixed with setting igntore to extend-ignore in .flake8 file Source
  • With ignore used it was enabling all the default ignored rules, include the ones below:
    • W503 line break before binary operator
    • W504 line break after binary operator
• CodeFactor issues are not from my changes.

Testing Artifacts:

Before code fix:

Error:

mealie-api       |   File "/app/mealie/services/scraper/cleaner.py", line 96, in clean_nutrition
mealie-api       |     if "sodiumContent" in nutrition and "m" not in nutrition["sodiumContent"] and "g" in nutrition["sodiumContent"]:
mealie-api       | TypeError: argument of type 'NoneType' is not iterable

After code fix:

Adds recipe.

Before flake8 fix

  if (
      "sodiumContent" in nutrition
      and type(nutrition["sodiumContent"]) == str
      and "m" not in nutrition["sodiumContent"]
      and "g" in nutrition["sodiumContent"]
  ):

mealie/services/scraper/cleaner.py:98:9: W503 line break before binary operator
mealie/services/scraper/cleaner.py:99:9: W503 line break before binary operator
mealie/services/scraper/cleaner.py:100:9: W503 line break before binary operator

Changed to this to fix W503

    if (
        "sodiumContent" in nutrition and
        type(nutrition["sodiumContent"]) == str and
        "m" not in nutrition["sodiumContent"] and 
        "g" in nutrition["sodiumContent"]
    ):

mealie/services/scraper/cleaner.py:102:38: W504 line break after binary operator
mealie/services/scraper/cleaner.py:103:49: W504 line break after binary operator
mealie/services/scraper/cleaner.py:104:47: W504 line break after binary operator

Then recieved 504.

After flake8 fix:

❯ make lint
poetry run black .
All done! ✨ 🍰 ✨
154 files left unchanged.
poetry run black . --check
All done! ✨ 🍰 ✨
154 files would be left unchanged.
poetry run flake8 mealie tests

[netlify]

✔️ Deploy Preview for mealie-docs ready!

🔨 Explore the source changes: d7793da

🔍 Inspect the deploy log: https://app.netlify.com/sites/mealie-docs/deploys/622d7dbd6f8e370008177efe

😎 Browse the preview: https://deploy-preview-1043--mealie-docs.netlify.app

[hay-kot]

Thanks for the detailed comments looks good to me!

I captured these changes for mealie-next in

• capture dev PRs #1043 #1055