[Solved] Extending Placement Restricter

[lub4095]

So what I am trying to do is add Oil as a new resource and make an Oil Refinery which can only be placed on oil tiles.
EntityType.cp - I added Oil at the bottom

Code Select Expand


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using UnityEngine; // Always needed
using AI;        // Needed when you do something with the AI
using Sound;     // Needed when you do something with the Sound
using UI;        // Needed when you do something with the GUI


namespace SpaceOdyssey
{
    public enum EntityType
    {
        Undefined,
        Pawn,
        Item,
        Apparel,
        Equipment,
        Filth,
        Fire,
        Corpse,
        Debris,
        Puddle,
        DropPod,
        Plant,
        Floor,
        Roof,
        Mote,
        Blueprint,
        Projectile,
        ItemSkyFalling,
        Rock,
        SteamGeyser,
        BuildingComplex,
        BuildingInert,
        Frame,
        Wall,
        Door,
        Bed,
        Building_WorkTable,
        Building_Table,
        Building_PowerPlantGeothermal,
        Building_PowerConduit,
        Building_Battery,
        Building_CommsConsole,
        Building_ResearchBench,
        Building_Turret,
        Building_Chair,
        Oil
    }
}


PlacementRestricter_OnOil.cp - I edited PlacementRestricter_OnSteamGeyser to fit the oil resource

Code Select Expand


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using UnityEngine; // Always needed
using AI;        // Needed when you do something with the AI
using Sound;     // Needed when you do something with the Sound
using UI;        // Needed when you do something with the GUI

namespace SpaceOdyssey
{
    public class PlacementRestricter_OnOil : PlacementRestricter
    {
        public override AcceptanceReport CanPlaceWithRestriction(EntityDef checkingDef, IntVec3 loc, IntRot rot)
        {
            Thing thing = Find.Grids.ThingAt(loc, EntityType.Oil);
            if (thing == null || thing.Position != loc)
            {
                return "Must be placed directly on Oil.";
            }
            return true;
        }
    }
}


So when I try to compile it I get these errors:
The best overloaded method match for 'ThingGrid.ThingAt(IntVec3, EntityType)' has some invalid arguments

Argument 2: cannot convert from 'SpaceOdyssey.EntityType' to 'EntityType' in PlacementRestricter_OnOil.cp line 16. What am I doing wrong? Please note that I am a complete newbie at these stuff and it is my first attempt to make an assembly.

[WorldOfIllusion]

Firstly, you will need to actually make an XML defined thing that is the oil (similar to how a steam geyser works).
Secondly, I don't believe the game will use your extended version of EntityType, it will just use the default version (and thus any calls you make to it won't work). This is what's causing your error (you are passing your version of EntityType which is different to the games EntityType).

[lub4095]

Firstly, you will need to actually make an XML defined thing that is the oil (similar to how a steam geyser works).
Secondly, I don't believe the game will use your extended version of EntityType, it will just use the default version (and thus any calls you make to it won't work). This is what's causing your error (you are passing your version of EntityType which is different to the games EntityType).

So how do I add a new entity type since I cannot overwrite the existing one from the game?

[WorldOfIllusion]

at this point I don't think you can, but I don't see why you need to?
If you make an XML defined 'oil well' or something like that and add it to worldgen you can check for it being present using its defName for example.

[lub4095]

at this point I don't think you can, but I don't see why you need to?
If you make an XML defined 'oil well' or something like that and add it to worldgen you can check for it being present using its defName for example.

How is that possible? The only XML tag that I found which checks where you can build was:

Code Select Expand


<placementRestricters>
      <li>PlacementRestricter_OnSteamGeyser</li>
</placementRestricters>


[WorldOfIllusion]

That XML tag is defining a DLL with the same name. You can implement your own placement restricter (just as you were trying to originally), that doesn't check based on EntityType (for example look at the next to hopper accepter).

[Cala13er]

Code Select Expand

public class PlacementRestricter_OnOil : PlacementRestricter
{
    public override AcceptanceReport CanPlaceWithRestriction(EntityDef Building, IntVec3 loc, IntRot rot)
    {
        ThingDef thingDef = ThingDef.Named("Oil");
        Thing thing = Find.Grids.ThingAt(loc, thingDef);
        if (thing == null || thing.Position != loc)
        {
            return "Must be placed directly on a steam geyser.";
        }
        return true;
    }
}

I quickly changed it for you, this should work.

[lub4095]

Code Select Expand

public class PlacementRestricter_OnOil : PlacementRestricter
{
    public override AcceptanceReport CanPlaceWithRestriction(EntityDef Building, IntVec3 loc, IntRot rot)
    {
        ThingDef thingDef = ThingDef.Named("Oil");
        Thing thing = Find.Grids.ThingAt(loc, thingDef);
        if (thing == null || thing.Position != loc)
        {
            return "Must be placed directly on a steam geyser.";
        }
        return true;
    }
}

I quickly changed it for you, this should work.

Haha it worked! And I thought I had to overwrite BuildingProperties (to add public bool wantsOilAdjacent;) and create a new Building_Oil class... I thank you! I am really new to this so it's really hard for me at the moment.

[Architect]

you might want to change what it returns first though Otherwise you're gonna get spammed with messages about steam geysers when you try to place it not on oil.

[Cala13er]

you might want to change what it returns first though Otherwise you're gonna get spammed with messages about steam geysers when you try to place it not on oil.

What?

[WorldOfIllusion]

you might want to change what it returns first though Otherwise you're gonna get spammed with messages about steam geysers when you try to place it not on oil.

Wait, steam geysers don't produce oil? *updates mod*

[Architect]

return "Must be placed directly on a steam geyser.";

should that not have something more to do with oil?

[lub4095]

you might want to change what it returns first though Otherwise you're gonna get spammed with messages about steam geysers when you try to place it not on oil.

Don't worry I changed that.