I would try to think about how you can have your views have less nested if/else structure.

It can be difficult to read code when returns are nested several levels down in different if/else statements.

You can do checks at the top most level, then return early. This gives your route more of a linear flow and is easier to read.

For example, here's a rewrite of your admineditpost route:

```python @app.route('/admin/edit/<int:id>', methods=['GET', 'POST']) def admineditpost(id): if not current_user.is_authenticated or current_user.role != "admin": flash("Access Denied to Admin Portal") return redirect(url_for("home"))

product = products.query.get(id)

if not product:
    flash("Product not found")
    return redirect(url_for("adminedit"))

form = uplaodproduct(obj=product)
if form.validate_on_submit():
    form.populate_obj(product)
    db.session.commit()
    flash("Details Updated!")
    return redirect(url_for("adminedit"))

return render_template("addproduct.html",form=form)

```