What is the difference between /characters/{character_id}/planets/{planet_id}/pins.schematic_id and /characters/{character_id}/planets/{planet_id}/pins.factory_details.schematic_id ?

[deadlybulb]

Question

Planetary Interaction endpoint defines a pin with structure:

get_characters_character_id_planets_planet_id_pin {
  latitude (number): latitude number ,
  longitude (number): longitude number ,
  pin_id (integer): pin_id integer ,
  type_id (integer): type_id integer ,
  schematic_id (integer, optional): schematic_id integer ,
  extractor_details (get_characters_character_id_planets_planet_id_extractor_details, optional): 
  extractor_details object ,
  factory_details (get_characters_character_id_planets_planet_id_factory_details, optional): 
  factory_details object ,
  contents (Array[get_characters_character_id_planets_planet_id_content], optional): contents array ,
  install_time (string, optional): install_time string ,
  expiry_time (string, optional): expiry_time string ,
  last_cycle_start (string, optional): last_cycle_start string
}

Here schematic_id is optional. So far so good. However, there is an optional factory_details object which also defines a schematic_id. Is there a difference? Are they the same? Can one be present without the other?

[ghost]

Hrm, this might be a straight up bug. Based on what I'm seeing it looks like the factory_details variation is never used. Has anyone ever seen a pin with a factory_details value?

[deadlybulb]

When I ported this bit for EveKit, only the top-level schematic_id was ever populated so I ended up ignoring factory details.

[EvilLivesHere]

I've also never noticed a pin with the factory_details object filled in. It seems to just populate the pin itself rather than add a sub object.

[warlof]

bump - for me it seems to be an inconsistency.

It seems the factory_details property is a duplicate of root level schematic_id property whereas factory_details is never used compared to schematic_id.

My suggestion would be to either remove factory_details part or improve the object by returning input/output and cycle information and remove the schematic_id from the root level.