0002-cards-create-related-card-in-a-popup-48534.patch

Lauréline Guérin, 26 mars 2021 09:33

Voir les différences: en ligne côte à côte

Subject: [PATCH 2/2] cards: create related card in a popup (#48534)

 tests/backoffice_pages/test_carddata.py       | 109 ++++++++++++++++++
 wcs/api.py                                    |   4 +-
 wcs/backoffice/data_management.py             |  41 ++++---
 wcs/backoffice/submission.py                  |   2 +
 wcs/carddef.py                                |   8 ++
 wcs/fields.py                                 |  17 ++-
 wcs/qommon/form.py                            |   3 +-
 wcs/qommon/static/css/dc2/admin.scss          |   9 ++
 wcs/qommon/static/js/popup_response.js        |   4 +
 wcs/qommon/static/js/qommon.admin.js          |  49 ++++++++
 .../qommon/forms/widgets/select_jsonp.html    |   7 ++
 .../wcs/backoffice/popup_response.html        |  11 ++
 wcs/views.py                                  |   1 +
 13 files changed, 243 insertions(+), 22 deletions(-)
 create mode 100644 wcs/qommon/static/js/popup_response.js
 create mode 100644 wcs/templates/wcs/backoffice/popup_response.html

         assert 'Associated User' in resp
         assert carddef.data_class().get(carddata.id).user_id == str(user.id)
         assert '/user-pending-forms' not in resp.text
     def test_carddata_add_related(pub):
         user = create_user(pub)
         BlockDef.wipe()
         block = BlockDef()
         block.name = 'child'
         block.fields = [
             fields.ItemField(
                 id='1',
                 label='Child',
                 type='item',
                 data_source={'type': 'carddef:child'},
                 display_mode='autocomplete',
             ),
+        ]
         block.store()
         CardDef.wipe()
         family = CardDef()
         family.name = 'Family'
         family.fields = [
             fields.ItemField(
                 id='1',
                 label='RL1',
                 type='item',
                 data_source={'type': 'carddef:adult'},
                 display_mode='autocomplete',
             ),
             fields.ItemField(
                 id='2',
                 label='RL2',
                 type='item',
                 data_source={'type': 'carddef:adult'},
                 display_mode='autocomplete',
             ),
             fields.BlockField(id='3', label='Children', type='block:child', max_items=42),
+        ]
         family.backoffice_submission_roles = user.roles
         family.workflow_roles = {'_editor': user.roles[0]}
         family.store()
         family.data_class().wipe()
         adult = CardDef()
         adult.name = 'Adult'
         adult.fields = [
             fields.ItemField(
                 id='1',
                 label='First name',
                 type='string',
             ),
             fields.ItemField(
                 id='2',
                 label='Last name',
                 type='string',
             ),
+        ]
         adult.backoffice_submission_roles = user.roles
         adult.workflow_roles = {'_editor': user.roles[0]}
         adult.store()
         adult.data_class().wipe()
         child = CardDef()
         child.name = 'Child'
         child.fields = [
             fields.ItemField(
                 id='1',
                 label='First name',
                 type='string',
             ),
             fields.ItemField(
                 id='2',
                 label='Last name',
                 type='string',
             ),
+        ]
         child.backoffice_submission_roles = user.roles
         child.workflow_roles = {'_editor': user.roles[0]}
         child.store()
         child.data_class().wipe()
         app = login(get_app(pub))
         resp = app.get('/backoffice/data/family/add/')
         assert 'Add another RL1' in resp
         assert 'Add another RL2' in resp
         assert 'Add another Child' in resp
         assert resp.text.count('/backoffice/data/adult/add/?_popup=1') == 2
         assert '/backoffice/data/child/add/?_popup=1' in resp
         # no autocompletion for RL1
         family.fields[0].display_mode = []
         family.store()
         resp = app.get('/backoffice/data/family/add/')
         assert 'Add another RL1' not in resp
         assert 'Add another RL2' in resp
         assert 'Add another Child' in resp
         assert resp.text.count('/backoffice/data/adult/add/?_popup=1') == 1
         assert '/backoffice/data/child/add/?_popup=1' in resp
         # user ha no creation rights on child
         child.backoffice_submission_roles = None
         child.store()
         resp = app.get('/backoffice/data/family/add/')
         assert 'Add another RL1' not in resp
         assert 'Add another RL2' in resp
         assert 'Add another Child' not in resp
         assert resp.text.count('/backoffice/data/adult/add/?_popup=1') == 1
         assert '/backoffice/data/child/add/?_popup=1' not in resp

             json_input = get_request().json
             formdata = self.formdef.data_class()()
             if not (user and self.can_user_add_cards()):
             if not (user and self.formdef.can_user_add_cards(user)):
                 raise AccessForbiddenError('cannot create card')
             if 'data' in json_input:
-...
             if get_request().get_method() != 'PUT':
                 raise MethodNotAllowedError(allowed_methods=['PUT'])
             get_request()._user = get_user_from_api_query_string()
             if not (get_request()._user and self.can_user_add_cards()):
             if not (get_request()._user and self.formdef.can_user_add_cards(get_request()._user)):
                 raise AccessForbiddenError('cannot import cards')
             afterjob = bool(get_request().form.get('async') == 'on')

     import csv
     import datetime
     import io
     import json
     from django.utils.encoding import force_text
     from quixote import get_publisher
     from quixote import get_request
     from quixote import get_response
     from quixote import redirect
     from quixote.html import TemplateIO
     from quixote.html import htmlescape
     from quixote.html import htmltext
     from wcs import fields
-...
         def add(self):
             return CardFillPage(self.formdef.url_name)
         def can_user_add_cards(self):
             if not self.formdef.backoffice_submission_roles:
                 return False
             for role in get_request().user.get_roles():
                 if role in self.formdef.backoffice_submission_roles:
                     return True
             return False
         def listing_top_actions(self):
             if not self.can_user_add_cards():
             if not self.formdef.can_user_add_cards(get_request().user):
                 return ''
             return htmltext('<span class="actions"><a href="./add/">%s</a></span>') % _('Add')
-...
         def get_formdata_sidebar_actions(self, qs=''):
             r = super(CardPage, self).get_formdata_sidebar_actions(qs=qs)
             if self.can_user_add_cards():
             if self.formdef.can_user_add_cards(get_request().user):
                 r += htmltext('<li><a rel="popup" href="import-csv">%s</a></li>') % _(
                     'Import data from a CSV file'
+                )
-...
             return output.getvalue()
         def import_csv(self):
             if not self.can_user_add_cards():
             if not self.formdef.can_user_add_cards(get_request().user):
                 raise errors.AccessForbiddenError()
             context = {'required_fields': []}
-...
             if self.formdef.user_support == 'optional':
                 self.has_user_support = True
         def submitted(self, form, *args):
             super(CardFillPage, self).submitted(form, *args)
         def redirect_after_submitted(self, form, filled):
             if get_request().form.get('_popup'):
                 popup_response_data = json.dumps(
+                    {
                         'value': str(filled.id),
                         'obj': str(filled.digest),
+                    }
+                )
                 return template.QommonTemplateResponse(
                     templates=['wcs/backoffice/popup_response.html'],
                     context={'popup_response_data': popup_response_data},
                     is_django_native=True,
+                )
             result = super().redirect_after_submitted(form, filled)
             if get_response().get_header('location').endswith('/backoffice/submission/'):
                 return redirect('..')
             return result
         def create_form(self, *args, **kwargs):
             form = super().create_form(*args, **kwargs)
             if get_request().form.get('_popup'):
                 form.add_hidden('_popup', 1)
             return form
     class CardBackOfficeStatusPage(FormBackOfficeStatusPage):

             self.set_tracking_code(filled)
             get_session().remove_magictoken(get_request().form.get('magictoken'))
             self.clean_submission_context()
             return self.redirect_after_submitted(form, filled)
         def redirect_after_submitted(self, form, filled):
             url = filled.perform_workflow()
             if url:
                 pass  # always redirect to an URL the workflow returned

             base_url = get_publisher().get_frontoffice_url()
             return '%s/api/cards/%s/' % (base_url, self.url_name)
         def can_user_add_cards(self, user):
             if not self.backoffice_submission_roles:
                 return False
             for role in user.get_roles():
                 if role in self.backoffice_submission_roles:
                     return True
             return False
         def store(self, comment=None):
             self.roles = self.backoffice_submission_roles
             return super().store(comment=comment)

             return self.display_mode
         def get_carddef(self):
             from wcs.carddef import CardDef
             try:
                 return CardDef.get_by_urlname(self.data_source['type'][8:])
             except KeyError:
                 return None
         def perform_more_widget_changes(self, form, kwargs, edit=True):
             data_source = data_sources.get_object(self.data_source)
             display_mode = self.get_display_mode(data_source)
             if display_mode == 'autocomplete' and data_source and data_source.can_jsonp():
                 self.url = kwargs['url'] = data_source.get_jsonp_url()
                 carddef = self.get_carddef()
                 if carddef and carddef.can_user_add_cards(get_request().user):
                     kwargs['add_related_url'] = carddef.get_backoffice_submission_url()
                 self.widget_class = JsonpSingleSelectWidget
                 return
-...
                 and self.data_source.get('type', '').startswith('carddef:')
             ):
                 return value
             from wcs.carddef import CardDef
             carddef = self.get_carddef()
             if not carddef:
                 return value
             try:
                 carddef = CardDef.get_by_urlname(self.data_source['type'][8:])
                 carddata = carddef.data_class().get(value_id)
             except KeyError:
                 return value

     class JsonpSingleSelectWidget(Widget):
         template_name = 'qommon/forms/widgets/select_jsonp.html'
         def __init__(self, name, value=None, url=None, **kwargs):
         def __init__(self, name, value=None, url=None, add_related_url=None, **kwargs):
             super(JsonpSingleSelectWidget, self).__init__(name, value=value, **kwargs)
             self.url = url
             self.add_related_url = add_related_url
         def add_media(self):
             get_response().add_javascript(['select2.js'])

     	display: inline-block;
+    }
     div.JsonpSingleSelectWidget a.add-related {
     	border-bottom: none;
     	&::before {
     		content: "\f067"; /* plus */
     		font-family: FontAwesome;
     		padding-right: 1ex;
+    	}
+    }
     aside#sidebar input.inline-input {
     	margin-right: 1em;
+    }

wcs/qommon/static/js/popup_response.js
	1	(function() {
	2	var initData = JSON.parse(document.getElementById('popup-response-constants').dataset.popupResponse);
	3	opener.dismissAddRelatedObjectPopup(window, initData.value, initData.obj);
	4	})();

+          }
         });
         $('[type=radio][name=display_mode]:checked').trigger('change');
         // IE doesn't accept periods or dashes in the window name, but the element IDs
         // we use to generate popup window names may contain them, therefore we map them
         // to allowed characters in a reversible way so that we can locate the correct
         // element when the popup window is dismissed.
         function id_to_windowname(text) {
             text = text.replace(/\./g, '__dot__');
             text = text.replace(/\-/g, '__dash__');
             return text;
+        }
         function windowname_to_id(text) {
             text = text.replace(/__dot__/g, '.');
             text = text.replace(/__dash__/g, '-');
             return text;
+        }
         function showAddRelatedObjectPopup(triggeringLink) {
             var name = triggeringLink.id.replace(/^add_/, '');
             name = id_to_windowname(name);
             console.log(name)
             var href = triggeringLink.href;
             console.log(href)
             var win = window.open(href, name, 'height=500,width=800,resizable=yes,scrollbars=yes');
             win.focus();
             return false;
+        }
         function dismissAddRelatedObjectPopup(win, newId, newRepr) {
             var name = windowname_to_id(win.name);
             var elem = document.getElementById(name);
             if (elem) {
                 var elemName = elem.nodeName.toUpperCase();
                 if (elemName === 'SELECT') {
                     elem.options[elem.options.length] = new Option(newRepr, newId, true, true);
+                }
                 // Trigger a change event to update related links if required.
                 $(elem).trigger('change');
+            }
             win.close();
+        }
         window.dismissAddRelatedObjectPopup = dismissAddRelatedObjectPopup;
         $('body').on('click', '.add-related', function(e) {
             e.preventDefault();
             if (this.href) {
                 showAddRelatedObjectPopup(this);
+            }
         });
     });

     {% extends "qommon/forms/widget.html" %}
     {% load i18n %}
     {% block widget-control %}
     <select id="form_{{widget.name}}" name="{{widget.name}}"
         data-select2-url="{{widget.get_select2_url}}"
-...
         data-required="{% if widget.is_required %}true{% endif %}"
         data-initial-display-value="{{widget.get_display_value|default_if_none:''}}">
     </select>
     {% if widget.add_related_url %}
     <a class="add-related" id="add_form_{{ widget.name }}"
         href="{{ widget.add_related_url }}?_popup=1"
         title="{% blocktrans with card=widget.get_title %}Add another {{ card }}{% endblocktrans %}">
     </a>
     {% endif %}
     {% endblock %}

     {% load i18n static %}<!DOCTYPE html>
     <html>
       <head><title>{% trans 'Popup closing...' %}</title></head>
       <body>
         <script type="text/javascript"
                 id="popup-response-constants"
                 src="{% static "/js/popup_response.js" %}"
                 data-popup-response="{{ popup_response_data }}">
         </script>
       </body>
     </html>

                     body.add_media()
                     if body.is_django_native:
                         self.template_names = body.templates[0]
                         context.update(body.context)
                     else:
                         body = template.render(body.templates, body.context)
                         self.template_names = None
+    -

Projet

Général

Profil

Produits Entr'ouvert » w.c.s.

0002-cards-create-related-card-in-a-popup-48534.patch