search

BlockySurvival / issue-tracker

A non-code repo for tracking issues w/ the Blocky Survival minetest server
12 stars 0 forks source link

PChests ownership and tubelib pushers #273

Closed Lanhild closed 3 years ago

Lanhild commented 3 years ago

Portable chests content can be moved with a tubelib pusher, even if the chest isn't yours.

oversword commented 3 years ago

sigh alright I'll add a checkbox so this issue doesn't keep coming up

fluxionary commented 3 years ago

sigh alright I'll add a damn checkbox so you'll shut up about it

please be a bit more civil, this is a public repo. you guys should be setting an example for the other players.

luk3yx commented 3 years ago

This is related to https://github.com/BlockySurvival/issue-tracker/issues/235, I'm not sure if it counts as a duplicate since this issue involves protected chests not locked chests.

oversword commented 3 years ago

Actually @luk3yx this is a duplicate of https://github.com/BlockySurvival/issue-tracker/issues/267

oversword commented 3 years ago

Ugh. Looks like this is actually custom code in bls_custom that makes pchests interact with tubelib, may still be possible, but messy. I would very much like this to come after https://github.com/fluxionary/hook/pull/1 so I at least have some space to work with without hacking the formspec too much. Maybe tubelib functionality should be added in as a PR to the main / flux's fork of the project, and this new switch can come with it

Lanhild commented 3 years ago

(No other chests affected, only pchests)

oversword commented 3 years ago

This can be closed, flux just merged my changes