style: fix lint (CI)

This commit is contained in:
usbharu 2024-08-13 11:16:14 +00:00 committed by github-actions[bot]
parent d5f08ef710
commit 6d997e8012
36 changed files with 99 additions and 98 deletions

View File

@ -58,13 +58,15 @@ class MigrationLocalActorApplicationService(
if (canAccountMigration.canMigration) { if (canAccountMigration.canMigration) {
fromActor.moveTo = toActorId fromActor.moveTo = toActorId
actorRepository.save(fromActor) actorRepository.save(fromActor)
} else when (canAccountMigration) { } else {
is AlreadyMoved -> throw IllegalArgumentException(canAccountMigration.message) when (canAccountMigration) {
is CanAccountMigration -> throw InternalServerException() is AlreadyMoved -> throw IllegalArgumentException(canAccountMigration.message)
is CircularReferences -> throw IllegalArgumentException(canAccountMigration.message) is CanAccountMigration -> throw InternalServerException()
is SelfReferences -> throw IllegalArgumentException("Self references are not supported") is CircularReferences -> throw IllegalArgumentException(canAccountMigration.message)
is AlsoKnownAsNotFound -> throw IllegalArgumentException(canAccountMigration.message) is SelfReferences -> throw IllegalArgumentException("Self references are not supported")
is MigrationCoolDown -> throw IllegalArgumentException(canAccountMigration.message) is AlsoKnownAsNotFound -> throw IllegalArgumentException(canAccountMigration.message)
is MigrationCoolDown -> throw IllegalArgumentException(canAccountMigration.message)
}
} }
} }

View File

@ -91,7 +91,6 @@ class RegisterApplicationApplicationService(
) )
} }
companion object { companion object {
private val logger = LoggerFactory.getLogger(RegisterApplicationApplicationService::class.java) private val logger = LoggerFactory.getLogger(RegisterApplicationApplicationService::class.java)
} }

View File

@ -36,15 +36,13 @@ class TimelineRelationshipFollowSubscriber(
relationship.targetActorId, relationship.targetActorId,
Visible.FOLLOWERS Visible.FOLLOWERS
) )
), it.body.principal ),
it.body.principal
) )
} }
} }
companion object { companion object {
private val logger = LoggerFactory.getLogger(TimelineRelationshipFollowSubscriber::class.java) private val logger = LoggerFactory.getLogger(TimelineRelationshipFollowSubscriber::class.java)
} }
}
}

View File

@ -11,4 +11,4 @@ class InternalServerException : RuntimeException {
enableSuppression, enableSuppression,
writableStackTrace writableStackTrace
) )
} }

View File

@ -11,4 +11,4 @@ class PermissionDeniedException : RuntimeException {
enableSuppression, enableSuppression,
writableStackTrace writableStackTrace
) )
} }

View File

@ -37,7 +37,6 @@ class UserRegisterFilterApplicationService(
) { ) {
override suspend fun internalExecute(command: RegisterFilter, principal: LocalUser): Filter { override suspend fun internalExecute(command: RegisterFilter, principal: LocalUser): Filter {
val filter = dev.usbharu.hideout.core.domain.model.filter.Filter.create( val filter = dev.usbharu.hideout.core.domain.model.filter.Filter.create(
id = FilterId(idGenerateService.generateId()), id = FilterId(idGenerateService.generateId()),
userDetailId = principal.userDetailId, userDetailId = principal.userDetailId,

View File

@ -16,18 +16,20 @@ class GetLocalInstanceApplicationService(
transaction: Transaction transaction: Transaction
) : ) :
AbstractApplicationService<Unit, Instance>( AbstractApplicationService<Unit, Instance>(
transaction, logger transaction,
logger
) { ) {
var cachedInstance: Instance? = null var cachedInstance: Instance? = null
override suspend fun internalExecute(command: Unit, principal: Principal): Instance { override suspend fun internalExecute(command: Unit, principal: Principal): Instance {
if (cachedInstance != null) { if (cachedInstance != null) {
return cachedInstance!! return cachedInstance!!
} }
val instance = (instanceRepository.findByUrl(applicationConfig.url.toURI()) val instance = (
?: throw InternalServerException("Local instance not found.")) instanceRepository.findByUrl(applicationConfig.url.toURI())
?: throw InternalServerException("Local instance not found.")
)
cachedInstance = Instance.of(instance) cachedInstance = Instance.of(instance)
return cachedInstance!! return cachedInstance!!
@ -36,4 +38,4 @@ class GetLocalInstanceApplicationService(
companion object { companion object {
private val logger = LoggerFactory.getLogger(GetLocalInstanceApplicationService::class.java) private val logger = LoggerFactory.getLogger(GetLocalInstanceApplicationService::class.java)
} }
} }

View File

@ -7,7 +7,10 @@ data class Instance(val id: Long, val name: String, val url: URI, val descriptio
companion object { companion object {
fun of(instance: Instance): dev.usbharu.hideout.core.application.instance.Instance { fun of(instance: Instance): dev.usbharu.hideout.core.application.instance.Instance {
return Instance( return Instance(
instance.id.instanceId, instance.name.name, instance.url, instance.description.description instance.id.instanceId,
instance.name.name,
instance.url,
instance.description.description
) )
} }
} }

View File

@ -1,6 +1,5 @@
package dev.usbharu.hideout.core.application.post package dev.usbharu.hideout.core.application.post
import dev.usbharu.hideout.core.domain.model.actor.Actor import dev.usbharu.hideout.core.domain.model.actor.Actor
import dev.usbharu.hideout.core.domain.model.instance.Instance import dev.usbharu.hideout.core.domain.model.instance.Instance
import dev.usbharu.hideout.core.domain.model.media.Media import dev.usbharu.hideout.core.domain.model.media.Media
@ -32,4 +31,4 @@ data class ActorDetail(
) )
} }
} }
} }

View File

@ -29,7 +29,8 @@ import org.springframework.stereotype.Service
@Service @Service
class DeleteLocalPostApplicationService( class DeleteLocalPostApplicationService(
private val postRepository: PostRepository, private val postRepository: PostRepository,
private val actorRepository: ActorRepository, transaction: Transaction, private val actorRepository: ActorRepository,
transaction: Transaction,
) : LocalUserAbstractApplicationService<DeleteLocalPost, Unit>(transaction, logger) { ) : LocalUserAbstractApplicationService<DeleteLocalPost, Unit>(transaction, logger) {
override suspend fun internalExecute(command: DeleteLocalPost, principal: LocalUser) { override suspend fun internalExecute(command: DeleteLocalPost, principal: LocalUser) {

View File

@ -25,7 +25,8 @@ class GetPostDetailApplicationService(
private val mediaRepository: MediaRepository, private val mediaRepository: MediaRepository,
private val iPostReadAccessControl: IPostReadAccessControl private val iPostReadAccessControl: IPostReadAccessControl
) : AbstractApplicationService<GetPostDetail, PostDetail>( ) : AbstractApplicationService<GetPostDetail, PostDetail>(
transaction, logger transaction,
logger
) { ) {
override suspend fun internalExecute(command: GetPostDetail, principal: Principal): PostDetail { override suspend fun internalExecute(command: GetPostDetail, principal: Principal): PostDetail {
val post = postRepository.findById(PostId(command.postId)) val post = postRepository.findById(PostId(command.postId))
@ -79,12 +80,15 @@ class GetPostDetailApplicationService(
val mediaList = mediaRepository.findByIds(post.mediaIds) val mediaList = mediaRepository.findByIds(post.mediaIds)
return PostDetail.of( return PostDetail.of(
post, first, second, third, mediaList post,
first,
second,
third,
mediaList
) )
} }
companion object { companion object {
private val logger = LoggerFactory.getLogger(GetPostDetailApplicationService::class.java) private val logger = LoggerFactory.getLogger(GetPostDetailApplicationService::class.java)
} }
} }

View File

@ -25,4 +25,4 @@ data class MediaDetail(
) )
} }
} }
} }

View File

@ -35,7 +35,6 @@ class UserAcceptFollowRequestApplicationService(
) : ) :
LocalUserAbstractApplicationService<AcceptFollowRequest, Unit>(transaction, logger) { LocalUserAbstractApplicationService<AcceptFollowRequest, Unit>(transaction, logger) {
override suspend fun internalExecute(command: AcceptFollowRequest, principal: LocalUser) { override suspend fun internalExecute(command: AcceptFollowRequest, principal: LocalUser) {
val actor = actorRepository.findById(principal.actorId) val actor = actorRepository.findById(principal.actorId)
?: throw InternalServerException("Actor ${principal.actorId} not found") ?: throw InternalServerException("Actor ${principal.actorId} not found")

View File

@ -36,8 +36,6 @@ class UserBlockApplicationService(
) : ) :
LocalUserAbstractApplicationService<Block, Unit>(transaction, logger) { LocalUserAbstractApplicationService<Block, Unit>(transaction, logger) {
override suspend fun internalExecute(command: Block, principal: LocalUser) { override suspend fun internalExecute(command: Block, principal: LocalUser) {
val actor = actorRepository.findById(principal.actorId) val actor = actorRepository.findById(principal.actorId)
?: throw IllegalStateException("Actor ${principal.actorId} not found") ?: throw IllegalStateException("Actor ${principal.actorId} not found")

View File

@ -46,14 +46,14 @@ class GetRelationshipApplicationService(
val target = actorRepository.findById(targetId) val target = actorRepository.findById(targetId)
?: throw IllegalArgumentException("Actor ${command.targetActorId} not found.") ?: throw IllegalArgumentException("Actor ${command.targetActorId} not found.")
val relationship = ( val relationship = (
relationshipRepository.findByActorIdAndTargetId(actor.id, targetId) relationshipRepository.findByActorIdAndTargetId(actor.id, targetId)
?: dev.usbharu.hideout.core.domain.model.relationship.Relationship.default(actor.id, targetId) ?: dev.usbharu.hideout.core.domain.model.relationship.Relationship.default(actor.id, targetId)
) )
val relationship1 = ( val relationship1 = (
relationshipRepository.findByActorIdAndTargetId(targetId, actor.id) relationshipRepository.findByActorIdAndTargetId(targetId, actor.id)
?: dev.usbharu.hideout.core.domain.model.relationship.Relationship.default(targetId, actor.id) ?: dev.usbharu.hideout.core.domain.model.relationship.Relationship.default(targetId, actor.id)
) )
val actorInstanceRelationship = val actorInstanceRelationship =
actorInstanceRelationshipRepository.findByActorIdAndInstanceId(actor.id, target.instance) actorInstanceRelationshipRepository.findByActorIdAndInstanceId(actor.id, target.instance)

View File

@ -15,4 +15,4 @@ abstract class LocalUserAbstractApplicationService<T : Any, R>(transaction: Tran
} }
protected abstract suspend fun internalExecute(command: T, principal: LocalUser): R protected abstract suspend fun internalExecute(command: T, principal: LocalUser): R
} }

View File

@ -13,7 +13,8 @@ class UserAddTimelineRelationshipApplicationService(
transaction: Transaction transaction: Transaction
) : ) :
LocalUserAbstractApplicationService<AddTimelineRelationship, Unit>( LocalUserAbstractApplicationService<AddTimelineRelationship, Unit>(
transaction, logger transaction,
logger
) { ) {
override suspend fun internalExecute(command: AddTimelineRelationship, principal: LocalUser) { override suspend fun internalExecute(command: AddTimelineRelationship, principal: LocalUser) {
timelineRelationshipRepository.save(command.timelineRelationship) timelineRelationshipRepository.save(command.timelineRelationship)
@ -22,4 +23,4 @@ class UserAddTimelineRelationshipApplicationService(
companion object { companion object {
private val logger = LoggerFactory.getLogger(UserAddTimelineRelationshipApplicationService::class.java) private val logger = LoggerFactory.getLogger(UserAddTimelineRelationshipApplicationService::class.java)
} }
} }

View File

@ -24,4 +24,4 @@ data class ApplicationConfig(
val url: URL, val url: URL,
val private: Boolean = true, val private: Boolean = true,
val keySize: Int = 2048, val keySize: Int = 2048,
) )

View File

@ -69,7 +69,6 @@ class Filter(
return id.hashCode() return id.hashCode()
} }
companion object { companion object {
fun isAllow(user: UserDetail, action: Action, resource: Filter): Boolean { fun isAllow(user: UserDetail, action: Action, resource: Filter): Boolean {
return when (action) { return when (action) {

View File

@ -241,25 +241,25 @@ class Post(
override fun toString(): String { override fun toString(): String {
return "Post(" + return "Post(" +
"id=$id, " + "id=$id, " +
"createdAt=$createdAt, " + "createdAt=$createdAt, " +
"url=$url, " + "url=$url, " +
"repostId=$repostId, " + "repostId=$repostId, " +
"replyId=$replyId, " + "replyId=$replyId, " +
"apId=$apId, " + "apId=$apId, " +
"actorId=$actorId, " + "actorId=$actorId, " +
"visibility=$visibility, " + "visibility=$visibility, " +
"visibleActors=$visibleActors, " + "visibleActors=$visibleActors, " +
"content=$content, " + "content=$content, " +
"overview=$overview, " + "overview=$overview, " +
"sensitive=$sensitive, " + "sensitive=$sensitive, " +
"text='$text', " + "text='$text', " +
"emojiIds=$emojiIds, " + "emojiIds=$emojiIds, " +
"mediaIds=$mediaIds, " + "mediaIds=$mediaIds, " +
"deleted=$deleted, " + "deleted=$deleted, " +
"hide=$hide, " + "hide=$hide, " +
"moveTo=$moveTo" + "moveTo=$moveTo" +
")" ")"
} }
companion object { companion object {
@ -315,6 +315,5 @@ class Post(
post.addDomainEvent(PostDomainEventFactory(post).createEvent(PostEvent.CREATE)) post.addDomainEvent(PostDomainEventFactory(post).createEvent(PostEvent.CREATE))
return post return post
} }
} }
} }

View File

@ -2,4 +2,4 @@ package dev.usbharu.hideout.core.domain.model.support.principal
import dev.usbharu.hideout.core.domain.model.actor.ActorId import dev.usbharu.hideout.core.domain.model.actor.ActorId
data object Anonymous : Principal(ActorId.ghost, null, null) data object Anonymous : Principal(ActorId.ghost, null, null)

View File

@ -12,4 +12,4 @@ class LocalUser(
actorId, actorId,
userDetailId, userDetailId,
acct acct
) )

View File

@ -4,4 +4,4 @@ import dev.usbharu.hideout.core.domain.model.actor.ActorId
import dev.usbharu.hideout.core.domain.model.support.acct.Acct import dev.usbharu.hideout.core.domain.model.support.acct.Acct
import dev.usbharu.hideout.core.domain.model.userdetails.UserDetailId import dev.usbharu.hideout.core.domain.model.userdetails.UserDetailId
sealed class Principal(open val actorId: ActorId, open val userDetailId: UserDetailId?, open val acct: Acct?) sealed class Principal(open val actorId: ActorId, open val userDetailId: UserDetailId?, open val acct: Acct?)

View File

@ -2,4 +2,4 @@ package dev.usbharu.hideout.core.domain.model.support.principal
interface PrincipalContextHolder { interface PrincipalContextHolder {
suspend fun getPrincipal(): Principal suspend fun getPrincipal(): Principal
} }

View File

@ -33,7 +33,7 @@ class TimelineObject(
lastUpdatedAt: Instant, lastUpdatedAt: Instant,
var warnFilters: List<TimelineObjectWarnFilter>, var warnFilters: List<TimelineObjectWarnFilter>,
) { ) {
var isPureRepost = isPureRepost var isPureRepost = isPureRepost
private set private set
var visibleActors = visibleActors var visibleActors = visibleActors
@ -125,9 +125,9 @@ class TimelineObject(
repostActorId = repost.actorId, repostActorId = repost.actorId,
visibility = post.visibility, visibility = post.visibility,
isPureRepost = repost.mediaIds.isEmpty() && isPureRepost = repost.mediaIds.isEmpty() &&
repost.overview == null && repost.overview == null &&
repost.content == PostContent.empty && repost.content == PostContent.empty &&
repost.replyId == null, repost.replyId == null,
mediaIds = post.mediaIds, mediaIds = post.mediaIds,
emojiIds = post.emojiIds, emojiIds = post.emojiIds,
visibleActors = post.visibleActors.toList(), visibleActors = post.visibleActors.toList(),

View File

@ -16,36 +16,37 @@ interface IPostReadAccessControl {
class DefaultPostReadAccessControl(private val relationshipRepository: RelationshipRepository) : class DefaultPostReadAccessControl(private val relationshipRepository: RelationshipRepository) :
IPostReadAccessControl { IPostReadAccessControl {
override suspend fun isAllow(post: Post, principal: Principal): Boolean { override suspend fun isAllow(post: Post, principal: Principal): Boolean {
// ポスト主は無条件で見れる
//ポスト主は無条件で見れる
if (post.actorId == principal.actorId) { if (post.actorId == principal.actorId) {
return true return true
} }
val relationship = (relationshipRepository.findByActorIdAndTargetId(post.actorId, principal.actorId) val relationship = (
?: Relationship.default(post.actorId, principal.actorId)) relationshipRepository.findByActorIdAndTargetId(post.actorId, principal.actorId)
?: Relationship.default(post.actorId, principal.actorId)
)
//ブロックされてたら見れない // ブロックされてたら見れない
if (relationship.blocking) { if (relationship.blocking) {
return false return false
} }
//PublicかUnlistedなら見れる // PublicかUnlistedなら見れる
if (post.visibility == Visibility.PUBLIC || post.visibility == Visibility.UNLISTED) { if (post.visibility == Visibility.PUBLIC || post.visibility == Visibility.UNLISTED) {
return true return true
} }
//principalがAnonymousなら見れない // principalがAnonymousなら見れない
if (principal is Anonymous) { if (principal is Anonymous) {
return false return false
} }
//DirectでvisibleActorsに含まれていたら見れる // DirectでvisibleActorsに含まれていたら見れる
if (post.visibility == Visibility.DIRECT && post.visibleActors.contains(principal.actorId)) { if (post.visibility == Visibility.DIRECT && post.visibleActors.contains(principal.actorId)) {
return true return true
} }
//Followersでフォロワーなら見れる // Followersでフォロワーなら見れる
if (post.visibility == Visibility.FOLLOWERS) { if (post.visibility == Visibility.FOLLOWERS) {
val inverseRelationship = val inverseRelationship =
relationshipRepository.findByActorIdAndTargetId(principal.actorId, post.actorId) ?: return false relationshipRepository.findByActorIdAndTargetId(principal.actorId, post.actorId) ?: return false
@ -53,8 +54,7 @@ class DefaultPostReadAccessControl(private val relationshipRepository: Relations
return inverseRelationship.following return inverseRelationship.following
} }
//その他の場合は見れない // その他の場合は見れない
return false return false
} }
}
}

View File

@ -34,4 +34,4 @@ class ExposedPrincipalQueryService : PrincipalQueryService, AbstractRepository()
companion object { companion object {
private val logger: Logger = LoggerFactory.getLogger(ExposedPrincipalQueryService::class.java) private val logger: Logger = LoggerFactory.getLogger(ExposedPrincipalQueryService::class.java)
} }
} }

View File

@ -22,7 +22,6 @@ class SpringSecurityFormLoginPrincipalContextHolder(
SecurityContextHolder.getContext().authentication?.principal as? HideoutUserDetails ?: return Anonymous SecurityContextHolder.getContext().authentication?.principal as? HideoutUserDetails ?: return Anonymous
return transaction.transaction { return transaction.transaction {
val userDetail = principalQueryService.findByUserDetailId(UserDetailId(hideoutUserDetails.userDetailsId)) val userDetail = principalQueryService.findByUserDetailId(UserDetailId(hideoutUserDetails.userDetailsId))
LocalUser( LocalUser(
userDetail.actorId, userDetail.actorId,
@ -30,6 +29,5 @@ class SpringSecurityFormLoginPrincipalContextHolder(
Acct(userDetail.username, userDetail.host) Acct(userDetail.username, userDetail.host)
) )
} }
} }
} }

View File

@ -33,4 +33,4 @@ class SpringSecurityOauth2PrincipalContextHolder(
) )
} }
} }
} }

View File

@ -30,4 +30,4 @@ data class InternalTimelineObjectOption(
val localOnly: Boolean? = null, val localOnly: Boolean? = null,
val remoteOnly: Boolean? = null, val remoteOnly: Boolean? = null,
val mediaOnly: Boolean? = null val mediaOnly: Boolean? = null
) )

View File

@ -38,7 +38,8 @@ class AuthController(
suspend fun signUp(@Validated @ModelAttribute signUpForm: SignUpForm, request: HttpServletRequest): String { suspend fun signUp(@Validated @ModelAttribute signUpForm: SignUpForm, request: HttpServletRequest): String {
val registerLocalActor = RegisterLocalActor(signUpForm.username, signUpForm.password) val registerLocalActor = RegisterLocalActor(signUpForm.username, signUpForm.password)
val uri = registerLocalActorApplicationService.execute( val uri = registerLocalActorApplicationService.execute(
registerLocalActor, Anonymous registerLocalActor,
Anonymous
) )
request.login(signUpForm.username, signUpForm.password) request.login(signUpForm.username, signUpForm.password)
return "redirect:$uri" return "redirect:$uri"

View File

@ -30,7 +30,6 @@ import org.springframework.web.bind.annotation.PathVariable
import java.nio.file.Path import java.nio.file.Path
import kotlin.io.path.name import kotlin.io.path.name
@Controller @Controller
@ConditionalOnProperty("hideout.storage.type", havingValue = "local", matchIfMissing = true) @ConditionalOnProperty("hideout.storage.type", havingValue = "local", matchIfMissing = true)
class LocalFileController( class LocalFileController(
@ -64,4 +63,4 @@ class LocalFileController(
.contentLength(pathResource.contentLength()) .contentLength(pathResource.contentLength())
.body(pathResource) .body(pathResource)
} }
} }

View File

@ -31,4 +31,4 @@ class PostsController(
return "postById" return "postById"
} }
} }

View File

@ -55,6 +55,6 @@ class PublishController(
) )
val id = userRegisterLocalPostApplicationService.execute(command, principal) val id = userRegisterLocalPostApplicationService.execute(command, principal)
return "redirect:/users/${principal.acct?.userpart}/posts/${id}" return "redirect:/users/${principal.acct?.userpart}/posts/$id"
} }
} }

View File

@ -3,4 +3,4 @@ package dev.usbharu.hideout.core.query.principal
import dev.usbharu.hideout.core.domain.model.actor.ActorId import dev.usbharu.hideout.core.domain.model.actor.ActorId
import dev.usbharu.hideout.core.domain.model.userdetails.UserDetailId import dev.usbharu.hideout.core.domain.model.userdetails.UserDetailId
data class PrincipalDTO(val userDetailId: UserDetailId, val actorId: ActorId, val username: String, val host: String) data class PrincipalDTO(val userDetailId: UserDetailId, val actorId: ActorId, val username: String, val host: String)

View File

@ -4,4 +4,4 @@ import dev.usbharu.hideout.core.domain.model.userdetails.UserDetailId
interface PrincipalQueryService { interface PrincipalQueryService {
suspend fun findByUserDetailId(userDetailId: UserDetailId): PrincipalDTO suspend fun findByUserDetailId(userDetailId: UserDetailId): PrincipalDTO
} }