bot/src/bot_data/abc/user_repository_abc.py

@@ -31,10 +31,6 @@ class UserRepositoryABC(ABC):
     def get_users_by_server_id(self, server_id: int) -> List[User]:
         pass
 
-    @abstractmethod
-    def get_users_with_activity_by_server_id(self, server_id: int) -> List[User]:
-        pass
-
     @abstractmethod
     def get_user_by_discord_id_and_server_id(self, discord_id: int, server_id: int) -> User:
         pass

bot/src/bot_data/model/user.py

@@ -32,8 +32,6 @@ class User(TableABC):
         self._birthday = birthday
         self._server = server
 
-        self._activity = 0
-
         TableABC.__init__(self)
         self._created_at = created_at if created_at is not None else self._created_at
         self._modified_at = modified_at if modified_at is not None else self._modified_at
@@ -122,14 +120,6 @@ class User(TableABC):
             return 0
         return float(result)
 
-    @property
-    def activity(self) -> int:
-        return self._activity
-
-    @activity.setter
-    def activity(self, value: int):
-        self._activity = value
-
     @property
     @ServiceProviderABC.inject
     def level(self, services: ServiceProviderABC) -> Level:

bot/src/bot_data/service/user_repository_service.py

@@ -1,8 +1,6 @@
-import datetime
 from typing import Optional
 
 from cpl_core.database.context import DatabaseContextABC
-from cpl_core.type import T, R
 from cpl_query.extension import List
 
 from bot_core.logging.database_logger import DatabaseLogger
@@ -40,9 +38,7 @@ class UserRepositoryService(UserRepositoryABC):
 
     def get_users(self) -> List[User]:
         self._logger.trace(__name__, f"Send SQL command: {User.get_select_all_string()}")
-        return List(
+        return List(User, [self._from_result(user) for user in self._context.select(User.get_select_all_string())])
-            User, [self._from_result(user) for user in self._context.select(User.get_select_all_string())]
-        ).for_each(lambda x: self._set_user_activity(x))
 
     def get_user_by_id(self, id: int) -> User:
         self._logger.trace(__name__, f"Send SQL command: {User.get_select_by_id_string(id)}")
@@ -81,16 +77,6 @@ class UserRepositoryService(UserRepositoryABC):
             [self._from_result(user) for user in self._context.select(User.get_select_by_server_id_string(server_id))],
         )
 
-    def get_users_with_activity_by_server_id(self, server_id: int) -> List[User]:
-        self._logger.trace(
-            __name__,
-            f"Send SQL command: {User.get_select_by_server_id_string(server_id)}",
-        )
-        return List(
-            User,
-            [self._from_result(user) for user in self._context.select(User.get_select_by_server_id_string(server_id))],
-        ).for_each(lambda x: self._set_user_activity(x))
-
     def get_user_by_discord_id_and_server_id(self, discord_id: int, server_id: int) -> User:
         self._logger.trace(
             __name__,
@@ -123,91 +109,3 @@ class UserRepositoryService(UserRepositoryABC):
     def delete_user(self, user: User):
         self._logger.trace(__name__, f"Send SQL command: {user.delete_string}")
         self._context.cursor.execute(user.delete_string)
-
-    def _set_user_activity(self, user):
-        days = (datetime.date.today() - (datetime.date.today() - datetime.timedelta(days=7))).days
-        query = f"""
-            SELECT (
-                (
-                    SELECT Count(UserGotAchievements.CreatedAt)
-                    FROM UserGotAchievements
-                    INNER JOIN Achievements ON UserGotAchievements.AchievementId = Achievements.Id
-                    INNER JOIN Users ON UserGotAchievements.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserGotAchievements.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                ) +
-                (
-                    SELECT SUM(
-                        UserMessageCountPerHour.XPCount / (
-                            SELECT XpPerMessage
-                            FROM CFG_Server
-                            WHERE ServerId = {user.server.id}
-                        )
-                    )
-                    FROM UserMessageCountPerHour
-                    INNER JOIN Users ON UserMessageCountPerHour.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserMessageCountPerHour.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                ) +
-                (
-                    SELECT Count(UserJoinedVoiceChannel.CreatedAt)
-                    FROM UserJoinedVoiceChannel
-                    INNER JOIN Users ON UserJoinedVoiceChannel.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserJoinedVoiceChannel.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                ) +
-                (
-                    SELECT IFNULL(ROUND(SUM(TIME_TO_SEC(
-                        TIMEDIFF(UserJoinedVoiceChannel.LeavedOn, UserJoinedVoiceChannel.JoinedOn)
-                    ) / 3600), 2), 0)
-                    FROM UserJoinedVoiceChannel
-                    INNER JOIN Users ON UserJoinedVoiceChannel.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserJoinedVoiceChannel.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                ) +
-                (
-                    SELECT Count(UserJoinedGameServer.CreatedAt)
-                    FROM UserJoinedGameServer
-                    INNER JOIN Users ON UserJoinedGameServer.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserJoinedGameServer.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                ) +
-                (
-                    SELECT IFNULL(ROUND(SUM(TIME_TO_SEC(
-                        TIMEDIFF(UserJoinedGameServer.LeavedOn, UserJoinedGameServer.JoinedOn)
-                    ) / 3600), 2), 0)
-                    FROM UserJoinedGameServer
-                    INNER JOIN Users ON UserJoinedGameServer.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserJoinedGameServer.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                ) -
-                (
-                    SELECT COUNT(UserWarnings.CreatedAt)
-                    FROM UserWarnings
-                    INNER JOIN Users ON UserWarnings.UserId = Users.UserId
-                    WHERE Users.ServerId = {user.server.id}
-                    AND Users.UserId = {user.id}
-                    AND UserWarnings.CreatedAt >= "{datetime.date.today() - datetime.timedelta(days=7)}"
-                )
-           ) / {days} * 1000 as count;
-        """
-        user.activity = self._cast_query_result(query, int)
-
-    def _cast_query_result(self, query: str, r_type: T) -> Optional[R]:
-        results = self._context.select(query)
-        if len(results) == 0 or len(results[0]) == 0:
-            return None
-        result = results[0][0]
-        default = None
-        if r_type is int or r_type is float:
-            default = 0
-        elif r_type is str:
-            default = ""
-
-        return r_type(result) if result is not None else default

bot/src/bot_graphql/abc/query_abc.py

@@ -64,9 +64,11 @@ class QueryABC(ObjectType):
             if user == "system" or user.auth_role == AuthRoleEnum.admin:
                 return self._resolve_collection(collection, *args, **kwargs)
 
-            return self._resolve_collection(
+            for x in collection.to_list():
-                collection.where(lambda x: self._can_user_see_element(user, x)), *args, **kwargs
+                if not self._can_user_see_element(user, x):
-            )
+                    collection.remove(x)
+
+            return self._resolve_collection(collection, *args, **kwargs)
 
         self.set_field(f"{name}s", wrapper)
         self.set_field(f"{name}Count", lambda *args: wrapper(*args).count())

bot/src/bot_graphql/graphql/user.gql

@@ -9,7 +9,6 @@ type User implements TableWithHistoryQuery {
     ontime: Float
     gameOntime: Float
     level: Level
-    activityScore: Int
 
     joinedServers(filter: UserJoinedServerFilter, page: Page, sort: Sort): [UserJoinedServer]
     joinedServerCount: Int

bot/src/bot_graphql/queries/server_query.py

@@ -86,7 +86,7 @@ class ServerQuery(DataQueryWithHistoryABC):
         )
         self.add_collection(
             "user",
-            lambda server, *_: self._users.get_users_with_activity_by_server_id(server.id),
+            lambda server, *_: self._users.get_users_by_server_id(server.id),
             UserFilter,
         )
         self.add_collection(

bot/src/bot_graphql/queries/server_statistic_query.py

@@ -114,9 +114,7 @@ class ServerStatisticQuery(QueryABC):
 
     def _resolve_voice_channel_ontime(self, server, *_):
         query = f"""
-            SELECT IFNULL(ROUND(SUM(
+            SELECT ROUND(SUM(TIME_TO_SEC(TIMEDIFF(UserJoinedVoiceChannel.LeavedOn, UserJoinedVoiceChannel.JoinedOn)) / 3600),{server.server.id}) FROM UserJoinedVoiceChannel
-                TIME_TO_SEC(TIMEDIFF(UserJoinedVoiceChannel.LeavedOn, UserJoinedVoiceChannel.JoinedOn)) / 3600
-            ), 2), 0) FROM UserJoinedVoiceChannel
             INNER JOIN Users ON UserJoinedVoiceChannel.UserId = Users.UserId
             WHERE Users.ServerId = {server.server.id}
               AND UserJoinedVoiceChannel.CreatedAt >= "{self._get_date(**server.kwargs)}";
@@ -134,10 +132,7 @@ class ServerStatisticQuery(QueryABC):
 
     def _resolve_game_server_ontime(self, server, *_):
         query = f"""
-            SELECT IFNULL(ROUND(SUM(
+            SELECT ROUND(SUM(TIME_TO_SEC(TIMEDIFF(UserJoinedGameServer.LeavedOn, UserJoinedGameServer.JoinedOn)) / 3600),2) FROM UserJoinedGameServer
-                TIME_TO_SEC(TIMEDIFF(UserJoinedGameServer.LeavedOn, UserJoinedGameServer.JoinedOn)) / 3600
-            ), 2), 0)
-            FROM UserJoinedGameServer
             INNER JOIN Users ON UserJoinedGameServer.UserId = Users.UserId
             WHERE Users.ServerId = {server.server.id}
               AND UserJoinedGameServer.CreatedAt >= "{self._get_date(**server.kwargs)}";

bot/src/bot_graphql/queries/user_query.py

@@ -59,7 +59,6 @@ class UserQuery(DataQueryWithHistoryABC):
         self.set_field("ontime", lambda user, *_: user.ontime)
         self.set_field("gameOntime", lambda user, *_: user.game_ontime)
         self.set_field("level", lambda user, *_: user.level)
-        self.set_field("activityScore", lambda user, *_: user.activity)
         self.add_collection(
             "joinedServer",
             lambda user, *_: self._ujs.get_user_joined_servers_by_user_id(user.id),

web/src/app/models/data/user.model.ts

@@ -18,7 +18,6 @@ export interface User extends DataWithHistory {
   ontime?: number;
   gameOntime?: number;
   level?: Level;
-  activityScore?: number;
   server?: Server;
   leftServer?: boolean;
 

web/src/app/models/graphql/queries.model.ts

@@ -362,7 +362,6 @@ export class Queries {
           xp
           ontime
           gameOntime
-          activityScore
           level {
             id
             name
@@ -389,7 +388,6 @@ export class Queries {
         birthday
         ontime
         gameOntime
-        activityScore
         level {
           id
           name

web/src/app/modules/view/server/members/members.component.html

@@ -74,13 +74,6 @@
             </div>
           </th>
 
-          <th hideable-th="activity" [parent]="this" [sortable]="true">
-            <div class="table-header-label">
-              <div class="table-header-text">{{'common.activity' | translate}}</div>
-              <p-sortIcon field="activityScore" class="table-header-icon"></p-sortIcon>
-            </div>
-          </th>
-
           <th hideable-th="left_server" [parent]="this" [sortable]="true">
             <div class="table-header-label">
               <div class="table-header-text">{{'common.left_server' | translate}}</div>
@@ -134,7 +127,6 @@
           <th hideable-th="xp" [parent]="this"></th>
           <th hideable-th="ontime" [parent]="this"></th>
           <th hideable-th="game_ontime" [parent]="this"></th>
-          <th hideable-th="activity" [parent]="this"></th>
           <th hideable-th="left_server" [parent]="this" class="table-header-small-dropdown">
             <form [formGroup]="filterForm">
               <p-dropdown formControlName="leftServer" [options]="leftServerOptions"
@@ -221,17 +213,6 @@
               </ng-template>
             </p-cellEditor>
           </td>
-          <td hideable-th="activity" [parent]="this">
-            <span class="p-column-title">{{'common.activity' | translate}}:</span>
-            <p-cellEditor>
-              <ng-template pTemplate="input">
-                {{member.activityScore}}
-              </ng-template>
-              <ng-template pTemplate="output">
-                {{member.activityScore}}
-              </ng-template>
-            </p-cellEditor>
-          </td>
           <td hideable-th="left_server" [parent]="this">
             <span class="p-column-title">{{'common.left_server' | translate}}:</span>
             <p-cellEditor>

web/src/app/modules/view/server/members/members.component.ts

@@ -89,7 +89,7 @@ export class MembersComponent extends ComponentWithTable implements OnInit, OnDe
     private data: DataService,
     private route: ActivatedRoute
   ) {
-    super("member", ["id", "discord_id", "name", "xp", "ontime", "game_ontime", "activity", "left_server", "level"]);
+    super("member", ["id", "discord_id", "name", "xp", "ontime", "game_ontime", "left_server", "level"]);
   }
 
   ngOnInit(): void {

web/src/app/modules/view/server/profile/profile.component.html

@@ -75,13 +75,6 @@
       </div>
     </div>
 
-    <div class="content-row">
-      <div class="content-column">
-        <div class="content-data-name">{{'common.activity' | translate}}:</div>
-        <div class="content-data-value">{{user.activityScore}}</div>
-      </div>
-    </div>
-
     <div class="content-row">
       <div class="content-column">
         <div class="content-data-name">{{'view.server.profile.level' | translate}}:</div>

web/src/assets/i18n/de.json

@@ -127,7 +127,6 @@
     "abort": "Abbrechen",
     "actions": "Aktionen",
     "active": "Aktiv",
-    "activity": "Aktivität",
     "add": "Hinzufügen",
     "attribute": "Attribut",
     "auth_role": "Rolle",

web/src/assets/i18n/en.json

@@ -127,7 +127,6 @@
     "abort": "Abort",
     "actions": "Actions",
     "active": "Active",
-    "activity": "Activity",
     "add": "Add",
     "attribute": "Attribute",
     "auth_role": "Role",