[Feature]: Spieler bannen #15

Open
SZUT-Dorian wants to merge 3 commits from TD-4_Admin_kann_spieler_bannen into trunk
Member

Ticket

TD-4

Beschreibung

Der Player Modell wurde um die Funktionalität erweitert, einen Spieler zu bannen und zu entbannen.

Weitere Infos

No response

### Ticket TD-4 ### Beschreibung Der Player Modell wurde um die Funktionalität erweitert, einen Spieler zu bannen und zu entbannen. ### Weitere Infos _No response_
SZUT-Dorian self-assigned this 2025-03-13 09:03:51 +00:00
SZUT-Mehdi was assigned by SZUT-Dorian 2025-03-13 09:03:51 +00:00
SZUT-Dorian added 3 commits 2025-03-13 09:03:51 +00:00
TD-4: Unban
Some checks failed
Quality Check / Validate OAS (push) Successful in 31s
Quality Check / Static Analysis (push) Failing after 54s
Quality Check / Linting (push) Successful in 57s
Quality Check / Testing (push) Failing after 58s
Quality Check / Validate OAS (pull_request) Successful in 32s
Quality Check / Static Analysis (pull_request) Failing after 57s
Quality Check / Testing (pull_request) Failing after 1m0s
Quality Check / Linting (pull_request) Successful in 1m3s
ca8942feba
requested review from snoweuph 2025-03-13 09:18:43 +00:00
SZUT-Rajbir requested changes 2025-03-13 09:20:59 +00:00
SZUT-Rajbir left a comment
Member

Test sind durch gefallen. Guckt euch das nochmal an

Test sind durch gefallen. Guckt euch das nochmal an
snoweuph requested changes 2025-03-13 09:28:05 +00:00
snoweuph left a comment
Owner

Bitte zu /player/login 403 hinzufügen für, wenn jemand gebannt ist

Bitte zu /player/login 403 hinzufügen für, wenn jemand gebannt ist
@ -259,6 +265,40 @@ paths:
type: array
items:
$ref: "#/components/schemas/AdministratablePlayer"
Owner

ihr habt beim /admin/players endpunkt kein 401, 500 und 503 mehr. sind benötigt

ihr habt beim /admin/players endpunkt kein 401, 500 und 503 mehr. sind benötigt
@ -262,0 +279,4 @@
- admin
summary: "Ban player by id"
description: "Bans a player by id, making them unable to play the game."
responses:
Owner

404 wenn spieler nicht gefunden

404 wenn spieler nicht gefunden
@ -262,0 +281,4 @@
description: "Bans a player by id, making them unable to play the game."
responses:
200:
description: "Successfully banned player!"
Owner

401,500 & 503 fehlen

401,500 & 503 fehlen
@ -262,0 +296,4 @@
- admin
summary: "Unban player by id"
description: "Unbans a player by id, allowing them to play the game."
responses:
Owner

404 hinzufügen, fals der speiler nicht gefunden wird

404 hinzufügen, fals der speiler nicht gefunden wird
@ -60,0 +64,4 @@
@Override
public ResponseEntity<Void> banPlayer(Long id) {
this.playerService.banPlayer(id);
return null;
Owner

ist null hier richtig? wollen wir nicht ein 200 zurückgeben?

ist null hier richtig? wollen wir nicht ein 200 zurückgeben?
@ -86,0 +97,4 @@
@Override
public ResponseEntity<Void> unbanPlayer(Long id) {
this.playerService.unbanPlayer(id);
return null;
Owner

st null hier richtig? wollen wir nicht ein 200 zurückgeben?

st null hier richtig? wollen wir nicht ein 200 zurückgeben?
@ -50,2 +51,4 @@
return md.digest(password);
}
public void banPlayer(Long id) {
Owner

Wenn der Spieler nicht existiert, sollte diese Funktion einen PlayerNotFound Exception schmeißen

Wenn der Spieler nicht existiert, sollte diese Funktion einen PlayerNotFound Exception schmeißen
@ -52,0 +58,4 @@
});
}
public void unbanPlayer(Long id) {
Owner

Wenn der Spieler nicht existiert, sollte diese Funktion einen PlayerNotFound Exception schmeißen

Wenn der Spieler nicht existiert, sollte diese Funktion einen PlayerNotFound Exception schmeißen
@ -45,7 +45,7 @@ public class ServerApiController implements ServerApi {
@Override
public ResponseEntity<Void> playerRegister(PlayerRegistrationData body) {
if(playerRepository.existsByUsername(body.getUsername())){
return new ResponseEntity<>(HttpStatus.CONFLICT);
Owner

warum die änderung?

warum die änderung?
@ -63,1 +63,4 @@
Player player = playerRepository.findByUsername(body.getUsername());
if(player.isBanned())
{
return new ResponseEntity<>(HttpStatus.CONFLICT);
Owner

403 währe hier richtig.

403 ist für wenn man sich authentiziert hat, aber keine zugriffsrechte hat

403 währe hier richtig. 403 ist für wenn man sich authentiziert hat, aber keine zugriffsrechte hat
Some checks failed
Quality Check / Validate OAS (push) Successful in 31s
Required
Details
Quality Check / Static Analysis (push) Failing after 54s
Required
Details
Quality Check / Linting (push) Successful in 57s
Required
Details
Quality Check / Testing (push) Failing after 58s
Required
Details
Quality Check / Validate OAS (pull_request) Successful in 32s
Required
Details
Quality Check / Static Analysis (pull_request) Failing after 57s
Required
Details
Quality Check / Testing (pull_request) Failing after 1m0s
Required
Details
Quality Check / Linting (pull_request) Successful in 1m3s
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin TD-4_Admin_kann_spieler_bannen:TD-4_Admin_kann_spieler_bannen
git checkout TD-4_Admin_kann_spieler_bannen
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TowerDefence/Server#15
No description provided.