Martin Trigaux [Wed, 24 Sep 2014 15:19:22 +0000 (17:19 +0200)]
[IMP] crm: remove TODO posted by past me
Martin Trigaux [Wed, 24 Sep 2014 15:10:14 +0000 (17:10 +0200)]
Forward port of branch 7.0 up to 1933e92
Martin Trigaux [Wed, 24 Sep 2014 15:00:24 +0000 (17:00 +0200)]
[INP] portal_sale: remove buggy context
Even if this should never be used in normal portal environment, if a user with enough access rights tries to create an invoice, this would crash. We don't need context here (opw 612591).
Guewen Baconnier [Tue, 16 Sep 2014 13:21:07 +0000 (15:21 +0200)]
[IMP] stock: Optimize stock_move.setlast_tracking()
When setlast_tracking is called on a large number of moves in a picking
(e.g. when splitting moves in a picking), the time to complete grows
exponentially. The reason is that it loops over all the moves of
a picking, even if it keeps only the last tracking.
The method now uses a search() with a limit so it doesn't need to browse
all the moves.
Added test to check the behaviour of setlast_tracking
Fixes #2448
Ravi Gohil [Mon, 9 Jun 2014 09:28:30 +0000 (14:58 +0530)]
[FIX] crm: correct open lead or opportunity view from calls
When opening a lead/opportunity from the phonecalls view, we did not open the correct view (always the lead).
This will use the type of the crm.lead to determine which view should be used, opw 608493.
Martin Trigaux [Wed, 24 Sep 2014 12:34:14 +0000 (14:34 +0200)]
[IMP] stock: do not generate empty backorders
If every line of a partial delivery is at zero, do not generate empty backorders (opw 608680)
dhr-odoo [Tue, 1 Jul 2014 11:20:59 +0000 (16:50 +0530)]
[FIX] point_of_sale: invoice not created in backend
Missing arguments that lead to the invoice not created in the backend (opw 609470).
Denis Ledoux [Wed, 24 Sep 2014 11:10:47 +0000 (13:10 +0200)]
[MERGE] forward port of branch 7.0 up to 5c2fb12
Denis Ledoux [Wed, 24 Sep 2014 11:07:26 +0000 (13:07 +0200)]
[FIX] web_calendar: reset dataset index if search result empty
Martin Trigaux [Wed, 24 Sep 2014 08:35:28 +0000 (10:35 +0200)]
Forward port of 7.0 up to rev 4a064b9
Martin Trigaux [Wed, 24 Sep 2014 08:27:40 +0000 (10:27 +0200)]
[FIX] stock: error on stock.partial.move creation
The stock_partial_move wizard removes the required attribute for the field picking_id on a stock.partial.move. This means that we could get moves without picking_id and the previous line was failing ('NoneType' object has no attribute 'currency_id'). opw 614531
Denis Ledoux [Tue, 23 Sep 2014 17:01:11 +0000 (19:01 +0200)]
[MERGE] forward port of branch 7.0 up to 9c77f79
Denis Ledoux [Tue, 23 Sep 2014 16:53:17 +0000 (18:53 +0200)]
[FIX] widget html: rendering issue in mail composer on Firefox
The cleditor width does not include the margins. Setting 100% will make the editable area too large (104%) on Firefox (opw 611700).
This issue was already fixed in newer releases, from commit
9247c37de7d67beb479905e218656665dc12391f
Nevertheless, it introduced a new issue: while editing the form, the content of widgets html which were located in hidden notebook pages weren't displayed when displaying the page content.
opw-614448
Denis Ledoux [Mon, 22 Sep 2014 14:33:09 +0000 (16:33 +0200)]
[FIX] website_blog: portal users should not be able to read unpublished blog posts
Denis Ledoux [Mon, 22 Sep 2014 13:58:24 +0000 (15:58 +0200)]
[FIX] account_voucher: import invoices, sign according to transaction type
Denis Ledoux [Mon, 22 Sep 2014 13:00:53 +0000 (15:00 +0200)]
[FIX] auth_ldap: remove unused sql column
Martin Trigaux [Mon, 22 Sep 2014 10:16:46 +0000 (12:16 +0200)]
[FIX] account_voucher: convertion rate of purchase receipt
The conversion of currencies in vouchers are made based on payment_rate_currency_id field (instead of usual currency_id).
This field was not present in the purchase view and not correctly set when we change the journal (and thus the currency) which was leading to rates of 1. With a rate of 1, the amount is equal to the currency amount (which is rarely correct). opw 611254
Géry Debongnie [Mon, 22 Sep 2014 09:18:41 +0000 (11:18 +0200)]
[FIX] correctly forward context in graph view
the new graph view silently ignored the context when doing its rpc
read_group. Usually, it's not really a problem, which is why it is only
now being fixed, but some models actually use the context in read_group.
(for ex, account_entries_report)
Martin Trigaux [Fri, 19 Sep 2014 13:28:02 +0000 (15:28 +0200)]
Forward port of 7.0 up to rev 9b87d6f
Martin Trigaux [Fri, 19 Sep 2014 12:29:43 +0000 (14:29 +0200)]
[FIX] purchase: keep fallback behaviour that was removed at rev 8973217
If no valid invocie is found on the purchase order line, use the price and currency of the line (coherent with help message)
Martin Trigaux [Fri, 19 Sep 2014 09:34:41 +0000 (11:34 +0200)]
[IMP] stock: display translated selection value in message
When a user tried to delete a done or canceled picking, the error messages used to display the key of the selection field ('done' or 'cancel') which was surprising in other languages than English. This patch takes the string value of the selection field, keeping the context to get the translated value (opw 613068)
Martin Trigaux [Fri, 19 Sep 2014 09:06:17 +0000 (11:06 +0200)]
[FIX] event: translate templates into user language
When sending an email of the registration/confirmation, the context was lost and the tempate sticked in en_US.
With the patch, the template will use the current user's language.
Denis Ledoux [Thu, 18 Sep 2014 14:05:34 +0000 (16:05 +0200)]
[MERGE] forward port of branch 7.0 up to a218a9e
Denis Ledoux [Thu, 18 Sep 2014 13:50:59 +0000 (15:50 +0200)]
[FIX] kanban: wrong condition to reset the dataset index
The dataset index should be reset either if the dataset is empty or the index is greater than the records size
rmu-odoo [Wed, 10 Sep 2014 07:45:51 +0000 (13:15 +0530)]
[FIX] hr_timesheet_sheet: avoid error when sort attendances
When an attendance line is edited, the tuple has no key 'name' which lead to a traceback.
Fixes #546
qdp-odoo [Wed, 17 Sep 2014 16:04:41 +0000 (18:04 +0200)]
[FIX] stock/purchase: dates and currency rates
Use the currency rate of the purchase date instead of the one of at the reception time (opw 610430)
Denis Ledoux [Wed, 17 Sep 2014 16:03:34 +0000 (18:03 +0200)]
[FIX] mail: chatter read more/less expand first oe_msg_body_long/short in the first oe_msg_body
Otherwise, if the mail itself contains a oe_msg_body_long (because sent in the mail history), it will open that one
Martin Trigaux [Tue, 16 Sep 2014 09:37:29 +0000 (11:37 +0200)]
[FIX] account_anglo_saxon: correctly set the accounts for returns
The generated moves on a return should be the reverse of the outgoing move.
Fixes #1817, opw 604417
Denis Ledoux [Wed, 17 Sep 2014 14:24:40 +0000 (16:24 +0200)]
Olivier Dony [Wed, 17 Sep 2014 13:13:42 +0000 (15:13 +0200)]
[MERGE] Forward-port 7.0 up to 938502a
Olivier Dony [Wed, 17 Sep 2014 13:09:55 +0000 (15:09 +0200)]
[FIX] loading: always process auto-installed modules for new databases
If the server was started without -i or -u and
happened to initialize a fresh database,
auto-installed modules that depend
on `base` only would stay in status "to install"
without actually being installed (until the next
installation round was triggered).
This was of little consequence in 7.0, but causes
a crash in 8.0.
Fixes #953
Denis Ledoux [Wed, 17 Sep 2014 11:39:13 +0000 (13:39 +0200)]
[MERGE] forward port of branch 7.0 up to f5f7609
Denis Ledoux [Wed, 17 Sep 2014 11:32:30 +0000 (13:32 +0200)]
[FIX] orm write: do not try to store computed & stored fields for relational records deleted by *2many fields
opw-613772
Martin Trigaux [Wed, 17 Sep 2014 10:28:21 +0000 (12:28 +0200)]
[FIX] website_sale: retrieve transactions as superuser
Due to additional security rules, the transactions made as public user will have a new partner_id. The transaction needs to be retrieved as admin to be set in the context.
The operations in payment_get_status are made as superuser but the session_id is checked in the assert above to avoid url manipulation.
Martin Trigaux [Wed, 17 Sep 2014 09:05:32 +0000 (11:05 +0200)]
[IMP] payment: simplify rev d99835e
The group public is defined in base so no need to add security rule in website_payment module (same as for portal)
Denis Ledoux [Tue, 16 Sep 2014 16:33:42 +0000 (18:33 +0200)]
[FIX] auth_ldap: avoid to try twice the authentication to the ldap server
See #1982
Denis Ledoux [Tue, 16 Sep 2014 15:55:14 +0000 (17:55 +0200)]
[FIX] point_of_sale: set journal as pos payment method on setting journal in the payment methods of a pos config when none is set.
Martin Trigaux [Tue, 16 Sep 2014 13:27:56 +0000 (15:27 +0200)]
[FIX] payment: access rights limitation
Do not allow everybody to access account.transactions.
Restrict by default to readonly and even restrict the access with a record rule, give access to salesman.
Denis Ledoux [Tue, 13 May 2014 08:15:01 +0000 (10:15 +0200)]
[FIX] sale: backport of rev
709e22c32aca
[FIX] sale: sale report view, wrong domain
Quotations are sale.order with state draft and sent
Sale orders are sale.order with state other than draft, sent and cancel
Martin Trigaux [Mon, 15 Sep 2014 13:24:43 +0000 (15:24 +0200)]
[FIX] orm: do not drop foreign keys of transient models
During the update of a module, the existing foreign keys are dropped if they have a different ondelete_rule than the one specified on the field.
The foreign keys for many2one transiant -> non-transiant are created with cascade rule by default (see `m2o_add_foreign_key_checked` method) so the check needs to be realised in the same conditions.
Denis Ledoux [Mon, 15 Sep 2014 13:22:04 +0000 (15:22 +0200)]
[FIX] web: do not check if db in session match filter if no db
Martin Trigaux [Mon, 15 Sep 2014 10:54:40 +0000 (12:54 +0200)]
Forward port of 7.0 up to revision 42680c9
Well actually it's an empty forward port to avoid forward porting last 7.0 commit in saas...
Denis Ledoux [Mon, 7 Apr 2014 08:24:45 +0000 (10:24 +0200)]
[FIX] web: backport of ebb826a and 2372d30
Fixing breadcrumb not being displayed after some action button (e.g. validate invoice, opw 612763)
Denis Ledoux [Mon, 15 Sep 2014 09:58:10 +0000 (11:58 +0200)]
[MERGE] forward port of branch 7.0 up to b601015
Denis Ledoux [Mon, 15 Sep 2014 09:55:53 +0000 (11:55 +0200)]
[FIX] tools: restrict available attributes
Olivier Dony [Fri, 12 Sep 2014 15:51:56 +0000 (17:51 +0200)]
[IMP] ir.model:.fields allow setting ondelete=restrict foncustom fields
Christophe Simonis [Fri, 12 Sep 2014 15:28:55 +0000 (17:28 +0200)]
[MERGE] forward port of branch 7.0 up to 9b1cdea
Denis Ledoux [Fri, 12 Sep 2014 14:25:48 +0000 (16:25 +0200)]
[FIX] mrp_repair: invoice the actual invoice partner id
Fekete Mihai [Fri, 12 Sep 2014 14:04:22 +0000 (16:04 +0200)]
[FIX] l10n_ro: backport of fix done in res_partner.py from aa76885
PR #720
Denis Ledoux [Thu, 11 Sep 2014 15:54:58 +0000 (17:54 +0200)]
[FIX] product: can be sold as default filter for templates
fka-odoo [Wed, 2 Jul 2014 08:38:41 +0000 (14:08 +0530)]
[FIX] web: sort with undefined value
In this awesome programming language called javascript, undefined is neither bigger nor smaller than a string. Previous code was then considering undefined values equal to any string.
This fix allows to sort in a column of a o2m field and group the undefined values together. opw 607704
Denis Ledoux [Thu, 11 Sep 2014 15:20:27 +0000 (17:20 +0200)]
[FIX] stock: return picking, pass serial number value when returning
Dharti Ratani [Thu, 11 Sep 2014 15:01:03 +0000 (17:01 +0200)]
[FIX] survey: access rights error when editing
The wizard 'Edit Survey' > 'Add Question' was trowing an access rights error as the id was passed as a string instead of a real id. opw 607854
Denis Ledoux [Thu, 11 Sep 2014 15:00:45 +0000 (17:00 +0200)]
[FIX] web_graph: pivot_table, use _.isEmpty to check if value_lookup is False
Martin Trigaux [Wed, 10 Sep 2014 16:06:38 +0000 (18:06 +0200)]
[FIX] hr_expense: tax amount computation
The tax_amount field should contain the basic amount (without tax) for base tax code. (opw 613470)
Denis Ledoux [Thu, 11 Sep 2014 07:58:38 +0000 (09:58 +0200)]
[FIX] web: *.openerp.com to *.odoo.com
Martin Trigaux [Wed, 10 Sep 2014 15:35:44 +0000 (17:35 +0200)]
[FIX] mail: encoding of sender name with unicode
When sending an email, both formats 'Name <email>' or '"Name" <email>' can be used for fields 'From', 'To' and others. If the name contains unicode characters, a regex only matching '"Name" <email>' was used to encode the name with RFC2047. That meant that the name was not encoded and eventually dropped, using only the email part.
Instead of using a limited regex, use the parseaddr method from email library.
Fixes lp:1272610, opw 607683
Denis Ledoux [Wed, 10 Sep 2014 15:25:56 +0000 (17:25 +0200)]
[FIX] mail: alias domain should be fetch as superuser
as ir.config.parameter has limited access
Frederic van der Essen [Wed, 10 Sep 2014 12:30:43 +0000 (14:30 +0200)]
[FIX] point_of_sale: chrome was dropping the cookie when loading the company logo, resulting in an incorrect logo in multi-db setups.
Frederic van der Essen [Wed, 10 Sep 2014 11:39:50 +0000 (13:39 +0200)]
[FIX] point_of_sale: fix some image chaching issues with the receipt logo. Using a random url at each fetch ensures that we don't get cached results
Denis Ledoux [Wed, 10 Sep 2014 10:18:14 +0000 (12:18 +0200)]
[FIX] ir_attachment: attachements removed on record unlink
Bhavik Bagdiya [Fri, 22 Aug 2014 09:29:39 +0000 (14:59 +0530)]
[FIX] web: stages positioned properly when there is no sheet
Fixes #271
Closes #2004
Denis Ledoux [Tue, 9 Sep 2014 17:17:23 +0000 (19:17 +0200)]
[FIX] email_template: a tag link rendering, do not process if no href
Denis Ledoux [Tue, 9 Sep 2014 17:00:58 +0000 (19:00 +0200)]
[FIX] website_event_sale: take taxes from product_id_change into account
Denis Ledoux [Tue, 9 Sep 2014 16:23:32 +0000 (18:23 +0200)]
[MERGE] forward port of branch 7.0 up to 8dff247
Denis Ledoux [Tue, 9 Sep 2014 16:02:30 +0000 (18:02 +0200)]
[MERGE] forward port of branch 7.0 up to 33e0910
Denis Ledoux [Tue, 9 Sep 2014 14:45:42 +0000 (16:45 +0200)]
[FIX] orm, read_group: read_group_fill_results handle no_leaf
Martin Trigaux [Tue, 9 Sep 2014 11:43:04 +0000 (13:43 +0200)]
[FIX] hr_expense: tax included computation
When a tax included has a several childs, the source line should not be duplicated for each tax.
Without the patch, the actual product price (without taxes) would be (total - tax amount) for the last tax only (and not the sum of each tax). (opw 609104)
Simon Lejeune [Tue, 9 Sep 2014 09:52:44 +0000 (11:52 +0200)]
[FIX] web: missing backport of rev
df3c6b750249
[FIX] web: do not try to close a destroyed dialog. trigger "closing" signal on close by top-right "X" button
Denis Ledoux [Mon, 8 Sep 2014 08:49:56 +0000 (10:49 +0200)]
[FIX] ir_attachment: ignore if model uninstalled
if the model of an attachement no longer exists (the according module have been uninstalled, for instance), ignore the security check
Mohammed Shekha [Fri, 5 Sep 2014 14:39:06 +0000 (16:39 +0200)]
[IMP] css: width of readonly list view field
readonly float fields in list view were not correctly placed when the focus was on the line (seen espacially if field was large, e.g. nb_register on registration)
Loïc Bellier [Tue, 29 Jul 2014 16:48:26 +0000 (18:48 +0200)]
[FIX] stock: show the UoS quantity on picking lines
The pickings already show the UoS, but not the associated quantity, which makes it useless.
Denis Ledoux [Fri, 5 Sep 2014 12:57:20 +0000 (14:57 +0200)]
[FIX] web: do not allow to access a database if not matching db-filter
Rifakathusen Haradwala [Mon, 1 Sep 2014 13:26:04 +0000 (18:56 +0530)]
[FIX] mail: translate tracked fields name in chatter
Status tracking messages are not translated into user's language while updating any document with tracked fields (opw 612847)
Denis Ledoux [Fri, 5 Sep 2014 09:44:20 +0000 (11:44 +0200)]
[FIX] tools,mail: improve parsing of references from rev.
0bee015dd67cb809fd235215b43d331074824e42
Backport of 8.0 fix was not enough for 7.0, because it still fails for the legacy reference match (model+res_id), which is not used in 8.0
Christophe Simonis [Thu, 4 Sep 2014 15:46:51 +0000 (17:46 +0200)]
[MERGE] forward port of branch 7.0 up to 59b76ee
Your Name [Thu, 21 Aug 2014 12:55:49 +0000 (18:25 +0530)]
[FIX] sale: keep page when confirming a sale order
To keep the pager and view changer after confirmation of a sale order, do not reload the view (quotation and sale order view are the same anyway)
Fixes #250
Somesh Khare [Tue, 26 Aug 2014 06:25:23 +0000 (11:55 +0530)]
[FIX] document: display Attachment(s) menu only on form view
The attachment menu (list and add) has no effect in tree view (not supporting multi-items upload and display), the menu should then only be displayed in these view (opw 612534)
Denis Ledoux [Thu, 4 Sep 2014 10:31:14 +0000 (12:31 +0200)]
[MERGE] forward port of branch 7.0 up to 5872c24
Denis Ledoux [Thu, 4 Sep 2014 10:28:16 +0000 (12:28 +0200)]
[FIX] stock: on partial picking, do no open a new window action if not needed
This is related to rev. dd4d72d
Not opening a new window action allows to keep the current list of stock picking being threated, and to use the next an previous arrow of the forom view of the web client.
Martin Trigaux [Thu, 4 Sep 2014 08:17:18 +0000 (10:17 +0200)]
[IMP] crm_partner_assign: remove inexistant onchange
This field is readonly so a onchange is useless anyway (opw 612593)
Denis Ledoux [Wed, 3 Sep 2014 16:26:53 +0000 (18:26 +0200)]
[MERGE] forward port of branch 7.0 up to 7307227
Denis Ledoux [Wed, 3 Sep 2014 16:25:19 +0000 (18:25 +0200)]
[FIX] mrp: do not reset back stock moves to confirm
For instance, setting a BOM Phantom with:
Finished product: stockable, MTO Manufacture
Components: stockable, MTS, Buy. Inventory set to 1000
Stock moves of components are directly set to assigned once the procurement confirmed thanks to JIT
The stock moves should not be set back to confirmed after they have been assigned
Denis Ledoux [Wed, 3 Sep 2014 13:19:26 +0000 (15:19 +0200)]
[FIX] payment_paypal: avoid json decode error
with post.pop('custom', '{}'), if the key 'custom' is not there, it will return '{}', but if the key is there, but the value is equivalent to False, the json decode will fail.
Olivier Dony [Tue, 2 Sep 2014 17:42:24 +0000 (19:42 +0200)]
[IMP] web: extra comments to explain commit 57b4860
Added some cryptic comments so we remember a bit
why we have a complicated dance with on_close.
Basically we do not want to reload the
original form view until the last popup is closed,
in the case where several wizard (steps) are opened
one after the other.
Denis Ledoux [Tue, 2 Sep 2014 16:43:22 +0000 (18:43 +0200)]
[FIX] account_payment: lin2bank correct fallback condition
Stéphane Wirtel [Thu, 12 Jun 2014 13:08:33 +0000 (15:08 +0200)]
[IMP] Optimize the schedulers of the calendar, fetchmail and base_action_rules modules
Disable the the crons when they'd have nothing to do, re-enable them when a
task is (probably) introduced
Denis Ledoux [Tue, 2 Sep 2014 15:32:37 +0000 (17:32 +0200)]
[FIX] web: chain close action on wizard confirm
If a wizard has several steps, or laucnh a second wizard, the view from where has been loaded the initial wizard is finally reloaded
Denis Ledoux [Mon, 1 Sep 2014 14:10:40 +0000 (16:10 +0200)]
[FIX] web: no_create, if loose focus do not open quick_create
Anael Closson [Wed, 18 Jun 2014 12:33:35 +0000 (14:33 +0200)]
[FIX] mail: more robust parsing of In-Reply-To/References (OPW 608919)
When parsing incoming messages, ignore white-space around
In-Reply-To headers, and extract message-id items inside
the References header using a regex.
This actually serves as a workaround for broken MTAs
mangling References (such as outlook.com nesting past ones
with commas, violating RFC2822).
Closes #516 as a manual rebase.
Denis Ledoux [Fri, 29 Aug 2014 18:54:15 +0000 (20:54 +0200)]
[MERGE] forward port of branch 7.0 up to 18d365c
Denis Ledoux [Fri, 29 Aug 2014 18:40:31 +0000 (20:40 +0200)]
"[REVERT][FIX] point_of_sale: field date of report.pos.order should be a datetime"
This reverts commit
97d097a2af8c3992d5108bd75b82fa8479d196cc.
As explained in the commit comments (on Github), this patch leads to an infinite loop in 7.0, the filter of the pos orders report using the '=' operator in its domain, which is not available for datetime fields, but is for date fields.
This should not be forward ported to newer release (saas-3)
Denis Ledoux [Fri, 29 Aug 2014 14:29:17 +0000 (16:29 +0200)]
[FIX] stock: display product variant name in the slip
Denis Ledoux [Fri, 29 Aug 2014 13:46:36 +0000 (15:46 +0200)]
[FIX] l10n_ma: proper chart of account name
Christophe Simonis [Thu, 28 Aug 2014 17:07:11 +0000 (19:07 +0200)]
[MERGE] forward port of branch 7.0 up to 15ee4d8
Christophe Simonis [Thu, 28 Aug 2014 17:04:14 +0000 (19:04 +0200)]
[FIX] email_template: remove group restriction on field mail.compose.message.template_id
This was added as a security check, but access to this field is need by the wizard itself.
As non employee users can't read/create email templates, this will not be problematic
Christophe Simonis [Thu, 28 Aug 2014 16:32:11 +0000 (18:32 +0200)]
[FIX] mail: read ir.config_parameter with SUPERUSER_ID
Christophe Simonis [Thu, 28 Aug 2014 16:15:10 +0000 (18:15 +0200)]
[MERGE] forward port of branch 7.0 up to f65c913
Christophe Simonis [Thu, 28 Aug 2014 16:14:08 +0000 (18:14 +0200)]
[FIX] base: security: employee must have access to res.groups
Christophe Simonis [Thu, 28 Aug 2014 15:21:54 +0000 (17:21 +0200)]
[FIX] account: use method has_group() instead of browsing the group directly.