Update to Terraria 1.4

[dark22]

Would it be possible to upgrade to the new version?

Apparently the items are not being saved when placed in the chest...

Cheers!

[Stealownz]

Hi @dark22, we're still trying to currently fully update the plugin. There's been some changes to the handling of plugins, and the order of the hooks being triggered. I've found that modifying the TShock Bouncer code somewhat resolves the items not being saved issue.

https://github.com/Pryaxis/TShock/blob/general-devel/TShockAPI/Bouncer.cs#L932

            if (args.Player.TPlayer.chest != id)
            {
                TShock.Log.ConsoleDebug("Bouncer / OnChestItemChange rejected from chest mismatch from {0}", args.Player.Name);
                args.Handled = true;
                return;
            }

to

            if (args.Player.TPlayer.chest != id)
            {
                TShock.Log.ConsoleDebug("Bouncer / OnChestItemChange rejected from chest mismatch from {0}", args.Player.Name);
                //args.Handled = true;
                //return;
            }

This change is not ideal, but I'm still trying to figure out the underlying changes that caused this.

[dark22]

Thank you for the temporary solution and quick response.

I had forgotten to activate Debug in the TShock settings and I didn't think it would be the problem. In addition to the referred line, testing here, apparently "RangeChecks", are also preventing communication from the client to the server. Keeping this option off, or commenting out the line, works normally.

https://github.com/Pryaxis/TShock/blob/general-devel/TShockAPI/Bouncer.cs#L951

Now that the chest limit has been increased to 8,000, would it be possible to disable Protector chests completely, if the user wishes? I know that it is currently possible with as 'MaxProtectorChests = 0' and not create the additional chests, but still animation of the chests when opening and closing are lost.