1 files changed, 73 insertions, 31 deletions

diff --git a/linden/indra/newview/llfloatersellland.cpp b/linden/indra/newview/llfloatersellland.cpp
index 5fac819..cc7df3b 100755
--- a/linden/indra/newview/llfloatersellland.cpp
+++ b/linden/indra/newview/llfloatersellland.cpp
@@ -55,15 +55,15 @@ private:
         virtual ~LLFloaterSellLandUI();
 
         LLViewerRegion* mRegion;
-        LLParcel*               mParcel;
-        bool                    mParcelIsForSale;
-        bool                    mSellToBuyer;
-        bool                    mChoseSellTo;
-        S32                             mParcelPrice;
-        S32                             mParcelActualArea;
-        LLUUID                  mParcelSnapshot;
-        LLUUID                  mAuthorizedBuyer;
-        bool                    mParcelSoldWithObjects;
+        LLParcelSelectionHandle mParcelSelection;
+        bool                                    mParcelIsForSale;
+        bool                                    mSellToBuyer;
+        bool                                    mChoseSellTo;
+        S32                                             mParcelPrice;
+        S32                                             mParcelActualArea;
+        LLUUID                                  mParcelSnapshot;
+        LLUUID                                  mAuthorizedBuyer;
+        bool                                    mParcelSoldWithObjects;
         
         void updateParcelInfo();
         void refreshUI();
@@ -87,17 +87,24 @@ public:
         
         static LLFloaterSellLandUI* soleInstance(bool createIfNeeded);
 
-        bool setParcel(LLViewerRegion* region, LLParcel* parcel);
+        bool setParcel(LLViewerRegion* region, LLParcelSelectionHandle parcel);
+
+private:
+        class SelectionObserver : public LLParcelObserver
+        {
+        public:
+                virtual void changed();
+        };
 };
 
 // static
 void LLFloaterSellLand::sellLand(
-        LLViewerRegion* region, LLParcel* parcel)
+        LLViewerRegion* region, LLParcelSelectionHandle parcel)
 {
         LLFloaterSellLandUI* ui = LLFloaterSellLandUI::soleInstance(true);
         if (ui->setParcel(region, parcel))
         {
-                ui->open();
+                ui->open();             /* Flawfinder: ignore */
         }
 }
 
@@ -115,12 +122,20 @@ LLFloaterSellLandUI* LLFloaterSellLandUI::soleInstance(bool createIfNeeded)
                 sInstance->center();
         }
         
+
+        static SelectionObserver* parcelSelectionObserver = NULL;
+        if (!parcelSelectionObserver)
+        {
+                parcelSelectionObserver = new SelectionObserver;
+                gParcelMgr->addObserver(parcelSelectionObserver);
+        }
+
         return sInstance;
 }
 
 LLFloaterSellLandUI::LLFloaterSellLandUI()
 :       LLFloater("Sell Land"),
-        mRegion(0), mParcel(0)
+        mRegion(0)
 {
 }
 
@@ -132,11 +147,27 @@ LLFloaterSellLandUI::~LLFloaterSellLandUI()
         }
 }
 
+void LLFloaterSellLandUI::SelectionObserver::changed()
+{
+        LLFloaterSellLandUI* ui = LLFloaterSellLandUI::soleInstance(false);
+        if (ui)
+        {
+                if (gParcelMgr->selectionEmpty())
+                {
+                        ui->close();
+                }
+                else {
+                        ui->setParcel(
+                                gParcelMgr->getSelectionRegion(),
+                                gParcelMgr->getParcelSelection());
+                }
+        }
+}
 
 void LLFloaterSellLandUI::onClose(bool app_quitting)
 {
         LLFloater::onClose(app_quitting);
-        delete this;
+        destroy();
 }
 
 BOOL LLFloaterSellLandUI::postBuild()
@@ -152,17 +183,18 @@ BOOL LLFloaterSellLandUI::postBuild()
         return TRUE;
 }
 
-bool LLFloaterSellLandUI::setParcel(LLViewerRegion* region, LLParcel* parcel)
+bool LLFloaterSellLandUI::setParcel(LLViewerRegion* region, LLParcelSelectionHandle parcel)
 {
-        if (!parcel) // || !can_agent_modify_parcel(parcel)) // can_agent_modify_parcel was deprecated by GROUPS
+        if (!parcel->getParcel()) // || !can_agent_modify_parcel(parcel)) // can_agent_modify_parcel was deprecated by GROUPS
         {
                 return false;
         }
 
         mRegion = region;
-        mParcel = parcel;
+        mParcelSelection = parcel;
         mChoseSellTo = false;
 
+
         updateParcelInfo();
         refreshUI();
 
@@ -171,14 +203,17 @@ bool LLFloaterSellLandUI::setParcel(LLViewerRegion* region, LLParcel* parcel)
 
 void LLFloaterSellLandUI::updateParcelInfo()
 {
-        mParcelActualArea = mParcel->getArea();
-        mParcelIsForSale = mParcel->getForSale();
+        LLParcel* parcelp = mParcelSelection->getParcel();
+        if (!parcelp) return;
+
+        mParcelActualArea = parcelp->getArea();
+        mParcelIsForSale = parcelp->getForSale();
         if (mParcelIsForSale)
         {
                 mChoseSellTo = true;
         }
-        mParcelPrice = mParcelIsForSale ? mParcel->getSalePrice() : 0;
-        mParcelSoldWithObjects = mParcel->getSellWithObjects();
+        mParcelPrice = mParcelIsForSale ? parcelp->getSalePrice() : 0;
+        mParcelSoldWithObjects = parcelp->getSellWithObjects();
         if (mParcelIsForSale)
         {
                 childSetValue("price", mParcelPrice);
@@ -197,16 +232,16 @@ void LLFloaterSellLandUI::updateParcelInfo()
                 childSetValue("sell_objects", "none");
         }
 
-        mParcelSnapshot = mParcel->getSnapshotID();
+        mParcelSnapshot = parcelp->getSnapshotID();
 
-        mAuthorizedBuyer = mParcel->getAuthorizedBuyerID();
+        mAuthorizedBuyer = parcelp->getAuthorizedBuyerID();
         mSellToBuyer = mAuthorizedBuyer.notNull();
 
         if(mSellToBuyer)
         {
                 LLString name;
-                char firstname[MAX_STRING];
-                char lastname[MAX_STRING];
+                char firstname[MAX_STRING];             /* Flawfinder: ignore */
+                char lastname[MAX_STRING];              /* Flawfinder: ignore */
                 gCacheName->getName(mAuthorizedBuyer, firstname, lastname);
                 name.assign(firstname);
                 name.append(" ");
@@ -238,13 +273,16 @@ void LLFloaterSellLandUI::setBadge(const char* id, Badge badge)
 
 void LLFloaterSellLandUI::refreshUI()
 {
+        LLParcel* parcelp = mParcelSelection->getParcel();
+        if (!parcelp) return;
+
         LLTextureCtrl* snapshot = LLViewerUICtrlFactory::getTexturePickerByName(this, "info_image");
         if (snapshot)
         {
                 snapshot->setImageAssetID(mParcelSnapshot);
         }
 
-        childSetText("info_parcel", mParcel->getName());
+        childSetText("info_parcel", parcelp->getName());
         childSetTextArg("info_size", "[AREA]", llformat("%d", mParcelActualArea));
 
         LLString price_str = childGetValue("price").asString();
@@ -377,7 +415,7 @@ void LLFloaterSellLandUI::doSelectAgent(void *userdata)
 void LLFloaterSellLandUI::callbackAvatarPick(const std::vector<std::string>& names, const std::vector<LLUUID>& ids, void* data)
 {       
         LLFloaterSellLandUI* floaterp = (LLFloaterSellLandUI*)data;
-        LLParcel* parcel = floaterp->mParcel;
+        LLParcel* parcel = floaterp->mParcelSelection->getParcel();
 
         if (names.empty() || ids.empty()) return;
         
@@ -402,7 +440,10 @@ void LLFloaterSellLandUI::doCancel(void *userdata)
 void LLFloaterSellLandUI::doShowObjects(void *userdata)
 {
         LLFloaterSellLandUI* self = (LLFloaterSellLandUI*)userdata;
-        send_parcel_select_objects(self->mParcel->getLocalID(), RT_SELL);
+        LLParcel* parcel = self->mParcelSelection->getParcel();
+        if (!parcel) return;
+
+        send_parcel_select_objects(parcel->getLocalID(), RT_SELL);
 
         LLNotifyBox::showXml("TransferObjectsHighlighted",
                                                  callbackHighlightTransferable,
@@ -420,7 +461,7 @@ void LLFloaterSellLandUI::doSellLand(void *userdata)
 {
         LLFloaterSellLandUI* self = (LLFloaterSellLandUI*)userdata;
 
-        LLParcel* parcel = self->mParcel;
+        LLParcel* parcel = self->mParcelSelection->getParcel();
 
         // Do a confirmation
         if (!parcel->getForSale())
@@ -472,7 +513,8 @@ void LLFloaterSellLandUI::onConfirmSale(S32 option, void *userdata)
                 return;
         }
 
-        LLParcel* parcel = self->mParcel;
+        LLParcel* parcel = self->mParcelSelection->getParcel();
+        if (!parcel) return;
 
         // can_agent_modify_parcel deprecated by GROUPS
 //      if (!can_agent_modify_parcel(parcel))
@@ -499,7 +541,7 @@ void LLFloaterSellLandUI::onConfirmSale(S32 option, void *userdata)
         }
 
         // Send update to server
-        gParcelMgr->sendParcelPropertiesUpdate( parcel, LLFloaterLand::sRequestReplyOnUpdate );
+        gParcelMgr->sendParcelPropertiesUpdate( parcel );
 
         self->close();
 }