Renaud Thiry
Renaud Thiry
Closed while in development for some reason, reopening.
@std-odoo Hey, I ended up making this a computed field. It should generally be a very quick compute and it seems to make more sense than having helpers returning dictionaries...
@tde-banana-odoo it avoided having either having a ``record.foo()[record.id]`` or separate methods for batch and single records. But I guess I can find some other way around it. -> Added a...
@std-odoo fair enough, updated then. Sorry for the delay, I was batteling with timezone bugs :)
Have a fix and a remove. After speaking with std, it should be fine to remove the MAKO pattern 'unsanitization'. But I leave the fix in fox now in case...
@std-odoo I'm also not sure what to do about 23e0ab1bbd99aea1451a8edbf634334b9aaf26c3 Should I maybe create a separate PR? Pretty sure the issue is in 16.0 too
Hello :wave: @std-odoo I moved it all* back to event. team js merged that other fix in another PR. @xmo-odoo applied changes based on your comments, left comments for those...
Note: it seems like the ``replace('&nbps;')`` under MAKO was not part of it (and was fixed recently to add ``cleaned =`` so that it actually does something), and a test...
Although @stefanorigano I see it's not validated by jija, I think she's available today/this week (?) so might be nice to try and get her to have a look. Maybe...