Added confirmation dialog before applying LightShare settings.

If you are using a preset other than the default preset or
the LightShare preset, the viewer will now ask you whether
to apply settings received from the server.

1 files changed, 34 insertions, 3 deletions

diff --git a/linden/indra/newview/lightshare.cpp b/linden/indra/newview/lightshare.cpp
index 24d4139..aa5443b 100644
--- a/linden/indra/newview/lightshare.cpp
+++ b/linden/indra/newview/lightshare.cpp
@@ -100,14 +100,45 @@ void WindlightMessage::processWindlight(LLMessageSystem* msg, void**)
 {
         if( gSavedSettings.getBOOL("UseServersideWindlightSettings") )
         {
-                WindlightMessage wl = WindlightMessage(msg);
-                if( wl.isValid() )
+                WindlightMessage* wl = new WindlightMessage(msg);
+                if( wl->isValid() )
                 {
-                        wl.apply();
+                        std::string water = LLWaterParamManager::instance()->mCurParams.mName;
+                        std::string sky = LLWLParamManager::instance()->mCurParams.mName;
+
+                        // If they are using the default or region settings, just apply
+                        // the new settings, don't bother asking.
+                        if((sky == "Default" || sky == sSkyPresetName) &&
+                           (water == "Default" || water == sWaterPresetName))
+                        {
+                                wl->apply();
+                                delete wl;
+                        }
+                        else
+                        {
+                                LLNotifications::instance()
+                                        .add("ConfirmLightShare",
+                                             LLSD(), LLSD(), 
+                                             boost::bind(&applyCallback, _1, _2, wl));
+                        }
                 }
         }
 }
 
+// static
+bool WindlightMessage::applyCallback(const LLSD& notification,
+                                     const LLSD& response,
+                                     WindlightMessage* wl)
+{
+        S32 option = LLNotification::getSelectedOption(notification, response);
+        if( option == 0 )
+        {
+                wl->apply();
+        }
+        delete wl;
+        return false;
+}
+
 
 bool WindlightMessage::apply()
 {