banking-addons-team team mailing list archive

[Alexandre Fayolle - camptocamp <alexandre.fayolle@xxxxxxxxxxxxxx>]

> My main concern is the generation of an invalid regular expression on lines
> 30-31 : what if the partner.bank_statement_label contains "special" characters
> such as .[]()^$+*? this could lead to a crash in the query.
> 
> Unless we are really really sure this is not possible, there should be some
> escaping performed on this string before converting it to a regex. I think
> there is a re.escape function available in Python to do just this.
> 
> line 37-38: useless, please remove
> 
> line 23: is context['label_memoizer']  used outside this function? If not,
> this could be simply a local variable.
> 
> line 101: get is a method -> use () instead of [] (or if you're sure the key
> is in there, just [] without get)

and l21: s/Follwing/Following/
-- 
https://code.launchpad.net/~camptocamp/banking-addons/improve_lookup/+merge/154730
Your team Banking Addons Team is subscribed to branch lp:banking-addons/bank-statement-reconcile-70.