Mercurial > mamba
changeset 277:82aa91664e27
Warn on save failures
author | Neil Muller <drnlmuller@gmail.com> |
---|---|
date | Thu, 15 Sep 2011 18:01:43 +0200 |
parents | 9c57025781e7 |
children | f10d06fc64b2 |
files | mamba/habitats/editor.py |
diffstat | 1 files changed, 14 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/mamba/habitats/editor.py Thu Sep 15 17:57:28 2011 +0200 +++ b/mamba/habitats/editor.py Thu Sep 15 18:01:43 2011 +0200 @@ -148,11 +148,14 @@ self.current_tool.prepare() def save(self, ev, widget): - if self.level.level_name in RESERVED_NAMES: - print "Can't save over reserved name" - return - if not self.level: - print "Need a level name" + messagebox = None + if not self.level.level_name: + messagebox = MessageBox((300, 300), 'Please enter a name') + elif self.level.level_name in RESERVED_NAMES: + messagebox = MessageBox((300, 300), 'Reserved level name') + if messagebox: + self.container.add(messagebox) + messagebox.grab_focus() return self.level.save_level() @@ -234,7 +237,12 @@ return True # No-op change if not new_name: messagebox = MessageBox((300, 300), 'Please enter a name') - if check_level_exists(new_name): + if new_name in RESERVED_NAMES: + # This case is caught by the existance check, but the + # importance of the reserved names means we use a different + # message + messagebox = MessageBox((300, 300), 'Reserved level name') + elif check_level_exists(new_name): messagebox = MessageBox((300, 300), 'Name already in use') if messagebox: self.container.add(messagebox)