> Hmm not sure if your fix is really ok, though: > > regexp_matches(string text, pattern text [, flags text]) > > it seems to me that you have either inverted the args to that function, or you > end up using re.escape on the wrong parameter. > > Automated test definitely needed.
my bad, the automated test is there. -- https://code.launchpad.net/~acsone-openerp/banking-addons/7.0-bug-1280814-base_completion/+merge/206617 Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0. -- Mailing list: https://launchpad.net/~banking-addons-team Post to : [email protected] Unsubscribe : https://launchpad.net/~banking-addons-team More help : https://help.launchpad.net/ListHelp
