Improve the wording of craft return messages

Author: fourZK

Entities/Activity.cpp

@@ -804,24 +804,28 @@ void Activity::Clear() {
 		}
 
 		float totalValue = orbitedCraft->GetTotalValue(0, foreignCostMult, nativeCostMult);
-		std::snprintf(messageString, sizeof(messageString), "Returned Craft + Cargo added %.0f oz to Funds!", totalValue);
 
+		std::string craftText = "Returned craft";
+		if (!orbitedCraft->IsInventoryEmpty()) { craftText += " + cargo"; }
+
+		if (totalValue > 0) {
+			m_TeamFunds[orbitedCraftTeam] += totalValue;
+			std::snprintf(messageString, sizeof(messageString), "%s added %.0f oz to funds!", craftText.c_str(), totalValue);
+		}
 		for (int player = Players::PlayerOne; player < Players::MaxPlayerCount; ++player) {
 			if (m_IsActive[player]) {
 				if (brainOnBoard && orbitedCraft == GetPlayerBrain(static_cast<Players>(player))) {
 					m_BrainEvacuated[player] = true;
-					std::snprintf(messageString, sizeof(messageString), "YOUR BRAIN HAS BEEN EVACUATED BACK INTO ORBIT!");
-				}
-
-				if (m_Team[player] == orbitedCraftTeam) {
+					g_FrameMan.ClearScreenText(ScreenOfPlayer(ScreenOfPlayer(static_cast<Players>(player))));
+					g_FrameMan.SetScreenText("YOUR BRAIN HAS BEEN EVACUATED BACK INTO ORBIT!", ScreenOfPlayer(static_cast<Players>(player)), 0, 3500);
+				} else if (m_Team[player] == orbitedCraftTeam && totalValue > 0) {
 					g_FrameMan.ClearScreenText(ScreenOfPlayer(ScreenOfPlayer(static_cast<Players>(player))));
 					g_FrameMan.SetScreenText(messageString, ScreenOfPlayer(static_cast<Players>(player)), 0, 3500);
 					m_MessageTimer[player].Reset();
 				}
 			}
 		}
 
-		m_TeamFunds[orbitedCraftTeam] += totalValue;
 		orbitedCraft->SetGoldCarried(0);
 		orbitedCraft->SetHealth(orbitedCraft->GetMaxHealth());