Make and test warning for blanked fields with no translation.

pyxform/survey.py

@@ -191,7 +191,7 @@ def xml(self, warnings=None):
 
         return node(
             "h:html",
-            node("h:head", node("h:title", self.title), self.xml_model()),
+            node("h:head", node("h:title", self.title), self.xml_model(warnings)),
             node("h:body", *self.xml_control(), **body_kwargs),
             **nsmap
         )
@@ -562,6 +562,7 @@ def _add_empty_translations(self, warnings=None):
         When translations are not provided "-" will be used.
         This disables any of the default_language fallback functionality.
         """
+
         if warnings is None:
             warnings = []
 
@@ -571,19 +572,54 @@ def _add_empty_translations(self, warnings=None):
                 paths[path] = paths.get(path, set()).union(content.keys())
 
         for lang, translation in self._translations.items():
+            this_path_has_warning = False
             for path, content_types in paths.items():
                 if path not in self._translations[lang]:
                     self._translations[lang][path] = {}
+                    # missing question/question thingy thats like a hint
+                    question_and_column = path.split(":")
+                    missing_warning = self._generate_missing_translation_warning(
+                        lang, question_and_column[0], question_and_column[1]
+                    )
+                    warnings.append(missing_warning)
+                    # no need to warn about content types missing translations since the
+                    # whole path has a warning now.
+                    this_path_has_warning = True
+
                 for content_type in content_types:
                     if content_type not in self._translations[lang][path]:
-                        if lang == 'default':
-                            warnings.append('\tDefault language not set,' +
-                                            ' with no default translations for ' + content_type
-                                            + ' Please consider setting a `default_language` in your'
-                                            + ' settings tab to insure questions and options appear'
-                                            + ' as expected.'
-                                            )
-                        self._translations[lang][path][content_type] = "-"
+                        self._translations[lang][path][content_type] = u"-"
+                        # missing question thingy thats like media
+                        if not this_path_has_warning:
+                            # the path has a translation but the content type is missing one.
+                            missing_warning = self._generate_missing_translation_warning(
+                                lang, path.split(":")[0], content_type
+                            )
+                            warnings.append(missing_warning)
+
+    def _generate_missing_translation_warning(self, lang, question_name, column_name):
+        found_on_msg = (
+            "\n--\n Question missing translation: "
+            + question_name
+            + "\n Column missing: "
+            + column_name
+        )
+
+        if lang == "default":
+            return (
+                "\tDefault language not set," + " with missing default translations."
+                " Please consider setting a `default_language` in your"
+                + " settings tab to insure questions and options appear"
+                + " as expected."
+                + found_on_msg
+            )
+
+        return (
+            "\tMissing field translations found for "
+            + lang
+            + " field may not appear as expected."
+            + found_on_msg
+        )
 
     def _setup_media(self):
         """
@@ -734,7 +770,7 @@ def _to_pretty_xml(self, warnings=None):
         # space to text
         # TODO: check out pyxml
         # http://ronrothman.com/public/leftbraned/xml-dom-minidom-toprettyxml-and-silly-whitespace/
-        xml_with_linebreaks = self.xml().toprettyxml(indent="  ")
+        xml_with_linebreaks = self.xml(warnings).toprettyxml(indent="  ")
         text_re = re.compile(r"(>)\n\s*(\s[^<>\s].*?)\n\s*(\s</)", re.DOTALL)
         output_re = re.compile(r"\n.*(<output.*>)\n(\s\s)*")
         pretty_xml = text_re.sub(
@@ -847,8 +883,6 @@ def print_xform_to_file(
         Print the xForm to a file and optionally validate it as well by
         throwing exceptions and adding warnings to the warnings array.
         """
-        print('print xform to file')
-        print('i has warnigns!')
         if warnings is None:
             warnings = []
         if not path:

pyxform/tests_v1/test_language_warnings.py

@@ -72,6 +72,66 @@ def test_label_with_unknown_subtag_should_warn(self):
         )
         os.unlink(tmp.name)
 
+    def test_missing_translation_no_default_lang_media_has_no_language(self):
+        # form should test media tag w NO default language set.
+        survey = self.md_to_pyxform_survey(
+            """
+            | survey  |                 |         |                     |             |
+            |         | type            | name    | label::English (en) | media::image|
+            |         | integer         | nums    | How many nums?      | opt1.jpg    |
+        """
+        )
+
+        warnings = []
+        tmp = tempfile.NamedTemporaryFile(suffix=".xml", delete=False)
+        tmp.close()
+        survey.print_xform_to_file(tmp.name, warnings=warnings)
+
+        # In this survey, when 'default' s selected the label of this question will be '-'.
+        # Also, when 'English (en)` is selected, the medial will be '-'
+        self.assertTrue(len(warnings) == 2)
+        os.unlink(tmp.name)
+
+    def test_missing_translation_media(self):
+        survey = self.md_to_pyxform_survey(
+            """
+            | survey  |                 |         |                     |                    |                          |
+            |         | type            | name    | label::English (en) | label::French (fr) | media::image::French (fr)|
+            |         | integer         | nums    | How many nums?      | Combien noms?      | opt1.jpg                 |
+        """
+        )
+
+        warnings = []
+        tmp = tempfile.NamedTemporaryFile(suffix=".xml", delete=False)
+        tmp.close()
+        survey.print_xform_to_file(tmp.name, warnings=warnings)
+        self.assertTrue(len(warnings) == 1)
+        self.assertIn(
+            "Question missing translation: /pyxform_autotestname/nums", warnings[0]
+        )
+        self.assertIn("Column missing: image", warnings[0])
+        os.unlink(tmp.name)
+
+    def test_missing_translation_hint(self):
+        survey = self.md_to_pyxform_survey(
+            """
+            | survey  |                 |         |                     |                    |                   |
+            |         | type            | name    | label::English (en) | label::French (fr) | hint::French (fr) |
+            |         | integer         | nums    | How many nums?      | Combien noms?      | noms est nombres  |
+        """
+        )
+
+        warnings = []
+        tmp = tempfile.NamedTemporaryFile(suffix=".xml", delete=False)
+        tmp.close()
+        survey.print_xform_to_file(tmp.name, warnings=warnings)
+        self.assertTrue(len(warnings) == 1)
+        self.assertIn(
+            "Question missing translation: /pyxform_autotestname/nums", warnings[0]
+        )
+        self.assertIn("Column missing: hint", warnings[0])
+        os.unlink(tmp.name)
+
     def test_default_language_only_should_not_warn(self):
         survey = self.md_to_pyxform_survey(
             """
@@ -82,7 +142,7 @@ def test_default_language_only_should_not_warn(self):
             |        | list_name       | name    | label  | fake          |
             |        | opts            | opt1    | Opt1   | 1             |
             |        | opts            | opt2    | Opt2   | 1             |
-            """
+        """
         )
 
         warnings = []