From a8583c8e6901b8014e8ab21aa2c8a098377344f9 Mon Sep 17 00:00:00 2001
From: Michael Genson <71845777+michael-genson@users.noreply.github.com>
Date: Sat, 9 Dec 2023 17:12:07 +0000
Subject: [PATCH 1/6] added backend translation support for plurals

---
 mealie/pkgs/i18n/json_provider.py             | 26 ++++++++++++++++---
 .../pkgs/i18n/test_locale_provider.py         | 23 ++++++++++++++++
 2 files changed, 46 insertions(+), 3 deletions(-)

diff --git a/mealie/pkgs/i18n/json_provider.py b/mealie/pkgs/i18n/json_provider.py
index 34cc7710c45..5fbeb10c902 100644
--- a/mealie/pkgs/i18n/json_provider.py
+++ b/mealie/pkgs/i18n/json_provider.py
@@ -1,6 +1,7 @@
 import json
 from dataclasses import dataclass
 from pathlib import Path
+from typing import cast
 
 
 @dataclass(slots=True)
@@ -13,6 +14,22 @@ def __init__(self, path: Path | dict):
         else:
             self.translations = path
 
+    def _parse_plurals(self, value: str, count: float):
+        # based off of: https://kazupon.github.io/vue-i18n/guide/pluralization.html
+
+        values = [v.strip() for v in value.split("|")]
+        if len(values) == 1:
+            return value
+        elif len(values) == 2:
+            return values[0] if count == 1 else values[1]
+        elif len(values) == 3:
+            if count == 0:
+                return values[0]
+            else:
+                return values[1] if count == 1 else values[2]
+        else:
+            return values[0]
+
     def t(self, key: str, default=None, **kwargs) -> str:
         keys = key.split(".")
 
@@ -30,9 +47,12 @@ def t(self, key: str, default=None, **kwargs) -> str:
 
             if i == last:
                 for key, value in kwargs.items():
-                    if not value:
+                    translation_value = cast(str, translation_value)
+                    if value is None:
                         value = ""
-                    translation_value = translation_value.replace("{" + key + "}", value)
-                return translation_value
+                    if key == "count":
+                        translation_value = self._parse_plurals(translation_value, float(value))
+                    translation_value = translation_value.replace("{" + key + "}", str(value))  # type: ignore
+                return translation_value  # type: ignore
 
         return default or key
diff --git a/tests/unit_tests/pkgs/i18n/test_locale_provider.py b/tests/unit_tests/pkgs/i18n/test_locale_provider.py
index 73100fb739a..f4bf3a55052 100644
--- a/tests/unit_tests/pkgs/i18n/test_locale_provider.py
+++ b/tests/unit_tests/pkgs/i18n/test_locale_provider.py
@@ -9,6 +9,29 @@ def test_json_provider():
     assert provider.t("test2", "DEFAULT") == "DEFAULT"
 
 
+def test_json_provider_plural():
+    provider = JsonProvider({"test": "test | tests"})
+    assert provider.t("test", count=0) == "tests"
+    assert provider.t("test", count=0.5) == "tests"
+    assert provider.t("test", count=1) == "test"
+    assert provider.t("test", count=1.5) == "tests"
+    assert provider.t("test", count=2) == "tests"
+
+    provider = JsonProvider({"test": "test 0 | test | tests"})
+    assert provider.t("test", count=0) == "test 0"
+    assert provider.t("test", count=0.5) == "tests"
+    assert provider.t("test", count=1) == "test"
+    assert provider.t("test", count=1.5) == "tests"
+    assert provider.t("test", count=2) == "tests"
+
+    provider = JsonProvider({"test": "zero tests | one test | {count} tests"})
+    assert provider.t("test", count=0) == "zero tests"
+    assert provider.t("test", count=0.5) == "0.5 tests"
+    assert provider.t("test", count=1) == "one test"
+    assert provider.t("test", count=1.5) == "1.5 tests"
+    assert provider.t("test", count=2) == "2 tests"
+
+
 def test_json_provider_nested_keys():
     nested_dict = {
         "root": {

From 2cfc63b3026e69b4b654180f915b6f02f4809b61 Mon Sep 17 00:00:00 2001
From: Michael Genson <71845777+michael-genson@users.noreply.github.com>
Date: Sat, 9 Dec 2023 17:19:06 +0000
Subject: [PATCH 2/6] added timedelta translations

---
 mealie/lang/messages/en-US.json | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/mealie/lang/messages/en-US.json b/mealie/lang/messages/en-US.json
index a4990159d5d..16bee3cc774 100644
--- a/mealie/lang/messages/en-US.json
+++ b/mealie/lang/messages/en-US.json
@@ -31,5 +31,14 @@
         "generic-updated-with-url": "{name} has been updated, {url}",
         "generic-duplicated": "{name} has been duplicated",
         "generic-deleted": "{name} has been deleted"
+    },
+    "datetime": {
+        "year": "year|years",
+        "day": "day|days",
+        "hour": "hour|hours",
+        "minute": "minute|minutes",
+        "second": "second|seconds",
+        "millisecond": "millisecond|milliseconds",
+        "microsecond": "microsecond|microseconds"
     }
 }

From 408df286fd0d8dbf883731ae0a3950d86066be60 Mon Sep 17 00:00:00 2001
From: Michael Genson <71845777+michael-genson@users.noreply.github.com>
Date: Sat, 9 Dec 2023 17:19:19 +0000
Subject: [PATCH 3/6] added translator to scraper

---
 mealie/routes/groups/controller_migrations.py |  1 +
 mealie/routes/recipe/recipe_crud_routes.py    |  6 +--
 mealie/services/migrations/_migration_base.py | 13 +++++-
 mealie/services/scraper/cleaner.py            | 40 ++++++++++---------
 .../services/scraper/recipe_bulk_scraper.py   |  8 +++-
 mealie/services/scraper/recipe_scraper.py     |  6 ++-
 mealie/services/scraper/scraper.py            |  5 ++-
 mealie/services/scraper/scraper_strategies.py |  4 +-
 8 files changed, 52 insertions(+), 31 deletions(-)

diff --git a/mealie/routes/groups/controller_migrations.py b/mealie/routes/groups/controller_migrations.py
index 3beba1083f4..3ec1ca26eb5 100644
--- a/mealie/routes/groups/controller_migrations.py
+++ b/mealie/routes/groups/controller_migrations.py
@@ -44,6 +44,7 @@ def start_data_migration(
             "user_id": self.user.id,
             "group_id": self.group_id,
             "add_migration_tag": add_migration_tag,
+            "translator": self.translator,
         }
 
         table: dict[SupportedMigrations, type[BaseMigrator]] = {
diff --git a/mealie/routes/recipe/recipe_crud_routes.py b/mealie/routes/recipe/recipe_crud_routes.py
index bace402ebd2..d479fe2f4ca 100644
--- a/mealie/routes/recipe/recipe_crud_routes.py
+++ b/mealie/routes/recipe/recipe_crud_routes.py
@@ -167,7 +167,7 @@ def handle_exceptions(self, ex: Exception) -> None:
     async def parse_recipe_url(self, req: ScrapeRecipe):
         """Takes in a URL and attempts to scrape data and load it into the database"""
         try:
-            recipe, extras = await create_from_url(req.url)
+            recipe, extras = await create_from_url(req.url, self.translator)
         except ForceTimeoutException as e:
             raise HTTPException(
                 status_code=408, detail=ErrorResponse.respond(message="Recipe Scraping Timed Out")
@@ -196,7 +196,7 @@ async def parse_recipe_url(self, req: ScrapeRecipe):
     @router.post("/create-url/bulk", status_code=202)
     def parse_recipe_url_bulk(self, bulk: CreateRecipeByUrlBulk, bg_tasks: BackgroundTasks):
         """Takes in a URL and attempts to scrape data and load it into the database"""
-        bulk_scraper = RecipeBulkScraperService(self.service, self.repos, self.group)
+        bulk_scraper = RecipeBulkScraperService(self.service, self.repos, self.group, self.translator)
         report_id = bulk_scraper.get_report_id()
         bg_tasks.add_task(bulk_scraper.scrape, bulk)
 
@@ -211,7 +211,7 @@ def parse_recipe_url_bulk(self, bulk: CreateRecipeByUrlBulk, bg_tasks: Backgroun
     async def test_parse_recipe_url(self, url: ScrapeRecipeTest):
         # Debugger should produce the same result as the scraper sees before cleaning
         try:
-            if scraped_data := await RecipeScraperPackage(url.url).scrape_url():
+            if scraped_data := await RecipeScraperPackage(url.url, self.translator).scrape_url():
                 return scraped_data.schema.data
         except ForceTimeoutException as e:
             raise HTTPException(
diff --git a/mealie/services/migrations/_migration_base.py b/mealie/services/migrations/_migration_base.py
index 1268a38fb7c..6cfa6fa3012 100644
--- a/mealie/services/migrations/_migration_base.py
+++ b/mealie/services/migrations/_migration_base.py
@@ -6,6 +6,7 @@
 
 from mealie.core import root_logger
 from mealie.core.exceptions import UnexpectedNone
+from mealie.lang.providers import Translator
 from mealie.repos.all_repositories import AllRepositories
 from mealie.schema.recipe import Recipe
 from mealie.schema.recipe.recipe_settings import RecipeSettings
@@ -35,12 +36,20 @@ class BaseMigrator(BaseService):
     helpers: DatabaseMigrationHelpers
 
     def __init__(
-        self, archive: Path, db: AllRepositories, session, user_id: UUID4, group_id: UUID, add_migration_tag: bool
+        self,
+        archive: Path,
+        db: AllRepositories,
+        session,
+        user_id: UUID4,
+        group_id: UUID,
+        add_migration_tag: bool,
+        translator: Translator,
     ):
         self.archive = archive
         self.db = db
         self.session = session
         self.add_migration_tag = add_migration_tag
+        self.translator = translator
 
         user = db.users.get_one(user_id)
         if not user:
@@ -225,6 +234,6 @@ def clean_recipe_dictionary(self, recipe_dict: dict) -> Recipe:
         with contextlib.suppress(KeyError):
             del recipe_dict["id"]
 
-        recipe_dict = cleaner.clean(recipe_dict, url=recipe_dict.get("org_url", None))
+        recipe_dict = cleaner.clean(recipe_dict, self.translator, url=recipe_dict.get("org_url", None))
 
         return Recipe(**recipe_dict)
diff --git a/mealie/services/scraper/cleaner.py b/mealie/services/scraper/cleaner.py
index f4bfc0ad708..294b5c99b42 100644
--- a/mealie/services/scraper/cleaner.py
+++ b/mealie/services/scraper/cleaner.py
@@ -10,6 +10,7 @@
 from slugify import slugify
 
 from mealie.core.root_logger import get_logger
+from mealie.lang.providers import Translator
 
 logger = get_logger("recipe-scraper")
 
@@ -32,7 +33,7 @@
 """ Matches multiple new lines and removes erroneous white space """
 
 
-def clean(recipe_data: dict, url=None) -> dict:
+def clean(recipe_data: dict, translator: Translator, url=None) -> dict:
     """Main entrypoint to clean a recipe extracted from the web
     and format the data into an accectable format for the database
 
@@ -45,9 +46,9 @@ def clean(recipe_data: dict, url=None) -> dict:
     recipe_data["description"] = clean_string(recipe_data.get("description", ""))
 
     # Times
-    recipe_data["prepTime"] = clean_time(recipe_data.get("prepTime"))
-    recipe_data["performTime"] = clean_time(recipe_data.get("performTime"))
-    recipe_data["totalTime"] = clean_time(recipe_data.get("totalTime"))
+    recipe_data["prepTime"] = clean_time(recipe_data.get("prepTime"), translator)
+    recipe_data["performTime"] = clean_time(recipe_data.get("performTime"), translator)
+    recipe_data["totalTime"] = clean_time(recipe_data.get("totalTime"), translator)
     recipe_data["recipeCategory"] = clean_categories(recipe_data.get("recipeCategory", []))
     recipe_data["recipeYield"] = clean_yield(recipe_data.get("recipeYield"))
     recipe_data["recipeIngredient"] = clean_ingredients(recipe_data.get("recipeIngredient", []))
@@ -332,7 +333,7 @@ def clean_yield(yld: str | list[str] | None) -> str:
     return yld
 
 
-def clean_time(time_entry: str | timedelta | None) -> None | str:
+def clean_time(time_entry: str | timedelta | None, translator: Translator) -> None | str:
     """_summary_
 
     Supported Structures:
@@ -358,11 +359,11 @@ def clean_time(time_entry: str | timedelta | None) -> None | str:
 
             try:
                 time_delta_instructionsect = parse_duration(time_entry)
-                return pretty_print_timedelta(time_delta_instructionsect)
+                return pretty_print_timedelta(time_delta_instructionsect, translator)
             except ValueError:
                 return str(time_entry)
         case timedelta():
-            return pretty_print_timedelta(time_entry)
+            return pretty_print_timedelta(time_entry, translator)
         case {"minValue": str(value)}:
             return clean_time(value)
         case [str(), *_]:
@@ -371,7 +372,7 @@ def clean_time(time_entry: str | timedelta | None) -> None | str:
             # TODO: Not sure what to do here
             return str(time_entry)
         case _:
-            logger.warning("[SCRAPER] Unexpected type or structure for time_entrys")
+            logger.warning("[SCRAPER] Unexpected type or structure for time_entries")
             return None
 
 
@@ -405,25 +406,25 @@ def parse_duration(iso_duration: str) -> timedelta:
     return timedelta(**times)
 
 
-def pretty_print_timedelta(t: timedelta, max_components=None, max_decimal_places=2):
+def pretty_print_timedelta(t: timedelta, translator: Translator, max_components=None, max_decimal_places=2):
     """
     Print a pretty string for a timedelta.
     For example datetime.timedelta(days=2, seconds=17280) will be printed as '2 days 4 Hours 48 Minutes'.
     Setting max_components to e.g. 1 will change this to '2.2 days', where the number of decimal
     points can also be set.
     """
-    time_scale_names_dict = {
-        timedelta(days=365): "year",
-        timedelta(days=1): "day",
-        timedelta(hours=1): "Hour",
-        timedelta(minutes=1): "Minute",
-        timedelta(seconds=1): "Second",
-        timedelta(microseconds=1000): "millisecond",
-        timedelta(microseconds=1): "microsecond",
+    time_scale_translation_keys_dict = {
+        timedelta(days=365): "datetime.year",
+        timedelta(days=1): "datetime.day",
+        timedelta(hours=1): "datetime.hour",
+        timedelta(minutes=1): "datetime.minute",
+        timedelta(seconds=1): "datetime.second",
+        timedelta(microseconds=1000): "datetime.millisecond",
+        timedelta(microseconds=1): "datetime.microsecond",
     }
     count = 0
     out_list = []
-    for scale, scale_name in time_scale_names_dict.items():
+    for scale, scale_translation_key in time_scale_translation_keys_dict.items():
         if t >= scale:
             count += 1
             n = t / scale if count == max_components else int(t / scale)
@@ -433,7 +434,8 @@ def pretty_print_timedelta(t: timedelta, max_components=None, max_decimal_places
             if n_txt[-2:] == ".0":
                 n_txt = n_txt[:-2]
 
-            out_list.append(f"{n_txt} {scale_name}{'s' if n > 1 else ''}")
+            scale_value = translator.t(scale_translation_key, count=n)
+            out_list.append(f"{n_txt} {scale_value}")
 
     if out_list == []:
         return "none"
diff --git a/mealie/services/scraper/recipe_bulk_scraper.py b/mealie/services/scraper/recipe_bulk_scraper.py
index ed701ecc388..bfd71494115 100644
--- a/mealie/services/scraper/recipe_bulk_scraper.py
+++ b/mealie/services/scraper/recipe_bulk_scraper.py
@@ -2,6 +2,7 @@
 
 from pydantic import UUID4
 
+from mealie.lang.providers import Translator
 from mealie.repos.repository_factory import AllRepositories
 from mealie.schema.recipe.recipe import CreateRecipeByUrlBulk, Recipe
 from mealie.schema.reports.reports import (
@@ -20,11 +21,14 @@
 class RecipeBulkScraperService(BaseService):
     report_entries: list[ReportEntryCreate]
 
-    def __init__(self, service: RecipeService, repos: AllRepositories, group: GroupInDB) -> None:
+    def __init__(
+        self, service: RecipeService, repos: AllRepositories, group: GroupInDB, translator: Translator
+    ) -> None:
         self.service = service
         self.repos = repos
         self.group = group
         self.report_entries = []
+        self.translator = translator
 
         super().__init__()
 
@@ -81,7 +85,7 @@ async def scrape(self, urls: CreateRecipeByUrlBulk) -> None:
         async def _do(url: str) -> Recipe | None:
             async with sem:
                 try:
-                    recipe, _ = await create_from_url(url)
+                    recipe, _ = await create_from_url(url, self.translator)
                     return recipe
                 except Exception as e:
                     self.service.logger.error(f"failed to scrape url during bulk url import {url}")
diff --git a/mealie/services/scraper/recipe_scraper.py b/mealie/services/scraper/recipe_scraper.py
index a9faeeccb6f..90a81c63602 100644
--- a/mealie/services/scraper/recipe_scraper.py
+++ b/mealie/services/scraper/recipe_scraper.py
@@ -1,3 +1,4 @@
+from mealie.lang.providers import Translator
 from mealie.schema.recipe.recipe import Recipe
 from mealie.services.scraper.scraped_extras import ScrapedExtras
 
@@ -14,11 +15,12 @@ class RecipeScraper:
     # List of recipe scrapers. Note that order matters
     scrapers: list[type[ABCScraperStrategy]]
 
-    def __init__(self, scrapers: list[type[ABCScraperStrategy]] | None = None) -> None:
+    def __init__(self, translator: Translator, scrapers: list[type[ABCScraperStrategy]] | None = None) -> None:
         if scrapers is None:
             scrapers = DEFAULT_SCRAPER_STRATEGIES
 
         self.scrapers = scrapers
+        self.translator = translator
 
     async def scrape(self, url: str) -> tuple[Recipe, ScrapedExtras] | tuple[None, None]:
         """
@@ -26,7 +28,7 @@ async def scrape(self, url: str) -> tuple[Recipe, ScrapedExtras] | tuple[None, N
         """
 
         for scraper_type in self.scrapers:
-            scraper = scraper_type(url)
+            scraper = scraper_type(url, self.translator)
             result = await scraper.parse()
 
             if result is not None:
diff --git a/mealie/services/scraper/scraper.py b/mealie/services/scraper/scraper.py
index eb8d415b88d..bcd240edd30 100644
--- a/mealie/services/scraper/scraper.py
+++ b/mealie/services/scraper/scraper.py
@@ -5,6 +5,7 @@
 from slugify import slugify
 
 from mealie.core.root_logger import get_logger
+from mealie.lang.providers import Translator
 from mealie.pkgs import cache
 from mealie.schema.recipe import Recipe
 from mealie.services.recipe.recipe_data_service import RecipeDataService
@@ -19,7 +20,7 @@ class ParserErrors(str, Enum):
     CONNECTION_ERROR = "CONNECTION_ERROR"
 
 
-async def create_from_url(url: str) -> tuple[Recipe, ScrapedExtras | None]:
+async def create_from_url(url: str, translator: Translator) -> tuple[Recipe, ScrapedExtras | None]:
     """Main entry point for generating a recipe from a URL. Pass in a URL and
     a Recipe object will be returned if successful.
 
@@ -29,7 +30,7 @@ async def create_from_url(url: str) -> tuple[Recipe, ScrapedExtras | None]:
     Returns:
         Recipe: Recipe Object
     """
-    scraper = RecipeScraper()
+    scraper = RecipeScraper(translator)
     new_recipe, extras = await scraper.scrape(url)
 
     if not new_recipe:
diff --git a/mealie/services/scraper/scraper_strategies.py b/mealie/services/scraper/scraper_strategies.py
index fd51498023a..a995ba38920 100644
--- a/mealie/services/scraper/scraper_strategies.py
+++ b/mealie/services/scraper/scraper_strategies.py
@@ -11,6 +11,7 @@
 from w3lib.html import get_base_url
 
 from mealie.core.root_logger import get_logger
+from mealie.lang.providers import Translator
 from mealie.schema.recipe.recipe import Recipe, RecipeStep
 from mealie.services.scraper.scraped_extras import ScrapedExtras
 
@@ -77,9 +78,10 @@ class ABCScraperStrategy(ABC):
 
     url: str
 
-    def __init__(self, url: str) -> None:
+    def __init__(self, url: str, translator: Translator) -> None:
         self.logger = get_logger()
         self.url = url
+        self.translator = translator
 
     @abstractmethod
     async def get_html(self, url: str) -> str:

From 3a30b3216ee69c34bfa960ed9e160dc76675cee9 Mon Sep 17 00:00:00 2001
From: Michael Genson <71845777+michael-genson@users.noreply.github.com>
Date: Sat, 9 Dec 2023 17:19:27 +0000
Subject: [PATCH 4/6] fixed tests

---
 .../scraper_tests/test_cleaner.py             |  7 +++++--
 .../scraper_tests/test_cleaner_parts.py       | 21 +++++++++++--------
 tests/unit_tests/test_recipe_parser.py        |  6 ++++--
 3 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/tests/unit_tests/services_tests/scraper_tests/test_cleaner.py b/tests/unit_tests/services_tests/scraper_tests/test_cleaner.py
index 395b943016d..7e7bf047fa6 100644
--- a/tests/unit_tests/services_tests/scraper_tests/test_cleaner.py
+++ b/tests/unit_tests/services_tests/scraper_tests/test_cleaner.py
@@ -4,6 +4,7 @@
 
 import pytest
 
+from mealie.lang.providers import local_provider
 from mealie.services.scraper import cleaner
 from mealie.services.scraper.scraper_strategies import RecipeScraperOpenGraph
 from tests import data as test_data
@@ -37,15 +38,17 @@
 
 @pytest.mark.parametrize("json_file,num_steps", test_cleaner_data)
 def test_cleaner_clean(json_file: Path, num_steps):
-    recipe_data = cleaner.clean(json.loads(json_file.read_text()))
+    translator = local_provider()
+    recipe_data = cleaner.clean(json.loads(json_file.read_text()), translator)
     assert len(recipe_data["recipeInstructions"]) == num_steps
 
 
 def test_html_with_recipe_data():
     path = test_data.html_healthy_pasta_bake_60759
     url = "https://www.bbc.co.uk/food/recipes/healthy_pasta_bake_60759"
+    translator = local_provider()
 
-    open_graph_strategy = RecipeScraperOpenGraph(url)
+    open_graph_strategy = RecipeScraperOpenGraph(url, translator)
 
     recipe_data = open_graph_strategy.get_recipe_fields(path.read_text())
 
diff --git a/tests/unit_tests/services_tests/scraper_tests/test_cleaner_parts.py b/tests/unit_tests/services_tests/scraper_tests/test_cleaner_parts.py
index 67cbf9b3486..5fedb073b7c 100644
--- a/tests/unit_tests/services_tests/scraper_tests/test_cleaner_parts.py
+++ b/tests/unit_tests/services_tests/scraper_tests/test_cleaner_parts.py
@@ -4,6 +4,7 @@
 
 import pytest
 
+from mealie.lang.providers import local_provider
 from mealie.services.scraper import cleaner
 
 
@@ -324,32 +325,32 @@ def test_cleaner_clean_yield_amount(case: CleanerCase):
     CleanerCase(
         test_id="timedelta",
         input=timedelta(minutes=30),
-        expected="30 Minutes",
+        expected="30 minutes",
     ),
     CleanerCase(
         test_id="timedelta string (1)",
         input="PT2H30M",
-        expected="2 Hours 30 Minutes",
+        expected="2 hours 30 minutes",
     ),
     CleanerCase(
         test_id="timedelta string (2)",
         input="PT30M",
-        expected="30 Minutes",
+        expected="30 minutes",
     ),
     CleanerCase(
         test_id="timedelta string (3)",
         input="PT2H",
-        expected="2 Hours",
+        expected="2 hours",
     ),
     CleanerCase(
         test_id="timedelta string (4)",
         input="P1DT1H1M1S",
-        expected="1 day 1 Hour 1 Minute 1 Second",
+        expected="1 day 1 hour 1 minute 1 second",
     ),
     CleanerCase(
         test_id="timedelta string (4)",
         input="P1DT1H1M1.53S",
-        expected="1 day 1 Hour 1 Minute 1 Second",
+        expected="1 day 1 hour 1 minute 1 second",
     ),
     CleanerCase(
         test_id="timedelta string (5) invalid",
@@ -366,7 +367,8 @@ def test_cleaner_clean_yield_amount(case: CleanerCase):
 
 @pytest.mark.parametrize("case", time_test_cases, ids=(x.test_id for x in time_test_cases))
 def test_cleaner_clean_time(case: CleanerCase):
-    result = cleaner.clean_time(case.input)
+    translator = local_provider()
+    result = cleaner.clean_time(case.input, translator)
     assert case.expected == result
 
 
@@ -536,10 +538,11 @@ def test_cleaner_clean_nutrition(case: CleanerCase):
 @pytest.mark.parametrize(
     "t,max_components,max_decimal_places,expected",
     [
-        (timedelta(days=2, seconds=17280), None, 2, "2 days 4 Hours 48 Minutes"),
+        (timedelta(days=2, seconds=17280), None, 2, "2 days 4 hours 48 minutes"),
         (timedelta(days=2, seconds=17280), 1, 2, "2.2 days"),
         (timedelta(days=365), None, 2, "1 year"),
     ],
 )
 def test_pretty_print_timedelta(t, max_components, max_decimal_places, expected):
-    assert cleaner.pretty_print_timedelta(t, max_components, max_decimal_places) == expected
+    translator = local_provider()
+    assert cleaner.pretty_print_timedelta(t, translator, max_components, max_decimal_places) == expected
diff --git a/tests/unit_tests/test_recipe_parser.py b/tests/unit_tests/test_recipe_parser.py
index b583543a55f..e7515d73d98 100644
--- a/tests/unit_tests/test_recipe_parser.py
+++ b/tests/unit_tests/test_recipe_parser.py
@@ -1,5 +1,6 @@
 import pytest
 
+from mealie.lang.providers import local_provider
 from mealie.services.scraper import scraper
 from tests.utils.recipe_data import RecipeSiteTestCase, get_recipe_test_cases
 
@@ -18,9 +19,10 @@
 @pytest.mark.parametrize("recipe_test_data", test_cases)
 @pytest.mark.asyncio
 async def test_recipe_parser(recipe_test_data: RecipeSiteTestCase):
-    recipe, _ = await scraper.create_from_url(recipe_test_data.url)
+    translator = local_provider()
+    recipe, _ = await scraper.create_from_url(recipe_test_data.url, translator)
 
     assert recipe.slug == recipe_test_data.expected_slug
-    assert len(recipe.recipe_instructions) == recipe_test_data.num_steps
+    assert len(recipe.recipe_instructions or []) == recipe_test_data.num_steps
     assert len(recipe.recipe_ingredient) == recipe_test_data.num_ingredients
     assert recipe.org_url == recipe_test_data.url

From 437f5c454f484f5f5fb097c204c17f0616b9e5d3 Mon Sep 17 00:00:00 2001
From: Michael Genson <71845777+michael-genson@users.noreply.github.com>
Date: Sat, 9 Dec 2023 22:04:21 +0000
Subject: [PATCH 5/6] fixed missing translator

---
 mealie/services/scraper/scraper_strategies.py | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/mealie/services/scraper/scraper_strategies.py b/mealie/services/scraper/scraper_strategies.py
index a995ba38920..2a9046b5843 100644
--- a/mealie/services/scraper/scraper_strategies.py
+++ b/mealie/services/scraper/scraper_strategies.py
@@ -105,7 +105,9 @@ async def get_html(self, url: str) -> str:
         return await safe_scrape_html(url)
 
     def clean_scraper(self, scraped_data: SchemaScraperFactory.SchemaScraper, url: str) -> tuple[Recipe, ScrapedExtras]:
-        def try_get_default(func_call: Callable | None, get_attr: str, default: Any, clean_func=None):
+        def try_get_default(
+            func_call: Callable | None, get_attr: str, default: Any, clean_func=None, **clean_func_kwargs
+        ):
             value = default
 
             if func_call:
@@ -121,7 +123,7 @@ def try_get_default(func_call: Callable | None, get_attr: str, default: Any, cle
                     self.logger.error(f"Error parsing recipe attribute '{get_attr}'")
 
             if clean_func:
-                value = clean_func(value)
+                value = clean_func(value, **clean_func_kwargs)
 
             return value
 
@@ -141,9 +143,9 @@ def get_instructions() -> list[RecipeStep]:
             except TypeError:
                 return []
 
-        cook_time = try_get_default(None, "performTime", None, cleaner.clean_time) or try_get_default(
-            None, "cookTime", None, cleaner.clean_time
-        )
+        cook_time = try_get_default(
+            None, "performTime", None, cleaner.clean_time, translator=self.translator
+        ) or try_get_default(None, "cookTime", None, cleaner.clean_time, translator=self.translator)
 
         extras = ScrapedExtras()
 
@@ -160,8 +162,8 @@ def get_instructions() -> list[RecipeStep]:
                 scraped_data.ingredients, "recipeIngredient", [""], cleaner.clean_ingredients
             ),
             recipe_instructions=get_instructions(),
-            total_time=try_get_default(None, "totalTime", None, cleaner.clean_time),
-            prep_time=try_get_default(None, "prepTime", None, cleaner.clean_time),
+            total_time=try_get_default(None, "totalTime", None, cleaner.clean_time, translator=self.translator),
+            prep_time=try_get_default(None, "prepTime", None, cleaner.clean_time, translator=self.translator),
             perform_time=cook_time,
             org_url=url,
         )

From 94342081f97f248794042974ea302de2c2654553 Mon Sep 17 00:00:00 2001
From: Michael Genson <71845777+michael-genson@users.noreply.github.com>
Date: Thu, 8 Feb 2024 14:43:13 +0000
Subject: [PATCH 6/6] I don't know why I changed this

---
 mealie/services/scraper/cleaner.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mealie/services/scraper/cleaner.py b/mealie/services/scraper/cleaner.py
index ba816e0365d..ee64bd3e394 100644
--- a/mealie/services/scraper/cleaner.py
+++ b/mealie/services/scraper/cleaner.py
@@ -375,7 +375,7 @@ def clean_time(time_entry: str | timedelta | None, translator: Translator) -> No
             # TODO: Not sure what to do here
             return str(time_entry)
         case _:
-            logger.warning("[SCRAPER] Unexpected type or structure for time_entries")
+            logger.warning("[SCRAPER] Unexpected type or structure for variable time_entry")
             return None