fix planned buildings getting bumped to the front of the line on load

myk002 · myk002 · commit edd7d2f920bc · 2024-01-17T16:19:51.000-08:00
diff --git a/docs/changelog.txt b/docs/changelog.txt
@@ -66,7 +66,8 @@ Template for new versions:
 - When passing map movement keys through to the map from DFHack tool windows, also pass fast z movements (shift-scroll by default)
 - `getplants`: fix crash when processing mod-added plants with invalid materials
 - Dreamfort: fix holes in the "Inside+" burrow on the farming level (burrow autoexpand is interrupted by the pre-dug miasma vents to the surface)
-- DFHack tabs (e.g. in `gui/control-panel`) are now rendered correctly when there is a vanilla tab behind them
+- DFHack tabs (e.g. in `gui/control-panel`) are now rendered correctly when there are certain vanilla screen elements behind them
+- `buildingplan`: when you save a game and load it again, newly planned buildings are now correctly placed in line after existing planned buildings of the same type
 
 ## Misc Improvements
 - `autochop`: better error output when target burrows are not specified on the commandline
diff --git a/plugins/buildingplan/plannedbuilding.cpp b/plugins/buildingplan/plannedbuilding.cpp
@@ -77,8 +77,10 @@ static set<string> get_specials(color_ostream &out, PersistentDataItem &bld_conf
         return ret;
     vector<string> specials;
     split_string(&specials, rawstrs[2], ",");
-    for (auto & special : specials)
-        ret.emplace(special);
+    for (auto & special : specials) {
+        if (special.size())
+            ret.emplace(special);
+    }
     return ret;
 }
 
