Merge pull request #716 from ModWorkshop/main

3.7.1

Author: Luffyyy

backend/app/Http/Controllers/FileController.php

@@ -376,7 +376,7 @@ public function getVersions(Request $request) {
      */
     public function registerDownload(Request $request, File $file)
     {
-        ModService::registerDownload($file);
+        return ModService::registerDownload($file);
     }
 
     /**

backend/app/Http/Controllers/TagController.php

@@ -33,6 +33,8 @@ public function index(FilteredRequest $request, Game $game=null)
             'global' => 'boolean|nullable'
         ]);
 
+        $val['limit'] ??= 50; // TODO: utilize m-select with fetch more for tags
+
         $cache = [
             'key' => 'tags:'.($game ? 'game:'.$game->name : ''),
             'ttl' => 120

backend/app/Services/ModService.php

@@ -396,12 +396,13 @@ public static function registerDownload(File|Link $downloadable)
          * @var Builder
          */
         $downloads = $downloadable->downloadsRelation();
-        $alreadyDownloadedFile = $downloads->when(isset($user), fn($q) => $q->where('user_id', $user->id))
-            ->orWhere('ip_address', $ip)
-            ->exists();
+        $alreadyDownloadedFile = $downloads->where(function($q) use ($user, $ip) {
+            $q->when(isset($user), fn($q) => $q->where('user_id', $user->id))
+                ->orWhere('ip_address', $ip);
+        })->exists();
 
         if ($alreadyDownloadedFile) {
-            return response()->noContent(201);
+            return response()->noContent(200);
         }
 
         // Create download for file or link
@@ -420,7 +421,7 @@ public static function registerDownload(File|Link $downloadable)
         })->where('mod_id', $mod->id)->exists();
 
         if ($alreadyDownloadedMod) {
-            return response()->noContent(201);
+            return response()->noContent(200);
         }
 
         $download = new ModDownload([

frontend/app/components/site/a-user.vue

@@ -7,7 +7,7 @@
 		:disabled="!showMiniProfile || static"
 	>
 		<m-flex inline :column="column" class="items-center" :gap="neededGap">
-			<NuxtLink v-if="avatar" class="inline-flex" :to="link">
+			<NuxtLink v-if="avatar" class="flex shrink-0" :to="link">
 				<m-avatar :size="avatarSize" :src="user?.avatar" :use-thumb="useThumb" :style="{ opacity: isBanned ? 0.6 : 1 }"/>
 			</NuxtLink>
 

frontend/app/components/site/the-comments.vue

@@ -51,12 +51,13 @@
 		</m-flex>
 		<m-dropdown v-model:open="showMentions" :style="{ left: `${mentionPos[0]}px`, top: `${mentionPos[1]+16}px`, position: 'fixed' }">
 			<template #content>
-				<m-flex v-if="users" column class="p-3" style="max-width: 300px; word-break: normal; overflow: hidden;">
-					<template v-if="users.data.length">
-						<a-user v-for="u in users.data" :key="u.id" :user="u" avatar static show-at class="cursor-pointer whitespace-pre" @click="e => onClickMention(e, u)"/>
-					</template>
-					<div v-else>{{ $t('no_users_found') }}</div>
-				</m-flex>
+				<template v-if="users?.data.length">
+					<m-dropdown-item v-for="u in users.data" :key="u.id">
+						<a-user :user="u" avatar static show-at @click="e => onClickMention(e, u)"/>
+					</m-dropdown-item>
+				</template>
+				<m-loading v-else-if="usersLoading"/>
+				<div v-else>{{ $t('no_users_found') }}</div>
 			</template>
 		</m-dropdown>
 		<transition>
@@ -91,7 +92,6 @@
 
 <script setup lang="ts">
 import type { Comment, Game, User } from '~/types/models';
-import { Paginator } from '~/types/paginator';
 import { vIntersectionObserver } from '@vueuse/components';
 import { useStore } from '~/store';
 import { remove } from '@antfu/utils';
@@ -142,7 +142,6 @@ const editingComment = ref<Comment>();
 const mentionPos = ref([0, 0]);
 const showMentions = ref(false);
 const mentionRange = ref([-1, -1]);
-const users = ref<Paginator<User>>();
 const usersCache: Record<string, User> = {};
 
 const posting = ref(false);
@@ -162,21 +161,30 @@ const { data: comments, execute: loadComments } = await useFetchMany<Comment>(pr
 
 watch(comments, () => isLoaded.value = !!comments.value, { immediate: true });
 
-let lastTimeout: NodeJS.Timeout;
-watch([commentContent, mentionRange], async () => {
-	if (lastTimeout) {
-		clearTimeout(lastTimeout);
+const query = ref('');
+const queryDebounced = refDebounced(query, 200);
+const firstTime = ref(true);
+const { data: users, loading: usersLoading, refresh } = await useFetchMany<User>('users', {
+	query: { query: queryDebounced },
+	watch: [queryDebounced],
+	immediate: false
+});
+
+watch(users, () => {
+	if (users.value) {
+		for (const user of users.value.data) {
+			usersCache[user.unique_name] = user;
+		}
 	}
-	if (showMentions.value) {
-		const query = commentContent.value.substring(mentionRange.value[0], mentionRange.value[1]);
-
-		lastTimeout = setTimeout(async () => {
-			users.value = undefined;
-			users.value = await getRequest<Paginator<User>>('users', { query: { query } });
-			for (const user of users.value.data) {
-				usersCache[user.unique_name] = user;
-			}
-		}, 150);
+});
+
+watch([commentContent, mentionRange], async () => {
+	if (mentionRange.value?.[0] !== undefined) {
+		query.value = commentContent.value.substring(mentionRange.value[0], mentionRange.value[1]);
+		if (firstTime.value) {
+			refresh();
+		}
+		firstTime.value = false;
 	}
 });
 

frontend/app/components/ui/layout/m-nav.vue

@@ -1,6 +1,6 @@
 <template>
 	<div class="nav">
-		<m-flex v-if="side" class="items-center hidden max-lg:block" @click="menuOpen = !menuOpen">
+		<m-flex v-if="side" class="items-center lg:hidden" @click="menuOpen = !menuOpen">
 			<m-link class="collapse-button">
 				<i-mdi-menu/>
 			</m-link>

frontend/app/components/ui/layout/m-tabs.vue

@@ -181,6 +181,7 @@ if (query) {
 	border-radius: var(--content-border-radius);
 	background-color: var(--secondary-content-bg-color);
 	align-self: start;
+	width: 100%;
 }
 
 .alt-background.tab-menu-bg {

frontend/app/i18n/locales/es.json

@@ -641,15 +641,15 @@
     "edit_mod_tip_2": "Una vez que estés satisfecho con el mod, puedes publicarlo y permitir que las personas puedan verlo y descargarlo.",
     "edit_mod_warn_1": "En algunos casos, los mods (como los de música) requieren aprobación. Si tu mod contiene música, envíalo para su aprobación (más abajo)",
     "edit_mod_warn_2": "Debes estar familiarizado con nuestras reglas. Puedes encontrarlas {here}",
-    "edit_mod_warn_new_user": "As a new user, your first upload must be approved by a moderator. Afterwards you should be able to upload freely.",
+    "edit_mod_warn_new_user": "Como usuario nuevo, tu primera subida debe ser aprobada por un moderador. Después podrás subir contenido libremente.",
     "auto_subscribe_to_mod": "Suscribirse automáticamente a los comentarios de los mods",
     "auto_subscribe_to_thread": "Suscribirse automáticamente a los hilos",
     "mark_as_answer": "Marcar como respuesta",
     "unmark_as_answer": "Desmarcar como respuesta",
     "thread_answer": "Este comentario fue elegido como respuesta para la publicación",
     "login_required": "Es necesario iniciar sesión",
     "manager_site_url": "Enlace del sitio",
-    "manager_site_url_desc": "Los enlaces al sitio web del Administrador de Mods o al lugar donde se puede conseguir",
+    "manager_site_url_desc": "Enlace al sitio web del Administrador de Mods o al lugar donde conseguirlo",
     "manager_download_url": "Enlace de descarga",
     "manager_download_url_desc": "Un enlace para manejar la descarga e instalación de los archivos de los mods. Parámetros disponibles actualmente: :mod_id, :file_id, :game_id, :manager_name y :game_short_name.",
     "status": "Estado",
@@ -675,8 +675,8 @@
     "unignore": "Dejar de ignorar",
     "ignored_games": "Juegos ignorados",
     "ignored_mods": "Mods ignorados",
-    "ignored_categories": "Ignored Categories",
-    "ignore_category": "Ignore Category",
+    "ignored_categories": "Categorías ignoradas",
+    "ignore_category": "Ignorar categoría",
     "audit_log": "Registro de auditoría",
     "copy_link": "Copiar enlace",
     "repository": "Repositorio",
@@ -686,11 +686,11 @@
     "hidden_tags_desc": "Oculta las etiquetas globales para que los usuarios no puedan seleccionarlas en los mods o hilos. Si algún mod ya usa estas etiquetas, será necesario actualizarlas manualmente.",
     "install_with": "Instalar con {modManager}",
     "mod_manager_not_installed": "¿No tienes {modManager}?",
-    "leave_members": "Leave Members",
-    "sessions": "Sessions",
-    "ip_address": "IP Address",
-    "ip_ban": "IP Ban",
-    "ip_ban_help": "Bans the user and their IP. Note: this shouldn't really be relied on too much, users can easily change IPs, this is more of a bandage solution.",
-    "pinned_mods": "Pinned Mods",
-    "pinned_mods_desc": "Pinned mods appear on top of your regular mod list. You may choose up to 5 mods you own or a mod you are a member of (except viewer)."
+    "leave_members": "Dejar de ser miembro",
+    "sessions": "Sesiones",
+    "ip_address": "Dirección IP",
+    "ip_ban": "Veto por IP",
+    "ip_ban_help": "Aplica un veto al usuario y a su IP. Nota: no es un método muy fiable, ya que los usuarios pueden cambiar de IP fácilmente; es más bien una solución temporal.",
+    "pinned_mods": "Mods fijados",
+    "pinned_mods_desc": "Los mods fijados aparecen al principio de tu lista de mods. Puedes fijar hasta 5 mods que poseas o de los que seas miembro (excepto como Espectador)."
 }

frontend/app/pages/user-settings/index.vue

@@ -15,7 +15,7 @@
 					<m-avatar size="md" :src="src"/>
 				</template>
 			</m-img-uploader>
-			<m-flex flex>
+			<m-flex flex wrap gap="3">
 				<m-input v-model="userForm.name" :label="$t('display_name')" maxlength="30"/>
 				<m-input v-model="userForm.custom_color" :disabled="!user.has_supporter_perks" :label="true" :desc="$t('custom_color_desc')" type="color">
 					<template #label>
@@ -77,11 +77,11 @@
 			</m-input>
 
 			<m-select v-model="userForm.show_tag" :options="showTagOptions" :label="$t('show_tag')" :desc="$t('show_tag_desc')"/>
-			<m-flex>
+			<m-flex wrap gap="3">
 				<m-input v-model="userForm.donation_url" :validity="donationValid" :label="$t('donation')" :desc="$t('donation_desc')"/>
 				<m-input v-model="userForm.custom_title" :label="$t('custom_title')"/>
 			</m-flex>
-			<m-flex>
+			<m-flex wrap gap="3">
 				<m-input v-model="userForm.private_profile" :label="$t('private_profile')" :desc="$t('private_profile_desc')" type="checkbox"/>
 				<m-input v-model="userForm.invisible" :label="$t('invisible')" :desc="$t('invisible_desc')" type="checkbox"/>
 			</m-flex>

frontend/app/utils/mod-helpers.ts

@@ -34,8 +34,13 @@ export function canSuperUpdate(mod: Mod) {
 	return (mod.user_id === user.id && hasPermission('create-mods', mod.game)) || hasPermission('manage-mods', mod.game);
 }
 
-export function registerDownload(downlaod_type: 'file' | 'link', download: File | Link) {
-	postRequest(`${downlaod_type}s/${download.id}/register-download`, null);
+export function registerDownload(downloadType: 'file' | 'link', download: File | Link) {
+	const { $mwsAPI } = useNuxtApp();
+
+	$mwsAPI(`${downloadType}s/${download.id}/register-download`, {
+		method: 'POST',
+		keepalive: true
+	});
 }
 
 // Bare minimum fields required for a list mod to function for optimization purposes.