{"action":"opened","issue":{"url":"https://api.github.com/repos/minetest/minetest/issues/4813","repository_url":"https://api.github.com/repos/minetest/minetest","labels_url":"https://api.github.com/repos/minetest/minetest/issues/4813/labels{/name}","comments_url":"https://api.github.com/repos/minetest/minetest/issues/4813/comments","events_url":"https://api.github.com/repos/minetest/minetest/issues/4813/events","html_url":"https://github.com/minetest/minetest/issues/4813","id":191819015,"number":4813,"title":"Create per player volatile inventory and enable callbacks for player inventory","user":{"login":"est31","id":8872119,"avatar_url":"https://avatars.githubusercontent.com/u/8872119?v=3","gravatar_id":"","url":"https://api.github.com/users/est31","html_url":"https://github.com/est31","followers_url":"https://api.github.com/users/est31/followers","following_url":"https://api.github.com/users/est31/following{/other_user}","gists_url":"https://api.github.com/users/est31/gists{/gist_id}","starred_url":"https://api.github.com/users/est31/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/est31/subscriptions","organizations_url":"https://api.github.com/users/est31/orgs","repos_url":"https://api.github.com/users/est31/repos","events_url":"https://api.github.com/users/est31/events{/privacy}","received_events_url":"https://api.github.com/users/est31/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-11-26T18:13:25Z","updated_at":"2016-11-26T18:13:25Z","closed_at":null,"body":"Issue #4403 has outilined some issues where modders used the detached inventory system to have per-player data. This is not what the system was designed for, and PR #4812 has only moved the problem into the server code, removing it from the network, which alleviates the problem, but doesn't solve it properly.\r\n\r\nThe reasons given for using detached inventories for stuff like bags or armor were the callbacks. Its true that we only have callbacks for detached inventories and for node metadata inventories.\r\n\r\nSo, the idea to properly resolve this is to add callbacks for the player inventory too.\r\n\r\nAnother use case for detached inventories was the creative inventory. It makes no sense to store it on disk, therefore it might make sense to have a second per-player inventory named like \"player_volatile:<name>\" which gets deleted once the player leaves the game (and mods can put stuff inside in the on_join callback)."}}
{"actor":{"display_login":"est31"}}