Hi @Wulf

There are a few things to think about

1. OnGrowableGather it would be nice to rename CanGrowableGather

2. OnGrowableGathered rename to OnGrowableGather and do next:

in code this strings:

player.GiveItem(item, BaseEntity.GiveItemReason.PickedUp);
Interface.CallHook("OnGrowableGathered", this, item, player);​

change to:

Interface.CallHook("OnGrowableGather", this, item, player);
player.GiveItem(item, BaseEntity.GiveItemReason.PickedUp);​

This allows it to display the item amount correctly for the player (then it was changed in hook)

3. For hook OnQuarryGather:

Now code is:

if (Interface.CallHook("OnQuarryGather", this, item) != null)
{
return;
}​

Need replace to:

if (Interface.CallHook("OnQuarryGather", this, item) != null)
{
continue;
}​

Because it placed in cycle

The first is not relevant, but there is an open PR regarding OnGrowableGathered. It won't be getting a rename, as we already have OnGrowableGather earlier in the original location.

The second can be done for OnQuarryGather.

thanks