aboutsummaryrefslogtreecommitdiff
path: root/budget/web.py
diff options
context:
space:
mode:
authorA.Avenel <avenel.alexandre@gmail.com>2013-10-12 16:56:25 +0200
committerA.Avenel <avenel.alexandre@gmail.com>2013-10-12 16:56:25 +0200
commit74e6f9f11db2fa460f2b6f3a8c7863bd46f90fd0 (patch)
tree373c1f31f9d6694130c86db46b5eb8fc914b112a /budget/web.py
parent8b64762f71f9a558985f4c90a820969ee6fb5ca0 (diff)
downloadihatemoney-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.py10
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