[email protected] wrote: > Except, what is the issue with the new_text variable? That's > my way of moving validity checks out of the transaction, since > they don't belong there and should be done before the transaction > is started. I could make the checks multiple times though, first > to rule out the error case, and then to distinguish between the > two update types. Just makes the code longer and less clear IMHO.
I should have been more explicit, sorry. I was referring to: if(new_text): This looks so much like C code ;-) Use this instead: if new_text: -- Remy
signature.asc
Description: OpenPGP digital signature
