Use SlotsSequenceElement to render pre-repair problems

src/allmydata/web/check_results.py

@@ -754,23 +754,20 @@ class DeepCheckAndRepairResultsRendererElement(Element, ResultsBase, ReloadMixin
             return tag
         return ""
 
-    # TODO: use SlotsSequenceElement
     @renderer
     def pre_repair_problems(self, req, tag):
         all_objects = self.monitor.get_status().get_all_results()
+        problems = []
+
         for path in sorted(all_objects.keys()):
             r = all_objects[path]
             assert ICheckAndRepairResults.providedBy(r)
             cr = r.get_pre_repair_results()
             if not cr.is_healthy():
-                yield path, cr
+                problem = self._join_pathstring(path), ": ", self._html(cr.get_summary())
+                problems.append(problem)
 
-    @renderer
-    def problem(self, req, tag):
-        # TODO: figure this out
-        # path, cr = data
-        return tag(self._join_pathstring(path), ": ",
-                   self._html(cr.get_summary()))
+        return SlotsSequenceElement(tag, problems)
 
     @renderer
     def post_repair_problems_p(self, req, tag):

src/allmydata/web/deep-check-and-repair-results.xhtml

@@ -35,8 +35,8 @@
 <h2>Files/Directories That Had Problems:</h2>
 
 <ul t:render="pre_repair_problems">
-  <li t:pattern="item" t:render="problem"/>
-  <li t:pattern="empty">None</li>
+  <li t:render="item"/>
+  <li t:render="empty">None</li>
 </ul>
 </div>