xazax

I agree with the changes, one should be able to reload questdb.

Agreed.

Of course, we can not avoid overflows completely, but 25 days sounds way to frequent to me =(.

Since we dropped TXT, I think it is not too hard to make it possible to modify offline player's variables, or read them. In fact, you can already do that with query_sql, however built in support would be much more elegant of course.

Maybe the best would be to have 3 different configurations? 1. max weight of the cart 2. max weight that is used for damamge 3. how much the weight influence the damage

Max level is also one of the most requested stuff to be moved to a config.

I'm happy with most of the staff and thanks for your great work, but I have only one concern. The tickcount overflow in 32 bit systems. Honestly I doubt that we need that big resolution for timers, so maybe it would worth to sacrafice a bit accuracy to avoid those overflows. Anyways huuuuge +1 for moving SQL operations to another thread.

Just a small notice, it would be nice to have an additional optional parameter to loadnpc to be able to avoid triggering oninit script just in case.

r2140

2118 O.o something strange happening.

r2108

r2089

2081

2049

This means, not just this patch, but also the current vending system needs some rework.

2036

I modified my diff, so it also checks flag (which should hold information about which button was pressed), now it should only trigger if one hit OK.

Gepard is working on reorganizing our svn. It will involve fixing this issue.

2021

I would prefer @loadnpc to trigger the OnInit label by default, and add an option (parameter) to avoid that. Opinion?

2017

I wonder how the official servers hande this. Maybe they send ZC_OPENSTORE with negative or 0 number, which forces the client to close the window? Just a blind guess though, I hope somebody will make some packet capture on official servers

Strange. What happens, if you do a @refresh before step 4? And I wonder if there are some novending cells on the map if they have the same effect (they should). Maybe it is yet another issue with supporting "new" clients.

Yeah. That also reflects, users probably want OnInit triggered by loadnpc.

2014