View Issue Details

IDProjectCategoryView StatusLast Update
0002404Industrial-Craft²general / otherpublic2018-08-11 17:26
Reporter016Nojr Assigned ToChocohead  
PrioritylowSeveritytweakReproducibilityalways
Status assignedResolutionopen 
Summary0002404: Timestamp in config on load
DescriptionI see there is a time stamp in the config that is set when the config reload? Is there a reason it's there, and if not, can it be removed?
TagsConfig
Minecraft Version1.12.2

Activities

Chocohead

2018-08-08 14:35

developer   ~0005768

The timestamp is just when it was last saved so there is an easy indicator to the last time it was changed directly by the game. It doesn't strictly have to be there as far as I'm aware, but I don't see the harm in it being there?

016Nojr

2018-08-08 15:03

reporter   ~0005770

I agree, there isnt any harm having it in there. But as a pack dev it gets a bit annoying when you end up with those configs that only shows as changed because the timestamp was changed, and this counts for the IC2 config too, there isnt anything changed in the config file when the timestamp is changed.

Not gonna be mad if it wont be removed, but I would appreciate it if it's removed.

Chocohead

2018-08-08 23:34

developer   ~0005771

I think changing it to only update the timestamp when a change is actually made is a logical fix for this. Right now it will change the stamp each launch whilst ensuring the config exists, which is certainly unnecessary. That way it retains the usefulness without updating redundantly.

016Nojr

2018-08-11 17:26

reporter   ~0005773

I would appreciate it if that could be changed to only update timestamp when config is changed.

Issue History

Date Modified Username Field Change
2018-08-08 00:07 016Nojr New Issue
2018-08-08 00:07 016Nojr Tag Attached: Config
2018-08-08 14:35 Chocohead Assigned To => Chocohead
2018-08-08 14:35 Chocohead Status new => feedback
2018-08-08 14:35 Chocohead Note Added: 0005768
2018-08-08 15:03 016Nojr Note Added: 0005770
2018-08-08 15:03 016Nojr Status feedback => assigned
2018-08-08 23:34 Chocohead Note Added: 0005771
2018-08-11 17:26 016Nojr Note Added: 0005773