Tryton - Issues

 

Issue5692

Title Allow to store Binary field in filestore
Priority feature Status resolved
Superseder Allow to reuse the attachment file storage outside of attachment
View: 5686
Nosy List ced, reviewbot, roundup-bot
Type feature request Components trytond
Assigned To ced Keywords review
Reviews 25471003
View: 25471003

Created on 2016-07-07.15:17:21 by ced, last changed by roundup-bot.

Messages
New changeset a8d3c527a445 by C?dric Krier in branch 'default':
Allow to store Binary field in filestore
http://hg.tryton.org/trytond/rev/a8d3c527a445
review25471003 updated at https://codereview.tryton.org/25471003/#ps40001
New review25471003 at https://codereview.tryton.org/25471003/#ps20001
msg26904 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2016-07-07.15:25:59
For the record, the code is written in such way that it is possible to switch from database to filestorage transparently. This means that if there is no filestore id, the field will try to read from the database.
The other way, is not supported. So it is not possible without a migration to change the configuration from filestore to database. But this is normally an option that should not be needed as users will switch to filestore because of performance needs.
msg26903 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2016-07-07.15:20:20
Here is review25471003 on top of review22141003
msg26902 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2016-07-07.15:17:18
With issue5686, it is now possible to use the filestore for any kind of data.
To ease this usage, we could add an option on Binary field to store in the filestore instead of the database. Then the store id should be stored in a Char field and read from this same field.
History
Date User Action Args
2016-07-29 10:53:14roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg27446
2016-07-14 07:00:24reviewbotsetmessages: + msg27074
2016-07-07 15:30:22reviewbotsetnosy: + reviewbot
messages: + msg26907
2016-07-07 15:30:18reviewbotsetreviews: 25471003
keyword: + review
2016-07-07 15:26:18cedlinkissue5693 superseder
2016-07-07 15:25:59cedsetmessages: + msg26904
2016-07-07 15:20:21cedsetstatus: in-progress -> testing
messages: + msg26903
2016-07-07 15:17:21cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)