Handle missing 2FAccount when deleting

This commit is contained in:
Bubka 2020-01-10 00:22:45 +01:00
parent 7b5ac91796
commit aaab76b7d3
2 changed files with 23 additions and 12 deletions

View File

@ -118,8 +118,12 @@ class TwoFAccountController extends Controller
* @param \App\TwoFAccount $twofaccount
* @return \Illuminate\Http\Response
*/
public function destroy(TwoFAccount $twofaccount)
public function destroy($id)
{
try {
$twofaccount = TwoFAccount::FindOrFail($id);
// delete icon
$storedIcon = 'public/icons/' . $twofaccount->icon;
@ -130,6 +134,13 @@ class TwoFAccountController extends Controller
$twofaccount->delete();
return response()->json(null, 204);
}
catch (\Exception $e) {
return response()->json('already gone', 404);
}
}
}

View File

@ -169,12 +169,12 @@
axios.defaults.headers.common['Content-Type'] = 'application/json'
axios.defaults.headers.common['Authorization'] = 'Bearer ' + this.token
axios.delete('/api/twofaccounts/' + id).then(response => {
axios.delete('/api/twofaccounts/' + id)
this.accounts.splice(this.accounts.findIndex(x => x.id === id), 1);
this.showAccounts = this.accounts.length > 0 ? true : false
this.showNoAccount = !this.showAccounts
})
}
},