Mark internal classes as internal #4

Merged
SKBotNL merged 3 commits from master into master 2025-06-11 15:29:54 -04:00
26 changed files with 31 additions and 32 deletions

No files matched your search

@@ -10,7 +10,7 @@ import org.bukkit.entity.Player
import org.bukkit.inventory.ItemStack
import kotlin.math.abs
object AutoCompress {
internal object AutoCompress {
fun compress(player: Player) {
DiamondBankOG.scope.launch {
val worldName = player.world.name
@@ -7,7 +7,7 @@ import org.bukkit.Material
import org.bukkit.entity.Item
import org.bukkit.entity.Player
object AutoDeposit {
internal object AutoDeposit {
fun deposit(player: Player, item: Item) {
val worldName = player.world.name
if (worldName != "world" && worldName != "world_nether" && worldName != "world_the_end") {
@@ -4,7 +4,7 @@ import org.bukkit.configuration.file.YamlConfiguration
import java.io.File
import kotlin.properties.Delegates
object Config {
internal object Config {
lateinit var prefix: String
lateinit var postgresUrl: String
lateinit var postgresUser: String
@@ -12,7 +12,7 @@ import org.bukkit.Bukkit
import org.bukkit.plugin.ServicePriority
import org.bukkit.plugin.java.JavaPlugin
class DiamondBankOG : JavaPlugin() {
internal class DiamondBankOG : JavaPlugin() {
companion object {
val scope = CoroutineScope(Dispatchers.Default + SupervisorJob())
@@ -106,6 +106,10 @@ class DiamondBankOG : JavaPlugin() {
}
override fun onDisable() {
redis.shutdown()
if (isPostgreSQLInitialised()) postgreSQL.pool.disconnect().get()
scope.cancel()
runBlocking {
@@ -113,10 +117,6 @@ class DiamondBankOG : JavaPlugin() {
}
transactionLock.removeAllLocks()
redis.shutdown()
if (isPostgreSQLInitialised()) postgreSQL.pool.disconnect().get()
}
}
}
@@ -3,7 +3,7 @@ package net.trueog.diamondbankog
import net.trueog.diamondbankog.PostgreSQL.PlayerShards
import java.util.*
object ErrorHandler {
internal object ErrorHandler {
class EconomyException(message: String) : RuntimeException(message)
/**
@@ -25,7 +25,7 @@ import org.bukkit.inventory.ItemStack
import org.bukkit.scheduler.BukkitRunnable
@OptIn(DelicateCoroutinesApi::class)
class Events : Listener {
internal class Events : Listener {
@EventHandler(priority = EventPriority.HIGHEST)
fun onPlayerJoin(event: PlayerJoinEvent) {
if (DiamondBankOG.economyDisabled) {
@@ -15,7 +15,7 @@ import org.bukkit.scheduler.BukkitRunnable
import kotlin.math.ceil
import kotlin.math.floor
object InventoryExtensions {
internal object InventoryExtensions {
private suspend fun Inventory.withdrawShards(shards: Int): Int {
val removeMap = runOnMainThread {
this.removeItem(Shard.createItemStack(shards))
@@ -6,7 +6,7 @@ import org.bukkit.Bukkit
import kotlin.coroutines.resume
import kotlin.coroutines.resumeWithException
object MainThreadBlock {
internal object MainThreadBlock {
@OptIn(ExperimentalCoroutinesApi::class)
suspend fun <T> runOnMainThread(block: () -> T): T {
return if (Bukkit.isPrimaryThread()) {
@@ -3,7 +3,7 @@ package net.trueog.diamondbankog
import kotlinx.coroutines.future.await
import java.util.*
object PlayerPrefix {
internal object PlayerPrefix {
private val legacyToMmMap = mapOf(
"0" to "<black>",
"1" to "<dark_blue>",
@@ -3,7 +3,7 @@ package net.trueog.diamondbankog
import io.lettuce.core.RedisClient
import io.lettuce.core.RedisConnectionException
class Redis {
internal class Redis {
private val redisClient: RedisClient = RedisClient.create(Config.redisUrl)
/**
@@ -10,7 +10,7 @@ import org.bukkit.inventory.ShapedRecipe
import org.bukkit.inventory.ShapelessRecipe
import org.bukkit.persistence.PersistentDataType
object Shard {
internal object Shard {
val namespacedKey = NamespacedKey(DiamondBankOG.plugin, "diamondshard")
/**
@@ -4,7 +4,7 @@ import java.util.*
import java.util.concurrent.ConcurrentHashMap
import java.util.concurrent.Semaphore
class TransactionLock {
internal class TransactionLock {
private val locks = ConcurrentHashMap<UUID, Semaphore>()
sealed class LockResult<out T> {
@@ -6,7 +6,7 @@ import net.trueog.diamondbankog.PostgreSQL.ShardType
import org.bukkit.entity.Player
import kotlin.math.floor
object WithdrawHelper {
internal object WithdrawHelper {
/**
* @return The amount of not removed shards, -1 if error
*/
@@ -7,7 +7,7 @@ import org.bukkit.command.CommandExecutor
import org.bukkit.command.CommandSender
import org.bukkit.entity.Player
class AutoCompress : CommandExecutor {
internal class AutoCompress : CommandExecutor {
override fun onCommand(
sender: CommandSender,
command: Command,
@@ -7,7 +7,7 @@ import org.bukkit.command.CommandExecutor
import org.bukkit.command.CommandSender
import org.bukkit.entity.Player
class AutoDeposit : CommandExecutor {
internal class AutoDeposit : CommandExecutor {
override fun onCommand(
sender: CommandSender,
command: Command,
@@ -14,8 +14,7 @@ import org.bukkit.entity.Player
import java.util.*
import kotlin.math.floor
class Balance : CommandExecutor {
internal class Balance : CommandExecutor {
@OptIn(DelicateCoroutinesApi::class)
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
DiamondBankOG.scope.launch {
@@ -14,7 +14,7 @@ import java.util.*
import kotlin.math.ceil
import kotlin.math.floor
class Balancetop : CommandExecutor {
internal class Balancetop : CommandExecutor {
@OptIn(DelicateCoroutinesApi::class)
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
DiamondBankOG.scope.launch {
@@ -19,7 +19,7 @@ import org.bukkit.inventory.ItemStack
import org.bukkit.inventory.meta.BlockStateMeta
import kotlin.math.abs
class Compress : CommandExecutor {
internal class Compress : CommandExecutor {
private data class RunOnMainThreadResult(
val shouldReturn: Boolean,
val changeInShards: Int?,
@@ -15,7 +15,7 @@ import org.bukkit.command.CommandSender
import org.bukkit.entity.Player
import kotlin.math.floor
class Deposit : CommandExecutor {
internal class Deposit : CommandExecutor {
@OptIn(DelicateCoroutinesApi::class)
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
DiamondBankOG.scope.launch {
@@ -6,7 +6,7 @@ import org.bukkit.command.Command
import org.bukkit.command.CommandExecutor
import org.bukkit.command.CommandSender
class DiamondBankHelp : CommandExecutor {
internal class DiamondBankHelp : CommandExecutor {
override fun onCommand(
sender: CommandSender,
command: Command,
@@ -8,7 +8,7 @@ import org.bukkit.command.Command
import org.bukkit.command.CommandExecutor
import org.bukkit.command.CommandSender
class DiamondBankReload : CommandExecutor {
internal class DiamondBankReload : CommandExecutor {
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
if (Config.load()) {
sender.sendMessage(DiamondBankOG.mm.deserialize("<red>Failed to reload the config. Check the console for more information."))
@@ -6,7 +6,7 @@ import org.bukkit.command.Command
import org.bukkit.command.CommandExecutor
import org.bukkit.command.CommandSender
class DisableEconomy : CommandExecutor {
internal class DisableEconomy : CommandExecutor {
override fun onCommand(
sender: CommandSender,
command: Command,
@@ -6,7 +6,7 @@ import org.bukkit.command.Command
import org.bukkit.command.CommandExecutor
import org.bukkit.command.CommandSender
class EnableEconomy : CommandExecutor {
internal class EnableEconomy : CommandExecutor {
override fun onCommand(
sender: CommandSender,
command: Command,
@@ -17,7 +17,7 @@ import org.bukkit.entity.Player
import java.util.*
import kotlin.math.floor
class Pay : CommandExecutor {
internal class Pay : CommandExecutor {
@OptIn(DelicateCoroutinesApi::class)
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
DiamondBankOG.scope.launch launch@{
@@ -14,7 +14,7 @@ import org.bukkit.command.CommandSender
import org.bukkit.entity.Player
import java.util.*
class SetBankBalance : CommandExecutor {
internal class SetBankBalance : CommandExecutor {
@OptIn(DelicateCoroutinesApi::class)
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
DiamondBankOG.scope.launch {
@@ -16,7 +16,7 @@ import org.bukkit.inventory.ItemStack
import kotlin.math.ceil
import kotlin.math.floor
class Withdraw : CommandExecutor {
internal class Withdraw : CommandExecutor {
@OptIn(DelicateCoroutinesApi::class)
override fun onCommand(sender: CommandSender, command: Command, label: String, args: Array<out String>?): Boolean {
DiamondBankOG.scope.launch {