search

stujones11 / minetest-3d_armor

Visible player armor & wielded items for minetest
Other
56 stars 98 forks source link

beter smart_inventory integration #83

Closed bell07 closed 7 years ago

bell07 commented 7 years ago

The first implementation did not work as expected because of wrong understanding on my site. A technical update was missed and it was not a good idea just press the page button in background if armor breaks.

The new implementation uses new implemented Methods in smart_inventory. For your documentation: smart_inventory.get_page_state("player", name) get the runtime object (=state) of the page "player" from inventory of player name. If the page is disabled, the method will return nil. The page contains a new hidden element "update_hook" that does all needed update things if submitted.

Previous it was the "inventory" runtime object with just submit the page selection button "player_button", if exists.

stujones11 commented 7 years ago

I do not really understand how your mod works so I will just take your word this fixes something. Besides,I like commits that remove more than they add ;-)