Chris/devopsexam
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refine Variable name & Comment for better readibility (#1)

Browse Source 
* Update app.py

* Update admin.html
This commit is contained in:
Xiaodong
2017-07-03 18:12:54 +08:00
committed by GitHub
parent 4a2b3fabab
commit 023a149be9
2 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app.py
View File

@@ -85,13 +85,13 @@ def FUN_add_user():
user_table = zip(range(1, len(user_list)+1),\
user_list,\
[x + y for x,y in zip(["/delete_user/"] * len(user_list), user_list)])
return(render_template("admin.html", id_is_duplicated = True, users = user_table))
return(render_template("admin.html", id_to_add_is_duplicated = True, users = user_table))
if " " in request.form.get('id') or "'" in request.form.get('id'):
user_list = list_users()
user_table = zip(range(1, len(user_list)+1),\
user_list,\
[x + y for x,y in zip(["/delete_user/"] * len(user_list), user_list)])
return(render_template("admin.html", id_is_invalid = True, users = user_table))
return(render_template("admin.html", id_to_add_is_invalid = True, users = user_table))
else:
add_user(request.form.get('id'), request.form.get('pw'))
return(redirect(url_for("FUN_admin")))

12
templates/admin.html
View File

@@ -2,14 +2,16 @@
{% block page_title %}Admin Dashboard{% endblock %}
{% block body %}
{{ super() }}
{% if id_is_duplicated %}
{# only invoked when failed adding new ID due to duplication #}
{% if id_to_add_is_duplicated %}
<div class="text-danger">
<strong>Warning!</strong> The account name already exists.
</div>
{% endif %}
{% if id_is_invalid %}
{# only invoked when failed adding new ID due to invalid character #}
{% if id_to_add_is_invalid %}
<div class="text-danger">
<strong>Warning!</strong> The account name is invalid.
</div>
@@ -67,4 +69,4 @@
{% endblock %}
{% endblock %}