If you have the /dev/null in off hand you can open the gui, however the you don't supply the item in the off hand, causing the gui to not load the item in the NBTTag. I have noticed that you have decided to not allow blocks to be placed when in off hand, so maybe add that same check into the the OnItemRightClicked method or pass the item in the hand that was used.