From 56da899bb5b4ea099cb0a50ef3a2eedcd03e01b3 Mon Sep 17 00:00:00 2001
From: Ansariel <none@none>
Date: Thu, 6 Feb 2014 11:15:59 +0100
Subject: [PATCH] FIRE-9045: Changed option to a general option to open the
 full avatar profile when clicking on an avatar inspect SLURL

---
 indra/newview/app_settings/settings.xml               |  4 ++--
 indra/newview/fschathistory.cpp                       |  2 +-
 indra/newview/llpanelprofile.cpp                      |  5 ++++-
 .../skins/default/xui/de/panel_preferences_chat.xml   |  1 -
 .../default/xui/de/panel_preferences_firestorm.xml    |  1 +
 .../skins/default/xui/en/panel_preferences_chat.xml   | 11 -----------
 .../default/xui/en/panel_preferences_firestorm.xml    | 10 ++++++++++
 7 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/indra/newview/app_settings/settings.xml b/indra/newview/app_settings/settings.xml
index 3787883b5e..0e25170222 100644
--- a/indra/newview/app_settings/settings.xml
+++ b/indra/newview/app_settings/settings.xml
@@ -21759,10 +21759,10 @@ Change of this parameter will affect the layout of buttons in notification toast
       <key>Value</key>
       <integer>1</integer>
     </map>
-    <key>FSChatnamesOpenFullProfile</key>
+    <key>FSInspectAvatarSlurlOpensProfile</key>
       <map>
       <key>Comment</key>
-      <string>Open the full profile of an avatar directly when clicking on its name in chat</string>
+      <string>Open the full profile of an avatar directly when clicking on its name</string>
       <key>Persist</key>
       <integer>1</integer>
       <key>Type</key>
diff --git a/indra/newview/fschathistory.cpp b/indra/newview/fschathistory.cpp
index dbb7dc5f64..11b1615df8 100644
--- a/indra/newview/fschathistory.cpp
+++ b/indra/newview/fschathistory.cpp
@@ -1140,7 +1140,7 @@ void FSChatHistory::appendMessage(const LLChat& chat, const LLSD &args, const LL
 				}
 				
 				name_params.is_name_slurl = true;
-				name_params.link_href = LLSLURL("agent", chat.mFromID, (gSavedSettings.getBOOL("FSChatnamesOpenFullProfile") ? "about" : "inspect")).getSLURLString();
+				name_params.link_href = LLSLURL("agent", chat.mFromID, "inspect").getSLURLString();
 
 				if (from_me && gSavedSettings.getBOOL("FSChatHistoryShowYou"))
 				{
diff --git a/indra/newview/llpanelprofile.cpp b/indra/newview/llpanelprofile.cpp
index bc743bb599..92d7cd28f4 100755
--- a/indra/newview/llpanelprofile.cpp
+++ b/indra/newview/llpanelprofile.cpp
@@ -126,7 +126,10 @@ public:
 		}
 
 		const std::string verb = params[1].asString();
-		if (verb == "about")
+		// <FS:Ansariel> FIRE-9045: Inspect links always open full profile
+		//if (verb == "about")
+		if (verb == "about" || (gSavedSettings.getBOOL("FSInspectAvatarSlurlOpensProfile") && verb == "inspect"))
+		// </FS:Ansariel>
 		{
 			LLAvatarActions::showProfile(avatar_id);
 			return true;
diff --git a/indra/newview/skins/default/xui/de/panel_preferences_chat.xml b/indra/newview/skins/default/xui/de/panel_preferences_chat.xml
index 14ab20741e..06ec235ad3 100755
--- a/indra/newview/skins/default/xui/de/panel_preferences_chat.xml
+++ b/indra/newview/skins/default/xui/de/panel_preferences_chat.xml
@@ -142,7 +142,6 @@
 			<check_box label="Chatleiste nach Senden deaktivieren" name="CloseChatOnReturn"/>
 			<check_box label="Chatverlauf ebenfalls deaktivieren" name="FSUnfocusChatHistoryOnReturn" left_delta="260"/>
 			<check_box name="FSMarkObjects" label="Objektename als (namenlos) anzeigen um Vortäuschungen zu unterbinden"/>
-			<check_box name="FSChatnamesOpenFullProfile" label="Vollständiges Avatarprofil beim Klicken auf Namen öffnen" tool_tip="Beim Klicken auf den Namen eines Avatars im Chatverlauf wird direkt das vollständige Profil geöffnet anstatt des Mini-Profils. Diese Einstellung hat keine Auswirkung auf Namen für bereits empfangenen Chat."/>
 
 			<!-- Groups -->
 			<text name="group_enhancements">
diff --git a/indra/newview/skins/default/xui/de/panel_preferences_firestorm.xml b/indra/newview/skins/default/xui/de/panel_preferences_firestorm.xml
index 160542a4a9..281fc7a795 100644
--- a/indra/newview/skins/default/xui/de/panel_preferences_firestorm.xml
+++ b/indra/newview/skins/default/xui/de/panel_preferences_firestorm.xml
@@ -23,6 +23,7 @@
 		<check_box label="Deaktiviere Leitstrahl, wenn Zielavatar erreicht wird (&lt;= 3m)" name="FSDisableAvatarTrackerAtCloseIn" tool_tip="Falls aktiviert, wird der Leitstrahl zum Zielavatar deaktiviert, wenn die Distanz weniger als 3m beträgt. (Standard)"/>
 		<check_box label="Unterschiedliche Suchbegriffe pro Registerkarte im Inventar erlauben" name="FSSplitInventorySearchOverTabs" tool_tip="Falls aktiviert, können im Inventarfenster für jede Registerkarte unterschiedliche Suchbegriffe eingegeben werden."/>
 		<check_box label="Partikeleffekte erzeugen, wenn Skripte kommunizieren" name="EffectScriptChatParticles" tool_tip="Falls aktiviert, werden Partikeleffekte um kommunizierende Objekte erzeugt."/>
+		<check_box label="Beim Klicken auf Avatarnamen direkt Profil öffnen" name="FSInspectAvatarSlurlOpensProfile" tool_tip="Falls aktiviert, wird bei einem Klick auf den Namenslink eines Avatars direkt dessen komplettes Profil geöffnet."/>
 		<check_box label="Aktiviere progressives Draw-Distance-Stepping" name="FSRenderFarClipStepping" tool_tip="Falls aktiviert wird Firestorm nach einem Teleport die Draw Distance progressiv erhöhen." width="270"/>
 		<slider name="progressive_draw_distance_interval" tool_tip="Intervall in Sekunden zwischen den jeweiligen Erhöhungen der Draw Distance."/>
 		<check_box name="UseLSLBridge" label="LSL-Client-Brücke aktivieren" tool_tip="Erlaub dem Viewer die Nutzung eines geskripteten Objektes (Brücke), um weitergehende Funktionen anzubieten."/>
diff --git a/indra/newview/skins/default/xui/en/panel_preferences_chat.xml b/indra/newview/skins/default/xui/en/panel_preferences_chat.xml
index d9850050db..b2450c480b 100755
--- a/indra/newview/skins/default/xui/en/panel_preferences_chat.xml
+++ b/indra/newview/skins/default/xui/en/panel_preferences_chat.xml
@@ -890,17 +890,6 @@
              name="FSMarkObjects"
              control_name="FSMarkObjects"
              label="Mark objects with (no name) when they speak to avoid spoofing"/>
-            <check_box
-             layout="topleft"
-             follows="left|top"
-             top_pad="0"
-             left="30"
-             width="400"
-             height="18"
-             name="FSChatnamesOpenFullProfile"
-             control_name="FSChatnamesOpenFullProfile"
-             label="Open full profile when clicking on avatar name"
-             tool_tip="Clicking on the name of an avatar in chat history will open its full profile instead of the mini profile. This option will not affect names in previously received chat."/>
 
             <!-- Groups -->
             <text
diff --git a/indra/newview/skins/default/xui/en/panel_preferences_firestorm.xml b/indra/newview/skins/default/xui/en/panel_preferences_firestorm.xml
index 95ae90cfca..9f8634e1ae 100644
--- a/indra/newview/skins/default/xui/en/panel_preferences_firestorm.xml
+++ b/indra/newview/skins/default/xui/en/panel_preferences_firestorm.xml
@@ -190,6 +190,16 @@
 		 initial_value="true"
 		 top_pad="5"
 		 width="350" />
+		<check_box
+		 control_name="FSInspectAvatarSlurlOpensProfile"
+		 height="16"
+		 label="Open avatar profile directly when clicking on its name"
+		 layout="topleft"
+		 name="FSInspectAvatarSlurlOpensProfile"
+		 tool_tip="If enabled, Firestorm will open the profile of an avatar directly when clicking on a link with its name"
+		 initial_value="true"
+		 top_pad="5"
+		 width="250" />
 		<check_box
 		 control_name="FSRenderFarClipStepping"
 		 height="16"