From 0b04b7478f054132044fe34bcf083287447fa2d6 Mon Sep 17 00:00:00 2001
From: FinnStutzenstein <finn.stutzenstein@hotmail.de>
Date: Fri, 2 Feb 2018 12:29:18 +0100
Subject: [PATCH] Fixed email translations, motion sorting, undefined in DOCX,
 reduced motion search

---
 .travis.yml                                   |  2 +-
 openslides/motions/static/js/motions/docx.js  |  4 ++--
 openslides/motions/static/js/motions/site.js  | 19 +------------------
 .../templates/motions/motion-detail.html      |  2 +-
 openslides/users/models.py                    |  6 +++---
 openslides/users/static/js/users/site.js      |  7 ++++++-
 openslides/users/views.py                     | 10 +++++++++-
 tests/integration/users/test_viewset.py       |  7 ++++++-
 8 files changed, 29 insertions(+), 28 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 5a2df9325..8740418cb 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -30,6 +30,6 @@ script:
   - coverage report --fail-under=44
 
   - DJANGO_SETTINGS_MODULE='tests.settings' coverage run ./manage.py test tests.integration
-  - coverage report --fail-under=74
+  - coverage report --fail-under=73
 
   - DJANGO_SETTINGS_MODULE='tests.settings' ./manage.py test tests.old
diff --git a/openslides/motions/static/js/motions/docx.js b/openslides/motions/static/js/motions/docx.js
index 0e6f92d15..172d3b72b 100644
--- a/openslides/motions/static/js/motions/docx.js
+++ b/openslides/motions/static/js/motions/docx.js
@@ -80,7 +80,7 @@ angular.module('OpenSlidesApp.motions.docx', ['OpenSlidesApp.core.docx'])
         var getMotionShortData = function (motions) {
             return _.map(motions, function (motion) {
                 return {
-                    identifier: motion.identifier,
+                    identifier: motion.identifier || '',
                     title: motion.getTitle(),
                 };
             });
@@ -113,7 +113,7 @@ angular.module('OpenSlidesApp.motions.docx', ['OpenSlidesApp.core.docx'])
                     sequential_enabled: sequential_enabled,
                     // Actual data
                     id: motion.id,
-                    identifier: motion.identifier,
+                    identifier: motion.identifier || '',
                     title: motion.getTitle(),
                     submitters: params.include.submitters ?  _.map(motion.submitters, function (submitter) {
                                     return submitter.get_full_name();
diff --git a/openslides/motions/static/js/motions/site.js b/openslides/motions/static/js/motions/site.js
index 547999e97..086d0c46f 100644
--- a/openslides/motions/static/js/motions/site.js
+++ b/openslides/motions/static/js/motions/site.js
@@ -782,21 +782,6 @@ angular.module('OpenSlidesApp.motions.site', [
                             },
                             hideExpression: "model.format !== 'csv'",
                         },
-                        {
-                            key: 'include',
-                            type: 'checkbox-buttons',
-                            templateOptions: {
-                                label: gettextCatalog.getString('Meta information'),
-                                options: getMetaInformationOptions({
-                                    state: true,
-                                    votingResult: true,
-                                    motionBlock: true,
-                                    origin: true,
-                                    recommendation: true,
-                                }),
-                            },
-                            hideExpression: "model.format !== 'docx'",
-                        },
                     ]);
                     if (commentsAvailable) {
                         fields.push({
@@ -855,6 +840,7 @@ angular.module('OpenSlidesApp.motions.site', [
             }
             if ($scope.params.format === 'docx') {
                 $scope.params.include.state = false;
+                $scope.params.include.submitter = true;
                 $scope.params.include.motionBlock = false;
                 $scope.params.include.origin = false;
                 $scope.params.include.recommendation = false;
@@ -1127,12 +1113,9 @@ angular.module('OpenSlidesApp.motions.site', [
         $scope.filter.propertyList = ['identifier', 'origin'];
         $scope.filter.propertyFunctionList = [
             function (motion) {return motion.getTitle();},
-            function (motion) {return motion.getText();},
-            function (motion) {return motion.getReason();},
             function (motion) {return motion.category ? motion.category.name : '';},
             function (motion) {return motion.motionBlock ? motion.motionBlock.name : '';},
             function (motion) {return motion.recommendation ? motion.getRecommendationName() : '';},
-            function (motion) {return _.filter(motion.comments).join(' ');},
         ];
         $scope.filter.propertyDict = {
             'submitters': function (submitter) {
diff --git a/openslides/motions/static/templates/motions/motion-detail.html b/openslides/motions/static/templates/motions/motion-detail.html
index d79590187..9cf34db85 100644
--- a/openslides/motions/static/templates/motions/motion-detail.html
+++ b/openslides/motions/static/templates/motions/motion-detail.html
@@ -239,7 +239,7 @@
                     <translate>Motion</translate>
                   </span>
                   <ul uib-dropdown-menu class="dropdown-menu" aria-labelledby="motion-dropdown">
-                    <li ng-repeat="m in motions">
+                    <li ng-repeat="m in motions | orderBy: identifier">
                       <a href ng-click="addMotionToRecommendationField(m)">
                         {{ m.identifier ? m.identifier + ':' : '' }} {{ m.getTitle() }}
                       </a>
diff --git a/openslides/users/models.py b/openslides/users/models.py
index 0fdba625b..a28f33719 100644
--- a/openslides/users/models.py
+++ b/openslides/users/models.py
@@ -240,7 +240,7 @@ class User(RESTModelMixin, PermissionsMixin, AbstractBaseUser):
         """
         raise RuntimeError('Do not use user.has_perm() but use openslides.utils.auth.has_perm')
 
-    def send_invitation_email(self, connection, skip_autoupdate=False):
+    def send_invitation_email(self, connection, subject, message, skip_autoupdate=False):
         """
         Sends an invitation email to the users. Returns True on success, False on failiure.
         May raise an ValidationError, if something went wrong.
@@ -260,10 +260,10 @@ class User(RESTModelMixin, PermissionsMixin, AbstractBaseUser):
             'url': config['users_pdf_url'],
             'username': self.username,
             'password': self.default_password})
-        message = config['users_email_body'].format(**message_format)
+        message = message.format(**message_format)
 
         subject_format = format_dict({'event_name': config['general_event_name']})
-        subject = config['users_email_subject'].format(**subject_format)
+        subject = subject.format(**subject_format)
 
         # Create an email and send it.
         email = mail.EmailMessage(subject, message, config['users_email_sender'], [self.email])
diff --git a/openslides/users/static/js/users/site.js b/openslides/users/static/js/users/site.js
index 30d502469..7ddc59acb 100644
--- a/openslides/users/static/js/users/site.js
+++ b/openslides/users/static/js/users/site.js
@@ -771,8 +771,13 @@ angular.module('OpenSlidesApp.users.site', [
                     return user.id;
                 })
                 .value();
+            var subject = gettextCatalog.getString(Config.get('users_email_subject').value);
+            var message = gettextCatalog.getString(Config.get('users_email_body').value);
+
             $http.post('/rest/users/user/mass_invite_email/', {
                 user_ids: user_ids,
+                subject: subject,
+                message: message,
             }).then(function (success) {
                 var numEmails = success.data.count;
                 var noEmailIds = success.data.no_email_ids;
@@ -1833,7 +1838,7 @@ angular.module('OpenSlidesApp.users.site', [
         gettext('Your login for {event_name}');
         gettext('You can use {event_name} as a placeholder.');
         gettext('Email body');
-        gettext('Dear {name},\n\nthis is your OpenSlides login for the event "{event_name}":\n    {url}\n    username: {username}\n    password: {password}\n\nThis email was generated automatically.');
+        gettext('Dear {name},\n\nthis is your OpenSlides login for the event "{event_name}":\n    {url}\n\n    username: {username}\n    password: {password}\n\nThis email was generated automatically.');
         gettext('Use these placeholders: {name}, {event_name}, {url}, {username}, {password}. The url referrs to the system url.');
     }
 ]);
diff --git a/openslides/users/views.py b/openslides/users/views.py
index aecfcd609..c3473d702 100644
--- a/openslides/users/views.py
+++ b/openslides/users/views.py
@@ -180,6 +180,14 @@ class UserViewSet(ModelViewSet):
         for user_id in user_ids:
             if not isinstance(user_id, int):
                 raise ValidationError({'detail': 'User_id has to be an int.'})
+        # Get subject and body from the response. Do not use the config values
+        # because they might not be translated.
+        subject = request.data.get('subject')
+        message = request.data.get('message')
+        if not isinstance(subject, str):
+            raise ValidationError({'detail': 'Subject has to be a string.'})
+        if not isinstance(message, str):
+            raise ValidationError({'detail': 'Message has to be a string.'})
         users = User.objects.filter(pk__in=user_ids)
 
         # Sending Emails. Keep track, which users gets an email.
@@ -199,7 +207,7 @@ class UserViewSet(ModelViewSet):
         try:
             for user in users:
                 if user.email:
-                    if user.send_invitation_email(connection, skip_autoupdate=True):
+                    if user.send_invitation_email(connection, subject, message, skip_autoupdate=True):
                         success_users.append(user)
                 else:
                     user_pks_without_email.append(user.pk)
diff --git a/tests/integration/users/test_viewset.py b/tests/integration/users/test_viewset.py
index 9f932457e..44011b49b 100644
--- a/tests/integration/users/test_viewset.py
+++ b/tests/integration/users/test_viewset.py
@@ -351,9 +351,14 @@ class UserSendIntivationEmail(TestCase):
         self.admin.save()
 
     def test_email_sending(self):
+        data = {
+            'user_ids': [self.admin.pk],
+            'subject': config['users_email_subject'],
+            'message': config['users_email_body']
+        }
         response = self.client.post(
             reverse('user-mass-invite-email'),
-            {'user_ids': [self.admin.pk]},
+            data,
             format='json')
         self.assertEqual(response.status_code, 200)
         self.assertEqual(response.data['count'], 1)