GuanM/tabby
1
0
Fork 0
mirror of https://github.com/Eugeny/tabby.git synced 2025-06-20 11:29:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

Deepclone profile before passing it to the configuration component and fix uncaught error on modal dismiss

Browse Source
This commit is contained in:
highfredo 2021-11-21 13:43:02 +01:00
parent 8ff130ebfd
commit 905e7ffc43
1 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tabby-settings/src/components/profilesSettingsTab.component.ts
View File

@ -100,10 +100,10 @@ export class ProfilesSettingsTabComponent extends BaseComponent {
if (!provider) { if (!provider) {
throw new Error('Cannot edit a profile without a provider') throw new Error('Cannot edit a profile without a provider')
} }
modal.componentInstance.profile = Object.assign({}, profile) modal.componentInstance.profile = deepClone(profile)
modal.componentInstance.profileProvider = provider modal.componentInstance.profileProvider = provider
try {
const result = await modal.result const result = await modal.result
// Fully replace the config // Fully replace the config
for (const k in profile) { for (const k in profile) {
// eslint-disable-next-line @typescript-eslint/no-dynamic-delete // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
@ -112,6 +112,7 @@ export class ProfilesSettingsTabComponent extends BaseComponent {
Object.assign(profile, result) Object.assign(profile, result)
profile.type = provider.id profile.type = provider.id
} catch (e) { }
} }
async deleteProfile (profile: PartialProfile<Profile>): Promise<void> { async deleteProfile (profile: PartialProfile<Profile>): Promise<void> {