Merge pull request #48 from uparma/feature/drop_non_keys

keys without translations are set to None
uparma · Mar 13, 2023 · c56d1f2 · c56d1f2
2 parents 66aafc5 + c6ef83b
commit c56d1f2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/tests/test_back_and_forward_mappings.py b/tests/test_back_and_forward_mappings.py
@@ -71,6 +71,10 @@ def test_simple_back_and_forward_mapping(test_id):
                 for k, v in forward_mapping.items():
                     if isinstance(v["translated_key"], list):
                         continue
+                    if v["original_key"].endswith("<DROP_KEY>"):
+                        continue
+                    if v["translated_key"] is None:
+                        continue
                     if v["was_translated"] is True:
                         new_input[v["translated_key"]] = v["translated_value"]
                         print("Reformatted forward_mapping", new_input)

diff --git a/uparma/uparma.py b/uparma/uparma.py
@@ -233,8 +233,8 @@ def translate(self, param_dict, original_style=None, translated_style=None):
                 }
             }
 
+        In cases where e.g. only a flag is set, the translated key will be None.
         """
-        cannot_be_translated = "{0} for {1} cannot be translated into {2}"
         translated_params = UParmaDict()
 
         for original_key, original_value in param_dict.items():
@@ -321,7 +321,8 @@ def translate(self, param_dict, original_style=None, translated_style=None):
                                     if _uparma_v == _uparma_vt:
                                         translated_value = _transtyle_v
                                         was_translated = True
-
+                    if translated_key.endswith("<DROP_KEY>"):
+                        translated_key = None
                     template_dict.update(
                         {
                             "translated_key": translated_key,