[Committed] MoveItem & AllowedClasses || Housing Vault

[Shursan]

Hi,

The patch for moveitem allow the client to move these allowedclasses in a vault inventory .. etc.

The patch for housing vault fix a recent problem, if you have a vault somewhere in a house, shutdown the server, and restart it, the vault add to the database each time .. if you have 50 vault, next restart you have 100

[Tolakram]

Oh hey, look at that. I'll review these later, but I think I like the vault fix.

[Tolakram]

Committed. Thanks for fixes!

[Shursan]

More good with the following patch, the last appears to broke the new vault to register in the db

[Tolakram]

That patch does not work with latest SVN. Are you sure there's a bug? Not sure what you are trying to fix.

[Shursan]

i think there's a bug yes, (or a svn error possibly when i have updated ..)
If you remove the first patch, test to load and shutdown your serer with one vault register in the db. Next restart if there's really an error, your chest are duplicated.

I test that tomorrow with a fresh svn, before the commit of my patch.

[Shursan]

bug confirmed with a svn rev 2718.

how to reproduce:

1: run this querry, juste for verify the count
select * from househookpointitem where itemtemplateid like "%vault";
2 start a dolserver rev 2718
3 when up shut down it, remake the query, all vault are duplicated in the db

my first patch fix it, so he's not reliable, because he don't allow new vault to be added to the db.

with my second patch all is fixed. i commit it.