Rename `DeleteArtistHandle` to `DeleteArtistHandler` for consistency and update related references. Clean up Protobuf comment in `UpdateArtistResponse`.
This commit is contained in:
parent
0df4b31a4d
commit
51a46c37e5
|
|
@ -11,12 +11,12 @@ import org.apache.logging.log4j.LogManager;
|
||||||
import org.apache.logging.log4j.Logger;
|
import org.apache.logging.log4j.Logger;
|
||||||
|
|
||||||
@Action("artist.delete")
|
@Action("artist.delete")
|
||||||
public class DeleteArtistHandle implements ActionHandler {
|
public class DeleteArtistHandler implements ActionHandler {
|
||||||
private static final Logger logger = LogManager.getLogger(DeleteArtistHandle.class);
|
private static final Logger logger = LogManager.getLogger(DeleteArtistHandler.class);
|
||||||
|
|
||||||
private final ArtistService artistService;
|
private final ArtistService artistService;
|
||||||
|
|
||||||
public DeleteArtistHandle(ArtistService artistService) {
|
public DeleteArtistHandler(ArtistService artistService) {
|
||||||
this.artistService = artistService;
|
this.artistService = artistService;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -54,9 +54,9 @@ public class DeleteArtistHandle implements ActionHandler {
|
||||||
.setSuccess(false)
|
.setSuccess(false)
|
||||||
.setMessage("Error: " + e.getMessage())
|
.setMessage("Error: " + e.getMessage())
|
||||||
.build();
|
.build();
|
||||||
return TransportProtocol.Response.newBuilder()
|
return TransportProtocol.Response.newBuilder()
|
||||||
.setStatusCode(500)
|
.setStatusCode(500)
|
||||||
.setPayload(deleteResponse.toByteString());
|
.setPayload(deleteResponse.toByteString());
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
@ -40,7 +40,7 @@ message UpdateArtistRequest {
|
||||||
}
|
}
|
||||||
|
|
||||||
message UpdateArtistResponse {
|
message UpdateArtistResponse {
|
||||||
Artist Artist = 1; // Genre atualizado
|
Artist Artist = 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
message DeleteArtistRequest {
|
message DeleteArtistRequest {
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue