Field "none" should not exist in "display" field of block model

[merak48763]

Due to the fix of #1733, the display field in block model now also accepts none as a key.
However the transform of none cannot be customized in game.

// net.minecraft.client.renderer.block.model.ItemTransforms#getTransform
return switch (itemDisplayContext) {
  case ItemDisplayContext.THIRD_PERSON_LEFT_HAND -> this.thirdPersonLeftHand;
  case ItemDisplayContext.THIRD_PERSON_RIGHT_HAND -> this.thirdPersonRightHand;
  case ItemDisplayContext.FIRST_PERSON_LEFT_HAND -> this.firstPersonLeftHand;
  case ItemDisplayContext.FIRST_PERSON_RIGHT_HAND -> this.firstPersonRightHand;
  case ItemDisplayContext.HEAD -> this.head;
  case ItemDisplayContext.GUI -> this.gui;
  case ItemDisplayContext.GROUND -> this.ground;
  case ItemDisplayContext.FIXED -> this.fixed;
  default -> ItemTransform.NO_TRANSFORM;
};

ItemDisplayContext.NONE goes to the default branch and yields a constant transform config.

[misode]

I'm a bit confused. Should the mentioned fix just be reverted back? @merak48763

[merak48763]

In my opinion it should be two separated enums, ItemDisplayContext and CustomizableItemDisplayContext.
CustomizableItemDisplayContext excludes none and replaces ItemDisplayContext in the keys of block model display field.