Apply suggestions from code review

Co-authored-by: ljf (zamentur) <zamentur@users.noreply.github.com>
YunoHost · Jan 31, 2025 · 0f1df76 · 0f1df76
1 parent 6eaa7ed
commit 0f1df76
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/share/config_global.toml b/share/config_global.toml
@@ -74,6 +74,7 @@ i18n = "global_settings_setting"
     [security.portal]
         [security.portal.portal_allow_edit_email]
         type = "boolean"
+        default = 0
 
         [security.portal.portal_allow_edit_email_alias]
         type = "boolean"

diff --git a/src/portal.py b/src/portal.py
@@ -231,7 +231,7 @@ def portal_update(
         else:
             # email already exist in the list we just move it on the first place
             new_mails.remove(mainemail)
-            new_mails.insert(0, mainemail)
+            new_mails = [mainemail] + new_mails[1:]
 
         new_attr_dict["mail"] = new_mails