Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VVPPアンインストール時とディレクトリ見つからない時のエラーハンドリング追加 #1047

Merged
merged 4 commits into from
Dec 17, 2022
Merged

VVPPアンインストール時とディレクトリ見つからない時のエラーハンドリング追加 #1047

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e09271f
VVPPアンインストール時とディレクトリ見つからない時のエラーハンドリング追加
Hiroshiba Dec 16, 2022
5f63b41
uninstallVvppEngineとかにコメントを書く
Hiroshiba Dec 17, 2022
9f871a7
UNINSTALL_VVPP_ENGINEの返り値をみてから再起動案内
Hiroshiba Dec 17, 2022
e9ea75a
Merge remote-tracking branch 'upstream/main' into VVPPアンインストール時とディレクト…
Hiroshiba Dec 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
46 changes: 28 additions & 18 deletions src/background.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -803,7 +803,7 @@ function openEngineDirectory(engineId: string) {

const engineDirectory = engineInfo.path;
if (engineDirectory == null) {
return;
throw new Error(`engineDirectory is null: engineId == ${engineId}`);
}

// Windows環境だとスラッシュ区切りのパスが動かない。
Expand All @@ -817,10 +817,10 @@ async function installVvppEngine(vvppPath: string) {
return true;
} catch (e) {
dialog.showErrorBox(
"読み込みエラー",
`${vvppPath} を読み込めませんでした。`
"インストールエラー",
`${vvppPath} をインストールできませんでした。`
);
log.error(`Failed to load ${vvppPath}, ${e}`);
log.error(`Failed to install ${vvppPath}, ${e}`);
return false;
}
}
Expand All @@ -830,23 +830,33 @@ async function uninstallVvppEngine(engineId: string) {
const engineInfo = engineInfos.find(
(engineInfo) => engineInfo.uuid === engineId
);
if (!engineInfo) {
throw new Error(`No such engineInfo registered: engineId == ${engineId}`);
}
try {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

installVvppEngine側はダイアログを出していたがuninstallVvppEngine側はダイアログを出しておらず、エラーが出ても何も表示されない状態になってたので合わせてみました

if (!engineInfo) {
throw new Error(`No such engineInfo registered: engineId == ${engineId}`);
}

if (engineInfo.type !== "vvpp") {
throw new Error(`engineInfo.type is not vvpp: engineId == ${engineId}`);
}
if (engineInfo.type !== "vvpp") {
throw new Error(`engineInfo.type is not vvpp: engineId == ${engineId}`);
}

const engineDirectory = engineInfo.path;
if (engineDirectory == null) {
throw new Error("engineDirectory == null");
}
const engineDirectory = engineInfo.path;
if (engineDirectory == null) {
throw new Error("engineDirectory == null");
}

// Windows環境だとエンジンを終了してから削除する必要がある。
// そのため、アプリの終了時に削除するようにする。
vvppManager.markWillDelete(engineId);
return true;
// Windows環境だとエンジンを終了してから削除する必要がある。
// そのため、アプリの終了時に削除するようにする。
vvppManager.markWillDelete(engineId);
return true;
} catch (e) {
const engineName = engineInfo?.name ?? engineId;
dialog.showErrorBox(
"アンインストールエラー",
`${engineName} をアンインストールできませんでした。`
);
log.error(`Failed to uninstall ${engineId}, ${e}`);
return false;
Hiroshiba marked this conversation as resolved.
Show resolved Hide resolved
}
}

// ディレクトリがエンジンとして正しいかどうかを判定する
Expand Down