[Konsole-devel] Review Request: konsole should never save a profile whose name is empty into disk

Jekyll Wu adaptee at gmail.com
Fri Sep 9 14:38:48 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102559/
-----------------------------------------------------------

Review request for Konsole.


Summary
-------

Currently it is possible to save profile whose name is empty into disk. Such profile will be saved as .profile and displayed in the ManageProfilesDialog with no name. This is confusing to the users. I think such profile should never saved into disk.

The patch contain 2 changes:
1). In the EditProfileDialog, when user ask to save the profile, pop up a warningbox if its name is empty.
2). In SessionManager, when asked to apply changes to a profile, never save it into disk if its name is empty.


This addresses bug 281268.
    http://bugs.kde.org/show_bug.cgi?id=281268


Diffs
-----

  src/EditProfileDialog.cpp d1edc00 
  src/EditProfileDialog.ui 8883298 
  src/SessionManager.cpp d782b71 

Diff: http://git.reviewboard.kde.org/r/102559/diff


Testing
-------

works well for me so far.


Thanks,

Jekyll

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20110909/9edfbb5c/attachment.html>


More information about the konsole-devel mailing list