Skip to content

Conversation

@JRoy
Copy link
Member

@JRoy JRoy commented Oct 4, 2020

Adds jail-online-time config option to only count a player's online time to the jail cap.

This also fixes memory leak in EssentialsTimer which I found in the process of testing.

Closes #429

@JRoy JRoy added type: enhancement Features and feature requests. hacktoberfest-accepted labels Oct 4, 2020
JRoy added 2 commits October 5, 2020 02:20
# Conflicts:
#	Essentials/src/com/earth2me/essentials/EssentialsTimer.java
#	Essentials/src/com/earth2me/essentials/Settings.java
#	Essentials/src/com/earth2me/essentials/User.java
#	Essentials/src/com/earth2me/essentials/UserData.java
#	Essentials/src/com/earth2me/essentials/commands/Commandtogglejail.java
#	Essentials/src/com/earth2me/essentials/utils/DateUtil.java
Octol1ttle
Octol1ttle previously approved these changes Oct 5, 2020
@JRoy JRoy added this to the 2.18.2 milestone Oct 5, 2020
@JRoy JRoy changed the title Add option to only count online jailed time Add option to only count online jailed time + Memory Leak Fix Oct 22, 2020
@mdcfe mdcfe self-assigned this Nov 9, 2020
@JRoy JRoy modified the milestones: 2.18.2, 2.19.0 Nov 9, 2020
# Conflicts:
#	Essentials/src/main/java/com/earth2me/essentials/User.java
@mdcfe mdcfe changed the title Add option to only count online jailed time + Memory Leak Fix Add option to only count online jailed time Jan 8, 2021
@mdcfe mdcfe merged commit 026d279 into EssentialsX:2.x Jan 8, 2021
@JRoy JRoy deleted the feature/jail-online-only branch January 11, 2021 13:59
ressidell pushed a commit to ressidell/Essentials that referenced this pull request Apr 27, 2025
Adds `jail-online-time` config option to only count a player's online time to the jail cap.

This also fixes memory leak in `EssentialsTimer` which I found in the process of testing.

Closes EssentialsX#429.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement Features and feature requests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to jail for online time only

4 participants