1 files changed, 56 insertions, 40 deletions

diff --git a/linden/indra/newview/llfloaterbuyland.cpp b/linden/indra/newview/llfloaterbuyland.cpp
index ca5f677..1b72d4e 100644
--- a/linden/indra/newview/llfloaterbuyland.cpp
+++ b/linden/indra/newview/llfloaterbuyland.cpp
@@ -75,7 +75,7 @@ private:
         virtual ~LLFloaterBuyLandUI();
 
         LLViewerRegion* mRegion;
-        LLParcel*               mParcel;
+        LLParcelSelectionHandle mParcel;
         bool                    mIsClaim;
         bool                    mIsForGroup;
 
@@ -146,7 +146,7 @@ public:
         static LLFloaterBuyLandUI* soleInstance(bool createIfNeeded);
 
         void setForGroup(bool is_for_group);
-        void setParcel(LLViewerRegion* region, LLParcel* parcel);
+        void setParcel(LLViewerRegion* region, LLParcelSelectionHandle parcel);
                 
         void updateAgentInfo();
         void updateParcelInfo();
@@ -205,7 +205,7 @@ static void cacheNameUpdateRefreshesBuyLand(const LLUUID&,
 
 // static
 void LLFloaterBuyLand::buyLand(
-        LLViewerRegion* region, LLParcel* parcel, bool is_for_group)
+        LLViewerRegion* region, LLParcelSelectionHandle parcel, bool is_for_group)
 {
         if(is_for_group && !gAgent.hasPowerInActiveGroup(GP_LAND_DEED))
         {
@@ -216,7 +216,7 @@ void LLFloaterBuyLand::buyLand(
         LLFloaterBuyLandUI* ui = LLFloaterBuyLandUI::soleInstance(true);
         ui->setForGroup(is_for_group);
         ui->setParcel(region, parcel);
-        ui->open();
+        ui->open();     /*Flawfinder: ignore*/
 }
 
 // static
@@ -350,7 +350,7 @@ void LLFloaterBuyLandUI::SelectionObserver::changed()
                 else {
                         ui->setParcel(
                                 gParcelMgr->getSelectionRegion(),
-                                gParcelMgr->getSelectedParcel());
+                                gParcelMgr->getParcelSelection());
                 }
         }
 }
@@ -368,7 +368,8 @@ void LLFloaterBuyLandUI::updateAgentInfo()
 
 void LLFloaterBuyLandUI::updateParcelInfo()
 {
-        mParcelValid = mParcel && mRegion;
+        LLParcel* parcel = mParcel->getParcel();
+        mParcelValid = parcel && mRegion;
         mParcelIsForSale = false;
         mParcelIsFirstLand = false;
         mParcelIsGroupLand = false;
@@ -391,50 +392,61 @@ void LLFloaterBuyLandUI::updateParcelInfo()
                 return;
         }
         
-        if (gParcelMgr->getMultipleOwners())
+        if (mParcel->getMultipleOwners())
         {
                 mCannotBuyReason = childGetText("multiple_parcels_selected");
                 return;
         }
 
+        const LLUUID& parcelOwner = parcel->getOwnerID();
         
-        const LLUUID& parcelOwner = mParcel->getOwnerID();
-        
-        mIsClaim = mParcel->isPublic();
+        mIsClaim = parcel->isPublic();
         if (!mIsClaim)
         {
-                mParcelActualArea = mParcel->getArea();
-                mParcelIsForSale = mParcel->getForSale();
-                mParcelIsFirstLand = mParcel->getReservedForNewbie();
-                mParcelIsGroupLand = mParcel->getIsGroupOwned();
-                mParcelPrice = mParcelIsForSale ? mParcel->getSalePrice() : 0;
+                mParcelActualArea = parcel->getArea();
+                mParcelIsForSale = parcel->getForSale();
+                mParcelIsFirstLand = parcel->getReservedForNewbie();
+                mParcelIsGroupLand = parcel->getIsGroupOwned();
+                mParcelPrice = mParcelIsForSale ? parcel->getSalePrice() : 0;
                 
                 if (mParcelIsGroupLand)
                 {
-                        LLUUID group_id = mParcel->getGroupID();
+                        LLUUID group_id = parcel->getGroupID();
                         mParcelGroupContribution = gAgent.getGroupContribution(group_id);
                 }
         }
         else
         {
-                mParcelActualArea = gParcelMgr->getClaimableArea();
+                mParcelActualArea = mParcel->getClaimableArea();
                 mParcelIsForSale = true;
-                mParcelPrice = mParcelActualArea * mParcel->getClaimPricePerMeter();
+                mParcelPrice = mParcelActualArea * parcel->getClaimPricePerMeter();
         }
 
         mParcelBillableArea =
                 llround(mRegion->getBillableFactor() * mParcelActualArea);
 
-        mParcelSupportedObjects = mParcel->getMaxPrimCapacity();
-        mParcelSoldWithObjects = mParcel->getSellWithObjects();
+        mParcelSupportedObjects = llround(
+                parcel->getMaxPrimCapacity() * parcel->getParcelPrimBonus()); 
+        // Can't have more than region max tasks, regardless of parcel 
+        // object bonus factor. 
+        LLViewerRegion* region = gParcelMgr->getSelectionRegion(); 
+        if(region) 
+        { 
+                S32 max_tasks_per_region = (S32)region->getMaxTasks(); 
+                mParcelSupportedObjects = llmin(
+                        mParcelSupportedObjects, max_tasks_per_region); 
+        } 
+
+        mParcelSoldWithObjects = parcel->getSellWithObjects();
+
         
-        LLVector3 center = mParcel->getCenterpoint();
+        LLVector3 center = parcel->getCenterpoint();
         mParcelLocation = llformat("%s %d,%d",
                                 mRegion->getName().c_str(),
                                 (int)center[VX], (int)center[VY]
                                 );
         
-        mParcelSnapshot = mParcel->getSnapshotID();
+        mParcelSnapshot = parcel->getSnapshotID();
         
         updateNames();
         
@@ -453,7 +465,7 @@ void LLFloaterBuyLandUI::updateParcelInfo()
 
         if (!mIsClaim)
         {
-                const LLUUID& authorizedBuyer = mParcel->getAuthorizedBuyerID();
+                const LLUUID& authorizedBuyer = parcel->getAuthorizedBuyerID();
                 const LLUUID buyer = gAgent.getID();
                 const LLUUID newOwner = mIsForGroup ? gAgent.getGroupID() : buyer;
 
@@ -492,7 +504,7 @@ void LLFloaterBuyLandUI::updateParcelInfo()
                         return;
                 }
 
-                if (gParcelMgr->hasOthersSelected())
+                if (mParcel->hasOthersSelected())
                 {
                         // Policy: Must not have someone else's land selected
                         mCannotBuyReason = childGetText("not_owned_by_you");
@@ -511,7 +523,7 @@ void LLFloaterBuyLandUI::updateParcelInfo()
         }
         */
         
-        if (mParcel->getReservedForNewbie())
+        if (parcel->getReservedForNewbie())
         {
                 if (mIsForGroup)
                 {
@@ -684,9 +696,10 @@ void LLFloaterBuyLandUI::updateWebSiteInfo()
 #endif
         
         LLXMLRPCValue keywordArgs = LLXMLRPCValue::createStruct();
-        keywordArgs.appendString("agentId", gAgent.getID().getString());
-        keywordArgs.appendString("secureSessionId",
-                gAgent.getSecureSessionID().getString());
+        keywordArgs.appendString("agentId", gAgent.getID().asString());
+        keywordArgs.appendString(
+                "secureSessionId",
+                gAgent.getSecureSessionID().asString());
         keywordArgs.appendInt("billableArea", mPreflightAskBillableArea);
         keywordArgs.appendInt("currencyBuy", mPreflightAskCurrencyBuy);
         
@@ -771,9 +784,10 @@ void LLFloaterBuyLandUI::runWebSitePrep(const std::string& password)
         }
         
         LLXMLRPCValue keywordArgs = LLXMLRPCValue::createStruct();
-        keywordArgs.appendString("agentId", gAgent.getID().getString());
-        keywordArgs.appendString("secureSessionId",
-                gAgent.getSecureSessionID().getString());
+        keywordArgs.appendString("agentId", gAgent.getID().asString());
+        keywordArgs.appendString(
+                "secureSessionId",
+                gAgent.getSecureSessionID().asString());
         keywordArgs.appendString("levelId", newLevel);
         keywordArgs.appendInt("billableArea",
                 mIsForGroup ? 0 : mParcelBillableArea);
@@ -820,7 +834,9 @@ void LLFloaterBuyLandUI::sendBuyLand()
 
 void LLFloaterBuyLandUI::updateNames()
 {
-        if (!mParcelValid)
+        LLParcel* parcelp = mParcel->getParcel();
+
+        if (!parcelp)
         {
                 mParcelSellerName = "";
                 return;
@@ -830,19 +846,19 @@ void LLFloaterBuyLandUI::updateNames()
         {
                 mParcelSellerName = "Linden Lab";
         }
-        else if (mParcel->getIsGroupOwned())
+        else if (parcelp->getIsGroupOwned())
         {
-                char groupName[DB_LAST_NAME_BUF_SIZE];
+                char groupName[DB_LAST_NAME_BUF_SIZE];  /*Flawfinder: ignore*/
                 
-                gCacheName->getGroupName(mParcel->getGroupID(), &groupName[0]);
+                gCacheName->getGroupName(parcelp->getGroupID(), &groupName[0]);
                 mParcelSellerName = groupName;
         }
         else
         {
-                char firstName[DB_LAST_NAME_BUF_SIZE];
-                char lastName[DB_LAST_NAME_BUF_SIZE];
+                char firstName[DB_LAST_NAME_BUF_SIZE];          /*Flawfinder: ignore*/
+                char lastName[DB_LAST_NAME_BUF_SIZE];           /*Flawfinder: ignore*/
                 
-                gCacheName->getName(mParcel->getOwnerID(), firstName, lastName);
+                gCacheName->getName(parcelp->getOwnerID(), firstName, lastName);
                 mParcelSellerName = llformat("%s %s", firstName, lastName);
         }
 }
@@ -939,7 +955,7 @@ BOOL LLFloaterBuyLandUI::postBuild()
         return TRUE;
 }
 
-void LLFloaterBuyLandUI::setParcel(LLViewerRegion* region, LLParcel* parcel)
+void LLFloaterBuyLandUI::setParcel(LLViewerRegion* region, LLParcelSelectionHandle parcel)
 {
         if (mTransaction &&  mTransactionType == TransactionBuy)
         {
@@ -996,7 +1012,7 @@ BOOL LLFloaterBuyLandUI::canClose()
 void LLFloaterBuyLandUI::onClose(bool app_quitting)
 {
         LLFloater::onClose(app_quitting);
-        delete this;
+        destroy();
 }