Bugfix: Save sorted document groups

History.md

@@ -4,6 +4,7 @@
 
 - Next to the status-dropdown for motions and amendments, there is now a link to a reference page, explaining the uses of these different statuses.
 - Bugfix: If a voting was set to be shown in a motion view, it was not shown on the dedicated voting page anymore.
+- Bugfix: Sorting document groups on the separate document page did not work properly.
 - Bugfix: Some tabs opened in parallel could interfere with logging in using a second factor.
 - Bugfix: When logging in using a second factor, one would always end up in the site home page, not the page where one started the login.
 - Bugfix: Some empty paragraphs could lead to problems with creating amendments if only one paragraph could be changed.

controllers/PagesController.php

@@ -534,10 +534,11 @@ public function actionDocuments(): ResponseInterface
             foreach ($this->consultation->fileGroups as $fileGroup) {
                 if (isset($idPosition[$fileGroup->id])) {
                     $fileGroup->position = $idPosition[$fileGroup->id];
-
-                    return new RedirectResponse(UrlHelper::createUrl('/pages/documents'));
+                    $fileGroup->save();
                 }
             }
+
+            return new RedirectResponse(UrlHelper::createUrl('/pages/documents'));
         }
 
         return new HtmlResponse($this->render('documents'));

views/consultation/index_layout_agenda.php

@@ -76,6 +76,7 @@
             $prefix = ($isResolutionList ? Yii::t('con', 'resolutions') . ': ' : '');
             echo '<h2 class="green">' . $prefix . Html::encode($agendaItem->title) . '</h2>';
             echo '<ul class="motionList motionListStd motionListBelowAgenda agenda' . $agendaItem->id . '">';
+            echo '<!-- Test -->';
             $itemImotions = MotionSorter::getSortedIMotionsFlat($consultation, $itemImotions);
             foreach ($itemImotions as $imotion) {
                 if (is_a($imotion, Motion::class)) {