{"action":"created","issue":{"url":"https://api.github.com/repos/OCA/bank-statement-import/issues/34","labels_url":"https://api.github.com/repos/OCA/bank-statement-import/issues/34/labels{/name}","comments_url":"https://api.github.com/repos/OCA/bank-statement-import/issues/34/comments","events_url":"https://api.github.com/repos/OCA/bank-statement-import/issues/34/events","html_url":"https://github.com/OCA/bank-statement-import/issues/34","id":98134711,"number":34,"title":"acc_number vs. sanitized_acc_number","user":{"login":"hbrunn","id":2563186,"avatar_url":"https://avatars.githubusercontent.com/u/2563186?v=3","gravatar_id":"","url":"https://api.github.com/users/hbrunn","html_url":"https://github.com/hbrunn","followers_url":"https://api.github.com/users/hbrunn/followers","following_url":"https://api.github.com/users/hbrunn/following{/other_user}","gists_url":"https://api.github.com/users/hbrunn/gists{/gist_id}","starred_url":"https://api.github.com/users/hbrunn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hbrunn/subscriptions","organizations_url":"https://api.github.com/users/hbrunn/orgs","repos_url":"https://api.github.com/users/hbrunn/repos","events_url":"https://api.github.com/users/hbrunn/events{/privacy}","received_events_url":"https://api.github.com/users/hbrunn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-30T10:22:37Z","updated_at":"2015-07-30T10:47:16Z","closed_at":null,"body":"I'm a bit bothered by the data duplication that `sanitized_acc_number` introduces and the hoops we've got to jump through in https://github.com/OCA/bank-statement-import/blob/8.0/account_bank_statement_import/res_partner_bank.py#L49 to make it work as users would expect.\r\n\r\nWhat is the rationale behind this?\r\n\r\nIf it's only for reliably finding the right account number, I'd suggest the following:\r\n\r\n- [ ] sanitize existing account numbers in an init hook\r\n- [ ] sanitize input during `create` and `write`\r\n- [ ] provide a form/tree widget for pretty formatting of IBANs (that is made is a way to be easily overridden in localization modules)\r\n- [ ] provide a search widget that sanitizes IBANs for searching\r\n\r\nWhat do you think?"},"comment":{"url":"https://api.github.com/repos/OCA/bank-statement-import/issues/comments/126271365","html_url":"https://github.com/OCA/bank-statement-import/issues/34#issuecomment-126271365","issue_url":"https://api.github.com/repos/OCA/bank-statement-import/issues/34","id":126271365,"user":{"login":"pedrobaeza","id":7165771,"avatar_url":"https://avatars.githubusercontent.com/u/7165771?v=3","gravatar_id":"","url":"https://api.github.com/users/pedrobaeza","html_url":"https://github.com/pedrobaeza","followers_url":"https://api.github.com/users/pedrobaeza/followers","following_url":"https://api.github.com/users/pedrobaeza/following{/other_user}","gists_url":"https://api.github.com/users/pedrobaeza/gists{/gist_id}","starred_url":"https://api.github.com/users/pedrobaeza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pedrobaeza/subscriptions","organizations_url":"https://api.github.com/users/pedrobaeza/orgs","repos_url":"https://api.github.com/users/pedrobaeza/repos","events_url":"https://api.github.com/users/pedrobaeza/events{/privacy}","received_events_url":"https://api.github.com/users/pedrobaeza/received_events","type":"User","site_admin":false},"created_at":"2015-07-30T10:47:16Z","updated_at":"2015-07-30T10:47:16Z","body":"I prefer also this approach to avoid a field duality with pretty same data, but requires all the tasks you have mentioned done."}}