Skip to content

Commit

Permalink
fix: ブランチが見つからなかった時のエラー文面を修正
Browse files Browse the repository at this point in the history
  • Loading branch information
eyemono-moe committed Jul 27, 2024
1 parent 6de0625 commit 376e3fa
Showing 1 changed file with 10 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,26 +14,27 @@ const BranchField: Component<Props> = (props) => {
const { formStore } = useApplicationForm()
const branches = useBranches(() => props.repo.id)

// 設定されていたブランチを取得できなかった場合(ブランチが削除された場合等)にエラーを表示し、
// 設定されていたブランチが見つからなかった場合(ブランチが削除された場合等)にエラーを表示し、
// 存在しない refName の代わりに空文字列を入れる
createEffect(() => {
const ref = getValue(
const refName = getValue(
untrack(() => formStore),
'form.refName',
)
if (ref !== undefined && !branches().includes(ref)) {
if (refName !== undefined && !branches().includes(refName)) {
// 下の refName に対する setValue により、もともと設定されていたブランチ名が空文字列に上書きされるため
// refName が空でない(=refName がもともと設定されていたブランチ名になっている)とき、それを使ってエラーを表示する
if (refName !== '') {
setError(formStore, 'form.refName', `設定されたブランチ(${refName})は存在しないブランチ名です`)
}

// refName を undefined にすると"更新しないプロパティ"という扱いになるため、
// 空文字列を設定してバリデーションエラーを発生させる
setValue(
untrack(() => formStore),
'form.refName',
'',
)
setError(
formStore,
'form.refName',
'設定されていたブランチの取得に失敗しました。リポジトリへのアクセス権を確認してください。',
)
} else {
clearError(formStore, 'form.refName')
}
Expand Down Expand Up @@ -62,6 +63,7 @@ const BranchField: Component<Props> = (props) => {
label: branch,
value: branch,
}))}
disabled={branches().length === 0}
value={field.value}
error={field.error}
readOnly={!props.hasPermission}
Expand Down

0 comments on commit 376e3fa

Please sign in to comment.