Code cleanup

This commit is contained in:
Steffen Schaefer
2018-12-19 12:35:52 +01:00
parent 3fef3cc691
commit 78e6afba72
14 changed files with 11 additions and 63 deletions
@@ -68,11 +68,7 @@ public class RegattaNameAndRaceName extends RegattaName implements RegattaAndRac
@Override
public TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(RegattaAndRaceIdentifier regattaAndRaceIdentifier) {
return new TypeRelativeObjectIdentifier(regattaAndRaceIdentifier.getRegattaName(), regattaAndRaceIdentifier.getRaceName());
return getTypeRelativeObjectIdentifier(getRegattaName(), getRaceName());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String regattaName, String raceName) {
@@ -180,11 +180,7 @@ public class MediaTrack implements Serializable, WithQualifiedObjectIdentifier {
}
public TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(MediaTrack mediaTrack) {
return new TypeRelativeObjectIdentifier(mediaTrack.dbId);
return getTypeRelativeObjectIdentifier(dbId);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String dbId) {
@@ -28,11 +28,7 @@ public interface Account extends WithQualifiedObjectIdentifier {
}
default TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Account account) {
return new TypeRelativeObjectIdentifier(account.getUser().getEmail());
return getTypeRelativeObjectIdentifier(getUser().getEmail());
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String email) {
@@ -138,11 +138,7 @@ public class BoatImpl extends RenamableImpl implements DynamicBoat {
}
public TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Boat boat) {
return new TypeRelativeObjectIdentifier(boat.getId().toString());
return getTypeRelativeObjectIdentifier(getId());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Serializable id) {
@@ -258,11 +258,7 @@ public class CompetitorImpl implements DynamicCompetitor {
}
public TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Competitor competitor) {
return new TypeRelativeObjectIdentifier(competitor.getId().toString());
return getTypeRelativeObjectIdentifier(getId());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Serializable id) {
@@ -384,11 +384,7 @@ public abstract class EventBaseImpl implements EventBase {
}
public TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(EventBase event) {
return new TypeRelativeObjectIdentifier(event.getId().toString());
return getTypeRelativeObjectIdentifier(getId());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(UUID id) {
@@ -72,11 +72,7 @@ public abstract class LeaderboardGroupBaseImpl implements LeaderboardGroupBase {
}
public TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(LeaderboardGroupBaseImpl leaderboarGroup) {
return new TypeRelativeObjectIdentifier(leaderboarGroup.getId().toString());
return getTypeRelativeObjectIdentifier(getId());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(UUID id) {
@@ -29,10 +29,6 @@ public interface SwissTimingArchiveConfiguration extends WithQualifiedObjectIden
return getTypeRelativeObjectIdentifier(getJsonURL());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(SwissTimingArchiveConfiguration config) {
return new TypeRelativeObjectIdentifier(config.getJsonURL());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String jsonUrl) {
return new TypeRelativeObjectIdentifier(jsonUrl);
}
@@ -41,10 +41,6 @@ public interface SwissTimingConfiguration extends WithQualifiedObjectIdentifier
return getTypeRelativeObjectIdentifier(getJsonURL());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(SwissTimingConfiguration config) {
return new TypeRelativeObjectIdentifier(config.getJsonURL());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String jsonUrl) {
return new TypeRelativeObjectIdentifier(jsonUrl);
}
@@ -53,10 +53,6 @@ public interface TracTracConfiguration extends WithQualifiedObjectIdentifier {
return getTypeRelativeObjectIdentifier(getJSONURL());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(TracTracConfiguration config) {
return new TypeRelativeObjectIdentifier(config.getJSONURL());
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String jsonUrl) {
return new TypeRelativeObjectIdentifier(jsonUrl);
}
@@ -252,10 +252,6 @@ public interface Regatta
return getTypeRelativeObjectIdentifier(getName());
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Regatta regatta) {
return new TypeRelativeObjectIdentifier(regatta.getName());
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String regattaName) {
return new TypeRelativeObjectIdentifier(regattaName);
}
@@ -626,17 +626,13 @@ public interface Leaderboard extends LeaderboardBase, HasRaceColumns {
}
default TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(this);
return getTypeRelativeObjectIdentifier(getName());
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(String name) {
return new TypeRelativeObjectIdentifier(name);
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(Leaderboard leaderboard) {
return new TypeRelativeObjectIdentifier(leaderboard.getName());
}
static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(RegattaName regattaName) {
return new TypeRelativeObjectIdentifier(regattaName.getRegattaName());
}
@@ -1138,15 +1138,11 @@ public interface TrackedRace
}
public static QualifiedObjectIdentifier getIdentifier(RegattaAndRaceIdentifier regattaAndRaceId) {
return getSecuredDomainType().getQualifiedObjectIdentifier(getTypeRelativeObjectIdentifier(regattaAndRaceId));
}
public static TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier(RegattaAndRaceIdentifier regattaAndRaceId) {
return regattaAndRaceId.getTypeRelativeObjectIdentifier();
return getSecuredDomainType().getQualifiedObjectIdentifier(regattaAndRaceId.getTypeRelativeObjectIdentifier());
}
default TypeRelativeObjectIdentifier getTypeRelativeObjectIdentifier() {
return getTypeRelativeObjectIdentifier(getRaceIdentifier());
return getRaceIdentifier().getTypeRelativeObjectIdentifier();
}
@Override
@@ -57,7 +57,7 @@ public class PermissionAwareRaceTrackingHandler extends DefaultRaceTrackingHandl
try {
return securityService.doWithTemporaryDefaultTenant(defaultTenant, () -> {
return securityService.setOwnershipCheckPermissionForObjectCreationAndRevertOnError(
SecuredDomainType.TRACKED_RACE, RegattaNameAndRaceName.getTypeRelativeObjectIdentifier(regattaAndRaceIdentifier),
SecuredDomainType.TRACKED_RACE, regattaAndRaceIdentifier.getTypeRelativeObjectIdentifier(),
regattaAndRaceIdentifier.toString(), () -> {
return innerAction.get();
});