Skip to content

Commit e175bcc

Browse files
committed
Fix Color Applicator model
Closes AE2-UEL#518
1 parent 127872b commit e175bcc

File tree

4 files changed

+47
-16
lines changed

4 files changed

+47
-16
lines changed

src/main/java/appeng/client/render/model/ColorApplicatorBakedModel.java

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,15 @@
11
package appeng.client.render.model;
22

33

4-
import com.google.common.collect.ImmutableMap;
54
import net.minecraft.block.state.IBlockState;
65
import net.minecraft.client.renderer.block.model.BakedQuad;
76
import net.minecraft.client.renderer.block.model.IBakedModel;
87
import net.minecraft.client.renderer.block.model.ItemCameraTransforms;
98
import net.minecraft.client.renderer.block.model.ItemOverrideList;
109
import net.minecraft.client.renderer.texture.TextureAtlasSprite;
1110
import net.minecraft.util.EnumFacing;
12-
import net.minecraftforge.client.model.PerspectiveMapWrapper;
13-
import net.minecraftforge.common.model.TRSRTransformation;
1411
import org.apache.commons.lang3.tuple.Pair;
12+
import org.jetbrains.annotations.NotNull;
1513

1614
import javax.annotation.Nullable;
1715
import javax.vecmath.Matrix4f;
@@ -24,15 +22,12 @@ class ColorApplicatorBakedModel implements IBakedModel {
2422

2523
private final IBakedModel baseModel;
2624

27-
private final ImmutableMap<ItemCameraTransforms.TransformType, TRSRTransformation> transforms;
28-
2925
private final EnumMap<EnumFacing, List<BakedQuad>> quadsBySide;
3026

3127
private final List<BakedQuad> generalQuads;
3228

33-
ColorApplicatorBakedModel(IBakedModel baseModel, ImmutableMap<ItemCameraTransforms.TransformType, TRSRTransformation> map, TextureAtlasSprite texDark, TextureAtlasSprite texMedium, TextureAtlasSprite texBright) {
29+
ColorApplicatorBakedModel(IBakedModel baseModel, TextureAtlasSprite texDark, TextureAtlasSprite texMedium, TextureAtlasSprite texBright) {
3430
this.baseModel = baseModel;
35-
this.transforms = map;
3631

3732
// Put the tint indices in... Since this is an item model, we are ignoring rand
3833
this.generalQuads = this.fixQuadTint(null, texDark, texMedium, texBright);
@@ -68,7 +63,7 @@ private List<BakedQuad> fixQuadTint(EnumFacing facing, TextureAtlasSprite texDar
6863
}
6964

7065
@Override
71-
public List<BakedQuad> getQuads(@Nullable IBlockState state, @Nullable EnumFacing side, long rand) {
66+
public @NotNull List<BakedQuad> getQuads(@Nullable IBlockState state, @Nullable EnumFacing side, long rand) {
7267
if (side == null) {
7368
return this.generalQuads;
7469
}
@@ -91,22 +86,22 @@ public boolean isBuiltInRenderer() {
9186
}
9287

9388
@Override
94-
public TextureAtlasSprite getParticleTexture() {
89+
public @NotNull TextureAtlasSprite getParticleTexture() {
9590
return this.baseModel.getParticleTexture();
9691
}
9792

9893
@Override
99-
public ItemCameraTransforms getItemCameraTransforms() {
94+
public @NotNull ItemCameraTransforms getItemCameraTransforms() {
10095
return this.baseModel.getItemCameraTransforms();
10196
}
10297

10398
@Override
104-
public ItemOverrideList getOverrides() {
99+
public @NotNull ItemOverrideList getOverrides() {
105100
return this.baseModel.getOverrides();
106101
}
107102

108103
@Override
109-
public Pair<? extends IBakedModel, Matrix4f> handlePerspective(ItemCameraTransforms.TransformType type) {
110-
return PerspectiveMapWrapper.handlePerspective(this, this.transforms, type);
104+
public @NotNull Pair<? extends IBakedModel, Matrix4f> handlePerspective(@NotNull ItemCameraTransforms.TransformType type) {
105+
return this.baseModel.handlePerspective(type);
111106
}
112107
}

src/main/java/appeng/client/render/model/ColorApplicatorModel.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ public IBakedModel bake(IModelState state, VertexFormat format, Function<Resourc
5555

5656
ImmutableMap<ItemCameraTransforms.TransformType, TRSRTransformation> map = PerspectiveMapWrapper.getTransforms(state);
5757

58-
return new ColorApplicatorBakedModel(baseModel, map, texDark, texMedium, texBright);
58+
return new ColorApplicatorBakedModel(baseModel, texDark, texMedium, texBright);
5959
}
6060

6161
private IBakedModel getBaseModel(IModelState state, VertexFormat format, Function<ResourceLocation, TextureAtlasSprite> bakedTextureGetter) {

src/main/resources/assets/appliedenergistics2/models/item/color_applicator_colored.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"parent": "item/generated",
2+
"parent": "appliedenergistics2:item/color_applicator_uncolored",
33
"textures": {
44
"layer0": "appliedenergistics2:items/color_applicator",
55
"layer1": "appliedenergistics2:items/color_applicator_tip_dark",
Lines changed: 37 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,42 @@
11
{
2-
"parent": "item/generated",
2+
"parent": "item/handheld",
33
"textures": {
44
"layer0": "appliedenergistics2:items/color_applicator"
5+
},
6+
"display": {
7+
"thirdperson_righthand": {
8+
"rotation": [
9+
0,
10+
90,
11+
-55
12+
],
13+
"translation": [
14+
0,
15+
4,
16+
0.5
17+
],
18+
"scale": [
19+
0.85,
20+
0.85,
21+
0.85
22+
]
23+
},
24+
"thirdperson_lefthand": {
25+
"rotation": [
26+
0,
27+
-90,
28+
55
29+
],
30+
"translation": [
31+
0,
32+
4,
33+
0.5
34+
],
35+
"scale": [
36+
0.85,
37+
0.85,
38+
0.85
39+
]
40+
}
541
}
642
}

0 commit comments

Comments
 (0)