cleaned up some more stuff in the code
This commit is contained in:
@@ -80,51 +80,6 @@ def update_user(mac_address, description, vlan_id):
|
|||||||
cursor.close()
|
cursor.close()
|
||||||
conn.close()
|
conn.close()
|
||||||
|
|
||||||
|
|
||||||
# def update_user_description(mac_address, description):
|
|
||||||
# """Update the description field of a user identified by MAC address."""
|
|
||||||
# conn = get_connection()
|
|
||||||
# cursor = conn.cursor()
|
|
||||||
# cursor.execute("UPDATE users SET description = %s WHERE mac_address = %s", (description, mac_address.lower()))
|
|
||||||
# conn.commit()
|
|
||||||
# cursor.close()
|
|
||||||
# conn.close()
|
|
||||||
|
|
||||||
# Note: This function seems redundant with update_user_description. Included as per instructions.
|
|
||||||
# def update_description(mac_address, description):
|
|
||||||
# """Update the description for a given MAC address in the users table."""
|
|
||||||
# conn = get_connection()
|
|
||||||
# cursor = conn.cursor()
|
|
||||||
# cursor.execute(
|
|
||||||
# "UPDATE users SET description = %s WHERE mac_address = %s",
|
|
||||||
# (description, mac_address.lower())
|
|
||||||
# )
|
|
||||||
# conn.commit()
|
|
||||||
# cursor.close()
|
|
||||||
# conn.close()
|
|
||||||
|
|
||||||
# def update_user_vlan(mac_address, vlan_id):
|
|
||||||
# """Update the VLAN ID for a given MAC address in the users table."""
|
|
||||||
# conn = get_connection()
|
|
||||||
# cursor = conn.cursor()
|
|
||||||
# cursor.execute("UPDATE users SET vlan_id = %s WHERE mac_address = %s", (vlan_id, mac_address.lower()))
|
|
||||||
# conn.commit()
|
|
||||||
# cursor.close()
|
|
||||||
# conn.close()
|
|
||||||
|
|
||||||
# # Note: This function seems redundant with update_user_vlan. Included as per instructions.
|
|
||||||
# def update_vlan(mac_address, vlan_id):
|
|
||||||
# """Update the VLAN ID for a given MAC address in the users table."""
|
|
||||||
# conn = get_connection()
|
|
||||||
# cursor = conn.cursor()
|
|
||||||
# cursor.execute(
|
|
||||||
# "UPDATE users SET vlan_id = %s WHERE mac_address = %s",
|
|
||||||
# (vlan_id, mac_address.lower())
|
|
||||||
# )
|
|
||||||
# conn.commit()
|
|
||||||
# cursor.close()
|
|
||||||
# conn.close()
|
|
||||||
|
|
||||||
def delete_user(mac_address):
|
def delete_user(mac_address):
|
||||||
"""Remove a user from the database by their MAC address."""
|
"""Remove a user from the database by their MAC address."""
|
||||||
conn = get_connection()
|
conn = get_connection()
|
||||||
|
|||||||
@@ -35,22 +35,6 @@ def update_user_route():
|
|||||||
update_user(mac, desc, vlan_id)
|
update_user(mac, desc, vlan_id)
|
||||||
return redirect(url_for('user.user_list'))
|
return redirect(url_for('user.user_list'))
|
||||||
|
|
||||||
# @user.route('/update_description', methods=['POST'])
|
|
||||||
# def update_description_route():
|
|
||||||
# mac = request.form['mac_address']
|
|
||||||
# desc = request.form.get('description', '')
|
|
||||||
# update_description(mac, desc)
|
|
||||||
# return redirect(url_for('user.user_list'))
|
|
||||||
|
|
||||||
|
|
||||||
# @user.route('/update_vlan', methods=['POST'])
|
|
||||||
# def update_vlan_route():
|
|
||||||
# mac = request.form['mac_address']
|
|
||||||
# group_id = request.form['group_id']
|
|
||||||
# update_vlan(mac, group_id)
|
|
||||||
# return redirect(url_for('user.user_list'))
|
|
||||||
|
|
||||||
|
|
||||||
@user.route('/delete', methods=['POST'])
|
@user.route('/delete', methods=['POST'])
|
||||||
def delete():
|
def delete():
|
||||||
mac = request.form['mac_address']
|
mac = request.form['mac_address']
|
||||||
|
|||||||
Reference in New Issue
Block a user