Very likely I was going to refactor this class, but then stopped since
it's abstract and thus can't have any constructor... but apparently I
forgot to revert that line.
Closes#377
If a player logs out then logs back in, and the task in the
PlayerJoinEvent rescues him first and the player has sat in the rescue
location for over 10 ticks (which should be within another portal), then
this task will also attempt to rescue the player. Because it was using
metadata in the log message though, it failed but also printed a
stacktrace since the player's metadata was already removed.
- Claim expiration collects all players (uuids) that own claims, and
iterates this set instead of iterating through every single claim.
- Individual claims had a "last modified" timestamp, so this method was
valid before, but makes no sense now since expiration is based on player
inactivity instead of claim inactivity.
New config option under the advanced node allows controlling the
frequency each player in the aforementioned set is checked for
inactivity.
[20:58:37] RoboMWM: hmmm it's a bit annoying how CB cuts off the
trailing 0 in my versions
[20:58:49] RoboMWM: e.g. 16.10 displays as 16.1
[20:59:02] +Choco: That seems... stupid? lol
[20:59:12] RoboMWM: very
[20:59:27] RoboMWM: plugin.yml inside jar reveals it to be correct
[20:59:43] RoboMWM: and afaik this is supposed to be a string so idk why
it's doing that
[21:00:49] RoboMWM: apparently wrapping it with single quotes does the
job
[21:01:33] +Choco: Maybe it assumes it's a double then converts it to a
String
[21:02:04] RoboMWM: I guess >_> since I stuck a test.16.10 without
quotes and it's all there
[21:02:09] RoboMWM: thank you yaml
[21:02:22] +Choco: What if you were to do 1.1.10?
[21:02:30] +Choco: Because that's not a valid number, maybe it would
assume String
[21:02:33] RoboMWM: the second dot probably makes it a string
[21:02:51] RoboMWM: maybe I should do 16.1O
[21:02:57] +Choco: lol that's cheating
1. Claim creation event... this is fired once all the claim creation checks have been passed. It can be cancelled. It contains the claim and the creating player
2. Claim modification event - this is fired as the claim is modified - it cannot be cancelled. It contains the claim and the modifier...which can be null.
Use Case:
This could be used so other plugins can hook gp and perform an action if a claim is created or changed. Something as simple as logging claim creations/modifications and deletions.
* Ignore Eclipse build/project artifacts.
* Fix Maven convention: rename src/tests/* to src/test/*
* Update Maven test dir. Remove directory specs for src, test, & rsrc.
${project.basedir} is the default directory. Only add directory
spec if different from convention.
According to #97 hyperconomy loads after GP. See #97 for discussion and etc. on a proper way to fix/avoid requiring this for any other economy plugins that happen to load before GP does.
* Configurable time until sieged claim is resecured
Fixes#113.
Makes the five minute period after winning a siege, during which the claim is
not secured, customizable.
* Make trapped words fully configurable.
Before, it contained hardcoded checks; the value of
`Messages.TrappedChatKeyword` was in addition to the hardcoded ones.
Make it more flexible, so you can override them entirely (and provide as many as
you want, comma-separated), and an empty value disables the message totally.
(Default value is 'trapped;stuck' to provide the same behaviour the hardcoded
checks did.)
In case they actually aren't and lagged out (perhaps due to chunk
loading), they'll have a few seconds to be able to walk out before being
automatically rescued on reconnect.
Also logs when a player has been rescued
Prevent players from creating a claim if _any_ part of their claim includes a worldguard region they cannot build in. (Previous behavior only prevented claiming if the entire claim was within a worldguard region the player could not build inside.)
Show the claim size to those with griefprevention.seeclaimsize when right-clicking with the investigation tool
(stick by default) - it's not exactly private information, the claim boundaries
are visualised so it would be easy to just count the blocks or compare the
coords of the corners to find out.
Necessarily this shouldn't be an issue anyways since the platform is
obsidian.
Also, if you have claims enable in the end, you should theoretically be
already protected.
- Witches can no longer apply potions to claimed animals (as if they
were an untrusted player)
- Protect claimed villagers as well
- Don't cancel entire event when we're protecting a single animal.
Not tested, but other than maintaining SQL storage "compatibility" I've
removed all usages of it. It doesn't seem to be used at all anyways
other than to be stored since commit
f935806b45 removed its purpose.
Addresses a request in #22
Seems like this check was intended for the
"portalCreationRequiresPermission," so when he threw in the portal trap
check, he didn't account for this check possibly breaking in worlds
where claims aren't enabled.
- Cancel existing rescue tasks on successful teleport
- Send the player a message when they are rescued (unfinished)
Someday, I will make this OOP-compliant
If _really_ desired, I could add an option for this. But more often than
not, unintended deaths could occur from players disconnecting from a
snowball fight.
Since we're also checking if location is the same, this really doesn't make much sense to store in the first place, unless of course we do some more string manipulation to store just the message and not the location.
This feature has value in anti-spam, but too many new GP installers are
confused by it. Disabling by default to save support cost - also I
figure at least some are installing the plugin, seeing this as a "bug"
and uninstalling without bothering to get educated, and that's bad for
their players.
Supports the new SpleefArena flag in GriefPrevention flags. Fired when
a player is prevented from breaking a block. If cancelled by another
plugin, the break will be allowed.
