Default to anychest if permission given #76

New Issue

Closed

opened 2017-08-31 10:32:09 -04:00 by lifehome · 4 comments

lifehome commented 2017-08-31 10:32:09 -04:00 (Migrated from github.com)

Copy Link

Can it be configurable default to anychest (on), if a player is given certain/specific permission?

Can it be configurable default to anychest (on), if a player is given certain/specific permission?

Jikoo commented 2017-09-01 04:46:02 -04:00 (Migrated from github.com)

Copy Link

Easy enough. I'll try to get to it this weekend, but no promises - my backlog's been piling up due to work.

Easy enough. I'll try to get to it this weekend, but no promises - my backlog's been piling up due to work.

🎉 1

Jikoo commented 2017-11-05 20:08:22 -05:00 (Migrated from github.com)

Copy Link

Two days, two months, pretty much the same thing.

Nodes are openinv.anychest.default and openinv.silentchest.default currently, default false to keep existing behavior.
I'm not sure that I'll keep the paths the same - currently all OpenInv permissions have OpenInv containing caps. Most permissions plugins I've touched in recent years only deal in lowercase anyway, but it would be nice to keep everything simple and consistent just in case. May end up lowercasing everything and adding backwards compatibility perms. I'll make a decision and a release tomorrow.

Two days, two months, pretty much the same thing. Nodes are `openinv.anychest.default` and `openinv.silentchest.default` currently, default false to keep existing behavior. I'm not sure that I'll keep the paths the same - currently all OpenInv permissions have OpenInv containing caps. Most permissions plugins I've touched in recent years only deal in lowercase anyway, but it would be nice to keep everything simple and consistent just in case. May end up lowercasing everything and adding backwards compatibility perms. I'll make a decision and a release tomorrow.

Jikoo commented 2017-11-07 18:06:02 -05:00 (Migrated from github.com)

Copy Link

Having reviewed the code, I'm not going to bother with lower case perms for now. It's simple enough to support both, but I don't want to do twice as many permissions checks just in case. When we next hit a major version bump I'll try to remember it. Nodes will be OpenInv.any.default and OpenInv.silent.default.

Having reviewed the code, I'm not going to bother with lower case perms for now. It's simple enough to support both, but I don't want to do twice as many permissions checks just in case. When we next hit a major version bump I'll try to remember it. Nodes will be `OpenInv.any.default` and `OpenInv.silent.default`.

Jikoo commented 2017-11-07 18:26:49 -05:00 (Migrated from github.com)

Copy Link

3.3.1

[3.3.1](https://github.com/lishid/OpenInv/releases/tag/3.3.1)

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#76

No description provided.