If you have a sales order come through it stores the ID of the vehicle. If the admin clears the schema & imports the fitments back in, that vehicle may have a new ID, so those orders may show the incorrect vehicle. I didn't think of this previously. I should probably store the name of the vehicle as a string instead of the ID of the vehicle.
Also if the vehicle is not found it throws an exception. I also need to rethink whether that situation should cause an exception/error, or just return null.
If you have a sales order come through it stores the ID of the vehicle. If the admin clears the schema & imports the fitments back in, that vehicle may have a new ID, so those orders may show the incorrect vehicle. I didn't think of this previously. I should probably store the name of the vehicle as a string instead of the ID of the vehicle.
Also if the vehicle is not found it throws an exception. I also need to rethink whether that situation should cause an exception/error, or just return null.