Skip to content
This repository has been archived by the owner on Apr 16, 2022. It is now read-only.

For sale #117

Open
ErgoAsh opened this issue Mar 10, 2014 · 9 comments
Open

For sale #117

ErgoAsh opened this issue Mar 10, 2014 · 9 comments

Comments

@ErgoAsh
Copy link
Contributor

ErgoAsh commented Mar 10, 2014

Hello. If there is possibility to make claims sell-able and buy-able? This should be similar to Towny /plot forsale Amount /plot claim. I really need this.

@markhughes
Copy link
Owner

This is a cool idea, however I may make this into a different plugin completely.

@ErgoAsh
Copy link
Contributor Author

ErgoAsh commented Mar 10, 2014

Why different plugin? Anyway, thanks for support that.

@markhughes
Copy link
Owner

Just it's something a bit more major, I might include it as part of FactionsPlus - I haven't decided yet.

@ErgoAsh
Copy link
Contributor Author

ErgoAsh commented Mar 10, 2014

Maybe you'll approach me those reason, I'll try understand.

For now, I don't see reason to make that external (I don't think that will affect on performance).

@ErgoAsh
Copy link
Contributor Author

ErgoAsh commented Mar 15, 2014

And what about taxes and plot(claim) taxes? It would be awesome too.

Why your plugin use PlayerMoveEvent so often?

@markhughes
Copy link
Owner

PlayerMoveEvent is ran whenever a player moves. But this is only doing one small check.

@ErgoAsh
Copy link
Contributor Author

ErgoAsh commented Mar 15, 2014

I know but I meant - for why you use that? It isn't small, for now FactionsPlus have one of the largest of CPU (more than Factions or Heroes, only lower than NCP). So, why?

http://prntscr.com/30ysz6

@markhughes
Copy link
Owner

PlayerMoveEvent is hugely used by most plugins, and it's used by FactionsPlus a lot due to the amount of features I give to you. It's average time is still very low? It's count is obviously going to be high, because as the event name suggests, it is fired whenever a player moves.

Do you have Jails enabled? Then the Jail listener is enabled. This has an event for all actions by a player (including PlayerMoveEvent) - which does a small check to see if they're in their jail and to move them to jail.

Do you have Disguise checks enabled? Then the Disguise listener is enabled. This checks for when a player is moving in and out of Faction land to un-disguise them.

Do you have Creative Gates checks enabled? Then the creative gates listener is enabled, this is experimental and checks if a player is in a creative gate.

Do you have Announcements enabled? Then the Announce Listener is enabled. This checks when a player enters their land to send the announcement to them.

As well as other features.. for example, in the core PlayerMoveEvent is used to check if the player has factionsplus.flightinownterritory.

@ErgoAsh
Copy link
Contributor Author

ErgoAsh commented Mar 17, 2014

That's response I understand. Now I'm only must wait for adding new awesome features.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants