fix: Fixed "This won't work here! This factory building is designed for Factory floor 1. Was instead Nauvis factory floor. ". Resolves #268

Author: notnotmelon

script/factory-buildings.lua

@@ -71,8 +71,8 @@ local function which_surface_should_this_new_factory_be_placed_on(layout, buildi
     end
 end
 
-local function is_legacy_factory_floor(surface)
-  return surface.name:match("^%d+%-factory%-floor$") ~= nil
+local function is_legacy_factory_floor(surface_name)
+    return surface_name:match("^%d+%-factory%-floor$") ~= nil
 end
 
 local function set_factory_active_or_inactive(factory)
@@ -86,8 +86,11 @@ local function set_factory_active_or_inactive(factory)
 
     local function can_place_factory_here()
         -- Check if a player is trying to cheat by moving factories between surfaces.
-        if factory.inside_surface.valid and factory.inside_surface.name ~= which_surface_should_this_new_factory_be_placed_on(factory.layout, building) then
-            if not is_legacy_factory_floor(factory.inside_surface) then
+        local surface_name = factory.inside_surface.name
+        -- https://github.com/notnotmelon/factorissimo-2-notnotmelon/issues/268
+        local surface_name = surface_name:gsub("%-factory%-floor%-factory%-floor", "-factory-floor")
+        if factory.inside_surface.valid and surface_name ~= which_surface_should_this_new_factory_be_placed_on(factory.layout, building) then
+            if not is_legacy_factory_floor(surface_name) then
                 flying_text = {"factory-connection-text.invalid-placement-surface", surface_localised_name(factory.inside_surface), surface_localised_name(surface)}
                 return false, flying_text, true
             end