diff options
| author | A.Avenel <avenel.alexandre@gmail.com> | 2013-10-12 16:56:25 +0200 |
|---|---|---|
| committer | A.Avenel <avenel.alexandre@gmail.com> | 2013-10-12 16:56:25 +0200 |
| commit | 74e6f9f11db2fa460f2b6f3a8c7863bd46f90fd0 (patch) | |
| tree | 373c1f31f9d6694130c86db46b5eb8fc914b112a /budget/web.py | |
| parent | 8b64762f71f9a558985f4c90a820969ee6fb5ca0 (diff) | |
| download | ihatemoney-mirror-74e6f9f11db2fa460f2b6f3a8c7863bd46f90fd0.zip ihatemoney-mirror-74e6f9f11db2fa460f2b6f3a8c7863bd46f90fd0.tar.gz ihatemoney-mirror-74e6f9f11db2fa460f2b6f3a8c7863bd46f90fd0.tar.bz2 | |
Add unit-test for bug #105.
Small code refactoring for better readability
Diffstat (limited to 'budget/web.py')
| -rw-r--r-- | budget/web.py | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/budget/web.py b/budget/web.py index 44105ca..77de026 100644 --- a/budget/web.py +++ b/budget/web.py @@ -72,12 +72,14 @@ def authenticate(project_id=None): if not form.id.data and request.args.get('project_id'): form.id.data = request.args['project_id'] project_id = form.id.data - project = False - if project_id: - project = Project.query.get(project_id) - else: + if project_id is None: + #User doesn't provide project identifier, return to authenticate form msg = _("You need to enter a project identifier") form.errors["id"] = [msg] + return render_template("authenticate.html", form=form) + else: + project = Project.query.get(project_id) + create_project = False # We don't want to create the project by default if not project: # But if the user try to connect to an unexisting project, we will |
