-
-
Notifications
You must be signed in to change notification settings - Fork 369
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
7.0 Conversion_dict is now created in the generic parser #23
Conversation
super(GenericFileParser, self).__init__(parse_name, ftype=ftype, **kwargs) | ||
conversion_dict = { | ||
'ref': unicode, | ||
'label': unicode, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to use ustr
from openerp.tools
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems better now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better to import
from openerp.tools import ustr
It may be the same function, but openerp.tools seems safer and makes more sense
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, thank you for the information
A new rule for automatic reconciliation and a cron [rebased]
LGTM 👍 |
'date': datetime.datetime, | ||
'amount': float_or_zero, | ||
} | ||
super(GenericFileParser, self).__init__(parse_name, ftype=ftype, extra_fields=conversion_dict, **kwargs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PEP8 80 cols please
Needs to be rebased in order to merge |
rebased! |
Unrelated Travis error, so 👍 |
[IMP] account_statement_base_import: 7.0 Conversion_dict is now created in the generic parser
Fix tax account on chart 1170 -> 2200
The conversion dict is displaced from the file_parser to the generic_parser in order to avoid the error of the validate function when creating a new parser which does not contain the 4 basc fields (ref, amount, date, label)
The old launchpad link :
https://code.launchpad.net/~akretion-team/banking-addons/account_statement_base_import_conversion_dict/+merge/198235