more TODO

Author: 1whohears

src/main/java/com/onewhohears/dscombat/data/aircraft/DefaultAircraftPresets.java

@@ -685,7 +685,7 @@ public class DefaultAircraftPresets {
 			.addEmptySlot("internal_2", SlotType.INTERNAL)
 			.addEmptySlot("internal_3", SlotType.ADVANCED_INTERNAL)
 			.setSlotItem("internal_3", ModItems.AXCEL_TRUCK_RADAR.getId())
-			.lockSlot("internal_3")
+			.lockSlot("internal_3") // TODO 2.5 make axcel truck radar removable
 			.addEmptySlot("internal_4", SlotType.ADVANCED_INTERNAL)
 			.build();
 

src/main/java/com/onewhohears/dscombat/data/parts/PartSlot.java

@@ -138,6 +138,7 @@ public boolean removePartData(EntityAircraft plane) {
 	public boolean isCompatible(PartData data) {
 		//System.out.println("is "+data+" compatible with "+this);
 		if (data == null) return false;
+		// TODO 7.4 check for duplicates
 		SlotType[] types = data.getCompatibleSlots();
 		for (int i = 0; i < types.length; ++i) if (types[i] == getSlotType()) return true;
 		return false;
@@ -159,6 +160,8 @@ public SlotType getSlotType() {
 		return type;
 	}
 
+	// FIXME 3 slot type should be read an a string id in preset data and not an int
+	// TODO 2.4 light/med/heavy turret slot type
 	public static enum SlotType {
 		SEAT("slottype.dscombat.seat"),
 		WING("slottype.dscombat.wing"),

src/main/java/com/onewhohears/dscombat/entity/aircraft/EntityAircraft.java

@@ -182,6 +182,10 @@ public abstract class EntityAircraft extends Entity implements IEntityAdditional
 	// TODO 3.1 vehicle armor plating that reduces damage 
 	// TODO 3.2 reduce damage passengers receive based on armor. make it configurable. especially explosive damage.
 	// TODO 8.2 creative mode middle click should give aircraft item
+	// TODO 5.2 bitchin betty
+	// TODO 5.3 ir/anti radar targeting tone
+	// TODO 5.4 aircraft breaks apart when damaged
+	// FIXME refactor EntityAircraft to EntityVehicle
 
 	public EntityAircraft(EntityType<? extends EntityAircraft> entityType, Level level, 
 			AircraftPreset defaultPreset,
@@ -285,6 +289,7 @@ else if (!AircraftPresets.get().has(preset)) {
 		item = ap.getItem();
 		CompoundTag presetNbt = ap.getDataAsNBT();
 		if (!nbt.getBoolean("merged_preset")) nbt.merge(presetNbt);
+		// FIXME 2 possibly all slot stats but definitely just slot positions should be forced and not configurable
 		partsManager.read(nbt);
 		weaponSystem.read(nbt);
 		radarSystem.read(nbt);

src/main/java/com/onewhohears/dscombat/entity/weapon/AntiRadarMissile.java

@@ -42,7 +42,7 @@ public boolean dieIfNoTargetOutsideTickRange() {
 	protected List<ARTarget> targets = new ArrayList<ARTarget>();
 
 	protected void findARTarget() {
-		// FIXME 3.2 make anti radar missile target entity type configurable so entities other mod entities can be targeted
+		// IDEA 7 make anti radar missile target entity type configurable so entities other mod entities can be targeted
 		targets.clear();
 		// planes
 		List<EntityAircraft> planes = level.getEntitiesOfClass(

src/main/java/com/onewhohears/dscombat/init/ModEntities.java

@@ -149,9 +149,10 @@ public static void register(IEventBus eventBus) {
 	 * OTHER
 	 * Weather Balloon
 	 * 
-	 * IDEA 9 AI for planes/turrets or an auto pilot module
+	 * IDEA 9.1 AI for planes/turrets or an auto pilot module
 	 * AI turret controllers need time and resources invested to level up
 	 * it would be funny if they were villagers
+	 * IDEA 9.2 pillager outposts with air defense
 	 */
 
 	public static final RegistryObject<EntityType<EntitySeat>> SEAT = ENTITIES.register("seat", 

src/main/java/com/onewhohears/dscombat/init/ModItems.java

@@ -62,7 +62,8 @@ public ItemStack makeIcon() {
 	// DISKS
 
 	// IDEA 8.1 Jupiter Missiles and Anadyr from blowback ost
-	// IDEA 8.1 disk 911?
+	// IDEA 8.2 disk 911?
+	// IDEA 8.3 the missile knows where it is disk
 
 	// PARTS
 	public static final RegistryObject<Item> TI83 = ITEMS.register("ti83", 
@@ -101,6 +102,7 @@ public ItemStack makeIcon() {
 			() -> new ItemRepairTool(20, 5));
 	public static final RegistryObject<Item> THICK_WRENCH = ITEMS.register("thick_wrench", 
 			() -> new ItemRepairTool(200, 5));
+	// TODO 2.3 parachute/emergency ejection
 
 	// CREATIVE WANDS
 	public static final RegistryObject<Item> NO_CONSUME_WAND = ITEMS.register("no_consume_wand",