From 6037e0939f41f962e8420a444b0d1efb689e74d1 Mon Sep 17 00:00:00 2001
From: skjnldsv <skjnldsv@protonmail.com>
Date: Fri, 25 Oct 2024 17:25:42 +0200
Subject: [PATCH] fix(systemtags): translations and etag method casing

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
---
 .../src/components/SystemTagPicker.vue        | 109 ++++++++++++------
 lib/private/SystemTag/SystemTag.php           |   2 +-
 lib/public/SystemTag/ISystemTag.php           |   2 +-
 3 files changed, 77 insertions(+), 36 deletions(-)

diff --git a/apps/systemtags/src/components/SystemTagPicker.vue b/apps/systemtags/src/components/SystemTagPicker.vue
index db93e6be6cfe0..b6a1b652cdc91 100644
--- a/apps/systemtags/src/components/SystemTagPicker.vue
+++ b/apps/systemtags/src/components/SystemTagPicker.vue
@@ -7,6 +7,7 @@
 	<NcDialog data-cy-systemtags-picker
 		:name="t('systemtags', 'Manage tags')"
 		:open="opened"
+		:class="'systemtags-picker--' + status"
 		class="systemtags-picker"
 		close-on-click-outside
 		out-transition
@@ -99,7 +100,7 @@ import { defineComponent } from 'vue'
 import { emit } from '@nextcloud/event-bus'
 import { sanitize } from 'dompurify'
 import { showError, showInfo } from '@nextcloud/dialogs'
-import { getLanguage, t } from '@nextcloud/l10n'
+import { getLanguage, n, t } from '@nextcloud/l10n'
 import escapeHTML from 'escape-html'
 
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
@@ -122,10 +123,10 @@ type TagListCount = {
 }
 
 enum Status {
-	BASE,
-	LOADING,
-	CREATING_TAG,
-	DONE,
+	BASE = 'base',
+	LOADING = 'loading',
+	CREATING_TAG = 'creating-tag',
+	DONE = 'done',
 }
 
 export default defineComponent({
@@ -190,15 +191,23 @@ export default defineComponent({
 
 		statusMessage(): string {
 			if (this.toAdd.length === 0 && this.toRemove.length === 0) {
+				// should not happen™
 				return ''
 			}
 
 			if (this.toAdd.length === 1 && this.toRemove.length === 1) {
-				return t('systemtags', '{tag1} will be set and {tag2} will be removed from {count} files.', {
-					tag1: this.formatTagChip(this.toAdd[0]),
-					tag2: this.formatTagChip(this.toRemove[0]),
-					count: this.nodes.length,
-				}, undefined, { escape: false })
+				return n(
+					'systemtags',
+					'{tag1} will be set and {tag2} will be removed from 1 file.',
+					'{tag1} and {tag2} will be set and removed from {count} files.',
+					this.nodes.length,
+					{
+						tag1: this.formatTagChip(this.toAdd[0]),
+						tag2: this.formatTagChip(this.toRemove[0]),
+						count: this.nodes.length,
+					},
+					{ escape: false },
+				)
 			}
 
 			const tagsAdd = this.toAdd.map(this.formatTagChip)
@@ -206,27 +215,55 @@ export default defineComponent({
 			const tagsRemove = this.toRemove.map(this.formatTagChip)
 			const lastTagRemove = tagsRemove.pop() as string
 
-			const addStringSingular = t('systemtags', '{tag} will be set to {count} files.', {
-				tag: lastTagAdd,
-				count: this.nodes.length,
-			}, undefined, { escape: false })
-
-			const removeStringSingular = t('systemtags', '{tag} will be removed from {count} files.', {
-				tag: lastTagRemove,
-				count: this.nodes.length,
-			}, undefined, { escape: false })
-
-			const addStringPlural = t('systemtags', '{tags} and {lastTag} will be set to {count} files.', {
-				tags: tagsAdd.join(', '),
-				lastTag: lastTagAdd,
-				count: this.nodes.length,
-			}, undefined, { escape: false })
-
-			const removeStringPlural = t('systemtags', '{tags} and {lastTag} will be removed from {count} files.', {
-				tags: tagsRemove.join(', '),
-				lastTag: lastTagRemove,
-				count: this.nodes.length,
-			}, undefined, { escape: false })
+			const addStringSingular = n(
+				'systemtags',
+				'{tag} will be set to 1 file.',
+				'{tag} will be set to {count} files.',
+				this.nodes.length,
+				{
+					tag: lastTagAdd,
+					count: this.nodes.length,
+				},
+				{ escape: false },
+			)
+
+			const removeStringSingular = n(
+				'systemtags',
+				'{tag} will be removed from 1 file.',
+				'{tag} will be removed from {count} files.',
+				this.nodes.length,
+				{
+					tag: lastTagRemove,
+					count: this.nodes.length,
+				},
+				{ escape: false },
+			)
+
+			const addStringPlural = n(
+				'systemtags',
+				'{tags} and {lastTag} will be set to 1 file.',
+				'{tags} and {lastTag} will be set to {count} files.',
+				this.nodes.length,
+				{
+					tags: tagsAdd.join(', '),
+					lastTag: lastTagAdd,
+					count: this.nodes.length,
+				},
+				{ escape: false },
+			)
+
+			const removeStringPlural = n(
+				'systemtags',
+				'{tags} and {lastTag} will be removed from 1 file.',
+				'{tags} and {lastTag} will be removed from {count} files.',
+				this.nodes.length,
+				{
+					tags: tagsRemove.join(', '),
+					lastTag: lastTagRemove,
+					count: this.nodes.length,
+				},
+				{ escape: false },
+			)
 
 			// Singular
 			if (this.toAdd.length === 1 && this.toRemove.length === 0) {
@@ -246,14 +283,14 @@ export default defineComponent({
 
 			// Mixed
 			if (this.toAdd.length > 1 && this.toRemove.length === 1) {
-				return `${addStringPlural}<br>${removeStringSingular}`
+				return `${addStringPlural} ${removeStringSingular}`
 			}
 			if (this.toAdd.length === 1 && this.toRemove.length > 1) {
-				return `${addStringSingular}<br>${removeStringPlural}`
+				return `${addStringSingular} ${removeStringPlural}`
 			}
 
 			// Both plural
-			return `${addStringPlural}<br>${removeStringPlural}`
+			return `${addStringPlural} ${removeStringPlural}`
 		},
 	},
 
@@ -452,6 +489,10 @@ export default defineComponent({
 	}
 }
 
+.systemtags-picker--done :deep(.empty-content__icon) {
+	opacity:  1;
+}
+
 // Rendered chip in note
 .nc-chip {
 	display: inline !important;
diff --git a/lib/private/SystemTag/SystemTag.php b/lib/private/SystemTag/SystemTag.php
index be97bed27ddb6..8c64f2389d01e 100644
--- a/lib/private/SystemTag/SystemTag.php
+++ b/lib/private/SystemTag/SystemTag.php
@@ -66,7 +66,7 @@ public function getAccessLevel(): int {
 	/**
 	 * {@inheritdoc}
 	 */
-	public function getEtag(): ?string {
+	public function getETag(): ?string {
 		return $this->etag;
 	}
 }
diff --git a/lib/public/SystemTag/ISystemTag.php b/lib/public/SystemTag/ISystemTag.php
index 77575494d16c6..593c127ba63c1 100644
--- a/lib/public/SystemTag/ISystemTag.php
+++ b/lib/public/SystemTag/ISystemTag.php
@@ -88,5 +88,5 @@ public function getAccessLevel(): int;
 	 *
 	 * @since 31.0.0
 	 */
-	public function getEtag(): ?string;
+	public function getETag(): ?string;
 }