feat: improve /display video error feedback

Author: arsmotorin

CHANGELOG.md

@@ -52,7 +52,7 @@ Plugin:
 - [x] Add `/display info` command for quick display information
 - [x] Add `/display list` filters (`mine`, `world <name>`, `owner <name>`, `sync`)
 - [x] Add translation for `/display list` command
-- [ ] Improve `/display video` error feedback (separate invalid URL/not owner/wrong target block)
+- [x] Improve `/display video` error feedback (separate invalid URL/not owner/wrong target block)
 - [x] Add total value output to `/display stats`
 - [ ] Add admin target mode for `/display on|off <player>`
 - [ ] Improve `/display reload` output with what was reloaded

spigot/src/main/kotlin/com/dreamdisplays/commands/subcommands/VideoCommand.kt

@@ -23,20 +23,26 @@ class VideoCommand : SubCommand {
             return
         }
 
+        val code = YouTubeUtils.extractVideoIdFromUri(args[1] ?: "")
+            ?: return Message.sendMessage(player, "invalidURL")
+
         val block = player.getTargetBlock(null, 32)
-        val data = isContains(block.location)
 
-        if (
-            block.type != Main.config.settings.baseMaterial ||
-            data == null ||
-            data.ownerId != player.uniqueId
-        ) {
+        if (block.type != Main.config.settings.baseMaterial) {
+            Message.sendMessage(player, "displayVideoWrongTargetBlock")
+            return
+        }
+
+        val data = isContains(block.location)
+        if (data == null) {
             Message.sendMessage(player, "noDisplay")
             return
         }
 
-        val code = YouTubeUtils.extractVideoIdFromUri(args[1] ?: "")
-            ?: return Message.sendMessage(player, "invalidURL")
+        if (data.ownerId != player.uniqueId) {
+            Message.sendMessage(player, "displayVideoNotOwner")
+            return
+        }
 
         data.apply {
             url = "https://youtube.com/watch?v=$code"

spigot/src/main/resources/assets/dreamdisplays/lang/be.json

@@ -17,6 +17,8 @@
   "noDisplay": "&7D |&f Дысплей не знойдзены!",
   "settedURL": "&7D |&f Відэа было загружана!",
   "invalidURL": "&7D |&f Ваша спасылка несапраўдная! Калі ласка, выкарыстоўвайце сапраўдную YouTube спасылку",
+  "displayVideoWrongTargetBlock": "&7D |&f Каб выкарыстаць гэтую каманду, глядзіце на блок дысплея!",
+  "displayVideoNotOwner": "&7D |&f Вы не ўладальнік гэтага дысплея!",
   "reportTooQuickly": "&7D |&f Вы адпраўляеце справаздачы занадта хутка! Пачакайце, калі ласка, перад адпраўкай наступнай",
   "reportSent": "&7D |&f Справаздача была адпраўлена!",
   "reportFailed": "&7D |&f Не ўдалося адправіць справаздачу. Калі ласка, паспрабуйце пазней.",

spigot/src/main/resources/assets/dreamdisplays/lang/cs.json

@@ -17,6 +17,8 @@
   "noDisplay": "&7D |&f Displej nebyl nalezen!",
   "settedURL": "&7D |&f Video bylo načteno!",
   "invalidURL": "&7D |&f Váš odkaz je neplatný! Použijte prosím platný YouTube odkaz",
+  "displayVideoWrongTargetBlock": "&7D |&f Pro použití tohoto příkazu se musíte dívat na blok displeje!",
+  "displayVideoNotOwner": "&7D |&f Nejste vlastník tohoto displeje!",
   "reportTooQuickly": "&7D |&f Odesíláte zprávy příliš rychle! Počkejte prosím před odesláním další",
   "reportSent": "&7D |&f Zpráva byla odeslána!",
   "reportFailed": "&7D |&f Nepodařilo se odeslat zprávu. Zkuste to prosím později.",

spigot/src/main/resources/assets/dreamdisplays/lang/de.json

@@ -17,6 +17,8 @@
   "noDisplay": "&7D |&f Display wurde nicht gefunden!",
   "settedURL": "&7D |&f Video wurde geladen!",
   "invalidURL": "&7D |&f Deine URL ist ungültig! Bitte verwende einen gültigen YouTube-Link",
+  "displayVideoWrongTargetBlock": "&7D |&f Du musst auf einen Display-Block schauen, um diesen Befehl zu nutzen!",
+  "displayVideoNotOwner": "&7D |&f Du bist nicht der Besitzer dieses Displays!",
   "reportTooQuickly": "&7D |&f Du sendest Berichte zu schnell! Warte bitte, bevor du einen weiteren sendest",
   "reportSent": "&7D |&f Bericht wurde gesendet!",
   "reportFailed": "&7D |&f Fehler beim Senden des Berichts. Bitte versuchen Sie es später erneut.",

spigot/src/main/resources/assets/dreamdisplays/lang/en.json

@@ -17,6 +17,8 @@
   "noDisplay": "&7D |&f Display has not been found!",
   "settedURL": "&7D |&f Video has been loaded!",
   "invalidURL": "&7D |&f Your URL is invalid! Please use a valid YouTube link",
+  "displayVideoWrongTargetBlock": "&7D |&f You must look at a display block to use this command!",
+  "displayVideoNotOwner": "&7D |&f You are not the owner of this display!",
   "reportTooQuickly": "&7D |&f You are sending reports too quickly! Please wait before sending another report",
   "reportSent": "&7D |&f Report has been sent!",
   "reportFailed": "&7D |&f Failed to send report. Please try again later.",

spigot/src/main/resources/assets/dreamdisplays/lang/he.json

@@ -17,6 +17,8 @@
   "noDisplay": "&7D |&f התצוגה לא נמצאה!",
   "settedURL": "&7D |&f הוידאו נטען!",
   "invalidURL": "&7D |&f הקישור שלך לא תקין! אנא השתמש בקישור YouTube תקין",
+  "displayVideoWrongTargetBlock": "&7D |&f כדי להשתמש בפקודה הזו, עליך להסתכל על בלוק תצוגה!",
+  "displayVideoNotOwner": "&7D |&f אינך הבעלים של התצוגה הזו!",
   "reportTooQuickly": "&7D |&f אתה שולח דיווחים מהר מדי! אנא המתן לפני שליחת דיווח נוסף",
   "reportSent": "&7D |&f הדיווח נשלח!",
   "reportFailed": "&7D |&f נכשל בשליחת הדיווח. אנא נסה שוב מאוחר יותר.",

spigot/src/main/resources/assets/dreamdisplays/lang/pl.json

@@ -14,6 +14,8 @@
   "noDisplay": "&7D |&f Wyświetlacz nie został znaleziony!",
   "settedURL": "&7D |&f Wideo załadowane!",
   "invalidURL": "&7D |&f Twój link jest nieprawidłowy! Proszę użyć prawidłowego linku YouTube",
+  "displayVideoWrongTargetBlock": "&7D |&f Aby użyć tej komendy, musisz patrzeć na blok wyświetlacza!",
+  "displayVideoNotOwner": "&7D |&f Nie jesteś właścicielem tego wyświetlacza!",
   "reportTooQuickly": "&7D |&f Wysyłasz raporty zbyt szybko! Poczekaj przed wysłaniem innego raportu",
   "reportSent": "&7D |&f Raport wysłany!",
   "reportFailed": "&7D |&f Nie udało się wysłać zgłoszenia. Spróbuj ponownie później.",

spigot/src/main/resources/assets/dreamdisplays/lang/ru.json

@@ -14,6 +14,8 @@
   "noDisplay": "&7D |&f Дисплей не найден!",
   "settedURL": "&7D |&f Видео загружено!",
   "invalidURL": "&7D |&f Ваша ссылка недействительна! Пожалуйста, используйте действительную ссылку YouTube",
+  "displayVideoWrongTargetBlock": "&7D |&f Чтобы использовать эту команду, смотрите на блок дисплея!",
+  "displayVideoNotOwner": "&7D |&f Вы не владелец этого дисплея!",
   "reportTooQuickly": "&7D |&f Вы отправляете отчёты слишком быстро! Подождите перед отправкой другого отчёта",
   "reportSent": "&7D |&f Отчёт отправлен!",
   "reportFailed": "&7D |&f Не удалось отправить отчёт. Пожалуйста, попробуйте позже.",

spigot/src/main/resources/assets/dreamdisplays/lang/uk.json

@@ -14,6 +14,8 @@
   "noDisplay": "&7D |&f Дисплей не знайдено!",
   "settedURL": "&7D |&f Відео завантажено!",
   "invalidURL": "&7D |&f Ваше посилання недійсне! Будь ласка, використовуйте дійсне посилання YouTube",
+  "displayVideoWrongTargetBlock": "&7D |&f Щоб використати цю команду, дивіться на блок дисплея!",
+  "displayVideoNotOwner": "&7D |&f Ви не власник цього дисплея!",
   "reportTooQuickly": "&7D |&f Ви надсилаєте звіти занадто швидко! Зачекайте перед надсиланням іншого звіту",
   "reportSent": "&7D |&f Звіт надіслано!",
   "reportFailed": "&7D |&f Не вдалося надіслати звіт. Будь ласка, спробуйте пізніше.",