Skip to content

Commit

Permalink
Merge pull request #182 from unipept/fix/deleting-studies
Browse files Browse the repository at this point in the history
Deleting a study works properly again
  • Loading branch information
pverscha authored Jun 1, 2022
2 parents 0d9014b + 8a63658 commit 52f94c0
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "unipept-desktop",
"version": "1.2.5",
"version": "1.2.6",
"private": true,
"author": "Unipept Team (Ghent University)",
"description": "A desktop equivalent of unipept.ugent.be. This app aims at high-throughput analysis of metaproteomics samples.",
Expand Down
2 changes: 1 addition & 1 deletion src/App.vue
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ export default class App extends Vue implements ErrorListener {
@Watch("activeAssay")
private activeAssayChanged(assay: AssayAnalysisStatus) {
if (assay.analysisReady) {
if (!assay || assay.analysisReady) {
electron.remote.BrowserWindow.getAllWindows()[0].setProgressBar(-1);
} else {
electron.remote.BrowserWindow.getAllWindows()[0].setProgressBar(
Expand Down
6 changes: 6 additions & 0 deletions src/components/navigation-drawers/StudyItem.vue
Original file line number Diff line number Diff line change
Expand Up @@ -313,6 +313,12 @@ export default class StudyItem extends Vue {
this.$store.getters.dbManager
);
await this.study.accept(studyDestroyer);
for (const assay of this.study.getAssays()) {
await this.$store.dispatch("removeAssay", assay);
}
await this.$store.dispatch("removeStudy", this.study);
}
private async onSelectAssay(assay: Assay) {
Expand Down

0 comments on commit 52f94c0

Please sign in to comment.