Removal of "uninhabited" attribute from planets is not handled correctly

[warp-core]

Is there an existing issue for this?

• I have searched the existing issues

Describe the bug

Thanks to @MidnightPlugins for reporting this on Discord.

https://discord.com/channels/251118043411775489/536900466655887360/1238291650925559898

When the "uninhabited" attribute is addd to Kraken Station, it is given the "default uninhabited port" that is, it has a spaceport button, but no other services (trade, job board, bank, hiring).
This is added by:

endless-sky/source/Planet.cpp

Lines 243 to 247 in 88850c6

	// For reverse compatibility, if this planet has a spaceport but it was not custom loaded,
	// and the planet has the "uninhabited" attribute, replace the spaceport with a special-case
	// uninhabited spaceport.
	if(attributes.count("uninhabited") && HasNamedPort() && !port.CustomLoaded())
	port.LoadUninhabitedSpaceport();

When the "uninhabited" attribute is removed, there is nothing that restores the full "port" services that it should have.

Steps to Reproduce

Load the attached save file.
See that Kraken Station doesn't have all the services it should.

Expected Behavior

The planet probably shouldn't keep the default assigned uninhabited spaceport after the "uninhabited" attribute is removed.

Screenshots

No response

Link to save file

Save file provided by @MidnightPlugins.
test_kraken_attributes.txt

Operating System

Windows 10 Pro 21H2

Game Source

Built from source

Game Version

1e19d35

Additional Information

No response