Implementing respawn location.

[svaningelgem]

Description

Adding respawn location attribute.

Creating this as a draft because I see there is a respawn function available (and tested). But I don't know exactly what .respawn() should do: should it go to the respawn location, or to the bed? What happens exactly?

Checklist

The following items should be checked before the pull request can be merged.

• Code follows existing style.
• Unit tests added (if applicable).

[Thorinwasher]

should it go to the respawn location, or to the bed? What happens exactly?

I recomend looking into the Paper code for this, as our behavior in mockbukkit is meant to copy that of paper. Anyhow for your specific question; it looks like there is only one variable that defines the respawn location in Paper. The bed respawn location is that location, here's the implementation of one of the methods:

    public Location getBedSpawnLocation() {
        return this.getRespawnLocation();
    }

I hope that clears that up for you

[svaningelgem]

Thanks for the hint @Thorinwasher! I have now a copy of the papers code on my computer.
The changes are now in this branch.

[Thorinwasher]

The changes are now in this branch.

Good! Now you need to change the formatting of the code to follow the same style as everything else.

[svaningelgem]

No ruff or black or anything like that for java? :'(

[Thorinwasher]

This here is what I meant reformat the code. Here's an example

Suggested change

	public @Nullable Location getRespawnLocation() { return this.respawnLocation; }
	public @Nullable Location getRespawnLocation()
	{
	return this.respawnLocation;
	}

[svaningelgem]

Ok, this should have been reformatted now.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud