Solved OnItemPickUp no longer working

Discussion in 'Rust Development' started by Shady757, Dec 4, 2015.

  1. Shady757

    Shady757

    private void OnItemPickup(BasePlayer player, Item item)

    This Hook no longer seems to be doing anything on my server, and it's also broken in GatherManager.

    I saw no replacement for this hook, or any mention of it being gone at all, any explanation as to why it's not working/why it's missing?
     
    Shady757, Dec 4, 2015
    #1
  2. Wulf

    Wulf Community Admin

    That was actually changed and I had forgotten about it when I was filtering through the recent changes to push. It needs to be deprecated though, but it was renamed to indicate more of what it handles.

    OnCollectiblePickup is the new hook name I believe, same parameters except swapped. OnWeaponThrown was also renamed to OnExplosiveThrown, though that wasn't intended to be the final name. Either way, they'll be deprecated.

    @Shady757, I edited my original post to reflect that the parameters of OnCollectiblePickup were actually swapped to match the other hooks.
     
    Last edited: Dec 4, 2015
    Wulf, Dec 4, 2015
    #2
  3. Shady757

    Shady757

    I'm getting this
    [Oxide] 7:32 PM [Error] Failed to call hook 'OnCollectiblePickup' on plugin 'Test' (InvalidCastException: Cannot cast from source type to de
    stination type.)

    When using the Hook at all, any idea why that is?


    EDIT: Nevermind, didn't see your edit.
     
    Shady757, Dec 4, 2015
    #3
  4. BiG|Raven

    BiG|Raven

    Is this one post the only place where this information is available? Not only does it say in the docs that OnWeaponThrown is deprecated, OnExplosiveThrown is also not described in the docs. I don't know where else to look.
     
    BiG|Raven, Apr 18, 2016
    #4
  5. Wulf

    Wulf Community Admin

    It's been here and in update notes. The Docs are somewhat outdated with hooks and have a lot I need to change on them.
     
    Wulf, Apr 18, 2016
    #5