Re: [Openerp-community-reviewer] [Question #257730]: Redmine - Odoo connector

2014-11-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Question #257730 on Redmine - OpenERP Connector changed:
https://answers.launchpad.net/redmine-openerp/+question/257730

Status: Open = Answered

Maxime Chambreuil (http://www.savoirfairelinux.com) proposed the following 
answer:
Hello Boujraf,

There is no plan yet.

-- 
You received this question notification because you are a member of
Redmine OpenERP Core Editors, which is an answer contact for Redmine -
OpenERP Connector.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0

2014-06-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read into 
lp:knowledge-addons/7.0.

-- 
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:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-14 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-fix-1319391/+merge/219530
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/openerp-manufacturing/70-add-mrp-repair-extension into lp:openerp-manufacturing

2014-04-26 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~akretion-team/openerp-manufacturing/70-add-mrp-repair-extension/+merge/211696
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:openerp-manufacturing.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing test

@ehdem

l205: Can you put CMIS versions (1.0 or 1.1, not sure) ? 1.7 is a Magento 
version.

Thanks
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing test

I got a stacktrace instead of an error message when testing the connection: 

Server Traceback (most recent call last):
  File /home/max/openerp/cmis/bzr/web/addons/web/session.py, line 89, in send
return openerp.netsvc.dispatch_rpc(service_name, method, args)
  File /opt/openerp/7.0/server/openerp/netsvc.py, line 296, in dispatch_rpc
result = ExportService.getService(service_name).dispatch(method, params)
  File /opt/openerp/7.0/server/openerp/service/web_services.py, line 626, in 
dispatch
res = fn(db, uid, *params)
  File /opt/openerp/7.0/server/openerp/osv/osv.py, line 190, in execute_kw
return self.execute(db, uid, obj, method, *args, **kw or {})
  File /opt/openerp/7.0/server/openerp/osv/osv.py, line 132, in wrapper
return f(self, dbname, *args, **kwargs)
  File /opt/openerp/7.0/server/openerp/osv/osv.py, line 199, in execute
res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File /opt/openerp/7.0/server/openerp/osv/osv.py, line 187, in execute_cr
return getattr(object, method)(cr, uid, *args, **kw)
  File /home/max/openerp/cmis/bzr/knowledge/cmis/cmis_model.py, line 68, in 
check_directory_of_write
repo = client.defaultRepository
  File 
/home/max/openerp/cmis/git/openerp/tools/eggs/cmislib-0.5.1-py2.7.egg/cmislib/model.py,
 line 179, in getDefaultRepository
doc = self.get(self.repositoryUrl, **self.extArgs)
  File 
/home/max/openerp/cmis/git/openerp/tools/eggs/cmislib-0.5.1-py2.7.egg/cmislib/model.py,
 line 206, in get
**kwargs)
  File 
/home/max/openerp/cmis/git/openerp/tools/eggs/cmislib-0.5.1-py2.7.egg/cmislib/net.py,
 line 145, in get
return opener.open(request)
  File /usr/lib/python2.7/urllib2.py, line 404, in open
response = self._open(req, data)
  File /usr/lib/python2.7/urllib2.py, line 422, in _open
'_open', req)
  File /usr/lib/python2.7/urllib2.py, line 382, in _call_chain
result = func(*args)
  File /usr/lib/python2.7/urllib2.py, line 1214, in http_open
return self.do_open(httplib.HTTPConnection, req)
  File /usr/lib/python2.7/urllib2.py, line 1184, in do_open
raise URLError(err)
URLError: urlopen error [Errno 111] Connexion refusée
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
test
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
test
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis_write into lp:knowledge-addons/7.0

2014-04-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_write/+merge/213940
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0

2014-04-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:knowledge-addons/7.0.

-- 
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:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
I just fixed the branch according to your reviews.

Thanks.
-- 
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744
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:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl.

Requested reviews:
  Sale Core Editors (sale-core-editors)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking/+merge/216323
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking/+merge/216323
Your team Sale Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl.
=== added directory 'sale_lot_tracking'
=== added file 'sale_lot_tracking/README'
=== added file 'sale_lot_tracking/__init__.py'
--- sale_lot_tracking/__init__.py	1970-01-01 00:00:00 +
+++ sale_lot_tracking/__init__.py	2014-04-17 13:51:05 +
@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Savoir-faire Linux (http://www.savoirfairelinux.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/.
+#
+##
+
+import sale
+

=== added file 'sale_lot_tracking/__openerp__.py'
--- sale_lot_tracking/__openerp__.py	1970-01-01 00:00:00 +
+++ sale_lot_tracking/__openerp__.py	2014-04-17 13:51:05 +
@@ -0,0 +1,51 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Savoir-faire Linux (http://www.savoirfairelinux.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/.
+#
+##
+
+# NOTE: The name of the supplied field was initially display_name, but it seems that OpenERP,
+# whenever it seems name in the field, returns the value for name. Well...
+
+{
+'name': 'sale_lot_tracking',
+'version': '1.0',
+'author': 'Savoir-faire Linux',
+'website': 'http://www.savoirfairelinux.com',
+'category': 'Generic Modules/Purchases',
+'description': 
+Lets you track sales using product's lots numbers
+
+
+This module lets you track sales using a product's lot
+number.
+
+
+,
+'depends': ['base', 
+'sale_landed_costs', 
+'purchase_landed_costs',
+'analytic'],
+'data': [
+
+],
+'demo': [],
+'test': [],
+'installable': True,
+'active': False,
+}

=== added file 'sale_lot_tracking/sale.py'
--- sale_lot_tracking/sale.py	1970-01-01 00:00:00 +
+++ sale_lot_tracking/sale.py	2014-04-17 13:51:05 +
@@ -0,0 +1,123 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Savoir-faire Linux (http://www.savoirfairelinux.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

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking 
into lp:sale-wkfl has been updated.

Description changed to:

[ADD] sale_lot_tracking

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking/+merge/216323
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking/+merge/216323
Your team Sale Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl.

-- 
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:~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast into lp:purchase-wkfl

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast into 
lp:purchase-wkfl.

Commit message:
[ADD] Add purchase profit forecast

Requested reviews:
  Purchase Core Editors (purchase-core-editors)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast/+merge/216326

[ADD] Add purchase profit forecast
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast/+merge/216326
Your team Purchase Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast into 
lp:purchase-wkfl.
=== added directory 'purchase_profit_forecast'
=== added file 'purchase_profit_forecast/__init__.py'
--- purchase_profit_forecast/__init__.py	1970-01-01 00:00:00 +
+++ purchase_profit_forecast/__init__.py	2014-04-17 13:53:57 +
@@ -0,0 +1,21 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (c) 2010-2013 Savoir-faire Linux. All Rights Reserved.
+#
+#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/.
+#
+##
+import report

=== added file 'purchase_profit_forecast/__openerp__.py'
--- purchase_profit_forecast/__openerp__.py	1970-01-01 00:00:00 +
+++ purchase_profit_forecast/__openerp__.py	2014-04-17 13:53:57 +
@@ -0,0 +1,40 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (c) 2010-2013 Savoir-faire Linux. All Rights Reserved.
+#
+#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': 'Purchase profit forecast',
+'version': '1.1',
+'category': 'Finance',
+'description': 
+,
+'author': 'Savoir-faire Linux',
+'website': 'http://www.savoirfairelinux.com/',
+'depends': ['account', 'purchase'],
+'data': [
+'report/purchase_profit_view.xml',
+],
+'test': [
+],
+'demo': [],
+'installable': True,
+'active': False,
+'certificate': False,
+}

=== added directory 'purchase_profit_forecast/report'
=== added file 'purchase_profit_forecast/report/__init__.py'
--- purchase_profit_forecast/report/__init__.py	1970-01-01 00:00:00 +
+++ purchase_profit_forecast/report/__init__.py	2014-04-17 13:53:57 +
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (c) 2010-2013 Savoir-faire Linux. All Rights Reserved.
+#
+#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 purchase
+from . import

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge into lp:account-invoicing

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge into 
lp:account-invoicing.

Commit message:
[ADD] account_invoice_purchase_origin module

Requested reviews:
  Account Core Editors (account-core-editors)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge/+merge/216327

[ADD] account_invoice_purchase_origin module
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge/+merge/216327
Your team Account Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge into 
lp:account-invoicing.
=== added directory 'account_invoice_purchase_origin'
=== added file 'account_invoice_purchase_origin/__init__.py'
--- account_invoice_purchase_origin/__init__.py	1970-01-01 00:00:00 +
+++ account_invoice_purchase_origin/__init__.py	2014-04-17 13:55:12 +
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (c) 2010-2013 Savoir-faire Linux. All Rights Reserved.
+#
+#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/.
+#
+##
+
+import purchase

=== added file 'account_invoice_purchase_origin/__openerp__.py'
--- account_invoice_purchase_origin/__openerp__.py	1970-01-01 00:00:00 +
+++ account_invoice_purchase_origin/__openerp__.py	2014-04-17 13:55:12 +
@@ -0,0 +1,42 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (c) 2010-2013 Savoir-faire Linux. All Rights Reserved.
+#
+#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': 'Account Invoice Purchase Origin',
+'version': '1.1',
+'category': 'Finance',
+'description': 
+This module displays the origin field of account invoice lines and populate
+them when an invoice is created from a purchase order
+,
+'author': 'Savoir-faire Linux',
+'website': 'http://www.savoirfairelinux.com/',
+'depends': ['account', 'purchase'],
+'data': [
+'account_invoice_view.xml',
+],
+'test': [
+],
+'demo': [],
+'installable': True,
+'active': False,
+'certificate': False,
+}

=== added file 'account_invoice_purchase_origin/account_invoice_view.xml'
--- account_invoice_purchase_origin/account_invoice_view.xml	1970-01-01 00:00:00 +
+++ account_invoice_purchase_origin/account_invoice_view.xml	2014-04-17 13:55:12 +
@@ -0,0 +1,14 @@
+?xml version=1.0 encoding=utf-8?
+openerp
+  data
+record id=view_invoice_line_tree model=ir.ui.view
+  field name=modelaccount.invoice/field
+  field name=inherit_id ref=account.invoice_supplier_form /
+  field name=arch type=xml
+xpath expr=//tree//field[@name='account_analytic_id'] position=after
+  field name=origin readonly=1 /
+/xpath
+  /field
+/record
+  /data
+/openerp

=== added file 'account_invoice_purchase_origin/purchase.py'
--- account_invoice_purchase_origin/purchase.py	1970-01-01 00:00:00 +
+++ account_invoice_purchase_origin/purchase.py	2014-04-17 13:55:12 +
@@ -0,0 +1,42 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/lp-community-utils/checkout-pep8 into lp:lp-community-utils

2014-03-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/checkout-pep8/+merge/211098
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:lp-community-utils.

-- 
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:~savoirfairelinux-openerp/server-env-tools/l10n into lp:server-env-tools

2014-03-14 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/server-env-tools/l10n/+merge/20
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:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406 into lp:openerp-canada

2014-02-13 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406 into 
lp:openerp-canada.

Requested reviews:
  OpenERP Community Reviewer/Maintainer (openerp-community-reviewer)
Related bugs:
  Bug #1213406 in OpenERP Canada: QPP - Employer Contribution (QPPERC) is 
include in Deduction Category
  https://bugs.launchpad.net/openerp-canada/+bug/1213406

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406/+merge/206286
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406/+merge/206286
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of lp:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406 
into lp:openerp-canada.
=== modified file 'l10n_ca_qc_hr_payroll/l10n_ca_qc_hr_payroll_data.xml'
--- l10n_ca_qc_hr_payroll/l10n_ca_qc_hr_payroll_data.xml	2013-03-08 15:20:04 +
+++ l10n_ca_qc_hr_payroll/l10n_ca_qc_hr_payroll_data.xml	2014-02-13 21:52:47 +
@@ -309,7 +309,7 @@
 field name=nameQPP - Employer Contribution (QPPERC)/field
 field name=sequence eval=138/
 field name=codeQPPERC/field
-field name=category_id ref=hr_payroll.DED/
+field name=category_id ref=l10n_ca_hr_payroll.EMP/
 field name=appears_on_payslip eval=0/
 field name=active eval=1/
 field name=condition_selectnone/field

=== modified file 'l10n_ca_qc_hr_payroll_account/__openerp__.py'
--- l10n_ca_qc_hr_payroll_account/__openerp__.py	2013-02-27 20:02:47 +
+++ l10n_ca_qc_hr_payroll_account/__openerp__.py	2014-02-13 21:52:47 +
@@ -47,7 +47,7 @@
 'data': [
 'l10n_ca_qc_hr_payroll_account_data.xml',
 ],
-'test': [],
+'test': ['test/l10n_ca_qc_hr_payroll_account.yml'],
 'demo': [],
 'installable': True,
 'active': False,

=== added directory 'l10n_ca_qc_hr_payroll_account/test'
=== added file 'l10n_ca_qc_hr_payroll_account/test/l10n_ca_qc_hr_payroll_account.yml'
--- l10n_ca_qc_hr_payroll_account/test/l10n_ca_qc_hr_payroll_account.yml	1970-01-01 00:00:00 +
+++ l10n_ca_qc_hr_payroll_account/test/l10n_ca_qc_hr_payroll_account.yml	2014-02-13 21:52:47 +
@@ -0,0 +1,10 @@
+-
+  Check salary rule QPP - Employer Contribution (QPPERC), it's supposed to be
+  define in category Employer Contribution.
+  https://bugs.launchpad.net/openerp-canada/+bug/1213406
+-
+  !python {model: hr.salary.rule}: |
+rule_id = ref('l10n_ca_qc_hr_payroll.rule_qc_qpp_er_c')
+rule = self.browse(cr, uid, rule_id)
+category_id = ref('l10n_ca_hr_payroll.EMP')
+assert (rule.category_id.id == category_id), Expected the category Employer Contribution

-- 
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:~savoirfairelinux-openerp/lp-community-utils/progressbar into lp:lp-community-utils

2014-02-06 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/progressbar/+merge/205252
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:lp-community-utils.

-- 
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:~savoirfairelinux-openerp/lp-community-utils/pep394 into lp:lp-community-utils

2014-02-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/pep394/+merge/204535
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:lp-community-utils.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/bug-1269036 into lp:partner-contact-management

2014-01-16 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
I don't know if it is technically possible, but could we use the name field to 
store the lastname value and have name+firstname displayed in a relationship ?
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/bug-1269036/+merge/201947
Your team Partner and Contact Core Editors is requested to review the proposed 
merge of lp:~savoirfairelinux-openerp/partner-contact-management/bug-1269036 
into lp:partner-contact-management.

-- 
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:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

2014-01-07 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension 
into lp:server-env-tools.

For more details, see:
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744

[ADD] import_odbc and base_external_dbsource from lp:openobject-extension/7.0

-- 
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744
Your team Server Environment And Tools Core Editors is requested to review the 
proposed merge of 
lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension 
into lp:server-env-tools.
=== added directory 'base_external_dbsource'
=== added file 'base_external_dbsource/__init__.py'
--- base_external_dbsource/__init__.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/__init__.py	2014-01-07 22:26:44 +
@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis
+#2011
+#
+#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 base_external_dbsource
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'base_external_dbsource/__openerp__.py'
--- base_external_dbsource/__openerp__.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/__openerp__.py	2014-01-07 22:26:44 +
@@ -0,0 +1,61 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis, 2011
+#Additional contributions by Maxime Chambreuil, Savoir-faire Linux
+#
+#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': 'External Database Sources',
+'version': '61.3',
+'category': 'Tools',
+'description': 
+This module allows you to define connections to foreign databases using ODBC,
+Oracle Client or SQLAlchemy.
+
+Database sources can be configured in Settings  Configuration - Data sources.
+
+Depending on the database, you need:
+ * to install unixodbc and python-pyodbc packages to use ODBC connections.
+ * to install FreeTDS driver (tdsodbc package) and configure it through ODBC to
+   connect to Microsoft SQL Server.
+ * to install and configure Oracle Instant Client and cx_Oracle python library
+   to connect to Oracle.
+,
+'author': 'Daniel Reis',
+'website': 'http://launchpad.net/addons-tko',
+'images': [
+'images/screenshot01.png',
+],
+'depends': [
+'base',
+],
+'data': [
+'base_external_dbsource_view.xml',
+'security/ir.model.access.csv',
+],
+'demo': [
+'base_external_dbsource_demo.xml',
+],
+'test': [
+'dbsource_connect.yml',
+],
+'installable': True,
+'active': False,
+}
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'base_external_dbsource/base_external_dbsource.py'
--- base_external_dbsource/base_external_dbsource.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/base_external_dbsource.py	2014-01-07 22:26:44 +
@@ -0,0 +1,175 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis
+#2011
+#
+#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 

[Openerp-community-reviewer] [Merge] lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

2014-01-07 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension 
into lp:server-env-tools.

Requested reviews:
  Server Environment And Tools Core Editors (server-env-tools-core-editors)
  Nicolas Bessi - Camptocamp (nbessi-c2c): code review, no tests
  Yannick Vaucher @ Camptocamp (yvaucher-c2c): code review, no tests

For more details, see:
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744

[ADD] import_odbc and base_external_dbsource from lp:openobject-extension/7.0
-- 
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744
Your team Server Environment And Tools Core Editors is requested to review the 
proposed merge of 
lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension 
into lp:server-env-tools.
=== added directory 'base_external_dbsource'
=== added file 'base_external_dbsource/__init__.py'
--- base_external_dbsource/__init__.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/__init__.py	2014-01-07 22:26:44 +
@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis
+#2011
+#
+#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 base_external_dbsource
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'base_external_dbsource/__openerp__.py'
--- base_external_dbsource/__openerp__.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/__openerp__.py	2014-01-07 22:26:44 +
@@ -0,0 +1,61 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis, 2011
+#Additional contributions by Maxime Chambreuil, Savoir-faire Linux
+#
+#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': 'External Database Sources',
+'version': '61.3',
+'category': 'Tools',
+'description': 
+This module allows you to define connections to foreign databases using ODBC,
+Oracle Client or SQLAlchemy.
+
+Database sources can be configured in Settings  Configuration - Data sources.
+
+Depending on the database, you need:
+ * to install unixodbc and python-pyodbc packages to use ODBC connections.
+ * to install FreeTDS driver (tdsodbc package) and configure it through ODBC to
+   connect to Microsoft SQL Server.
+ * to install and configure Oracle Instant Client and cx_Oracle python library
+   to connect to Oracle.
+,
+'author': 'Daniel Reis',
+'website': 'http://launchpad.net/addons-tko',
+'images': [
+'images/screenshot01.png',
+],
+'depends': [
+'base',
+],
+'data': [
+'base_external_dbsource_view.xml',
+'security/ir.model.access.csv',
+],
+'demo': [
+'base_external_dbsource_demo.xml',
+],
+'test': [
+'dbsource_connect.yml',
+],
+'installable': True,
+'active': False,
+}
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'base_external_dbsource/base_external_dbsource.py'
--- base_external_dbsource/base_external_dbsource.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/base_external_dbsource.py	2014-01-07 22:26:44 +
@@ -0,0 +1,175 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis
+#2011

Re: [Openerp-community-reviewer] [Merge] lp:~initos.com/account-analytic/7.0 into lp:account-analytic

2014-01-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Hello Markus,

I don't know if it has an impact but the PO files still contains the previous 
name of the module.

Otherwise I would approve it.

Thanks.
-- 
https://code.launchpad.net/~initos.com/account-analytic/7.0/+merge/195362
Your team Account Core Editors is subscribed to branch lp:account-analytic.

-- 
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:~initos.com/account-analytic/7.0 into lp:account-analytic

2014-01-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~initos.com/account-analytic/7.0/+merge/195362
Your team Account Core Editors is subscribed to branch lp:account-analytic.

-- 
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:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report

2014-01-02 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:account-invoice-report.

-- 
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] [Blueprint resume-management] Resume Management

2013-12-31 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Blueprint changed by Maxime Chambreuil (http://www.savoirfairelinux.com):

Definition Status: New = Approved

-- 
Resume Management
https://blueprints.launchpad.net/openerp-hr/+spec/resume-management

-- 
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:~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067 into lp:openerp-fiscal-rules

2013-12-30 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067 into 
lp:openerp-fiscal-rules.

Commit message:
[MRG] Fix _fiscal_position_map() got multiple values for keyword argument 'uid'

Requested reviews:
  Account Core Editors (account-core-editors)
Related bugs:
  Bug #1265067 in openerp-fiscal-rules: [7.0] _fiscal_position_map() got 
multiple values for keyword argument 'uid'
  https://bugs.launchpad.net/openerp-fiscal-rules/+bug/1265067

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067/+merge/200202

[FIX] _fiscal_position_map() got multiple values for keyword argument 'uid'
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067/+merge/200202
Your team Account Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067 into 
lp:openerp-fiscal-rules.
=== modified file 'account_fiscal_position_rule_sale/sale.py'
--- account_fiscal_position_rule_sale/sale.py	2013-12-04 07:52:33 +
+++ account_fiscal_position_rule_sale/sale.py	2013-12-30 22:14:10 +
@@ -54,6 +54,7 @@
 
 values = result['value']
 kwargs = context.copy()
+del kwargs['uid']
 kwargs.update({
 'shop_id': context['shop_id'],
 'partner_id': partner_id,

-- 
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:~serpentcs/server-env-tools/7.0-base_synchro into lp:server-env-tools

2013-12-28 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Hi GRaeme,

Shall we reject this MP ?
-- 
https://code.launchpad.net/~serpentcs/server-env-tools/7.0-base_synchro/+merge/183102
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:~camptocamp/stock-logistic-flows/7.0-add-stock_recompute_availability_on_force-afe into lp:stock-logistic-flows/7.0

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~camptocamp/stock-logistic-flows/7.0-add-stock_recompute_availability_on_force-afe
 into lp:stock-logistic-flows/7.0 has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-add-stock_recompute_availability_on_force-afe/+merge/187772
-- 
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-add-stock_recompute_availability_on_force-afe/+merge/187772
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows/7.0.

-- 
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:~parthiv-patel/web-addons/web-addons into lp:web-addons

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~parthiv-patel/web-addons/web-addons into 
lp:web-addons has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~parthiv-patel/web-addons/web-addons/+merge/179846
-- 
https://code.launchpad.net/~parthiv-patel/web-addons/web-addons/+merge/179846
Your team Web-Addons Core Editors is subscribed to branch lp:web-addons.

-- 
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:~lin-yu/account-invoicing/add_account_invoce_line into lp:account-invoicing

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~lin-yu/account-invoicing/add_account_invoce_line into 
lp:account-invoicing has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~lin-yu/account-invoicing/add_account_invoce_line/+merge/182531
-- 
https://code.launchpad.net/~lin-yu/account-invoicing/add_account_invoce_line/+merge/182531
Your team Account Core Editors is subscribed to branch lp:account-invoicing.

-- 
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:~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix into lp:ocb-addons

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix into lp:ocb-addons 
has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix/+merge/185331
-- 
https://code.launchpad.net/~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix/+merge/185331
Your team OpenERP Community Backports Team is subscribed to branch 
lp:ocb-addons.

-- 
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:~bcim/account-invoicing/7.0-account_invoice_line_default_account into lp:account-invoicing

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~bcim/account-invoicing/7.0-account_invoice_line_default_account into 
lp:account-invoicing has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~bcim/account-invoicing/7.0-account_invoice_line_default_account/+merge/186078
-- 
https://code.launchpad.net/~bcim/account-invoicing/7.0-account_invoice_line_default_account/+merge/186078
Your team Account Core Editors is subscribed to branch lp:account-invoicing.

-- 
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:~bcim/account-invoicing/7.0-account_invoice_line_default_account into lp:account-invoicing

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

l282,283: can you add from . import... ?
-- 
https://code.launchpad.net/~bcim/account-invoicing/7.0-account_invoice_line_default_account/+merge/186078
Your team Account Core Editors is subscribed to branch lp:account-invoicing.

-- 
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/account-financial-tools/70-fiscal-position-check-vat into lp:account-financial-tools

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~akretion-team/account-financial-tools/70-fiscal-position-check-vat into 
lp:account-financial-tools has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~akretion-team/account-financial-tools/70-fiscal-position-check-vat/+merge/186515
-- 
https://code.launchpad.net/~akretion-team/account-financial-tools/70-fiscal-position-check-vat/+merge/186515
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:account-financial-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:~camptocamp/report-print-send/7.0-base_report_to_printer_port-imp-yvr-2 into lp:report-print-send

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~camptocamp/report-print-send/7.0-base_report_to_printer_port-imp-yvr-2/+merge/188825
Your team Report Printing and Sending Core Editors is subscribed to branch 
lp:report-print-send.

-- 
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:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing into lp:stock-logistic-flows

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Hello Sébastien,

Few changes :

l21: from . import ...
l140: Move AUTHORS content in the module description.
l166: from . import ...

You need to add the pot files for both modules.
-- 
https://code.launchpad.net/~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing/+merge/189182
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows.

-- 
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:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing into lp:stock-logistic-flows

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing
 into lp:stock-logistic-flows has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing/+merge/189182
-- 
https://code.launchpad.net/~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing/+merge/189182
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows.

-- 
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:~acsone-openerp/ocb-web/active_domain-7.0-backport into lp:ocb-web

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Abstain


-- 
https://code.launchpad.net/~acsone-openerp/ocb-web/active_domain-7.0-backport/+merge/189279
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-web.

-- 
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:~camptocamp/partner-contact-management/add-base_partner_merge into lp:partner-contact-management

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Please fix your PEP8 issues:

$ flake8 base_partner_merge/ --ignore=E501,F401
base_partner_merge/base_partner_merge.py:47:1: F812 list comprehension 
redefines 'email' from line 39
base_partner_merge/base_partner_merge.py:66:1: E302 expected 2 blank lines, 
found 1
base_partner_merge/base_partner_merge.py:266:9: E301 expected 1 blank line, 
found 0
base_partner_merge/base_partner_merge.py:322:46: E228 missing whitespace around 
modulo operator
base_partner_merge/base_partner_merge.py:451:29: E128 continuation line 
under-indented for visual indent
base_partner_merge/base_partner_merge.py:452:33: E128 continuation line 
under-indented for visual indent
base_partner_merge/base_partner_merge.py:738:46: W291 trailing whitespace
base_partner_merge/base_partner_merge.py:739:55: W291 trailing whitespace
base_partner_merge/validate_email.py:29:5: E301 expected 1 blank line, found 0
base_partner_merge/validate_email.py:29:33: E701 multiple statements on one 
line (colon)
base_partner_merge/validate_email.py:48:13: E126 continuation line 
over-indented for hanging indent
base_partner_merge/validate_email.py:50:17: E126 continuation line 
over-indented for hanging indent
base_partner_merge/validate_email.py:52:22: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:55:21: E126 continuation line 
over-indented for hanging indent
base_partner_merge/validate_email.py:57:14: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:61:53: E261 at least two spaces before 
inline comment
base_partner_merge/validate_email.py:63:17: E126 continuation line 
over-indented for hanging indent
base_partner_merge/validate_email.py:65:22: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:67:33: E126 continuation line 
over-indented for hanging indent
base_partner_merge/validate_email.py:70:26: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:73:22: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:75:34: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:78:18: E121 continuation line indentation 
is not a multiple of four
base_partner_merge/validate_email.py:84:1: E302 expected 2 blank lines, found 1
base_partner_merge/validate_email.py:84:41: E231 missing whitespace after ','
base_partner_merge/validate_email.py:97:23: E701 multiple statements on one 
line (colon)
base_partner_merge/validate_email.py:99:45: E226 missing whitespace around 
arithmetic operator
base_partner_merge/validate_email.py:105:34: E701 multiple statements on one 
line (colon)
base_partner_merge/validate_email.py:107:37: E701 multiple statements on one 
line (colon)
base_partner_merge/validate_email.py:110:37: E701 multiple statements on one 
line (colon)
base_partner_merge/validate_email.py:112:55: E261 at least two spaces before 
inline comment
base_partner_merge/validate_email.py:112:56: E262 inline comment should start 
with '# '
base_partner_merge/validate_email.py:116:42: W291 trailing whitespace
base_partner_merge/wizard/portal_wizard.py:25:1: E302 expected 2 blank lines, 
found 1
base_partner_merge/wizard/portal_wizard.py:32:57: E128 continuation line 
under-indented for visual indent
-- 
https://code.launchpad.net/~camptocamp/partner-contact-management/add-base_partner_merge/+merge/189616
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~serpentcs/account-financial-report/account-serpentcs into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/account-financial-report/account-serpentcs 
into lp:account-financial-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~serpentcs/account-financial-report/account-serpentcs/+merge/190320
-- 
https://code.launchpad.net/~serpentcs/account-financial-report/account-serpentcs/+merge/190320
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
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:~therp-nl/ocb-addons/6.1_lp1237832 into lp:ocb-addons/6.1

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~therp-nl/ocb-addons/6.1_lp1237832/+merge/190312
Your team OpenERP Community Backports Team is subscribed to branch 
lp:ocb-addons/6.1.

-- 
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:~serpentcs/purchase-report/purchase-serpentcs into lp:purchase-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/purchase-report/purchase-serpentcs into 
lp:purchase-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~serpentcs/purchase-report/purchase-serpentcs/+merge/190323
-- 
https://code.launchpad.net/~serpentcs/purchase-report/purchase-serpentcs/+merge/190323
Your team Purchase Core Editors is requested to review the proposed merge of 
lp:~serpentcs/purchase-report/purchase-serpentcs into lp:purchase-report.

-- 
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:~serpentcs/sale-reports/sale-serpentcs into lp:sale-reports

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Hello Nishant,

Can you use local import (from . import ...) in your __init__.py files ?

I am not familiar with webkit report. Don't they need a translation file ?

Thanks.
-- 
https://code.launchpad.net/~serpentcs/sale-reports/sale-serpentcs/+merge/190321
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:sale-reports.

-- 
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:~serpentcs/webkit-utils/utils-serpentcs into lp:webkit-utils

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/webkit-utils/utils-serpentcs into 
lp:webkit-utils has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~serpentcs/webkit-utils/utils-serpentcs/+merge/190327
-- 
https://code.launchpad.net/~serpentcs/webkit-utils/utils-serpentcs/+merge/190327
Your team Webkit Utils Core Editors is subscribed to branch lp:webkit-utils.

-- 
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:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Hello Lorenzo,

Please remove the AUTHORS.txt file and add its content to the module 
description.

Thanks.
-- 
https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:account-invoice-report.

-- 
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:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into 
lp:account-invoice-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230
-- 
https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:account-invoice-report.

-- 
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:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 into lp:openerp-sage-50

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 into 
lp:openerp-sage-50 has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50/+merge/191893
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50/+merge/191893
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of lp:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 
into lp:openerp-sage-50.

-- 
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:~vauxoo/account-financial-report/7.0-report_webkit_afr-_data_dev-jorge into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~vauxoo/account-financial-report/7.0-report_webkit_afr-_data_dev-jorge into 
lp:account-financial-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~vauxoo/account-financial-report/7.0-report_webkit_afr-_data_dev-jorge/+merge/192050
-- 
https://code.launchpad.net/~vauxoo/account-financial-report/7.0-report_webkit_afr-_data_dev-jorge/+merge/192050
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
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:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls/+merge/192250
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
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:~luc-demeyer/account-financial-tools/7.0-account_pain into lp:account-financial-tools

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~luc-demeyer/account-financial-tools/7.0-account_pain 
into lp:account-financial-tools has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~luc-demeyer/account-financial-tools/7.0-account_pain/+merge/192256
-- 
https://code.launchpad.net/~luc-demeyer/account-financial-tools/7.0-account_pain/+merge/192256
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:account-financial-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:~pedro.baeza/stock-logistic-flows/7.0-stock_picking_show_returns into lp:stock-logistic-flows/7.0

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~pedro.baeza/stock-logistic-flows/7.0-stock_picking_show_returns/+merge/193075
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows/7.0.

-- 
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:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Please fix the conflicts.
-- 
https://code.launchpad.net/~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations/+merge/200119
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
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:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations
 into lp:account-financial-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations/+merge/200119
-- 
https://code.launchpad.net/~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations/+merge/200119
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
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:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations
 into lp:account-financial-report.

Requested reviews:
  Pedro Manuel Baeza (pedro.baeza): code review, no test
  Guewen Baconnier @ Camptocamp (gbaconnier-c2c)
  Jacques-Etienne Baudoux (jbaudoux): code review, no test

For more details, see:
https://code.launchpad.net/~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations/+merge/200119

Added Dutch translation. Only thing is missing is the Clearance Analysis/ 
Clearance date. I don't understand the explanation. If someone could explain 
it, i will add translations.
-- 
https://code.launchpad.net/~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations/+merge/200119
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.
=== added file 'account_financial_report_webkit/i18n/nl.po'
--- account_financial_report_webkit/i18n/nl.po	1970-01-01 00:00:00 +
+++ account_financial_report_webkit/i18n/nl.po	2013-12-27 19:48:28 +
@@ -0,0 +1,746 @@
+# Translation of OpenERP Server.
+# This file contains the translation of the following modules:
+# 	* account_financial_report_webkit
+#
+msgid 
+msgstr 
+Project-Id-Version: OpenERP Server 7.0\n
+Report-Msgid-Bugs-To: \n
+POT-Creation-Date: 2013-10-30 11:59+\n
+PO-Revision-Date: 2013-10-30 14:58+0100\n
+Last-Translator: Erwin van der Ploeg (BAS Solutions) erwin@bas-solutions.
+nl\n
+Language-Team: \n
+MIME-Version: 1.0\n
+Content-Type: text/plain; charset=UTF-8\n
+Content-Transfer-Encoding: 8bit\n
+Plural-Forms: \n
+X-Generator: Poedit 1.5.7\n
+
+#. module: account_financial_report_webkit
+#: help:account.account,centralized:0
+msgid 
+If flagged, no details will be displayed in the General Ledger report (the 
+webkit one only), only centralized amounts per period.
+msgstr 
+Indien aangevinkt worden er geen details weergegeven in het grootboek, maar 
+alleen totaal bedragen per periode.
+
+#. module: account_financial_report_webkit
+#: constraint:account.common.balance.report:0
+#: constraint:general.ledger.webkit:0 constraint:open.invoices.webkit:0
+#: constraint:partner.balance.webkit:0 constraint:partners.ledger.webkit:0
+#: constraint:print.journal.webkit:0 constraint:trial.balance.webkit:0
+msgid 
+When no Fiscal year is selected, you must choose to filter by periods or by 
+date.
+msgstr 
+Wanneer geen boekjaar is geselecteerd dient u te kiezen voor een filter op 
+datum.
+
+#. module: account_financial_report_webkit
+#: field:account.common.balance.report,account_ids:0
+#: field:general.ledger.webkit,account_ids:0
+#: field:partner.balance.webkit,account_ids:0
+#: field:trial.balance.webkit,account_ids:0
+msgid Filter on accounts
+msgstr Filter op rekeningen
+
+#. module: account_financial_report_webkit
+#: field:open.invoices.webkit,group_by_currency:0
+msgid Group Partner by currency
+msgstr Groepeer relaties per valuta
+
+#. module: account_financial_report_webkit
+#: view:open.invoices.webkit:0
+msgid onchange_date_to(fiscalyear_id, period_to, date_to, until_date)
+msgstr onchange_date_to(fiscalyear_id, period_to, date_to, until_date)
+
+#. module: account_financial_report_webkit
+#: view:partner.balance.webkit:0
+msgid 
+This report is an analysis done by a partner, It is a PDF report containing 
+one line per partner representing the cumulative credit balance
+msgstr 
+Dit rapport is een aanalyse op relatie. Het is een PDF rapport met één regel 
+per relatie welke de opgetelde credit balans weergeeft.
+
+#. module: account_financial_report_webkit
+#: help:general.ledger.webkit,amount_currency:0
+#: help:open.invoices.webkit,amount_currency:0
+#: help:partners.ledger.webkit,amount_currency:0
+#: help:print.journal.webkit,amount_currency:0
+msgid It adds the currency column
+msgstr Voekgt de valuta kolom toe
+
+#. module: account_financial_report_webkit
+#: field:general.ledger.webkit,centralize:0
+msgid Activate Centralization
+msgstr Activeer totalen
+
+#. module: account_financial_report_webkit
+#: help:account.common.balance.report,account_ids:0
+#: help:general.ledger.webkit,account_ids:0
+#: help:partner.balance.webkit,account_ids:0
+#: help:trial.balance.webkit,account_ids:0
+msgid 
+Only selected accounts will be printed. Leave empty to print all accounts.
+msgstr 
+Alleen de geselecteerde rekeningen worden afgedrukt. Laat leeg om alle 
+rekeningen af te drukken.
+
+#. module: account_financial_report_webkit
+#: model:ir.model,name:account_financial_report_webkit.model_trial_balance_webkit
+msgid Trial Balance Report
+msgstr Proefbalans rapport
+
+#. module: account_financial_report_webkit
+#: view:general.ledger.webkit:0 view:open.invoices.webkit:0
+#: view:partner.balance.webkit:0 view:partners.ledger.webkit:0
+#: view:trial.balance.webkit:0
+msgid True
+msgstr Waar
+
+#. module: account_financial_report_webkit

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into lp:openerp-hr/6.1

2013-12-23 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review and test


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763
Your team HR Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into 
lp:openerp-hr/6.1.

-- 
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:~therp-nl/server-env-tools/7.0-my_OpenERP_is_not_unsupported into lp:server-env-tools

2013-12-21 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve


-- 
https://code.launchpad.net/~therp-nl/server-env-tools/7.0-my_OpenERP_is_not_unsupported/+merge/199896
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:~lin-yu/purchase-wkfl/add_purchase_control_suuplier into lp:purchase-wkfl

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~lin-yu/purchase-wkfl/add_purchase_control_suuplier 
into lp:purchase-wkfl has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~lin-yu/purchase-wkfl/add_purchase_control_suuplier/+merge/180795
-- 
https://code.launchpad.net/~lin-yu/purchase-wkfl/add_purchase_control_suuplier/+merge/180795
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.

-- 
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:~jean-lelievre/purchase-report/product_stock_type into lp:purchase-report

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~jean-lelievre/purchase-report/product_stock_type into 
lp:purchase-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~jean-lelievre/purchase-report/product_stock_type/+merge/182050
-- 
https://code.launchpad.net/~jean-lelievre/purchase-report/product_stock_type/+merge/182050
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
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:~jean-lelievre/purchase-report/move_reports into lp:purchase-report

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~jean-lelievre/purchase-report/move_reports into 
lp:purchase-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~jean-lelievre/purchase-report/move_reports/+merge/182044
-- 
https://code.launchpad.net/~jean-lelievre/purchase-report/move_reports/+merge/182044
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
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:~jean-lelievre/stock-logistic-report/stock_batch_move into lp:stock-logistic-report

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~jean-lelievre/stock-logistic-report/stock_batch_move 
into lp:stock-logistic-report has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~jean-lelievre/stock-logistic-report/stock_batch_move/+merge/182060
-- 
https://code.launchpad.net/~jean-lelievre/stock-logistic-report/stock_batch_move/+merge/182060
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-report.

-- 
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:~pedro.baeza/account-financial-report/6.1-balance-reporting into lp:account-financial-report/6.1

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~pedro.baeza/account-financial-report/6.1-balance-reporting into 
lp:account-financial-report/6.1 has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~pedro.baeza/account-financial-report/6.1-balance-reporting/+merge/183114
-- 
https://code.launchpad.net/~pedro.baeza/account-financial-report/6.1-balance-reporting/+merge/183114
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report/6.1.

-- 
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:~serpentcs/server-env-tools/7.0-base_synchro into lp:server-env-tools

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/server-env-tools/7.0-base_synchro into 
lp:server-env-tools has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~serpentcs/server-env-tools/7.0-base_synchro/+merge/183102
-- 
https://code.launchpad.net/~serpentcs/server-env-tools/7.0-base_synchro/+merge/183102
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:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Thanks Yannick.

I made the changes according to your review.
-- 
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/183539
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:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into lp:margin-analysis

2013-12-12 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge/+merge/198551
Your team Margin Analysis Core Editors is requested to review the proposed 
merge of lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into 
lp:margin-analysis.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-12-12 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Don't forget where you are from. There is no state in France ;)
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into lp:margin-analysis

2013-12-11 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review and test

The new field Cost price is on the Information tab in the product form. It 
would be better on the Procurements one, with the standard price field.

I have tested the calculation of the cost price based on components cost price 
and the inventory valuation in the reporting. It is working fine on my side.

Thank you very much Joël.
-- 
https://code.launchpad.net/~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge/+merge/198551
Your team Margin Analysis Core Editors is requested to review the proposed 
merge of lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into 
lp:margin-analysis.

-- 
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:~savoirfairelinux-openerp/openerp-hr/hr_employee_phone_extension into lp:openerp-hr

2013-12-11 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/hr_employee_phone_extension/+merge/198620
Your team HR Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_phone_extension into 
lp:openerp-hr.

-- 
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:~savoirfairelinux-openerp/account-invoicing/account_invoice_purchase_origin into lp:account-invoicing

2013-12-09 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/account-invoicing/account_invoice_purchase_origin/+merge/198321
Your team Account Core Editors is subscribed to branch lp:account-invoicing.

-- 
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:~savoirfairelinux-openerp/openerp-hr/unique_code into lp:openerp-hr

2013-12-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/unique_code into lp:openerp-hr.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/unique_code/+merge/196184

Added a unique constraint for department codes per company.
Added name_search and name_get as '[CODE] Department name'
Added french locale

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/unique_code/+merge/196184
Your team HR Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/unique_code into lp:openerp-hr.
=== modified file 'hr_department_sequence/hr_department.py'
--- hr_department_sequence/hr_department.py	2013-09-23 13:55:07 +
+++ hr_department_sequence/hr_department.py	2013-12-03 19:24:38 +
@@ -37,3 +37,20 @@
 _parent_store = True
 _parent_order = 'sequence, name'
 _order = 'parent_left'
+_sql_constraints = [
+('code_uniq', 'unique(code, company_id)', 'The code for the department must be unique per company !'),
+]
+
+def name_get(self, cr, uid, ids, context=None):
+
+Show department code with name
+
+if isinstance(ids, (int, long)):
+ids = [ids]
+return [(record.id, '[%s] %s' % (record.code, record.name) if record.code else record.name)
+for record in self.browse(cr, uid, ids, context=context or {})]
+
+def name_search(self, cr, uid, name='', args=None, operator='ilike',  context=None, limit=100):
+ids = self.search(cr, uid, ['|', ('code', 'ilike', name), ('name', 'ilike', name)] + args,
+  limit=limit, context=context)
+return self.name_get(cr, uid, ids, context=context)

=== added directory 'hr_department_sequence/i18n'
=== added file 'hr_department_sequence/i18n/fr.po'
--- hr_department_sequence/i18n/fr.po	1970-01-01 00:00:00 +
+++ hr_department_sequence/i18n/fr.po	2013-12-03 19:24:38 +
@@ -0,0 +1,52 @@
+# Translation of OpenERP Server.
+# This file contains the translation of the following modules:
+# 	* hr_department_sequence
+#
+msgid 
+msgstr 
+Project-Id-Version: OpenERP Server 7.0\n
+Report-Msgid-Bugs-To: \n
+POT-Creation-Date: 2013-11-21 19:08+\n
+PO-Revision-Date: 2013-11-21 14:12-0500\n
+Last-Translator: Sandy Carter sandy.car...@savoirfairelinux.com\n
+Language-Team: Savoir-faire Linux http://www.savoirfairelinux.com\n
+MIME-Version: 1.0\n
+Content-Type: text/plain; charset=UTF-8\n
+Content-Transfer-Encoding: 8bit\n
+X-Generator: Poedit 1.5.7\n
+Language: fr\n
+
+#. module: hr_department_sequence
+#: field:hr.department,parent_left:0
+msgid Left Parent
+msgstr Parent de gauche
+
+#. module: hr_department_sequence
+#: field:hr.department,code:0
+msgid Code
+msgstr Code
+
+#. module: hr_department_sequence
+#: field:hr.department,sequence:0
+msgid Sequence
+msgstr Séquence
+
+#. module: hr_department_sequence
+#: help:hr.department,sequence:0
+msgid Gives the sequence order when displaying a list of departments.
+msgstr Donne l'ordre dans la liste des départements.
+
+#. module: hr_department_sequence
+#: sql_constraint:hr.department:0
+msgid The code for the department must be unique per company !
+msgstr Le code d'un département doit être unique par compagnie !
+
+#. module: hr_department_sequence
+#: model:ir.model,name:hr_department_sequence.model_hr_department
+msgid Department
+msgstr Département
+
+#. module: hr_department_sequence
+#: field:hr.department,parent_right:0
+msgid Right Parent
+msgstr Parent de droite

=== added file 'hr_department_sequence/i18n/hr_department_sequence.pot'
--- hr_department_sequence/i18n/hr_department_sequence.pot	1970-01-01 00:00:00 +
+++ hr_department_sequence/i18n/hr_department_sequence.pot	2013-12-03 19:24:38 +
@@ -0,0 +1,52 @@
+# Translation of OpenERP Server.
+# This file contains the translation of the following modules:
+#	* hr_department_sequence
+#
+msgid 
+msgstr 
+Project-Id-Version: OpenERP Server 7.0\n
+Report-Msgid-Bugs-To: \n
+POT-Creation-Date: 2013-11-21 19:08+\n
+PO-Revision-Date: 2013-11-21 19:08+\n
+Last-Translator: \n
+Language-Team: \n
+MIME-Version: 1.0\n
+Content-Type: text/plain; charset=UTF-8\n
+Content-Transfer-Encoding: \n
+Plural-Forms: \n
+
+#. module: hr_department_sequence
+#: field:hr.department,parent_left:0
+msgid Left Parent
+msgstr 
+
+#. module: hr_department_sequence
+#: field:hr.department,code:0
+msgid Code
+msgstr 
+
+#. module: hr_department_sequence
+#: field:hr.department,sequence:0
+msgid Sequence
+msgstr 
+
+#. module: hr_department_sequence
+#: help:hr.department,sequence:0
+msgid Gives the sequence order when displaying a list of departments.
+msgstr 
+
+#. module: hr_department_sequence
+#: sql_constraint:hr.department:0
+msgid The code for the department must be unique per company !
+msgstr 
+
+#. module: hr_department_sequence
+#: model:ir.model,name:hr_department_sequence.model_hr_department
+msgid Department
+msgstr 

[Openerp-community-reviewer] [Merge] lp:~ovnicraft/web-addons/6.1-web_report_name into lp:web-addons/6.1

2013-12-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~ovnicraft/web-addons/6.1-web_report_name into 
lp:web-addons/6.1 has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~ovnicraft/web-addons/6.1-web_report_name/+merge/169662
-- 
https://code.launchpad.net/~ovnicraft/web-addons/6.1-web_report_name/+merge/169662
Your team Web-Addons Core Editors is requested to review the proposed merge of 
lp:~ovnicraft/web-addons/6.1-web_report_name into lp:web-addons/6.1.

-- 
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:~bruno-bottacini/account-financial-tools/7.0-add_account_invoice_template into lp:account-financial-tools

2013-12-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~bruno-bottacini/account-financial-tools/7.0-add_account_invoice_template 
into lp:account-financial-tools has been updated.

Status: Needs review = Rejected

For more details, see:
https://code.launchpad.net/~bruno-bottacini/account-financial-tools/7.0-add_account_invoice_template/+merge/170010
-- 
https://code.launchpad.net/~bruno-bottacini/account-financial-tools/7.0-add_account_invoice_template/+merge/170010
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch 
lp:account-financial-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:~openerp-community/purchase-wkfl/6.1-purchase_internal_validation into lp:purchase-wkfl/6.1

2013-12-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~openerp-community/purchase-wkfl/6.1-purchase_internal_validation into 
lp:purchase-wkfl/6.1 has been updated.

Status: Needs review = Work in progress

For more details, see:
https://code.launchpad.net/~openerp-community/purchase-wkfl/6.1-purchase_internal_validation/+merge/170305
-- 
https://code.launchpad.net/~openerp-community/purchase-wkfl/6.1-purchase_internal_validation/+merge/170305
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl/6.1.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-11-22 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Hello Pedro,

Your module is based on the scenario where the user enters the zip, and other 
fields (city, state and country) are set automatically.

Our scenario is the user chooses among a list of cities in our travel module. 
We don't care about the zip.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-11-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-11-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Pedro,

base_location deals with ZIP code. We only want to have a dropdown list for 
cities in the address. 

That's exactly what this module is providing. Maybe base_location should depend 
on this one...
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-11-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
https://www.openerp.com/apps/7.0/base_location/
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-11-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
@Sandy

Let's test base_location tomorrow morning if it does the job.

@Pedro

We will get back to you after our tests.

Thanks.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 into lp:openerp-sage-50

2013-11-18 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 into 
lp:openerp-sage-50.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50/+merge/191893

[UPG] Upgrade to version 7.0

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50/+merge/191893
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of lp:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 
into lp:openerp-sage-50.
=== added directory 'exportsage50'
=== added directory 'exportsage50/exportsage50'
=== added file 'exportsage50/exportsage50/AUTHORS.txt'
--- exportsage50/exportsage50/AUTHORS.txt	1970-01-01 00:00:00 +
+++ exportsage50/exportsage50/AUTHORS.txt	2013-11-15 16:29:49 +
@@ -0,0 +1,2 @@
+EL HADJI DEM elhadji@savoirfairelinux.com
+Savoir-faire Linux cont...@savoirfairelinux.com
\ No newline at end of file

=== added file 'exportsage50/exportsage50/__init__.py'
--- exportsage50/exportsage50/__init__.py	1970-01-01 00:00:00 +
+++ exportsage50/exportsage50/__init__.py	2013-11-15 16:29:49 +
@@ -0,0 +1,23 @@
+# -*- encoding: utf-8 -*-
+###
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Gestion-Ressources (http://www.gestion-ressources.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/.
+#
+###
+
+import wizard
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file

=== added file 'exportsage50/exportsage50/__init__.pyc'
Binary files exportsage50/exportsage50/__init__.pyc	1970-01-01 00:00:00 + and exportsage50/exportsage50/__init__.pyc	2013-11-15 16:29:49 + differ
=== added file 'exportsage50/exportsage50/__openerp__.py'
--- exportsage50/exportsage50/__openerp__.py	1970-01-01 00:00:00 +
+++ exportsage50/exportsage50/__openerp__.py	2013-11-15 16:29:49 +
@@ -0,0 +1,125 @@
+# -*- encoding: utf-8 -*-
+###
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Gestion-Ressources (http://www.gestion-ressources.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': 'Export to Sage50',
+'version': '1.0',
+category: 'Accounting  Finance',
+'complexity': easy,
+'description': 
+French version below
+
+Export accounting data from OpenErp to Sage50.
+
+ Export accounting data from OpenErp to Sage50. The exportation generates the imp file to import in Sage50.
+
+
+Documentation : layout of the import file (.IMP)
+
+A PDF document (in the /doc repository) gives more details about the .IMP file layout that can be imported
+into Sage 50.This document is part of the Sage 50 SDK (Software Development Kit) available for download at
+the following address : http://na.sage.com/sage-simply-accounting/lp/partners/sdk/?isnow=ssa.
+
+
+Sage 50: After creating .IMP file
+
+Solution:
+Importing purchase invoices, purchase quotes, sales invoices and sales orders into Sage 50
+How to import purchase invoices, purchase quotes, sales invoices and sales orders?
+You can import purchase invoices, purchase quotes, sales invoices and sales orders into Sage 50.
+The transaction(s) details should be in a text file with extension .IMP.
+

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into lp:openerp-hr/6.1

2013-11-18 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into 
lp:openerp-hr/6.1 has been updated.

Description changed to:

[FIX] PEP8 issue
[IMP] Cleanup hr_skill
[IMP] Demo data installable.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763
Your team HR Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into 
lp:openerp-hr/6.1.

-- 
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:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into lp:openerp-hr/6.1

2013-11-18 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into 
lp:openerp-hr/6.1 has been updated.

Description changed to:

[FIX] PEP8 issue
[IMP] Cleanup hr_skill
[IMP] Demo data installable.
[ADD] hr_experience_analytic

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763
Your team HR Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into 
lp:openerp-hr/6.1.

-- 
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:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into lp:openerp-hr/6.1

2013-11-18 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Hello Pedro,

We have moved modules to other branches.

There is only hr_experience_analytic left here with some cleanup of hr_skill.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763
Your team HR Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into 
lp:openerp-hr/6.1.

-- 
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:~savoirfairelinux-openerp/openerp-mgmtsystem/openupgrade70 into lp:openerp-mgmtsystem

2013-11-18 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-mgmtsystem/openupgrade70 into 
lp:openerp-mgmtsystem.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-mgmtsystem/openupgrade70/+merge/192730

This patch adds openupgrade migration scripts which should migrate mgmtsystem 
documents, actions and nonconformities properly.

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-mgmtsystem/openupgrade70/+merge/192730
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of lp:~savoirfairelinux-openerp/openerp-mgmtsystem/openupgrade70 
into lp:openerp-mgmtsystem.
=== added directory 'document_page_environment_manual/migrations'
=== added directory 'document_page_environment_manual/migrations/7.0.1.0'
=== added file 'document_page_environment_manual/migrations/7.0.1.0/pre-migration.py'
--- document_page_environment_manual/migrations/7.0.1.0/pre-migration.py	1970-01-01 00:00:00 +
+++ document_page_environment_manual/migrations/7.0.1.0/pre-migration.py	2013-11-07 22:30:27 +
@@ -0,0 +1,54 @@
+# -*- encoding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#This module copyright (C) 2013 Savoir-faire Linux
+#(http://www.savoirfairelinux.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/.
+#
+##
+
+try:
+from openupgrade.openupgrade import logged_query
+except ImportError:
+import logging
+logger = logging.getLogger('upgrade')
+
+def logged_query(cr, query, args=None):
+if args is None:
+args = []
+res = cr.execute(query, args)
+logger.debug('Running %s', query % tuple(args))
+logger.debug('%s rows affected', cr.rowcount)
+return cr.rowcount
+
+
+def pre_migrate_environment_manual_category(cr, version):
+logged_query(cr, \
+UPDATE document_page
+SET parent_id = (SELECT id FROM document_page WHERE name = 'Manuals' LIMIT 1),
+name = name || ' (' || %s || ')'
+WHERE parent_id = (SELECT id FROM document_page WHERE name = 'Environment Manual' AND type = 'category')
+ AND type = 'content';, [version])
+logged_query(cr, \
+UPDATE document_page
+SET name = name || ' (' || %s || ')'
+WHERE name = 'Environment Manual' AND type = 'category';, [version])
+
+
+def migrate(cr, version):
+pre_migrate_environment_manual_category(cr, version)
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added directory 'document_page_environmental_aspect/migrations'
=== added directory 'document_page_environmental_aspect/migrations/7.0.1.0'
=== added file 'document_page_environmental_aspect/migrations/7.0.1.0/post-migration.py'
--- document_page_environmental_aspect/migrations/7.0.1.0/post-migration.py	1970-01-01 00:00:00 +
+++ document_page_environmental_aspect/migrations/7.0.1.0/post-migration.py	2013-11-07 22:30:27 +
@@ -0,0 +1,64 @@
+# -*- encoding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#This module copyright (C) 2013 Savoir-faire Linux
+#(http://www.savoirfairelinux.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/.
+#
+##
+
+try:
+from openupgrade.openupgrade import logged_query
+except ImportError:
+import logging
+logger = logging.getLogger('upgrade')
+
+def logged_query(cr, query, args=None):
+if args is None:
+

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-accountedge/6.1-fix-email-addresses-notifications into lp:openerp-accountedge

2013-11-18 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-accountedge/6.1-fix-email-addresses-notifications
 into lp:openerp-accountedge.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-accountedge/6.1-fix-email-addresses-notifications/+merge/194052



-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-accountedge/6.1-fix-email-addresses-notifications/+merge/194052
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of 
lp:~savoirfairelinux-openerp/openerp-accountedge/6.1-fix-email-addresses-notifications
 into lp:openerp-accountedge.
=== modified file 'hr_expense_accountedge/__openerp__.py'
--- hr_expense_accountedge/__openerp__.py	2013-08-08 19:16:49 +
+++ hr_expense_accountedge/__openerp__.py	2013-11-18 22:10:04 +
@@ -19,17 +19,17 @@
 #
 ##
 {
-name : Harmonization of expenses with AccountEdge,
-version : 1.0,
-author : Savoir-faire Linux,
-website : http://www.savoirfairelinux.com;,
-category : Human Resources,
+name: Harmonization of expenses with AccountEdge,
+version: 1.0,
+author: Savoir-faire Linux,
+website: http://www.savoirfairelinux.com;,
+category: Human Resources,
 description: 
 This module generates the csv reports for the exportation
 of expenses in AccountEdge.
 It also modifies the workflow of the expenses.
 ,
-depends : [
+depends: [
 'l10n_ca',
 'hr_employee_accountedge',
 'hr_expense_line_supplier',
@@ -39,14 +39,13 @@
 'account_account_halftax',
 'account_tax_accountedge'
 ],
-init_xml : [],
-update_xml : [
+init_xml: [],
+update_xml: [
 'hr_expense_accountedge.xml',
 'security/ir_rule.xml',
 ],
-demo_xml : [],
-installable : True,
-certificate : ''
+demo_xml: [],
+installable: True,
+certificate: ''
 }
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-

=== modified file 'hr_expense_accountedge/edi/__init__.py'
--- hr_expense_accountedge/edi/__init__.py	2013-07-16 19:21:49 +
+++ hr_expense_accountedge/edi/__init__.py	2013-11-18 22:10:04 +
@@ -18,8 +18,5 @@
 #along with this program.  If not, see http://www.gnu.org/licenses/.
 #
 ##
-
 import hr_expense_expense
-
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-

=== modified file 'hr_expense_accountedge/edi/hr_expense_expense.py'
--- hr_expense_accountedge/edi/hr_expense_expense.py	2013-07-16 19:21:49 +
+++ hr_expense_accountedge/edi/hr_expense_expense.py	2013-11-18 22:10:04 +
@@ -22,28 +22,22 @@
 from osv import fields, osv, orm
 from edi import EDIMixin
 
+
 class hr_expense_expense(osv.osv, EDIMixin):
 _inherit = 'hr.expense.expense'
 
 def _edi_get_web_url_view(self, cr, uid, ids, field_name, arg, context):
-res  = {}
-
+res = {}
 for id in ids:
 web_root_url = self.pool.get('ir.config_parameter').get_param(cr, uid, 'web.base.url')
 res[id] = %s/web/webclient/home#id=%sview_type=pagemodel=hr.expense.expense % (web_root_url, id)
-
 return res
 
 _columns = {
-'web_url_view' : fields.function(
-_edi_get_web_url_view,
-string='Url of the expense view',
-type='char',
-size=255,
-readonly=True),
+'web_url_view': fields.function(_edi_get_web_url_view,
+string='Url of the expense view',
+type='char',
+size=255,
+readonly=True),
 }
-
-
-
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-

=== modified file 'hr_expense_accountedge/hr_expense_accountedge.py'
--- hr_expense_accountedge/hr_expense_accountedge.py	2013-08-12 22:45:27 +
+++ hr_expense_accountedge/hr_expense_accountedge.py	2013-11-18 22:10:04 +
@@ -29,56 +29,52 @@
 class hr_expense_expense(osv.osv):
 _inherit = 'hr.expense.expense'
 
-def _create_csv_report(self,cr,uid,ids,context={}):
+def _create_csv_report(self, cr, uid, ids, context={}):
 res = {}
 for id in ids:
-this= self.browse(cr, uid, id)
-empl_id = self.pool.get('hr.employee').search(cr, uid, [('user_id','=', this.user_id.id)])
-empl= self.pool.get('hr.employee').browse(cr, uid, empl_id)[0]
+this = self.browse(cr, uid, id)
 
-output  = this.employee_id.name
+output = this.employee_id.name
 output += \r\n
 output += Employee\tCard ID\tDate\tVendor Invoice #\tAccount Number\tAmount\tDescription\
 \tTax 

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-sage-50/6.1-exportsage50 into lp:openerp-sage-50/6.1

2013-11-16 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-sage-50/6.1-exportsage50 into 
lp:openerp-sage-50/6.1.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/6.1-exportsage50/+merge/158246

[ADD] exportsage50

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/6.1-exportsage50/+merge/158246
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of lp:~savoirfairelinux-openerp/openerp-sage-50/6.1-exportsage50 
into lp:openerp-sage-50/6.1.
=== added directory 'exportsage50'
=== added file 'exportsage50/AUTHORS.txt'
--- exportsage50/AUTHORS.txt	1970-01-01 00:00:00 +
+++ exportsage50/AUTHORS.txt	2013-11-15 15:41:37 +
@@ -0,0 +1,2 @@
+EL HADJI DEM elhadji@savoirfairelinux.com
+Savoir-faire Linux cont...@savoirfairelinux.com
\ No newline at end of file

=== added file 'exportsage50/__init__.py'
--- exportsage50/__init__.py	1970-01-01 00:00:00 +
+++ exportsage50/__init__.py	2013-11-15 15:41:37 +
@@ -0,0 +1,23 @@
+# -*- encoding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Gestion-Ressources (http://www.gestion-ressources.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/.
+#
+##
+
+import wizard
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'exportsage50/__openerp__.py'
--- exportsage50/__openerp__.py	1970-01-01 00:00:00 +
+++ exportsage50/__openerp__.py	2013-11-15 15:41:37 +
@@ -0,0 +1,128 @@
+# -*- encoding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Gestion-Ressources (http://www.gestion-ressources.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': 'Export to Sage50',
+'version': '1.0',
+category: 'Accounting  Finance',
+'complexity': easy,
+'description': 
+French version below
+
+Export accounting data from OpenErp to Sage50.
+
+ Export accounting data from OpenErp to Sage50. The exportation generates the imp file to import in Sage50.
+
+
+Documentation : layout of the import file (.IMP)
+
+A PDF document (in the /doc repository) gives more details about the .IMP file layout that can be imported
+into Sage 50.This document is part of the Sage 50 SDK (Software Development Kit) available for download at
+the following address : http://na.sage.com/sage-simply-accounting/lp/partners/sdk/?isnow=ssa.
+
+
+
+Sage 50: After creating .IMP file
+
+Solution:
+Importing purchase invoices, purchase quotes, sales invoices and sales orders into Sage 50
+How to import purchase invoices, purchase quotes, sales invoices and sales orders?
+You can import purchase invoices, purchase quotes, sales invoices and sales orders into Sage 50.
+The transaction(s) details should be in a text file with extension .IMP.
+
+
+After you have created the .IMP file, you can import the transaction(s) into Sage 50 by following these steps:
+(Account information is not included in the .IMP format because when importing the file, you will receive
+a pop-up screen to ask you Select an Account to match.)
+
+From the Home Window, go to File, Import/Export
+Click on Import Transactions (the Import Transactions Wizard appears)
+Select on 

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/geospatial-addons/geoengine_geocoder_ca_geocoder into lp:geospatial-addons

2013-11-14 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/geospatial-addons/geoengine_geocoder_ca_geocoder 
into lp:geospatial-addons.

Requested reviews:
  Geospatial Addons Core Editors (geospatial-addons-core-editors)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/geospatial-addons/geoengine_geocoder_ca_geocoder/+merge/195296
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/geospatial-addons/geoengine_geocoder_ca_geocoder/+merge/195296
Your team Geospatial Addons Core Editors is requested to review the proposed 
merge of 
lp:~savoirfairelinux-openerp/geospatial-addons/geoengine_geocoder_ca_geocoder 
into lp:geospatial-addons.
=== added directory 'geoengine_geocoder_ca_geocoder'
=== added file 'geoengine_geocoder_ca_geocoder/__init__.py'
--- geoengine_geocoder_ca_geocoder/__init__.py	1970-01-01 00:00:00 +
+++ geoengine_geocoder_ca_geocoder/__init__.py	2013-11-14 19:37:29 +
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2012 Savoir-faire Linux (http://www.savoirfairelinux.com).
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU 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 General Public License for more details.
+#
+#You should have received a copy of the GNU General Public License
+#along with this program.  If not, see http://www.gnu.org/licenses/.
+#
+##
+import geocoder_ca_coder
+import wizard

=== added file 'geoengine_geocoder_ca_geocoder/__openerp__.py'
--- geoengine_geocoder_ca_geocoder/__openerp__.py	1970-01-01 00:00:00 +
+++ geoengine_geocoder_ca_geocoder/__openerp__.py	2013-11-14 19:37:29 +
@@ -0,0 +1,33 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Savoir-faire Linux (http://www.savoirfairelinux.com).
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU 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 General Public License for more details.
+#
+#You should have received a copy of the GNU General Public License
+#along with this program.  If not, see http://www.gnu.org/licenses/.
+#
+##
+{'name': 'Auto Geocoding of partners using geocoder.ca',
+ 'version': '0.1',
+ 'category': 'GeoBI',
+ 'description':  Automatically geocode addresses using http://www.geocoder.ca/ api. ,
+ 'data': ['wizard/bulk_encode_view.xml'],
+ 'author': 'Savoir-faire Linux',
+ 'license': 'AGPL-3',
+ 'website': 'http://www.savoirfairelinux.com',
+ 'depends': ['geoengine_geoname_geocoder'],
+ 'installable': True,
+ 'active': False,
+ 'icon': '/base_geoengine/static/src/images/map_icon.png'}
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'geoengine_geocoder_ca_geocoder/geocoder_ca_coder.py'
--- geoengine_geocoder_ca_geocoder/geocoder_ca_coder.py	1970-01-01 00:00:00 +
+++ geoengine_geocoder_ca_geocoder/geocoder_ca_coder.py	2013-11-14 19:37:29 +
@@ -0,0 +1,118 @@
+# -*- coding: utf-8 -*-
+##
+#
+#OpenERP, Open Source Management Solution
+#Copyright (C) 2013 Savoir-faire Linux (http://www.savoirfairelinux.com).
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU 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 General Public License for more details.
+#
+#You should have received a copy of the GNU General Public License
+#along with this program.  If not, see http://www.gnu.org/licenses

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/lp-community-utils/openerp-travel into lp:lp-community-utils

2013-11-12 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging 
lp:~savoirfairelinux-openerp/lp-community-utils/openerp-travel into 
lp:lp-community-utils.

Commit message:
[IMP] Add lp:openerp-travel project

Requested reviews:
  OpenERP Community Reviewer/Maintainer (openerp-community-reviewer)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/openerp-travel/+merge/194899
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/openerp-travel/+merge/194899
Your team OpenERP Community Reviewer/Maintainer is requested to review the 
proposed merge of 
lp:~savoirfairelinux-openerp/lp-community-utils/openerp-travel into 
lp:lp-community-utils.
=== modified file 'projects'
--- projects	2013-11-11 14:47:04 +
+++ projects	2013-11-12 16:53:16 +
@@ -39,6 +39,7 @@
 openerp-product-variant
 openerp-reporting-engines
 openerp-sage-50
+openerp-travel
 partner-contact-management
 product-kitting
 project-reporting

-- 
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:~lin-yu/purchase-wkfl/add_purchase_price_list into lp:purchase-wkfl

2013-11-09 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Thanks for your contribution Yu.

l163: please use orm.Model instead of osv.osv. Update the import accordingly.
l208, 463: no need to instantiate
l417: 2 lines are enough to comply with PEP8. Run flake8 to check for PEP8 
issues
-- 
https://code.launchpad.net/~lin-yu/purchase-wkfl/add_purchase_price_list/+merge/180796
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/partner_isp into lp:partner-contact-management

2013-11-05 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Thanks for your feedback.

Module was moved to a more specific project lp:openerp-isp.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner_isp/+merge/185095
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/partner-contact-management/partner_isp into lp:partner-contact-management

2013-11-05 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge 
lp:~savoirfairelinux-openerp/partner-contact-management/partner_isp into 
lp:partner-contact-management has been updated.

Status: Needs review = Rejected

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner_isp/+merge/185095
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner_isp/+merge/185095
Your team Partner and Contact Core Editors is subscribed to branch 
lp:partner-contact-management.

-- 
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:~savoirfairelinux-openerp/bazaar-extractor/trunk-improve-shell-script into lp:bazaar-extractor

2013-11-05 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/bazaar-extractor/trunk-improve-shell-script into 
lp:bazaar-extractor.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/bazaar-extractor/trunk-improve-shell-script/+merge/186950



-- 
https://code.launchpad.net/~savoirfairelinux-openerp/bazaar-extractor/trunk-improve-shell-script/+merge/186950
Your team OpenERP Community Reviewer is requested to review the proposed merge 
of lp:~savoirfairelinux-openerp/bazaar-extractor/trunk-improve-shell-script 
into lp:bazaar-extractor.
=== added file 'README'
--- README	1970-01-01 00:00:00 +
+++ README	2013-09-23 14:30:52 +
@@ -0,0 +1,6 @@
+README
+##
+
+Run bzr-replay.sh for more information on how to use it.
+
+The script has not been tested with multiple modules.

=== renamed file 'bzr-replay-readme.txt' = 'bzr-replay.sh' (properties changed: -x to +x)
--- bzr-replay-readme.txt	2012-11-28 12:54:21 +
+++ bzr-replay.sh	2013-09-23 14:30:52 +
@@ -1,20 +1,46 @@
-#!bin/bash
-
-
-mkdir $BRANCH_DEST_NAME
+#!/bin/bash
+
+BRANCH_SRC_NAME=$1
+BRANCH_DEST_NAME=$2
+MODULE_LIST=$3
+LP_BRANCH_SRC_NAME=$4
+LP_BRANCH_DEST_NAME=$5
+REL_PATH=`dirname $0`
+PATH_TO=`readlink -e $REL_PATH`
+
+if [ $# != 5 ]; then
+	echo Usage: $0 BRANCH_SRC_NAME BRANCH_DEST_NAME MODULE_LIST LP_BRANCH_SRC_NAME LP_BRANCH_DEST_NAME
+	exit 1
+fi
+
+if [ ! -d $BRANCH_DEST_NAME ]; then
+	mkdir -p $BRANCH_DEST_NAME
+	cd $BRANCH_DEST_NAME
+	bzr init .
+	cd -
+fi
+
+SRC_ABS_PATH=`readlink -e $BRANCH_SRC_NAME`
 cd $BRANCH_DEST_NAME
-bzr init .
-ruby $PATH_TO/bzr-super-replay2.rb $BRANCH_SRC_NAME 1 --module $MODULE_LIST
+bzr unbind
+ruby $PATH_TO/bzr-super-replay.rb $SRC_ABS_PATH 1 --module $MODULE_LIST
+cd -
 
 # Check diff
 
+for MODULE in $MODULE_LIST; do
+	diff -r $BRANCH_SRC_NAME/$MODULE $BRANCH_DEST_NAME/$MODULE
+	[ $? == 1 ]  echo $MODULE is different in the 2 repositories  exit 1
+done
+
 # Push LP_BRANCH_DEST_NAME
-
 cd $BRANCH_SRC_NAME
 bzr unbind
 bzr rm $MODULE_LIST
-bzr ci -m[DEL] Remove module cause we move them to LP_BRANCH_DEST_NAME. Module list: MODULE_LIST
+bzr ci -m [DEL] Remove module cause we move them to $LP_BRANCH_DEST_NAME. Module list: $MODULE_LIST
 bzr push $LP_BRANCH_SRC_NAME
 bzr bind $LP_BRANCH_SRC_NAME
 
+exit 0
+
 # Warning: This is really brute force and still has some bugs. Don't forget to make a diff after using it to verify that nothing has been lost.

-- 
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:~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report into lp:openerp-manufacturing

2013-11-05 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report into 
lp:openerp-manufacturing.

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report/+merge/191493

Don't delete the original report id, just update the rml location.

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report/+merge/191493
Your team Manufacture Core Editors is requested to review the proposed merge of 
lp:~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report into 
lp:openerp-manufacturing.
=== modified file 'mrp_industrial_design_bom/__openerp__.py'
--- mrp_industrial_design_bom/__openerp__.py	2013-04-08 15:44:53 +
+++ mrp_industrial_design_bom/__openerp__.py	2013-10-16 19:58:45 +
@@ -30,7 +30,9 @@
 'depends' : ['mrp'],
 data : ['mrp_industrial_design.xml'],
 'description': 
-Add the fields 'Bubble Number' and 'RefDes' (reference description) to a component in BoM view.
+This module adds the fields 'Bubble Number' and 'RefDes' (reference description) to a component in BoM view.
+
+It also point the BOM Structure report to a new version that uses the new fields.
 ,
 'auto_install': False,
 'installable': True

=== modified file 'mrp_industrial_design_bom/industrial_design.py'
--- mrp_industrial_design_bom/industrial_design.py	2013-04-08 15:44:53 +
+++ mrp_industrial_design_bom/industrial_design.py	2013-10-16 19:58:45 +
@@ -2,7 +2,7 @@
 ##
 #
 #OpenERP, Open Source Management Solution
-#Copyright (C) 2010 Savoir-faire Linux (http://www.savoirfairelinux.com).
+#Copyright (C) 2013 Savoir-faire Linux (http://www.savoirfairelinux.com).
 #
 #This program is free software: you can redistribute it and/or modify
 #it under the terms of the GNU General Public License as
@@ -15,18 +15,19 @@
 #GNU General Public License for more details.
 #
 #You should have received a copy of the GNU General Public License
-#along with this program.  If not, see http://www.gnu.org/licenses/. 
+#along with this program.  If not, see http://www.gnu.org/licenses/.
 #
 ##
 
 from openerp.osv import fields, orm
 
+
 class mrp_bom(orm.Model):
 _inherit = 'mrp.bom'
 _columns = {
 'bubble_number': fields.integer('No'),
 'refdes': fields.text('Notes'),
 }
-
+
 _order = bubble_number
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== modified file 'mrp_industrial_design_bom/mrp_industrial_design.xml'
--- mrp_industrial_design_bom/mrp_industrial_design.xml	2013-04-26 13:44:37 +
+++ mrp_industrial_design_bom/mrp_industrial_design.xml	2013-10-16 19:58:45 +
@@ -1,15 +1,10 @@
 ?xml version=1.0 ?
 openerp
 data
-delete model=ir.actions.report.xml search=[('name', '=', 'bom.structure')] /
-report auto=True 
-header=False
-id=report_industrial_design_bom_structure
-model=mrp.bom
-name=industrialdesign.bom.structure
-rml=mrp_industrial_design_bom/report/bom_structure_industrial_design.rml
-string=BOM Structure/
-
+record model=ir.actions.report.xml id=mrp.report_bom_structure
+field name=report_filemrp_industrial_design_bom/report/bom_structure_industrial_design.rml/field
+field name=report_rmlmrp_industrial_design_bom/report/bom_structure_industrial_design.rml/field
+/record
 record model=ir.ui.view id=mrp_bom_form_view_inherit
 field name=namemrp.bom.form.inherit/field
 field name=modelmrp.bom/field

-- 
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:~elbati/ocb-addons/7.0_fix_1181291_elbati into lp:ocb-addons

2013-11-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review

When merging, please add a space l10 after !=


-- 
https://code.launchpad.net/~elbati/ocb-addons/7.0_fix_1181291_elbati/+merge/169352
Your team OpenERP Community Backports Team is subscribed to branch 
lp:ocb-addons.

-- 
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:~bruno-bottacini/account-financial-tools/7.0-add_account_invoice_template into lp:account-financial-tools

2013-11-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing

Thanks Bruno,

I agree with Pedro. This module would better fit in the lp:account-invoicing 
project.
-- 
https://code.launchpad.net/~bruno-bottacini/account-financial-tools/7.0-add_account_invoice_template/+merge/170010
Your team OpenERP Community Reviewer is subscribed to branch 
lp:account-financial-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:~luc-demeyer/account-financial-report/7.0-account_financial_report_webkit-fixes into lp:account-financial-report

2013-11-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Disapprove

Luc,

Please refer to the community guidelines to get your contributions merged:
https://doc.openerp.com/contribute/15_guidelines/contribution_guidelines/#merge-proposal-guidelines

Work on a separate feature/bug/whatever at a time. Do not mix lots of changes 
in one merge proposal, as it will be too complicated to review, thus refused.
-- 
https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_financial_report_webkit-fixes/+merge/172359
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
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:~ovnicraft/account-financial-tools/account-financial-tools-inv-voucher-constraints into lp:account-financial-tools

2013-11-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Please update the module description according to the constraint provided, or 
implement the second constraint.

Thanks.
-- 
https://code.launchpad.net/~ovnicraft/account-financial-tools/account-financial-tools-inv-voucher-constraints/+merge/175417
Your team OpenERP Community Reviewer is subscribed to branch 
lp:account-financial-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:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

2013-10-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
You have been requested to review the proposed merge of 
lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension 
into lp:server-env-tools.

For more details, see:
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/183539

[ADD] import_odbc and base_external_dbsource from lp:openobject-extension/7.0

-- 
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/183539
Your team Server Environment And Tools Core Editors is requested to review the 
proposed merge of 
lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension 
into lp:server-env-tools.
=== added directory 'base_external_dbsource'
=== added file 'base_external_dbsource/__init__.py'
--- base_external_dbsource/__init__.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/__init__.py	2013-09-07 19:40:07 +
@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis
+#2011
+#
+#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 base_external_dbsource
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'base_external_dbsource/__openerp__.py'
--- base_external_dbsource/__openerp__.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/__openerp__.py	2013-09-07 19:40:07 +
@@ -0,0 +1,61 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis, 2011
+#Additional contributions by Maxime Chambreuil, Savoir-faire Linux
+#
+#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': 'External Database Sources',
+'version': '61.3',
+'category': 'Tools',
+'description': 
+This module allows you to define connections to foreign databases using ODBC,
+Oracle Client or SQLAlchemy.
+
+Database sources can be configured in Settings  Configuration - Data sources.
+
+Depending on the database, you need:
+ * to install unixodbc and python-pyodbc packages to use ODBC connections.
+ * to install FreeTDS driver (tdsodbc package) and configure it through ODBC to
+   connect to Microsoft SQL Server.
+ * to install and configure Oracle Instant Client and cx_Oracle python library
+   to connect to Oracle.
+,
+'author': 'Daniel Reis',
+'website': 'http://launchpad.net/addons-tko',
+'images': [
+'images/screenshot01.png',
+],
+'depends': [
+'base',
+],
+'data': [
+'base_external_dbsource_view.xml',
+'security/ir.model.access.csv',
+],
+'demo': [
+'base_external_dbsource_demo.xml',
+],
+'test': [
+'dbsource_connect.yml',
+],
+'installable': True,
+'active': False,
+}
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'base_external_dbsource/base_external_dbsource.py'
--- base_external_dbsource/base_external_dbsource.py	1970-01-01 00:00:00 +
+++ base_external_dbsource/base_external_dbsource.py	2013-09-07 19:40:07 +
@@ -0,0 +1,180 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Daniel Reis
+#2011
+#
+#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 

Re: [Openerp-community-reviewer] [Merge] lp:~elbati/stock-logistic-warehouse/adding_stock_lot_costing into lp:stock-logistic-warehouse

2013-10-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review

Thanks Lorenzo.

Can you please add the translation file and run flake8 on your code ?
-- 
https://code.launchpad.net/~elbati/stock-logistic-warehouse/adding_stock_lot_costing/+merge/164766
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
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


  1   2   >