Two more TODOs gone

scripts/api/entity/shiptemplatebasedobject.lua

@@ -53,6 +53,9 @@ function Entity:setTemplate(template_name)
             crew.components.internal_repair_crew = {}
         end
     end
+    if comp.shields and template.__type ~= "station" then
+        comp.shields.frequency = irandom(0, 20)
+    end
     return self
 end
 --- [DEPRECATED]

src/script/enum.h

@@ -90,11 +90,11 @@ template<> struct Convert<ScanState::State> {
     }
     static ScanState::State fromLua(lua_State* L, int idx) {
         string str = string(luaL_checkstring(L, idx)).lower();
-        if (str == "none")
+        if (str == "none" || str == "notscanned" || str == "not")
             return ScanState::State::NotScanned;
-        else if (str == "fof")
+        else if (str == "fof" || str == "friendorfoeidentified")
             return ScanState::State::FriendOrFoeIdentified;
-        else if (str == "simple")
+        else if (str == "simple" || str == "simplescan")
             return ScanState::State::SimpleScan;
         else if (str == "full" || str == "fullscan")
             return ScanState::State::FullScan;

src/spaceObjects/spaceObject.cpp

@@ -1,17 +0,0 @@
-/*TODO
-template<> void convert<ScanState::State>::param(lua_State* L, int& idx, ScanState::State& ss)
-{
-    ss = ScanState::State::NotScanned;
-    if (!lua_isstring(L, idx))
-        return;
-    string str = string(luaL_checkstring(L, idx++)).lower();
-    if (str == "notscanned" || str == "not")
-        ss = ScanState::State::NotScanned;
-    else if (str == "friendorfoeidentified")
-        ss = ScanState::State::FriendOrFoeIdentified;
-    else if (str == "simple" || str == "simplescan")
-        ss = ScanState::State::SimpleScan;
-    else if (str == "full" || str == "fullscan")
-        ss = ScanState::State::FullScan;
-}
-*/

src/spaceObjects/spaceship.cpp

@@ -29,16 +29,6 @@
 #include <SDL_assert.h>
 
 
-/*TODO
-SpaceShip::SpaceShip(string multiplayerClassName, float multiplayer_significant_range)
-: ShipTemplateBasedObject(50, multiplayerClassName, multiplayer_significant_range)
-{
-    auto shields = entity.getComponent<Shields>();
-    if (shields)
-        shields->frequency = irandom(0, BeamWeaponSys::max_frequency);
-}
-*/
-
 /*TODO
 void SpaceShip::draw3DTransparent()
 {