Storage Multiplier

[Donnchadh]

All,

taking a break from battle code, but I will return, I promise.

Just speculating on the interest for the ability to control the storage sizes via config?

Thinking:

define("STORAGE_MULTIPLIER",5);
define("STORAGE_BASE",800*STORAGE_MULTIPLIER);

And then strip out hard coded 800's (default maxstore and maxcrop) and modify the display and update engine to multiply the building attri variable by STORAGE_MULTIPLIER.

Only reason I've done it is because I'm running 100x server for battle code testing and it's handy to have the extra storage.

It would have to be carefully released as I can't set the config.php entry in GitHub, so people would need to be able to read.

Thoughts?

Regards,
Donnchadh.

[MentaL]

[masyaf]

It would be good, because i have to change 40 values if i want to increase size of warehouse

[rectifier]

Yes it can be a good idea

[Dzoki]

Love the idea!

[arafarn12]

you are DPStokes ? *[]*

[ZZJHONS]

you are DPStokes ? *[]*

DPStokes is Donnchadh

define("STORAGE_MULTIPLIER",5);
define("STORAGE_BASE",800*STORAGE_MULTIPLIER);

It would have to be carefully released as I can't set the config.php entry in GitHub, so people would need to be able to read.

Put it in install/data/constant_format.tpl

[Donnchadh]

DPStokes is Donnchadh



Put it in install/data/constant_format.tpl

Yes, but that wouldn't help for people who are doing upgrades that miss the config.php change and get their storage completely broken when everything gets multiplied by 0

Donnchadh aka DPStokes.

Git'd.

If someone would like to update the installer to cater, that would be useful.

Regards,
Donnchadh.

[rectifier]

STORAGE MULTIPLIER has wrong colculation

Look at this then you'll understand :






Don't forget that my STORAGE MULTIPLIER is x2
 

[skysea]

STORAGE MULTIPLIER has wrong colculation

Look at this then you'll understand :






Don't forget that my STORAGE MULTIPLIER is x2
 

I also present the same problem

[Donnchadh]

What are we looking at? There's a blank between "understand" and "Don't"

Donn.

[rectifier]

What are we looking at? There's a blank between "understand" and "Don't"

Donn.

It seems your browser have problem or some your browser setting changed

We can see pic here

Try check your problem dude
 

[Donnchadh]

OK, I can see the images when I browse this site from my home PC.

So, question.... did you start these villages from scratch after the storage multiplier was configured? ALL the storage multiplier does is (a) multiply the current and next storage values displayed in the granary/warehouse and (b) multiplies the difference between storage levels when you upgrade and adds that to maxstore and maxcrop.

If you had existing villages with maxstore/maxcrop set for standard non-multiplied levels, it will not amend the existing values, merely increment them at upgrade time with the multiplied differentials.

None of this code is being written to retrofit existing installations. Feel free to write a recountStorage function.

Regards,
Donnchadh.

[rectifier]

OK, I can see the images when I browse this site from my home PC.

So, question.... did you start these villages from scratch after the storage multiplier was configured? ALL the storage multiplier does is (a) multiply the current and next storage values displayed in the granary/warehouse and (b) multiplies the difference between storage levels when you upgrade and adds that to maxstore and maxcrop.

If you had existing villages with maxstore/maxcrop set for standard non-multiplied levels, it will not amend the existing values, merely increment them at upgrade time with the multiplied differentials.

None of this code is being written to retrofit existing installations. Feel free to write a recountStorage function.

Regards,
Donnchadh.

I made a new village and upgrade it again then i saw there is no problem

But i found where is the previous problem

When we upgrade for example warehouse then if we instantly finish it with gold then problem will be appear (If finish it instantly, Storage will act like normal)

You can test

Even one special problem in latest release of github (Downloaded in this post date)

If you look at map you'll understand the problem of new release

[Donnchadh]

DOH! forgot to update the finishAll function in Building.php.

Giting shortly.

Donn.

[rectifier]

DOH! forgot to update the finishAll function in Building.php.

Giting shortly.

Donn.

OK, Good luck

[Tookie]

Where to add this? :P
If its works right? Or its just Beta fixed?

[Dzoki]

Where to add this? :P
If its works right? Or its just Beta fixed?

It's on github-.-

[Tookie]

It's on github-.-

Sorry then..