Merge pull request #804 from nrybowski/fix_802

[frontend/settings] Remove useless spaces in ACL
UCL-INGI · Apr 15, 2022 · 391d1f0 · 391d1f0
2 parents 15a948f + 772f0f2
commit 391d1f0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/inginious/frontend/pages/course_admin/settings.py b/inginious/frontend/pages/course_admin/settings.py
@@ -76,7 +76,7 @@ def POST_AUTH(self, courseid):  # pylint: disable=arguments-differ
                 errors.append(_('Invalid ACL value'))
             if course_content['registration_ac'] == "None":
                 course_content['registration_ac'] = None
-            course_content['registration_ac_list'] = data['registration_ac_list'].splitlines()
+            course_content['registration_ac_list'] = [line.strip() for line in data['registration_ac_list'].splitlines()]
 
             course_content['is_lti'] = 'lti' in data and data['lti'] == "true"
             course_content['lti_url'] = data.get("lti_url", "")

diff --git a/inginious/frontend/templates/course_admin/settings.html b/inginious/frontend/templates/course_admin/settings.html
@@ -291,7 +291,7 @@ <h2>{{_("Course settings")}}</h2>
     <div class="form-group row no-lti">
         <label for="registration_ac_list" class="col-sm-2 control-label">{{_("Access control list")}}</label>
         <div class="col-sm-10">
-            <textarea class="form-control" id="registration_ac_list" name="registration_ac_list" rows="10" placeholder="{{ _('Only used if access control is activated. Separate users by new lines.') }}"> {{ course.get_access_control_list() | join('\n') }} </textarea>
+            <textarea class="form-control" id="registration_ac_list" name="registration_ac_list" rows="10" placeholder="{{ _('Only used if access control is activated. Separate users by new lines.') }}">{{ course.get_access_control_list() | join('\n') }}</textarea>
         </div>
     </div>
     <div class="form-group row lti">