Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Hi I have finish the storage refactor, so please give me your feedback ! Also I have move this MP on github so let's comment here = https://github.com/OCA/server-tools/pull/1 -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Review: Resubmit pr created on github -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Hello Sébastien, some PEP8 improvements here https://code.launchpad.net/~agilebg/server-env-tools/binary_field_pep8/+merge/223935 -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Hi I am asking myself about adding an object storage.configuration and a field storing_configuration_id on the model ir.model.fields. The idea will to use a default storing configuration for every field (on file sytem for exemple) but after we will be able to customise on every field from the backend where where you want to store each binary/image fields (database, S3, ftp, SFTP, Filestore) Do you like the idea? (not so hard to implement) -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Hi Georges, Sorry I fix the description. In my case I want to store every binary/image field on the file system and not in postgresql. And this not only for attachement but for every image/binary field (image field on product, on category) I want this because I want to serve my image directly with nginx without using Postgresql, OpenERP. You storage on Large Object seem really interesting and as I can pass a custom storage class to my field maybe we can create a new custom class for storing in Large Object and reuse you work. I add a doc string and update the description, give me your feedback Thanks -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Hi Sébastien, could you be a bit more specific in the branch description, what's the intended scope/purpose, that kind of thing ? Maybe as a short, committed doc file ? Am I right to believe that you did a new field parly to avoid the base64 roundtrips ? Maybe I overlooked it, but the controllers need serious work too (current OpenERP 7 impl is feeble to say the least. BTW, I'm interested because of the Postgres Large Object support we published on Bitbucket (https://bitbucket.org/anybox/advanced_attachment/src/37ba6c51432bb106b30c500a00b89ff91c9e5ccc/attachment_large_object/?at=default) Thanks ! -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
[Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
The proposal to merge lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is requested to review the proposed merge of lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
[Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Sébastien BEAU - http://www.akretion.com has proposed merging
lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools.
Requested reviews:
Server Environment And Tools Core Editors (server-env-tools-core-editors)
For more details, see:
https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91
Here is a proposal too add new fields for image and binary
--
https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91
Your team Server Environment And Tools Core Editors is requested to review the
proposed merge of lp:~akretion-team/server-env-tools/server-env-tools into
lp:server-env-tools.
=== added directory 'binary_field'
=== added file 'binary_field/__init__.py'
--- binary_field/__init__.py 1970-01-01 00:00:00 +
+++ binary_field/__init__.py 2014-06-06 09:01:45 +
@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+###
+#
+# Module for OpenERP
+# Copyright (C) 2013 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU sebastien.b...@akretion.com
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program. If not, see http://www.gnu.org/licenses/.
+#
+###
+
+from . import fields
=== added file 'binary_field/__openerp__.py'
--- binary_field/__openerp__.py 1970-01-01 00:00:00 +
+++ binary_field/__openerp__.py 2014-06-06 09:01:45 +
@@ -0,0 +1,50 @@
+# -*- coding: utf-8 -*-
+###
+#
+# Module for OpenERP
+# Copyright (C) 2013 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU sebastien.b...@akretion.com
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program. If not, see http://www.gnu.org/licenses/.
+#
+###
+
+{'name': 'Binary Field',
+ 'version': '0.0.1',
+ 'author': 'Akretion',
+ 'website': 'www.akretion.com',
+ 'license': 'AGPL-3',
+ 'category': 'Framework',
+ 'description': This module extend the fields class in order to add 3 new
+type of fields
+- BinaryStore
+- ImageStore
+- ImageRezise
+
+Now let's do it ;)
+
+ ,
+ 'depends': [
+ 'base',
+ ],
+ 'data': [
+ 'data.xml',
+ ],
+ 'installable': True,
+ 'application': True,
+}
+
+
+
+
=== added file 'binary_field/data.xml'
--- binary_field/data.xml 1970-01-01 00:00:00 +
+++ binary_field/data.xml 2014-06-06 09:01:45 +
@@ -0,0 +1,9 @@
+?xml version=1.0 encoding=UTF-8?
+openerp
+data
+record id=binary_storage_parameter model=ir.config_parameter
+field name=keybinary.location/field
+field name=valuefile:///filestore/field
+/record
+/data
+/openerp
=== added file 'binary_field/fields.py'
--- binary_field/fields.py 1970-01-01 00:00:00 +
+++ binary_field/fields.py 2014-06-06 09:01:45 +
@@ -0,0 +1,249 @@
+# -*- coding: utf-8 -*-
+###
+#
+# Module for OpenERP
+# Copyright (C) 2014 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU sebastien.b...@akretion.com
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the
[Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
The proposal to merge lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools has been updated. Description changed to: Here is a proposal too add new fields for image and binary. There is a example module so you can try it. But before merging I propose to remove it. For more details, see: https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 -- https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91 Your team Server Environment And Tools Core Editors is requested to review the proposed merge of lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community-reviewer More help : https://help.launchpad.net/ListHelp
[Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools
Sébastien BEAU - http://www.akretion.com has proposed merging
lp:~akretion-team/server-env-tools/server-env-tools into lp:server-env-tools.
Requested reviews:
Server Environment And Tools Core Editors (server-env-tools-core-editors)
For more details, see:
https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91
Here is a proposal too add new fields for image and binary
--
https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91
Your team Server Environment And Tools Core Editors is requested to review the
proposed merge of lp:~akretion-team/server-env-tools/server-env-tools into
lp:server-env-tools.
=== added directory 'binary_field'
=== added file 'binary_field/__init__.py'
--- binary_field/__init__.py 1970-01-01 00:00:00 +
+++ binary_field/__init__.py 2014-06-06 09:01:45 +
@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+###
+#
+# Module for OpenERP
+# Copyright (C) 2013 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU sebastien.b...@akretion.com
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program. If not, see http://www.gnu.org/licenses/.
+#
+###
+
+from . import fields
=== added file 'binary_field/__openerp__.py'
--- binary_field/__openerp__.py 1970-01-01 00:00:00 +
+++ binary_field/__openerp__.py 2014-06-06 09:01:45 +
@@ -0,0 +1,50 @@
+# -*- coding: utf-8 -*-
+###
+#
+# Module for OpenERP
+# Copyright (C) 2013 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU sebastien.b...@akretion.com
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program. If not, see http://www.gnu.org/licenses/.
+#
+###
+
+{'name': 'Binary Field',
+ 'version': '0.0.1',
+ 'author': 'Akretion',
+ 'website': 'www.akretion.com',
+ 'license': 'AGPL-3',
+ 'category': 'Framework',
+ 'description': This module extend the fields class in order to add 3 new
+type of fields
+- BinaryStore
+- ImageStore
+- ImageRezise
+
+Now let's do it ;)
+
+ ,
+ 'depends': [
+ 'base',
+ ],
+ 'data': [
+ 'data.xml',
+ ],
+ 'installable': True,
+ 'application': True,
+}
+
+
+
+
=== added file 'binary_field/data.xml'
--- binary_field/data.xml 1970-01-01 00:00:00 +
+++ binary_field/data.xml 2014-06-06 09:01:45 +
@@ -0,0 +1,9 @@
+?xml version=1.0 encoding=UTF-8?
+openerp
+data
+record id=binary_storage_parameter model=ir.config_parameter
+field name=keybinary.location/field
+field name=valuefile:///filestore/field
+/record
+/data
+/openerp
=== added file 'binary_field/fields.py'
--- binary_field/fields.py 1970-01-01 00:00:00 +
+++ binary_field/fields.py 2014-06-06 09:01:45 +
@@ -0,0 +1,249 @@
+# -*- coding: utf-8 -*-
+###
+#
+# Module for OpenERP
+# Copyright (C) 2014 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU sebastien.b...@akretion.com
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the
