Suggestions for inventories #92

New Issue

Closed

opened 2018-10-08 03:58:53 -04:00 by ArtelGG · 7 comments

ArtelGG commented 2018-10-08 03:58:53 -04:00 (Migrated from github.com)

Copy Link

Here's a list of things you guys should try to add for online players' inventories:

1. Show what the player has attached to their cursor;
   

1.a Just in case it's not already a thing, the player's off-hand slot for newer versions.

2. Show what the user has in their crafting inventory, including the result slot;
   
3. Have an item formatted with the user's potion effects, UUID, perhaps their hunger and saturation, gamemode, location, etc.

**Here's a list of things you guys should try to add for online players' inventories:** 1. Show what the player has attached to their cursor;  > 1.a Just in case it's not already a thing, the player's off-hand slot for newer versions. 2. Show what the user has in their crafting inventory, including the result slot;  3. Have an item formatted with the user's potion effects, UUID, perhaps their hunger and saturation, gamemode, location, etc.

RoboMWM commented 2018-10-08 04:20:04 -04:00 (Migrated from github.com)

Copy Link

Are users really hiding things in these (temporary) slots?

3. IMO is outside the scope of the plugin, this generic info can be found in many other plugins.

Are users really hiding things in these (temporary) slots? 3. IMO is outside the scope of the plugin, this generic info can be found in many other plugins.

ArtelGG commented 2018-10-08 04:27:24 -04:00 (Migrated from github.com)

Copy Link

There are instances where users who are trying to hide an item will pick the item up in their cursor to hide it from others (most likely staff) trying to find said item until they give up. Also, some modified clients can make it so the four crafting slots can be changed to store items rather than drop them when the inventory is closed.

There are instances where users who are trying to hide an item will pick the item up in their cursor to hide it from others (most likely staff) trying to find said item until they give up. Also, some modified clients can make it so the four crafting slots can be changed to store items rather than drop them when the inventory is closed.

Jikoo commented 2018-10-08 05:55:16 -04:00 (Migrated from github.com)

Copy Link

1 and 2 I'll look into. 3 is included in #54 to a degree, probably more in line for an addon. Offhand slot is already handled.
I won't show the crafting result slot - that should not be insertable by the client. If it is, that's really on the server implementation to prevent. I believe actual anticheat features to be outside of the scope of OI.

1 and 2 I'll look into. 3 is included in #54 to a degree, probably more in line for an addon. Offhand slot is already handled. I won't show the crafting result slot - that should not be insertable by the client. If it is, that's really on the server implementation to prevent. I believe actual anticheat features to be outside of the scope of OI.

ArtelGG commented 2018-10-08 06:35:09 -04:00 (Migrated from github.com)

Copy Link

1 and 2 I'll look into. 3 is included in #54 to a degree, probably more in line for an addon. Offhand slot is already handled.
I won't show the crafting result slot - that should not be insertable by the client. If it is, that's really on the server implementation to prevent. I believe actual anticheat features to be outside of the scope of OI.

Sounds good. If the crafting result slot is just a simple grab of a slot ID think you could implement it anyway?

> > > 1 and 2 I'll look into. 3 is included in #54 to a degree, probably more in line for an addon. Offhand slot is already handled. > I won't show the crafting result slot - that should not be insertable by the client. If it is, that's really on the server implementation to prevent. I believe actual anticheat features to be outside of the scope of OI. Sounds good. If the crafting result slot is just a simple grab of a slot ID think you could implement it anyway?

RoboMWM commented 2018-10-08 14:42:56 -04:00 (Migrated from github.com)

Copy Link

There's no reason to show the crafting result slot if the client can't store an item in there, which I see no reason why it would be able to.

There's no reason to show the crafting result slot if the client can't store an item in there, which I see no reason why it would be able to.

👍 1

Jikoo commented 2018-10-19 08:38:57 -04:00 (Migrated from github.com)

Copy Link

Sounds good. If the crafting result slot is just a simple grab of a slot ID think you could implement it anyway?

After some thought, no.
It looks simple, but then you start to consider what do we do if the viewer modifies the result. Do we rewrite all of OpenInv's click handling so the viewer can't modify the result slot? Do we "craft" the item, and do we then have to manually include all crafting logic? Gets messy and time-consuming very quickly.

The server should be rejecting inserts to that slot, so there's no need for it. It's not a simple addition.

>Sounds good. If the crafting result slot is just a simple grab of a slot ID think you could implement it anyway? After some thought, no. It looks simple, but then you start to consider what do we do if the viewer modifies the result. Do we rewrite all of OpenInv's click handling so the viewer can't modify the result slot? Do we "craft" the item, and do we then have to manually include all crafting logic? Gets messy and time-consuming very quickly. The server should be rejecting inserts to that slot, so there's no need for it. It's not a simple addition.

Jikoo commented 2024-07-03 10:34:17 -04:00 (Migrated from github.com)

Copy Link

OI now offers a view of the crafting slots and cursor. If the player's inventory is the expected size, a view of the result is also available. The result view is only for clarity of what the player is doing, and is not editable.

OI now offers a view of the crafting slots and cursor. If the player's inventory is the expected size, a view of the result is also available. The result view is only for clarity of what the player is doing, and is not editable.

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

master

feature/cursorcrafting

uuid-rework

4.1.8

4.1.7

4.1.6

4.1.5

4.1.4

4.1.3

4.1.2

4.1.1

4.1.0

4.0.8

4.0.7

4.0.6

4.0.5

4.0.4

4.0.3

4.0.2

4.0.1

4.0.0

3.3.7

3.3.6

3.3.5

3.3.4

3.3.4-A1

3.3.3

3.3.2

3.3.1

3.3.0

3.2.1

3.2.0

3.1.2

3.1.1

3.1.0

3.0.6

3.0.5

3.0.4

3.0.3

3.0.2

3.0.1

3.0.0

2.5.3

2.5.3-alpha-1

2.5.2

2.5.1

2.5.0

2.4.9

2.4.8

2.4.7

2.4.5

2.4.4

2.4.3

2.4.2

2.4.1

2.4.0

2.3.0

2.2.10

2.2.9

2.2.8

2.2.7

2.2.6

2.2.5

Labels

Clear labels

Bug

Cannot Reproduce

Enhancement

Fixed

Invalid

Low Priority

Need More Information

Pending

Question

No Label

Enhancement

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

minster586

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: minster586/OpenInv#92

No description provided.