Make reassinging the renamed profile shortcut conditional
ClosedPublic

Authored by ahmadsamir on Dec 14 2017, 9:23 PM.

Details

Summary

Following up on D9272; this diff changes reassigning the old profile shortcut to the newly renamed profile if deleting the old profile returns true. This was if the profile is in read-only dir the shortcut isn't reassigned.

Diff Detail

Repository
R319 Konsole
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ahmadsamir created this revision.Dec 14 2017, 9:23 PM
Restricted Application added a project: Konsole. · View Herald TranscriptDec 14 2017, 9:23 PM
Restricted Application added a subscriber: Konsole. · View Herald Transcript
ahmadsamir requested review of this revision.Dec 14 2017, 9:23 PM
ahmadsamir added a subscriber: hindenburg.

With the patch, I still see the shortcut on both profiles in the manage profile dialog.

Once you close the dialog, only the r/o profile has the shortcut.

With the patch, I still see the shortcut on both profiles in the manage profile dialog.

Once you close the dialog, only the r/o profile has the shortcut.

Both profiles have the same name? we still haven't fixed the issue where konsole writes/saves a profile read from /usr/share/konsole to ~/.local/share/konsole with the same name.

This diff is more of a safe-guard, if for whatever reason deleteProfile() fails, the shortcut should stay with the old/orig profile.

I'm still working on a patch for profiles under /usr/share/konsole.

hindenburg accepted this revision.Dec 15 2017, 1:48 PM

OK I see know

This revision is now accepted and ready to land.Dec 15 2017, 1:48 PM
This revision was automatically updated to reflect the committed changes.