Replace picked up item with another items

[K_Abbage]

So I am trying to make a wood type changer and I made a script to do that but it didn't work.
My code:

on pickup of oak logs :
    set {inv.%player%} to player's inventory
    set {woodcount.%player%} to amount of oak logs in {inv.%player%}
    if {woodtype.%player%} is not 1:
        if {woodtype.%player%} is 2:
            give {woodcount.%player%} of spruce logs to the player
        if {woodtype.%player%} is 3:
            give {woodcount.%player%} of birch logs to the player
        if {woodtype.%player%} is 4:
            give {woodcount.%player%} of jungle logs to the player
        if {woodtype.%player%} is 5:
            give {woodcount.%player%} of acacia logs to the player
        if {woodtype.%player%} is 6:
            give {woodcount.%player%} of dark oak logs to the player
        send {woodcount.%player%}

but when i picked up the logs nothing happened (my woodtype variable is set to 2)
any help?
p/s: also woodtype 1 is oak so i used if not for that

 

[timeis]

So I am trying to make a wood type changer and I made a script to do that but it didn't work.
My code:

on pickup of oak logs :
    set {inv.%player%} to player's inventory
    set {woodcount.%player%} to amount of oak logs in {inv.%player%}
    if {woodtype.%player%} is not 1:
        if {woodtype.%player%} is 2:
            give {woodcount.%player%} of spruce logs to the player
        if {woodtype.%player%} is 3:
            give {woodcount.%player%} of birch logs to the player
        if {woodtype.%player%} is 4:
            give {woodcount.%player%} of jungle logs to the player
        if {woodtype.%player%} is 5:
            give {woodcount.%player%} of acacia logs to the player
        if {woodtype.%player%} is 6:
            give {woodcount.%player%} of dark oak logs to the player
        send {woodcount.%player%}

but when i picked up the logs nothing happened (my woodtype variable is set to 2)
any help?
p/s: also woodtype 1 is oak so i used if not for that

Do some debugging with adding like

send "%{woodcount.%player%}%"

 

[K_Abbage]

umm it doesn't do anything..

 

[Zaneti4]

i guess player's inventory is a list