defendable homes into loottable

conflict with loottable and defendable homes. 
 

Calling hook OnItemSplit resulted in a conflict between the following plugins: Loottable - Item.flarex2.39732035 (Item), DefendableHomes (Item.flarex2.39732037 (Item))

conflict in console above

reply from other dev below


"you need to edit the Loottable plugin to do this

[2:06 AM]

in the likeness of how I have already published changes for the Stack Modifier plugin

[2:06 AM]

https://discord.com/channels/814762568366686218/1023885899962007614/1048139046574096415

[2:07 AM]

or if the configuration of this plugin allows you to create exceptions for custom items, then you can use it, but this is if there is such an opportunity in this plugin"

 

Stack Modifier plugin
A new if statement should be specified for Item OnItemSplit(Item item, int amount):

if (item.skin == 2591851360 || item.skin == 2817854052 || item.skin == 2817854377 || item.skin == 2817854677 || item.skin == 2888602635 || item.skin == 2888602942 || item.skin == 2888603247) return null;

The updated code should look like this once you've added the new statement:

private Item OnItemSplit(Item item, int amount) { if (item.skin == 2591851360 || item.skin == 2817854052 || item.skin == 2817854377 || item.skin == 2817854677 || item.skin == 2888602635 || item.skin == 2888602942 || item.skin == 2888603247) return null; ... }

Need to change the return value for CanCombineDroppedItem hook. In all lines where return false is used, it must be replaced with return true, without changing return null



how can i add this into your loottable addon please to fix this conflict. or is there another way to do this?
thanks

 

I will fix this issue with the next update

Changed Status from Pending to Work in Progress

Changed Status from Work in Progress to Closed

Fixed it

Changed Fixed In to 1.0.24