Animated-Java / animated-java

A Blockbench plugin that makes complex animation a breeze in Minecraft: Java Edition.
Other
165 stars 27 forks source link

[FR] Remove Texture File Extension From JSON Export #298

Closed Dearliet closed 4 days ago

Dearliet commented 2 months ago

Current Format:

"f021e3e9-bdc5-dc67-7f7c-9be97a16fed7": {
   "name": "texture.png",
   "src": "src"
}

Proposed Format:

"f021e3e9-bdc5-dc67-7f7c-9be97a16fed7": {
   "name": "texture",
   "src": "src"
}
IanSSenne commented 2 months ago

wouldn't leaving the extension be more future proof then without in case minecraft ever adds support for other image formats?

Dearliet commented 2 months ago

In that case, you can retrieve the image format from the source field. The primary reason I want to exclude the extension is related to sanitization though.

IanSSenne commented 2 months ago

ah, that makes sense.

Dearliet commented 4 days ago

Closing this because of #338