Mercurial > getan > getan
diff getan/controller.py @ 135:ce707fbb9666
Change coding style of if clauses
author | Björn Ricks <bjoern.ricks@intevation.de> |
---|---|
date | Wed, 31 Oct 2012 14:21:44 +0100 |
parents | d85b2a25797c |
children | 92b7582b8f44 |
line wrap: on
line diff
--- a/getan/controller.py Wed Oct 31 09:57:20 2012 +0100 +++ b/getan/controller.py Wed Oct 31 14:21:44 2012 +0100 @@ -57,8 +57,10 @@ def update_entries(self, project): logger.debug("GetanController: update entries.") - if project: entries = self.backend.load_entries(project.id) - else: entries = [] + if project: + entries = self.backend.load_entries(project.id) + else: + entries = [] self.entries_view.set_rows(entries) self.view.update_view() @@ -68,7 +70,8 @@ try: while self.entries_view.selection: node = self.entries_view.selection.pop() - if node.selected: node.select() + if node.selected: + node.select() entries.append(node.item) logger.info("GetanController: move entry '%s' (id = %d, "\ "project id = %d) to project '%s'" @@ -79,7 +82,8 @@ old_project = self.project_by_id(node.item.project_id) finally: self.backend.move_entries(entries, project.id) - if not old_project: return + if not old_project: + return project.entries = self.backend.load_entries(project.id) old_project.entries = self.backend.load_entries(old_project.id) self.update_entries(old_project) @@ -115,7 +119,8 @@ return None def start_project(self, project): - if not project: return + if not project: + return self.running.append(project) project.start = datetime.now() logger.info("Start project '%s' at %s." @@ -124,9 +129,11 @@ logger.debug('All running projects: %r' % self.running) def stop_project(self, desc='-no description-', display=True): - if not self.running: return + if not self.running: + return project = self.running.pop() - if not project: return + if not project: + return logger.info("Stop project '%s' at %s." % (project.desc, format_time(datetime.now()))) project.stop = datetime.now()