From 666ed6e0f499e238a32ea68b0c38073c4a76ab3e Mon Sep 17 00:00:00 2001
From: Elias Showk <eshowk@entrouvert.com>
Date: Thu, 3 May 2018 10:50:13 +0200
Subject: [PATCH] add CheckboxesWidget.get_options and checkboxes.html (#23413)

---
 tests/test_widgets.py                         | 26 +++++++++++++++++
 wcs/qommon/form.py                            | 29 +++++--------------
 .../qommon/forms/widgets/checkboxes.html      | 18 ++++++++++++
 3 files changed, 51 insertions(+), 22 deletions(-)
 create mode 100644 wcs/qommon/templates/qommon/forms/widgets/checkboxes.html

diff --git a/tests/test_widgets.py b/tests/test_widgets.py
index b58e6a32..68b487c0 100644
--- a/tests/test_widgets.py
+++ b/tests/test_widgets.py
@@ -2,6 +2,7 @@ import datetime
 import sys
 import shutil
 import copy
+import collections
 
 from quixote import cleanup
 from quixote.http_request import parse_query
@@ -411,6 +412,31 @@ def test_checkboxes_widget():
     mock_form_submission(req, widget, {'test$elementpeach': ['yes'], 'test$elementpear': ['yes']})
     assert widget.parse() == ['pear', 'peach']
 
+def test_checkboxes_widget_submission_with_options():
+    test_dt = datetime.datetime.now()
+    widget = CheckboxesWidget('test',
+        options_with_attributes=[
+            ('apple', 'Apple', 'apple', {'datetime': test_dt}),
+            ('pear', 'Pear', 'pear', {'datetime': test_dt}),
+            ('peach', 'Peach', 'peach', {'datetime': test_dt})])
+    mock_form_submission(req, widget, {'test$elementpeach': ['yes'], 'test$elementpear': ['yes']})
+    assert widget.parse() == ['pear', 'peach']
+
+def test_checkboxes_get_widgets():
+    test_dt = datetime.datetime.now()
+    widget = CheckboxesWidget('test',
+        options_with_attributes=[
+            ('apple', 'Apple', 'apple', {'datetime': test_dt}),
+            ('pear', 'Pear', 'pear', {'datetime': test_dt}),
+            ('peach', 'Peach', 'peach', {'datetime': test_dt})])
+    assert isinstance(widget.get_widgets(), collections.Iterator)
+    iter_options = list(widget.get_widgets())
+    assert len(iter_options) == 3
+    for widget in iter_options:
+        assert isinstance(widget, CheckboxWidget)
+        assert isinstance(widget.extra_options, dict)
+        assert widget.extra_options['datetime'] == test_dt
+
 
 def test_composite_widget():
     widget = CompositeWidget('compotest')
diff --git a/wcs/qommon/form.py b/wcs/qommon/form.py
index 49df467e..947914d0 100644
--- a/wcs/qommon/form.py
+++ b/wcs/qommon/form.py
@@ -1109,28 +1109,13 @@ class CheckboxesWidget(CompositeWidget):
     def has_error(self, request=None):
         return Widget.has_error(self, request=request)
 
-    def render_content(self):
-        r = TemplateIO(html=True)
-        if self.inline:
-            r += htmltext('<ul class="inline">')
-        else:
-            r += htmltext('<ul>')
-        for widget in self.get_widgets():
-            if widget.attrs and 'disabled' in widget.attrs:
-                r += htmltext('<li class="disabled"><label>')
-            else:
-                r += htmltext('<li><label>')
-            if self.readonly:
-                widget.attrs['disabled'] = 'disabled'
-                if widget.value:
-                    r += htmltext('<input type="hidden" name="%s" value="yes" >') % widget.name
-                widget.name = widget.name + 'xx'
-            r += widget.render_content()
-            r += htmltext('<span>%s</span>') % widget.title
-            r += htmltext('</label>')
-            r += htmltext('</li>')
-        r += htmltext('</ul>')
-        return r.getvalue()
+    def get_widgets(self):
+        '''yield every widget with extra information
+        '''
+        for idx, widget in enumerate(super(CheckboxesWidget, self).get_widgets()):
+            widget.extra_options = self.options_with_attributes[idx][-1] if self.options_with_attributes else None
+            yield widget
+
 
 class ValidatedStringWidget(StringWidget):
     '''StringWidget which checks the value entered is correct according to a regex'''
diff --git a/wcs/qommon/templates/qommon/forms/widgets/checkboxes.html b/wcs/qommon/templates/qommon/forms/widgets/checkboxes.html
new file mode 100644
index 00000000..b13bbd81
--- /dev/null
+++ b/wcs/qommon/templates/qommon/forms/widgets/checkboxes.html
@@ -0,0 +1,18 @@
+{% extends "qommon/forms/widget.html" %}
+{% block widget-control %}
+<ul {% if widget.inline %}class="inline"{% endif %}>
+  {% for w in widget.get_widgets %}
+    <li {% if "disabled" in w %}class="disabled"{% endif %}>
+      <label>
+        {% if widget.readonly or "disabled" in w %}
+        <input type="hidden" name="{{w.name}}" value="yes" >
+        {% endif %}
+        <input type="checkbox" value="yes" name="{{w.name}}"
+           {% if widget.readonly or "disabled" in w %}disabled{% endif %}
+           {% if w.value %}checked="checked"{% endif %}
+           {% for key, value in w.attrs.items %}{{key}}="{{value}}"{% endfor %}/>
+        <span>{{ w.title }}</span></label>
+    </li>
+  {% endfor %}
+</ul>
+{% endblock %}
\ No newline at end of file
-- 
2.17.0