From 2872cd437c6c5cbdbd6cdb01326d2c3e62f9dbd7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Norman=20J=C3=A4ckel?= <openslides@normanjaeckel.de>
Date: Thu, 26 Jan 2017 21:03:10 +0100
Subject: [PATCH] Fixed bug in motion set_identifier method. Fixed #2911.

---
 openslides/motions/models.py | 28 +++++++++++++++++++---------
 openslides/motions/views.py  |  2 +-
 2 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/openslides/motions/models.py b/openslides/motions/models.py
index 3a5d6a87e..3700ec01b 100644
--- a/openslides/motions/models.py
+++ b/openslides/motions/models.py
@@ -304,16 +304,16 @@ class Motion(RESTModelMixin, models.Model):
         Sets the motion identifier automaticly according to the config value if
         it is not set yet.
         """
-        # The identifier is already set or should be set manually
+        # The identifier is already set or should be set manually.
         if config['motions_identifier'] == 'manually' or self.identifier:
             # Do not set an identifier.
             return
 
-        # The motion is an amendment
+        # The motion is an amendment.
         elif self.is_amendment():
             motions = self.parent.amendments.all()
 
-        # The motions should be counted per category
+        # The motions should be counted per category.
         elif config['motions_identifier'] == 'per_category':
             motions = Motion.objects.filter(category=self.category)
 
@@ -321,25 +321,35 @@ class Motion(RESTModelMixin, models.Model):
         else:
             motions = Motion.objects.all()
 
+        # Get biggest number.
         number = motions.aggregate(Max('identifier_number'))['identifier_number__max'] or 0
+
+        # If MOTION_IDENTIFIER_WITHOUT_BLANKS is set, don't use blanks when building identifier.
+        without_blank = hasattr(settings, 'MOTION_IDENTIFIER_WITHOUT_BLANKS') and settings.MOTION_IDENTIFIER_WITHOUT_BLANKS
+
+        # Build prefix.
         if self.is_amendment():
             parent_identifier = self.parent.identifier or ''
-            prefix = '%s %s ' % (parent_identifier, config['motions_amendments_prefix'])
+            if without_blank:
+                prefix = '%s%s' % (parent_identifier, config['motions_amendments_prefix'])
+            else:
+                prefix = '%s %s ' % (parent_identifier, config['motions_amendments_prefix'])
         elif self.category is None or not self.category.prefix:
             prefix = ''
         else:
-            prefix = '%s ' % self.category.prefix
+            if without_blank:
+                prefix = '%s' % self.category.prefix
+            else:
+                prefix = '%s ' % self.category.prefix
 
+        # Calculate new identifier.
         number += 1
         identifier = '%s%s' % (prefix, self.extend_identifier_number(number))
         while Motion.objects.filter(identifier=identifier).exists():
             number += 1
             identifier = '%s%s' % (prefix, self.extend_identifier_number(number))
 
-        # remove all whitespaces from identifier if MOTION_IDENTIFIER_WITHOUT_BLANKS is set
-        if hasattr(settings, 'MOTION_IDENTIFIER_WITHOUT_BLANKS') and settings.MOTION_IDENTIFIER_WITHOUT_BLANKS:
-            identifier = identifier.replace(' ', '')
-
+        # Set identifier and identifier_number.
         self.identifier = identifier
         self.identifier_number = number
 
diff --git a/openslides/motions/views.py b/openslides/motions/views.py
index 9c94efe95..3e91505c3 100644
--- a/openslides/motions/views.py
+++ b/openslides/motions/views.py
@@ -274,7 +274,7 @@ class MotionViewSet(ModelViewSet):
             motion.reset_state()
 
         # Save motion.
-        motion.save(update_fields=['state', 'identifier'])
+        motion.save(update_fields=['state', 'identifier', 'identifier_number'])
         message = _('The state of the motion was set to %s.') % motion.state.name
 
         # Write the log message and initiate response.