<div dir="auto"><div>Hello, </div><div dir="auto"><br></div><div dir="auto">Assuming itʼs WTForms, you might want to take a look at form.populate_obj() (in the if branch) and form = Form(obj=ir) (before the if statement), while completely dropping tho else clause.<div dir="auto"><br></div><div dir="auto">The problem with your approach is that your else branch will trigger if a POST operation is in place, but form validation fails. In such a case, right now you drop user data and change it to the data stared in ir.</div><div dir="auto"><br></div><div dir="auto">If you have multiple SQLAlchemy models you may also want to look into the wforms-sqlalchemy package.</div><div dir="auto"><br></div><div dir="auto">Best,</div><div dir="auto">Gergely</div><br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Mon, 9 Sep 2019, 15:41 Corey Boyle, <<a href="mailto:coreybrett@gmail.com">coreybrett@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I have this in one of my views...<br><br><font face="monospace">        if form.validate_on_submit():<br>            ir.notes = form.notes.data<br>            ir.direction = form.direction.data<br>            ir.insp_date = form.insp_date.data<br>            ir.in_out_date = form.in_out_date.data<br>            ir.customer_pk = form.customer.data<br>            ir.branch_pk = form.branch.data<br>            db.session.add(ir)<br>            db.session.commit()<br>            return redirect(url_for('sales.inspectionreport', irpk=<a href="http://ir.pk" target="_blank" rel="noreferrer">ir.pk</a>))<br>        else:<br>            form.notes.data = ir.notes<br>            form.direction.data = ir.direction<br>            form.insp_date.data = ir.insp_date<br>            form.in_out_date.data = ir.in_out_date<br>            form.customer.data = ir.customer_pk<br>            form.branch.data = ir.branch_pk</font><div><br></div><div>The else does all the same assignments as the if, but in reverse order.</div><div><br></div><div>Is there a better way to write this?</div></div>
_______________________________________________<br>
Flask mailing list<br>
<a href="mailto:Flask@python.org" target="_blank" rel="noreferrer">Flask@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/flask" rel="noreferrer noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/flask</a><br>
</blockquote></div></div></div>