Aidan Taylor || Client	2017/12/11 (16:31)
Hi,

The configuration editor when saved removes all values not in the template, this breaks a lot of config values that we cannot have an editor for.

The file type affected is ini files.

It should not be removing any settings.

IP Address: 81.98.18.155

Luis Alvarenga || Staff	2017/12/11 (16:41)
This only happens if you have "Read variable values from file" unchecked. When it's unchecked it will use the config file template every time you use the config editor.

Luis Alvarenga
TCAdmin - The Game Hosting Control Panel
http://www.tcadmin.com

How would you rate this reply? 	Poor 						 Excellent


Aidan Taylor || Client	2017/12/13 (06:39)
Hi,

That's already checked

Luis Alvarenga || Staff	2017/12/16 (08:47)
Hi, can you send the link to a game server where I can see this and the admin password?

Luis Alvarenga
TCAdmin - The Game Hosting Control Panel
http://www.tcadmin.com

How would you rate this reply? 	Poor 						 Excellent


Aidan Taylor || Client	2017/12/17 (18:20)
Hi,

https://tcadmin.logicserver.co.uk:8881/Interface/GameHosting/ServiceHome.aspx?serviceid=813

Test Game.ini config: https://pastebin.com/U5sLnEdt

User: tcatmp
Password: 43n5n34o523@

Aidan Taylor || Client	2017/12/17 (18:21)
Sorry password is 43n5n34o523@A

Luis Alvarenga || Staff	2017/12/17 (23:14)
When you click on config editor for Game.ini it says it can't find values for lots of variables. That is why it uses the default template.

If it can read the values it won't show this message and it will keep the original file when writing the updated values.

The following variables could not be read: AutoPvEStartTimeSeconds, AutoPvEStopTimeSeconds, BabyCuddleGracePeriodMultiplier, BabyCuddleLoseImprintQualitySpeedMultiplier, BabyFoodConsumptionSpeedMultiplier, BabyImprintingStatScaleMultiplier, bAutoPvETimer, bAutoPvEUseSystemTime, bDisableDinoRiding, bDisableDinoTaming, bDisableLootCrates, bDisableStructurePlacementCollision, bFlyerPlatformAllowUnalignedDinoBasing, bIncreasePvPRespawnInterval, bOnlyAllowSpecifiedEngrams, bPvEAllowTribeWar, bPvEAllowTribeWarCancel, bPvEDisableFriendlyFire, CropDecaySpeedMultiplier, CropGrowthSpeedMultiplier, CustomRecipeEffectivenessMultiplier, CustomRecipeSkillMultiplier, DinoHarvestingDamageMultiplier, DinoTurretDamageMultiplier, HairGrowthSpeedMultiplier, IncreasePvPRespawnIntervalBaseAmount, IncreasePvPRespawnIntervalCheckPeriod, IncreasePvPRespawnIntervalMultiplier, KickIdlePlayersPeriod, MaxNumberOfPlayersInTribe, PlayerHarvestingDamageMultiplier, PoopIntervalMultiplier, PvPZoneStructureDamageMultiplier, StructureDamageRepairCooldown, MaxTribeLogs. The default value for these variables and the default template will be used. Previous configuration will be overwritten.

Luis Alvarenga
TCAdmin - The Game Hosting Control Panel
http://www.tcadmin.com

How would you rate this reply? 	Poor 						 Excellent


Aidan Taylor || Client	2017/12/18 (19:52)
Hi,

The game removes all variables set to default values.

It needs to ignore that and use set values.

Luis Alvarenga || Staff	2017/12/19 (12:00)
I see it keeps the current values of variables that it is able to find. These values are shown in the text editor and they are loaded when using the config editor:
GlobalSpoilingTimeMultiplier=5
GlobalItemDecompositionTimeMultiplier=3
GlobalCorpseDecompositionTimeMultiplier=3
ResourceNoReplenishRadiusPlayers=1
ResourceNoReplenishRadiusStructures=1
MatingIntervalMultiplier=0.1
EggHatchSpeedMultiplier=16.0
BabyMatureSpeedMultiplier=16.0
LayEggIntervalMultiplier=0.3
BabyCuddleIntervalMultiplier=0.10

Luis Alvarenga
TCAdmin - The Game Hosting Control Panel
http://www.tcadmin.com

How would you rate this reply? 	Poor 						 Excellent


Aidan Taylor || Client	2017/12/19 (12:02)
Hi,

I'm aware of that.

The fact is it's removing everything that it cannot find, which should not be the case. These config files contain variables that cannot have a configuration editor for them.

This never happened before the last update.

Luis Alvarenga || Staff	2017/12/19 (12:24)
It has always worked that way. You can downgrade if you want to test. If TCAdmin can't find a value for all variables used in the config editor it's going to use the default template.

Luis Alvarenga
TCAdmin - The Game Hosting Control Panel
http://www.tcadmin.com

How would you rate this reply? 	Poor 						 Excellent


Aidan Taylor || Client	2017/12/19 (13:48)
Hi,

I'm not wasting my time downgrading.

Can we please have an option to disable using the default template when not all variables are found, while still being able to read the variables from the file?
The configs are ini files, you can read all the variables and merge the files together with minimal code.

TCAdmin also seems to have issues in configs such as Terraria, where a config value may use two tcadmin variables, e.g. world save which needs the path to the world, and the world name.

When will we see more performance improvements?
As I'm sure you've noticed our panel is unbearable and we've already established in the past the hardware we use for TCAdmin and our Database is overly sufficient. We are grateful that you got some load times from 8 second to 4 seconds, but that's an extremely slow speed. Websites should respond in milliseconds, not whole seconds.

Will there be an updated version of the WHMCS plugin? It's very buggy in WHMCS 7.3/7.4, the username field is never filled in, suspensions/terminations constantly fail and it's still using mysql instead of Carbon (using mysql instead of mysqli too).

Aidan Taylor || Client	2018/03/09 (13:07)
Hi,

This really really needs fixed.

Why can't the configuration editor just read the file and only change what it has to?

This is happening with all configs, json, xml, ini. It's not that hard to fix this.