Extending item_db.conf

[Cydh 9]

First, please don't feel strange if I make new post as suggestion here. Thank you.

 

We know that on of many differences between Hercules and other emu is on item_db.conf. I feel 'the new' format file & item_db structure can provides some item_db values that currently are separated, like item_buyingstore.txt, item_delay.txt, item_trade.txt, item_nouse.txt, item_stack.txt, and item_avail.txt.

 

I have suggestion to move those separated file to item_db.conf format:

• Buyingstore: Yes //item_buyingstore.txt
• Delay: <duration> //item_delay.txt
• Trade: <flag>,<override> //item_trade.txt
• Nouse: <flag>,<override> //item_nouse.txt
• Stack: <amount>,<flag> //or sperated values instead of use a flag, <inventory>,<cart>,<storage>,<guild storage> //item_stack.txt
• Sprite: <item_id> //item_avail.txt
  

When those files are merged, then, the sql table for item_db can stores those values too.

And maybe, later some db files can be merge to be one file like mob_db, skill, item creation, etc.

 

(actually I'm doing something on 'other' emu related SQL - TXT databases then found that Herc's item_db.conf the only one can merges some separated item dbs)

Edited May 11, 2014 by Cydh

[Dastgir 1246]

First, please don't feel strange if I make new post as suggestion here. Thank you.

 

We know that on of many differences between Hercules and other emu is on item_db.conf. I feel 'the new' format file & item_db structure can provides some item_db values that currently are separated, like item_buyingstore.txt, item_delay.txt, item_trade.txt, item_nouse.txt, item_stack.txt, and item_avail.txt.

 

I have suggestion to move those separated file to item_db.conf format:

• Buyingstore: Yes //item_buyingstore.txt
  
• Delay: <duration> //item_delay.txt
  
• Trade: <flag>,<override> //item_trade.txt
  
• Nouse: <flag>,<override> //item_nouse.txt
  
• Stack: <amount>,<flag> //item_stack.txt
  
• Sprite: <item_id> //item_avail.txt
  

When those files are merged, then, the sql table for item_db can stores those values too.

And maybe, later some db files can be merge to be one file like mob_db, skill, item creation, etc.

 

(actually I'm doing something on 'other' emu related SQL - TXT databases then found that Herc's item_db.conf the only one can merges some separated item dbs)

Everyone is welcome to give suggestion

+1 For Additional options on item_db.conf.

[hemagx 69]

+1 i like this.

[Mystery 594]

That doesn't seem like a bad idea. We can get ride of the other text files and just add it into the item_db o-o

[evilpuncker 504]

that would be awesome!

[Mumbles 193]

I hope you realise we aren't bothered by any affiliation with (or the mention of) *Athena or other emulators. Suggestions are welcome, and I personally appreciate this one; something I've always looked forward to seeing was an eventual merge or centralisation of item data into the item_db.conf file.

 

+1, all the way. Thanks for bringing it to our attention, Cydh. :^)

[Dastgir 1246]

we would have to save those item_trade and all those in sql's too if we decide to centralize them. which is good. FluxCP can be extended , to show that its available on buyingstore, and its non-tradable, and all those..

[Cydh 9]

we would have to save those item_trade and all those in sql's too if we decide to centralize them. which is good. FluxCP can be extended , to show that its available on buyingstore, and its non-tradable, and all those..

yup, that's my idea in "other world", so we can put some sperated item db files for item_db table, can be used for CP.

I already done this sql thing in "other world", also fine in testing.

 

Seems, I can try that on Herc too, and try make .diff or PR maybe

[Dastgir 1246]

 

we would have to save those item_trade and all those in sql's too if we decide to centralize them. which is good. FluxCP can be extended , to show that its available on buyingstore, and its non-tradable, and all those..

yup, that's my idea in "other world", so we can put some sperated item db files for item_db table, can be used for CP.

I already done this sql thing in "other world", also fine in testing.

 

Seems, I can try that on Herc too, and try make .diff or PR maybe

I have the server-side part ready(so, i maybe will open pull request), to read all things, but SQL part is not done till now...

[Jguy 215]

This is an excellent idea, Cydh.

 

Thank you for the suggestion. Not sure why we didn't think of it before

[Cydh 9]

This is an excellent idea, Cydh.

 

Thank you for the suggestion. Not sure why we didn't think of it before

 

welcome. /ok

 

btw, thank's to Dastgir who added it here. /no1

[Mystery 594]

Yup. Being moved since it has been implemented:

 

Part 1: https://github.com/HerculesWS/Hercules/commit/49500a5e4055673d082b6f9c4d92cd4a9f04c230

 

Part 2: https://github.com/HerculesWS/Hercules/commit/244985a25e5147517f53c887877f7ab9e4977ad4