Merge lp:~therp-nl/openerp-web/7.0-lp1167429-binary_fields_in_o2m_representation_and_unnecessary_preloading into lp:openerp-web/7.0
Proposed by
Stefan Rijnhart (Opener)
Status: | Needs review |
---|---|
Proposed branch: | lp:~therp-nl/openerp-web/7.0-lp1167429-binary_fields_in_o2m_representation_and_unnecessary_preloading |
Merge into: | lp:openerp-web/7.0 |
Diff against target: |
48 lines (+8/-8) 3 files modified
addons/web/static/src/js/data.js (+1/-1) addons/web/static/src/js/view_form.js (+4/-0) addons/web/static/src/js/view_list.js (+3/-7) |
To merge this branch: | bzr merge lp:~therp-nl/openerp-web/7.0-lp1167429-binary_fields_in_o2m_representation_and_unnecessary_preloading |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
OpenERP Core Team | Pending | ||
Review via email: mp+210418@code.launchpad.net |
Description of the change
Fix binary fields in one2many widgets:
- Don't retrieve the binary contents just to display the size, but pass context with bin_size=True instead
- Always pass filename in download link
Combination of patches from the bug report from Enrico Ganzaroli, Cedric Le Brouster and the last commit from myself which actually prevents the preloading of the binaries.
To post a comment you must log in.
Unmerged revisions
- 4160. By Stefan Rijnhart (Opener)
-
[FIX] o2m dataset's get_context to take additional context as does _super()
- 4159. By Enrico Ganzaroli
-
[FIX] Pass options containing context to read_ids()
- 4158. By Cedric Le Brouster(OpenFire)
-
[FIX] Filename of downloaded binary from list view