Renamed role table into groups, and debug
[cavote.git] / main.py
diff --git a/main.py b/main.py
index e2e7d81..b3f50ba 100755 (executable)
--- a/main.py
+++ b/main.py
@@ -65,7 +65,7 @@ def get_userid():
     user = session.get('user')
     if user is None: 
         return -1
-    elif not user.get('id') > 0:
+    elif user.get('id') < 0:
         return -1
     else:
         return user.get('id')
@@ -121,7 +121,7 @@ def login_key(userid, key):
 def user(userid):
     if int(userid) != get_userid():
         abort(401)
-    groups = query_db('select * from roles join user_role on id=id_role where id_user = ?', userid)
+    groups = query_db('select * from groups join user_group on id=id_group where id_user = ?', userid)
     return render_template('user.html', groups=groups)
 
 @app.route('/user/settings/<userid>', methods=['GET', 'POST'])
@@ -167,18 +167,18 @@ def user_password(userid):
 def admin_users():
     if not session.get('user').get('is_admin'):
         abort(401)
-    tuples = query_db('select *, roles.name as rolename from (select *, id as userid, name as username from users join user_role on id=id_user order by id desc) join roles on id_role=roles.id')
+    tuples = query_db('select *, groups.name as groupname from (select *, id as userid, name as username from users join user_group on id=id_user order by id desc) join groups on id_group=groups.id')
     users = dict()
     for t in tuples:
         if t['userid'] in users:
-            users[t['userid']]['roles'].append(t["rolename"])
+            users[t['userid']]['groups'].append(t["groupname"])
         else:
             users[t['userid']] = dict()
             users[t['userid']]['userid'] = t['userid']
             users[t['userid']]['email'] = t['email']
             users[t['userid']]['username'] = t['username']
             users[t['userid']]['is_admin'] = t['is_admin']
-            users[t['userid']]['roles'] = [t['rolename']]
+            users[t['userid']]['groups'] = [t['groupname']]
 
     return render_template('admin_users.html', users=users.values())
 
@@ -198,11 +198,11 @@ def admin_user_add():
             g.db.commit()
             user = query_db('select * from users where email = ?', [request.form["email"]], one=True)
             if user:
-              for role in request.form.getlist('roles'):
-                  # :TODO:maethor:120528: Check if this role exist
-                  if query_db('select id from roles where id = ?', role, one=True) is None:
+              for group in request.form.getlist('groups'):
+                  # :TODO:maethor:120528: Check if this group exist
+                  if query_db('select id from groups where id = ?', group, one=True) is None:
                       abort(401)
-                  g.db.execute('insert into user_role values (?, ?)', [user['id'], role])
+                  g.db.execute('insert into user_group values (?, ?)', [user['id'], group])
                   g.db.commit()
               # :TODO:maethor:120528: Send mail
               flash(u'Le nouvel utilisateur a été créé avec succès', 'success')
@@ -211,43 +211,43 @@ def admin_user_add():
                 flash(u'Une erreur s\'est produite.', 'error')
         else:
             flash(u"Vous devez spécifier une adresse email.", 'error')
-    groups = query_db('select * from roles where system=0') 
+    groups = query_db('select * from groups where system=0') 
     return render_template('admin_user_new.html', groups=groups)
 
 #-------------
 # Roles admin
 
-@app.route('/admin/roles')
-def admin_roles():
+@app.route('/admin/groups')
+def admin_groups():
     if not session.get('user').get('is_admin'):
         abort(401)
-    roles = query_db('select * from roles')
-    return render_template('admin_roles.html', roles=roles)
+    groups = query_db('select * from groups')
+    return render_template('admin_groups.html', groups=groups)
 
-@app.route('/admin/roles/add', methods=['POST'])
-def admin_role_add():
+@app.route('/admin/groups/add', methods=['POST'])
+def admin_group_add():
     if not session.get('user').get('is_admin'):
         abort(401)
     if request.method == 'POST':
         if request.form['name']:
-            g.db.execute('insert into roles (name) values (?)', [request.form['name']])
+            g.db.execute('insert into groups (name) values (?)', [request.form['name']])
             g.db.commit()
         else:
             flash(u"Vous devez spécifier un nom.", "error")
-    return redirect(url_for('admin_roles'))
+    return redirect(url_for('admin_groups'))
 
-@app.route('/admin/roles/delete/<idrole>')
-def admin_role_del(idrole):
+@app.route('/admin/groups/delete/<idgroup>')
+def admin_group_del(idgroup):
     if not session.get('user').get('is_admin'):
         abort(401)
-    role = query_db('select * from roles where id = ?', [idrole], one=True)
-    if role is None:
+    group = query_db('select * from groups where id = ?', [idgroup], one=True)
+    if group is None:
         abort(404)
-    if role['system']:
+    if group['system']:
         abort(401)
-    g.db.execute('delete from roles where id = ?', [idrole])
+    g.db.execute('delete from groups where id = ?', [idgroup])
     g.db.commit()
-    return redirect(url_for('admin_roles'))
+    return redirect(url_for('admin_groups'))
 
 #------------
 # Votes list
@@ -256,7 +256,7 @@ def admin_role_del(idrole):
 def votes(votes):
     today = date.today()
     active_button = votes
-    basequery = 'select *, roles.name as rolename from votes join roles on roles.id=votes.id_role where is_open=1'
+    basequery = 'select *, votes.id as voteid, groups.name as groupname from votes join groups on groups.id = id_group where is_open=1'
     if votes == 'all':
         votes = query_db(basequery + ' order by id desc')
     elif votes == 'archive':
@@ -293,7 +293,7 @@ def has_voted(idvote, iduser=-1):
 
 @app.route('/vote/<idvote>', methods=['GET', 'POST'])
 def vote(idvote):
-    vote = query_db('select *, roles.name as rolename from votes join roles on roles.id=votes.id_role where votes.id=?', [idvote], one=True)
+    vote = query_db('select votes.*, groups.name as groupname from votes join groups on groups.id=votes.id_group where votes.id=?', [idvote], one=True)
     if vote is None:
         abort(404)
     if can_see_vote(idvote, get_userid()):
@@ -319,15 +319,15 @@ def vote(idvote):
                 users[t['userid']]['userid'] = t['userid']
                 users[t['userid']]['username'] = t['username']
                 users[t['userid']]['choices'] = [t['choiceid']]
-        choices = query_db('select choices.name, choices.id, choices.name, choices.id_vote, count(id_choice) as nb from choices left join user_choice on id_choice = choices.id where id_vote = ? group by id_choice, name, id_vote', [idvote])
+        choices = query_db('select choices.name, choices.id, choices.name, choices.id_vote, count(id_choice) as nb from choices left join user_choice on id_choice = choices.id where id_vote = ? group by id_choice, name, id_vote order by id', [idvote])
         attachments = query_db('select * from attachments where id_vote=?', [idvote])
         return render_template('vote.html', vote=vote, attachments=attachments, choices=choices, users=users.values(), can_vote=can_vote(idvote, get_userid()))
     flash('Vous n\'avez pas le droit de voir ce vote, désolé.')
     return(url_for('home'))
 
 @app.route('/vote/deletechoices/<idvote>/<iduser>')
-def vote_deletechoices(idvote):
-    if (not has_voted(idvote, get_userid())) or iduser != get_userid():
+def vote_deletechoices(idvote, iduser):
+    if int(iduser) != get_userid():
         abort(401)
     g.db.execute('delete from user_choice where id_user = ? and id_choice in (select id from choices where id_vote = ?)',
         [iduser, idvote])
@@ -341,7 +341,7 @@ def vote_deletechoices(idvote):
 def admin_votes():
     if not session.get('user').get('is_admin'):
         abort(401)
-    votes = query_db('select *, roles.name as rolename from votes join roles on roles.id=votes.id_role order by id desc')
+    votes = query_db('select *, votes.id as voteid, groups.name as groupname from votes join groups on groups.id=votes.id_group order by id desc')
     return render_template('admin_votes.html', votes=votes)
 
 @app.route('/admin/votes/add', methods=['GET', 'POST'])
@@ -361,11 +361,11 @@ def admin_vote_add():
                 public = 1
             if 'multiplechoice' in request.form.keys():
                 multiplechoice = 1
-            role = query_db('select id from roles where name = ?', [request.form['role']], one=True)
-            if role is None:
-                role[id] = 1
-            g.db.execute('insert into votes (title, description, category, date_begin, date_end, is_transparent, is_public, is_multiplechoice, id_role, id_author) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)',
-                    [request.form['title'], request.form['description'], request.form['category'], date_begin, date_end, transparent, public, multiplechoice, role['id'], session['user']['id']])
+            group = query_db('select id from groups where name = ?', [request.form['group']], one=True)
+            if group is None:
+                group[id] = 1
+            g.db.execute('insert into votes (title, description, category, date_begin, date_end, is_transparent, is_public, is_multiplechoice, id_group, id_author) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)',
+                    [request.form['title'], request.form['description'], request.form['category'], date_begin, date_end, transparent, public, multiplechoice, group['id'], session['user']['id']])
             g.db.commit()
             vote = query_db('select * from votes where title = ? and date_begin = ? order by id desc', 
                     [request.form['title'], date_begin], one=True) # :DEBUG:maethor:120528: Bug possible car le titre n'est pas unique
@@ -377,7 +377,7 @@ def admin_vote_add():
                 return redirect(url_for('admin_vote_edit', voteid=vote['id']))
         else:
             flash(u'Vous devez spécifier un titre.', 'error')
-    groups = query_db('select * from roles') 
+    groups = query_db('select * from groups') 
     return render_template('admin_vote_new.html', groups=groups)
 
 @app.route('/admin/votes/edit/<voteid>', methods=['GET', 'POST'])
@@ -409,7 +409,7 @@ def admin_vote_edit(voteid):
 
     # :TODO:maethor:120529: Calculer la durée du vote (différence date_end - date_begin) 
     vote['duration'] = 15
-    group = query_db('select name from roles where id = ?', [vote['id_role']], one=True) 
+    group = query_db('select name from groups where id = ?', [vote['id_group']], one=True) 
     choices = query_db('select * from choices where id_vote = ?', [voteid])
     attachments = query_db('select * from attachments where id_vote = ?', [voteid])
     return render_template('admin_vote_edit.html', vote=vote, group=group, choices=choices, attachments=attachments)