Merge branch 'user-menu-fixes' into 'master'

User Menu Template Fixes

See merge request allianceauth/allianceauth!1728

allianceauth/framework/api/user.py

@@ -10,39 +10,24 @@ from allianceauth.authentication.models import CharacterOwnership
 from allianceauth.eveonline.models import EveCharacter
 
 
-def get_all_characters_from_user(user: User, main_first: bool = False) -> list:
+def get_all_characters_from_user(user: User) -> list:
     """
-    Get all characters from a user
-    This function retrieves all characters associated with a given user, optionally ordering them
-    with the main character first.
-    If the user is None, an empty list is returned.
+    Get all characters from a user or an empty list
+    when no characters are found for the user or the user is None
 
-    :param user: The user whose characters are to be retrieved
-    :type user: User
-    :param main_first: If True, the main character will be listed first
-    :type main_first: bool
-    :return: A list of EveCharacter objects associated with the user
-    :rtype: list[EveCharacter]
+    :param user:
+    :type user:
+    :return:
+    :rtype:
     """
 
     if user is None:
         return []
 
     try:
-        if main_first:
-            characters = [
-                char.character
-                for char in CharacterOwnership.objects.filter(user=user).order_by(
-                    "-character__userprofile", "character__character_name"
-                )
-            ]
-        else:
-            characters = [
-                char.character
-                for char in CharacterOwnership.objects.filter(user=user).order_by(
-                    "character__character_name"
-                )
-            ]
+        characters = [
+            char.character for char in CharacterOwnership.objects.filter(user=user)
+        ]
     except AttributeError:
         return []
 

allianceauth/services/modules/discord/discord_client/client.py

@@ -675,7 +675,7 @@ class DiscordClient:
         )
         logger.debug('%s: response:\n%s', uid, r.text)
         if not r.ok:
-            logger.error(
+            logger.warning(
                 '%s: Discord API returned error code %d and this response: %s',
                 uid,
                 r.status_code,

docs/development/custom/framework/api.md

@@ -43,7 +43,7 @@ if the `EveCharacter` has no user.
 
 ### get_all_characters_from_user
 
-This is to get all character objects (`EveCharacter`) of a user (alphabetically sorted).
+This is to get all character objects (`EveCharacter`) of a user.
 
 Given we have a `User` object called `my_user` and we want to get all characters:
 
@@ -51,16 +51,12 @@ Given we have a `User` object called `my_user` and we want to get all characters
 # Alliance Auth
 from allianceauth.framework.api.user import get_all_characters_from_user
 
-characters = get_all_characters_from_user(user=my_user, main_first=False)
+characters = get_all_characters_from_user(user=my_user)
 ```
 
 Now, `characters` is a `list` containing all `EveCharacter` objects of the user.
 If the user is `None`, an empty `list` will be returned.
 
-The second parameter `main_first` is optional and defaults to `False`.
-If set to `True`, the function will return the main character as the first
-item in the list of characters.
-
 ### get_main_character_from_user
 
 This is to get the main character object (`EveCharacter`) of a user.