[FIX] score-documents: handle document titles with newlines

orangecontrib/text/widgets/owscoredocuments.py

@@ -283,6 +283,13 @@ def __init__(self, parent=None):
         super().__init__(parent=parent)
         self._extremes = {}
 
+    @staticmethod
+    def simplify(s):
+        """Remove tab and newline characters from the string"""
+        for ch in "\n\t\r":
+            s = s.replace(ch, " ")
+        return s
+
     def data(self, index, role=Qt.DisplayRole):
         if index.column() > 0 and role == gui.BarRatioRole and index.isValid():
             # for all except first columns return ratio for distribution bar
@@ -291,6 +298,9 @@ def data(self, index, role=Qt.DisplayRole):
             return (value - vmin) / ((vmax - vmin) or 1)
         if role in (gui.BarRatioRole, Qt.DisplayRole):
             dat = super().data(index, Qt.EditRole)
+            if role == Qt.DisplayRole and index.column() == 0:
+                # in document title column remove newline characters from titles
+                dat = self.simplify(dat)
             return dat
         if role == Qt.BackgroundColorRole and index.column() == 0:
             return TableModel.ColorForRole[TableModel.Meta]

orangecontrib/text/widgets/tests/test_owscoredocuments.py

@@ -433,6 +433,15 @@ def test_output_unique(self):
         output = self.get_output(self.widget.Outputs.selected_documents)
         self.assertTrue("Word count (1)" in output.domain)
 
+    def test_titles_no_newline(self):
+        corpus = Corpus.from_file("andersen")
+        corpus.metas[0, 0] = corpus.metas[0, 0] + "\ntest"
+        corpus.set_title_variable("Title")
+        self.send_signal(self.widget.Inputs.corpus, corpus)
+        self.assertEqual(
+            "The Little Match-Seller test", self.widget.view.model().index(0, 0).data()
+        )
+
 
 if __name__ == "__main__":
     unittest.main()