[Getan-commits] [PATCH 12 of 32] We already have the project here therefore pass it directly to the methods

Wald Commits scm-commit at wald.intevation.org
Fri Oct 11 14:33:56 CEST 2013


# HG changeset patch
# User Björn Ricks <bjoern.ricks at intevation.de>
# Date 1381482322 -7200
# Node ID b97d18d58f85efb1074797631fb3ca0c903154d7
# Parent  afe36c18417fb1f495499a306b4b0bbd7bea07a2
We already have the project here therefore pass it directly to the methods

Using item_in_focus may be dangerous because a user may have already selected
another element.

diff -r afe36c18417f -r b97d18d58f85 getan/states.py
--- a/getan/states.py	Fri Oct 11 11:03:50 2013 +0200
+++ b/getan/states.py	Fri Oct 11 11:05:22 2013 +0200
@@ -168,9 +168,8 @@
         if proj:
             self.reset()
             self.view.select_project(proj)
-            self.controller.start_project(self.view.item_in_focus())
-            self.controller.update_entries(
-                self.view.item_in_focus())
+            self.controller.start_project(proj)
+            self.controller.update_entries(proj)
             self.set_next_state(
                 RunningProjectsState(self.controller, self.view,
                                      proj))


More information about the Getan-commits mailing list