Skip to content

Fix offlineplayer issues#40

Open
cerqiest wants to merge 4 commits intomainfrom
fix-offlineplayer-issues
Open

Fix offlineplayer issues#40
cerqiest wants to merge 4 commits intomainfrom
fix-offlineplayer-issues

Conversation

@cerqiest
Copy link
Collaborator

These are some dumb issues and other fixed that I probably could've caught in testing (or in review!) but nevertheless need to be fixed.

@cerqiest cerqiest requested a review from zNotChill as a code owner April 21, 2025 03:54
@cerqiest
Copy link
Collaborator Author

I did not mean to open this as an actual PR and cannot figure out how to convert it to a draft.

@github-actions
Copy link

Qodana Community for JVM

47 new problems were found

Inspection name Severity Problems
Unused symbol 🔶 Warning 20
Unused import directive 🔶 Warning 11
Redundant call of conversion method 🔶 Warning 2
Unresolved reference in KDoc 🔶 Warning 1
Usage of redundant or deprecated syntax or deprecated symbols 🔶 Warning 1
Redundant nullable return type 🔶 Warning 1
Class member can have 'private' visibility ◽️ Notice 6
If-Null return/break/... foldable to '?:' ◽️ Notice 2
Variable declaration could be moved inside 'when' ◽️ Notice 1
Redundant empty primary constructor ◽️ Notice 1
Accessor call that can be replaced with property access syntax ◽️ Notice 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@cerqiest
Copy link
Collaborator Author

I did not mean to open this as an actual PR and cannot figure out how to convert it to a draft.

Ready to merge.

}
}
open val location
get() = offlinePlayer.location?.let { LuaLocation.fromBukkit(it) }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Surely this doesn't work since the player is offline...

get() = offlinePlayer.player
@TwineNativeProperty("player")
open val playerLuaProperty
get() = offlinePlayer.player?.let { LuaPlayer(it) }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same question

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants