From dfaa903f3927e661534c49100c0d177a9b203a1b Mon Sep 17 00:00:00 2001
From: =?utf8?q?Tim=20D=C3=BCsterhus?= <duesterhus@woltlab.com>
Date: Sun, 15 Dec 2013 16:54:06 +0100
Subject: [PATCH] Validate userOnlineMarking

---
 wcfsetup/install/files/acp/templates/userGroupAdd.tpl          | 2 +-
 wcfsetup/install/files/lib/acp/form/UserGroupAddForm.class.php | 3 +++
 wcfsetup/install/lang/de.xml                                   | 1 +
 wcfsetup/install/lang/en.xml                                   | 1 +
 4 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/wcfsetup/install/files/acp/templates/userGroupAdd.tpl b/wcfsetup/install/files/acp/templates/userGroupAdd.tpl
index a771e38ea3..736f69560b 100644
--- a/wcfsetup/install/files/acp/templates/userGroupAdd.tpl
+++ b/wcfsetup/install/files/acp/templates/userGroupAdd.tpl
@@ -103,7 +103,7 @@
 						<input type="text" id="userOnlineMarking" name="userOnlineMarking" value="{$userOnlineMarking}" class="long" />
 						{if $errorType.userOnlineMarking|isset}
 							<small class="innerError">
-								{lang}wcf.acp.group.userOnlineMarking.error.{@$errorType.priority}{/lang}
+								{lang}wcf.acp.group.userOnlineMarking.error.{@$errorType.userOnlineMarking}{/lang}
 							</small>
 						{/if}
 						<small>{lang}wcf.acp.group.userOnlineMarking.description{/lang}</small>
diff --git a/wcfsetup/install/files/lib/acp/form/UserGroupAddForm.class.php b/wcfsetup/install/files/lib/acp/form/UserGroupAddForm.class.php
index 3c225c93d7..9a5bf9d9e3 100755
--- a/wcfsetup/install/files/lib/acp/form/UserGroupAddForm.class.php
+++ b/wcfsetup/install/files/lib/acp/form/UserGroupAddForm.class.php
@@ -120,6 +120,9 @@ class UserGroupAddForm extends AbstractOptionListForm {
 			if (!I18nHandler::getInstance()->validateValue('groupName')) {
 				throw new UserInputException('groupName');
 			}
+			if (mb_strpos($this->userOnlineMarking, '%s') === false) {
+				throw new UserInputException('userOnlineMarking', 'notValid');
+			}
 		}
 		catch (UserInputException $e) {
 			$this->errorType[$e->getField()] = $e->getType();
diff --git a/wcfsetup/install/lang/de.xml b/wcfsetup/install/lang/de.xml
index ee256988fd..b9f1e1fb2f 100644
--- a/wcfsetup/install/lang/de.xml
+++ b/wcfsetup/install/lang/de.xml
@@ -346,6 +346,7 @@
 		<item name="wcf.acp.group.priority.description"><![CDATA[Bestimmt u.a. die Reihenfolge auf der Team-Seite sowie die Auswahl von Benutzerrängen und „Wer ist online“-Darstellungen auf Basis der höchsten Priorität.]]></item>
 		<item name="wcf.acp.group.userOnlineMarking"><![CDATA[„Benutzer online“-Darstellung]]></item>
 		<item name="wcf.acp.group.userOnlineMarking.description"><![CDATA[Sie können die HTML-Formatierung für Mitglieder dieser Benutzergruppe in der „Wer ist online“-Anzeige anpassen. <em>&lt;strong&gt;%s&lt;/strong&gt;</em> stellt Mitglieder dieser Gruppe beispielsweise in Fettdruck dar.]]></item>
+		<item name="wcf.acp.group.userOnlineMarking.error.notValid"><![CDATA[Die Darstellung muss „%s“ enthalten]]></item>
 		<item name="wcf.acp.group.showOnTeamPage"><![CDATA[Mitglieder dieser Benutzergruppe auf der Team-Seite anzeigen]]></item>
 		<item name="wcf.acp.group.option.admin.user.canEnableUser"><![CDATA[Kann Benutzer aktivieren]]></item>
 		<item name="wcf.acp.group.option.user.profile.renamePeriod"><![CDATA[Umbenennung]]></item>
diff --git a/wcfsetup/install/lang/en.xml b/wcfsetup/install/lang/en.xml
index 9bf7cb16e4..3e1599997a 100644
--- a/wcfsetup/install/lang/en.xml
+++ b/wcfsetup/install/lang/en.xml
@@ -345,6 +345,7 @@ Examples for medium ID detection:
 		<item name="wcf.acp.group.priority.description"><![CDATA[Determines show order on team page, user rank and “Who is Online” marking based on highest priority.]]></item>
 		<item name="wcf.acp.group.userOnlineMarking"><![CDATA[“Who is Online” Marking]]></item>
 		<item name="wcf.acp.group.userOnlineMarking.description"><![CDATA[Adjust the HTML formatting for members of this group in the “Who is Online” list. <em>&lt;strong&gt;%s&lt;/strong&gt;</em> results in a bolder appearance.]]></item>
+		<item name="wcf.acp.group.userOnlineMarking.error.notValid"><![CDATA[There must be an occurrence of “%s”]]></item>
 		<item name="wcf.acp.group.showOnTeamPage"><![CDATA[Show members on team page]]></item>
 		<item name="wcf.acp.group.option.admin.user.canEnableUser"><![CDATA[Can approve users]]></item>
 		<item name="wcf.acp.group.option.user.profile.renamePeriod"><![CDATA[Rename]]></item>
-- 
2.20.1