- Aug 19, 2014
-
-
Sébastien Beau authored
-
David Arnold authored
The type 'general' no longer exists on a account.voucher. As in general context, removed the default value on this menu. Fixes #1261, it produces a traceback in 8.0 where the selection fields are less fault-tolerant.
-
- Aug 18, 2014
-
-
Martin Trigaux authored
The field tax_amount is fieled with onchanges and the compute_tax method. Setting a different value than the one computed by the system may lead to unbalanced move (which is obviously wrong). In the future, handeling these operations by setting the correct value to the tax accounts would be better.
-
Martin Trigaux authored
The field partner_id is not required on an account.voucher but the validation was failing if none was set (opw 611663). This patch makes a fallback on the account of the voucher if neither a partner nor a writeoff account is specified.
-
Thomas Groutars authored
When uninstalling/updating a module, we may execute unlink method on product.template before product.product. In such cases, the product is already removed after removeing the template (_inherits) and the chained unlink of the product would fail (traceback when browsing).
-
- Aug 14, 2014
-
-
Denis Ledoux authored
The date_order field of the pos.order is a datetime, not a date As, in report.pos.order, the date field is related to date_order of the pos.order We usually do not commit fixes altering the model fields structure in 7.0, but this one is retro-compatible, as the database structure won't change
-
Denis Ledoux authored
Sometimes, focus_field could be a button, and in such cases, we should not focus on it opw-612045
-
Rifakat Haradwala authored
Inventory lines without production lot number should be compared with the stock level of products without a production lot. Otherwise the final result of a validated inventory is wrong as soon as there are lines with and without production lot for the same product. Adds corresponding regression test. Fixes https://bugs.launchpad.net/openobject-addons/+bug/1008099 Manual rebase of #1658, courtesy of Numerigraphe
-
Olivier Dony authored
Avoid revalidating the complete account moves that contain the lines being reconciled. The reconciliation does not change the validity of those moves anyway. This represents a very important speed up of reconciliation when moves with several hundred lines are involved.
-
Martin Trigaux authored
The title field should not have a domain by default. When a new record is created or the 'is company' field is changed, the onchange_type is triggered to restrict the domain accordingly to is_company field. When an existing record is modified, there will have no domain on title field. It's not ideal but at least companies will be able to change the title without the need to modify twice the field is_company. Fixes #1713
-
Olivier Dony authored
See also https://github.com/odoo/odoo/wiki/GitHub-Transition#translations
-
Olivier Dony authored
Total new terms: 168 Total deleted terms: 95 Total identical terms: 16329 (Some modules skipped, typically all l10n_* modules)
-
Olivier Dony authored
When the addons_path config contained the default path, the system was working with an empty addons_path in the middle of the other paths. This empty one matched for all files, making all files appear to belong to the root path component, e.g. `home` if the root path is in /home. Refactored a bit to avoid duplicates and redundant path calculations.
-
- Aug 13, 2014
-
-
Vo Minh Thu authored
On Launchpad, as commented on bug 933496, PO entries (and their comments) are shared between series. This mean that e.g. the 7.0 series can have the wrong `reference` comments (those beginning with #:) as they would be copied from say the trunk series. Those `reference` comments are used to import translations and look them up. This patch adds a few lines of code to tools.translate so that targets defined in the POT `reference` comments are used in addition to those from the PO file. Also adds a test module to validate the new behavior. This patch stems from: - the 6.1 branch by Vo Minh Thu: https://code.launchpad.net/+branch/~openerp-dev/openobject-server/6.1-fix-po-targets-933496-vmt - the 7.0 port by Numerigraphe: https://code.launchpad.net/~numerigraphe-team/openobject-server/7.0-fix-po-targets-933496-vmt
-
Olivier Dony authored
Pricelist access is necessary for being able to read the unit price of sale order lines, and does not pose major security risks, as it does not permit reading the actual pricelist definition.
-
Olivier Dony authored
Portal access rights need to be associated with security rules, or not granted at all, as soon as they grant access to non-trivial data.
-
Martin Trigaux authored
The title of the partner is restricted with a context that will change in the onchange_type. Having twice the same field will not work well. Remove the invisible attribute as the domain is handled by the domain.
-
Cecile Tonglet authored
-
Cecile Tonglet authored
-
Denis Ledoux authored
In some cases, the data dispalyed in a list depends on the context This context needs to be passed to the export method, so the exported data reflects correctly the data from the list view
-
Olivier Dony authored
The residual amount is typically needed to render the online payment forms (payment acquirers). Payments on the other hand rely on account.move.line, something that portal users should never be allowed to read. Removing the field from the view by setting a model-level group permission ensures they will not see an error.
-
Olivier Dony authored
The payment form needs to be rendered for non-employee users, and may require accessing extra financial data not available for those users.
-
Olivier Dony authored
-
- Aug 12, 2014
-
-
Denis Ledoux authored
-
Ruchir Shukla authored
When computing the price difference lines, in move_line_get of account_anglo_saxon, we loop on the result of super call for each lines (n * n times) to compute the price difference. The product_id was used to match the returned line and the original invoice line. This was wrong as we could get several lines with the same product_id (and then get n * n price difference lines). This patch adds the line id to the result of move_line_get (from account) so that account_anglo_saxon can filter more efficiently and only get one price difference per invoice line. Fixes #704
-
Martin Trigaux authored
The self is needed in the _value() method below, opw 608878
-
Denis Ledoux authored
If partner_id is False, do not try to name_get This fix is related to the rev 095be21a
-
Denis Ledoux authored
Use formataddr method from email.utils lib, which do the job correctly
-
Denis Ledoux authored
-
Denis Ledoux authored
If the mrp scheduler fails during the treatment of a procurement (due to a postgresql error, like an update lock), we let it try the other procurements, instead of stopping at the first procurement which fails. Besides, we retry to treat the procurements which failed, until the list of the remaining procurements is always the same (meaning something is wrong and block the procurements treatment).
-
- Aug 11, 2014
-
-
Denis Ledoux authored
By default, on binary images read, the server returns the binary size This is possible that two images have the exact same size Therefore we trigger the change in case the image value hasn't changed So the image is re-rendered correctly
-
Denis Ledoux authored
This is related to commit d31faceb (This is to avoid the runbot being yellow if on_change methods have extra keys in the returned value, this is not wrong, the web client will simply ignore them).
-
Jérome Maes authored
-
- Aug 08, 2014
-
-
Cecile Tonglet authored
When you set the date of a cron the July 1st at midnight, if the user time zone has a positive offset, then the converted UTC date is the June 30th and adding 1 month will end up on July 30th translating to July 31th instead of September 1st. To solve this issue we use the super user time zone for the date calculation.
-
Denis Ledoux authored
-
Martin Trigaux authored
[FIX] account_anglo_savon: round prices according to account precision to avoid getting unbalanced move The standard move_line_get from account.invoice returns prices rounded (price_subtotal field as account precision) while the anglo-saxon module could return dict with non-rounded prices. With amounts with bigger precision, we could get a difference of a few cents when comparing the debit/credits and then not be able to balance the invoice (opw 597208).
-
Denis Ledoux authored
In the return result of an on_change, having keys which are not ine the view is not that bad, it is not an actual problem. Display warning instead of asserting all the keys are in the views.
-
Hardik Ansodariya authored
The id of the journal is passed in the context. Avoid trying to browse on jounral_id=False Fixes #821
-
- Aug 07, 2014
-
-
Dharti Ratani authored
The fields procure_method and supply_method are not present if we just install project_timesheet which makes the tests fail (opw 610590).
-
Denis Ledoux authored
in view_form.js, in do_show method, if the dataset index is not set, the on_button_new method is triggered, opening a new record In regular kanbans (non-grouped kanban), the dataset index wasn't set, and, therefore, when switching to the view mode, it opened a form for a new record instead of opening the first record of the list, like this is the case for the view list and the grouped kanban
-