Commit Graph
103591 Commits
Author SHA1 Message Date
Yannick Tivisse 3106d5a3f2 [IMP] web_settings_dashboard,web_tour: usability improvement from user tests
1/ The tip should be on the textarea, not on the button.

2/ The access right field on the quick user creation form is confusing.
   I propose to put it in "Technical Feature" so that it's not visible for normal users.

3/ dissociate les settings tour from project tour
2016-06-29 12:02:19 +02:00
Christophe Simonis 2a3f37806f [FIX] im_livechat: use new api when calling cleaning methods 2016-06-28 19:01:53 +02:00
Christophe Simonis 30edf6fa8f [FIX] website_payment: use renamed fields/variables 2016-06-28 18:59:46 +02:00
Christophe Simonis 93e6316029 [MERGE] forward port of branch saas-11 up to 7153077 2016-06-28 18:05:25 +02:00
Christophe Simonis 8eb2be4668 [REM] mail_tip
Since the deletion of `web_tip` module (6416c46d85),
this link module is useless.
2016-06-28 17:22:49 +02:00
Christophe Simonis 1747e12bee [REM] hr_expense: remove unused data file.
This file has wrongly been reintroduced by 717f9217bb
2016-06-28 17:12:51 +02:00
Christophe Simonis 71530771ea [FIX] web: re-add now usefull variable self 2016-06-28 17:05:11 +02:00
Martin Trigaux cf1e8dcb92 [IMP] point_of_sale: migration of the action
Was removed at 2e4b82f4 (for good reasons) but an explicit empty value is better
for migration as it prevents needing to explicitly removed the domain.
2016-06-28 16:24:21 +02:00
Christophe Simonis 5166fbd21b [MERGE] forward port of branch saas-10 up to 6d51bce 2016-06-28 15:12:06 +02:00
Nicolas Martinelli 6d51bce057 [FIX] account: money in/out in PoS
It is not possible to take money out or put money in for a PoS since it
leads to a traceback.

The decorator `@api.one` returns a list, which is something we don't
want when the method returns a dictionary.

opw-681417
2016-06-28 14:17:39 +02:00
Yannick Tivisse c7bde46d64 [IMP] project_issue: remove 'team_id' on project issue + code cleaning
The sales team defined on the project issue is never used.

Furtermore, in the method 'stage_find' contains a parameter team_id BUT
a project_id is given when called.

That way, we can also remove the dependency to sales_team which doesn't make
sense anymore, and remove 2 useless ir.model.access rules
2016-06-28 14:13:20 +02:00
Christophe Simonis 841aa4e208 [MERGE] forward port of branch 9.0 up to 3e15b86 2016-06-28 14:10:35 +02:00
Yannick Tivisse 675f6a0f3f [IMP] crm: Activity Analysis: Confusing labels
Author: User who logged the activity in the chatter ----> Created By
Responsible: User assigned on the lead ----> Salesperson
Salesperson filer = Author. --> Same label : Created By
2016-06-28 13:26:40 +02:00
Martin Trigaux 3e15b86428 [FIX] point_of_sale: do not insert related fields on one2many
fields ref, journal_id and date on account.move.line are all related fields on
the account.move.
Setting a value here is then useless and triggers write on the account.move
after the creation of the account.move.line (inverse values in write)
2016-06-28 13:23:41 +02:00
Martin Trigaux 5d9e4b7858 [FIX] account: performances of map_tax
break on first match instead of checking other Tax Source that will not match
anyway.
2016-06-28 13:23:41 +02:00
Martin Trigaux 173b76c7bb [FIX] point_of_sale: performance and invalidation
Function fields, old-api style, that are not stored do not have cache
invalidation mechanisms. This means that they are recomputed every time the
field is accessed.
Due to the way the cache behaves, the records that are present in the cache will
be included when the prefetching is done.

In the scenario of the a POS session being closed, this implies that the cache
will contains more and more account.move.line records and try to prefetch all
(hopefully max 1000) without ever relying on the computed result for the next
access. This makes the computation of fields `amount_total` and `price_subtotal`
very slow, even for orders with a very low number of lines.

Backport the new API migration of these computed field to benefit from depends
mechanism and keep the computed value into cache when needed.

In master these fields should be stored.
2016-06-28 13:23:41 +02:00
Christophe Simonis 10550aa5a5 [MERGE] forward port of branch saas-6 up to e68ed41 2016-06-28 13:11:35 +02:00
Jainik Patel 633ff1df67 [IMP] website: Snippets images are quite old, so replace with new images. 2016-06-28 13:05:15 +02:00
Christophe Simonis e68ed417b6 [MERGE] forward port of branch 8.0 up to 3e965ec 2016-06-28 12:23:54 +02:00
Martin Trigaux cbbc90fc2e [FIX] website_payment: access parent payment method
On the other routes (e.g. /my/, /my/contract/) payment information on the parent
contact are displayed but can not be access in the my payment route.
You get in a unconsistant situation where you can access the contract of the
company and its payment information but can not use this payment information.

opw-681491, opw-681502
2016-06-28 12:16:41 +02:00
Mahendra barad 6ba6213fa9 [ADD] account_taxcloud: Integrate TaxCloud API to get Sales Tax
- add a new checkbox on fiscal position: TaxCloud API
- when computing this fiscal position, it will:
        - return an existing tax if one exists for the same tax
          percentage, state, zip and TIC Category
        - or create and return this tax if it does not exists yet
- Create a data fiscal position: country U.S., tax cloud API
- In l10n_us, add a settings in accounting: compute tax automatically using tax cloud.

NB:
- The signature of map_tax has been modified to optionally specify the product
  and the partner on which the tax is applied
- The avoid the validationerrors that may occur while evaluation the tax with Taxcloud
  when a public user is on the frontend shop (website_sale), we have to execute the
  error silently by logging it.
2016-06-28 12:12:30 +02:00
Christophe Simonis b355377a25 [IMP] web: visible default color on tags now visible in kanban view
un-revert of fe66472e2c which have
wrongly been reverted with 977db823c7
2016-06-28 12:01:27 +02:00
Goffin Simon 8ec03fde14 [FIX] point_of_sale: _check_unicity
A violated python constraint must raise a validation error.
This is expected by function _validate_fields.

opw:681212
2016-06-28 10:43:30 +02:00
Nicolas Martinelli 84b0fb5bbc [FIX] sale, purchase: add back analytic groups
The commit 2277b8955d removed these groups
while they should be kept.

opw-681848
2016-06-28 10:32:00 +02:00
Raphael Collet c78a3811b8 [IMP] models: make records.mapped(arg) return records when arg is falsy
This extension of the method `mapped` is a consistent extrapolation of mapping
a recordset through an optional sequence of relational fields.  It removes the
necessity of special-casing the empty sequence, and also avoids an obscure
exception in that case.
2016-06-28 10:16:26 +02:00
Nicolas Martinelli 77212863b1 [FIX] stock, sale_stock, portal_stock: picking access
An access error occurs when a the delivery address of a SO is set to a
different partner than the SO partner. In this case, the picking
created is associated to the partner of the delivery address. This is an
issue when the SO partner tries to view the picking from the front-end
("My Account"), since he is not allowed to access it.

opw-681341
2016-06-28 09:10:55 +02:00
Fabien Pinckaers 62d6f52312 [IMP] project: tour: improved sentence 2016-06-27 11:24:59 -07:00
Fabien Pinckaers df3631ee3a [IMP] project: tour: add a Done stage to be able to close tasks. 2016-06-27 10:25:24 -07:00
Martin Trigaux cc4f8df99f [FIX] account: compute total invoice in batch
The computation of the total_invoiced field did not work in batch mode.
When displaying a company in a form view (field invoice_total comptued), the
children of this company are present in the cache and the value of the field
invoice_total will be prefetched for these children as well (at least in 9.0).
Most of the time this is redudant information and the same value is computed
for each child while most of the invoices are on the parent.

Batching the computation per partner allows to make one global query for all
partners.

Add test to make sure the total increase correctly after a test invoice
confirmation.
2016-06-27 17:20:05 +02:00
Denis Ledoux 9062c01c02 [FIX] web: pager page on next/previous record
When browsing a form with a one2many / many2many field
with several pages, the one2many pager page was not reset
when going to the previous/next record, while the next/previous
record could not have multiple pages on its one2many/many2many,
and the page on which the user was on the former record could
not exist in the newly displayed record.

Besides, even if the page does exist, there is no reason
to display this new record X2many value from its first page.

It's the case anyway when you edit/save, it comes back to the first page.

opw-679092
2016-06-27 16:50:47 +02:00
Joren Van Onder 86cf147f2e [IMP] pos_mercury: move automatic Mercury token deletion to vacuum cron 2016-06-27 16:24:31 +02:00
Martin Geubelle f8ca7634ca [FIX] base: rate with correct current user's company
Bug introduced in 7eab8e2.

As `self.env.user` is browsed with `sudo`, the result of this call
will always return the admin's current company and not the user's current
company.

opw-681809
2016-06-27 16:20:47 +02:00
Aaron Bohy 246b9a29ae [FIX] mail: tour: tip's position
Tips are now positionned in their closest scrollable parent (b70c990b), which
breaks some tips in discuss as they overflow that parent.
2016-06-27 16:04:25 +02:00
Goffin Simon d577eeaa6e [FIX] product: pricelist items
With the pricelist, it is possible to define a pricelist just for
a variant. But in the product.product.form, only the pricelist items
linked to the product template could be displayed.
The field item_ids has been created in product.product model to display
all the pricelist applied on a product.

opw:674700
2016-06-27 15:47:58 +02:00
Aaron Bohy 488e9f299d [IMP] calendar: remove /calendar/notify RPC done every 5 minutes
Basicaly, do this RPC once, at webclient startup, to get the next notification.
Do the RPC again when the notification is displayed to get the next one.
When an event with alarm is created, edited or removed, trigger an event on the
bus for all involved users (the attendees) with an update of their next
calendar notification.
2016-06-27 15:30:45 +02:00
Aaron Bohy d9cc893dbc [IMP] openerp: models.py: use LOG_ACCESS_COLUMNS constant 2016-06-27 15:30:45 +02:00
Christophe Matthieu 5aac400e33 [IMP] qweb: use CDATA in test and remove stringify_children method 2016-06-27 14:29:04 +02:00
Géry Debongnie 96bf5b2656 [FIX] web: fix label for selection fields in grouped list view
Before this commit, the list view, grouped by a selection field, displayed the
'technical' value of the field as group label.  It was basically forced
to do that, because the result of the read_group only contains the
technical value.

This commit changes the list view in a way that it now requires a full field
get, so it can use it to find the correct string.
2016-06-27 14:03:54 +02:00
Nicolas Martinelli 3229629579 [FIX] stock: reserved quants in lots
Let's take the following case:
- Enable lot tracking on a product
- Costing Method is "Real Price", Inventory Valuation is "Perpetual"
- Create a picking with 2 stock moves, each stock move is linked to the
  same product

When the picking is validated, 4 account moves are created instead of 2.

This occurs because the list of stock moves and the list of reserved
quants is sent to the method `_move_quants_by_lot`, but there is no
check to which stock move is related a given quant.

For most cases, the issue doesn't show up since there isn't usually 2
stock moves with the same product in a given picking. There is at least
one use case where this happens: selling a pack of two identical
products. In this specific case, 2 stock moves of the same product are
created, but one pack operation will merge the 2 stock moves.

opw-680308
2016-06-27 12:19:08 +02:00
Martin Trigaux 5abcbf0eb2 [FIX] js: translations and string substitutions
_('foo '+bar) does not work as `bar` is a variable with undefinied content
during translation export (and variable content during evaluation) so a match
will not be found.
Also trailing spaces are stripped during synchronisation with transifex.

The proper syntax would be
_.str.sprintf(_t("foo %s"), bar)
but to avoid breaking existing translations (need to reexport .pot)
use the syntax
_("foo ") + " " + bar
that is more stable friendly.

The 'No value found for the field...' string is not possible to be fixed without
breaking the translations should be split at least twice.
Write it the propre way directly.
2016-06-27 12:05:36 +02:00
Denis Ledoux 0fa052cc1e [FIX] l10n_generic_coa: remove liquidity account templates
These accounts are created automatically on the journals creation
e.g. On the creation of a Bank Journal, a Bank account is automatically
created, linked to this journal.

Before this revision, when installing the generic chart of account
(or l10n_us), you had two Bank accounts (101400, 101401),
and two Cash accounts (101500, 101501),
by default,
which is not right.

opw-676950
2016-06-27 11:21:35 +02:00
Aaron Bohy 8d61316486 [FIX] web_tour: load consumed tours in the frontend
Also deactivate unnecessary magic fields for the web_tour model.
2016-06-27 10:59:39 +02:00
Stéphane Bidoul e30ea5647d [IMP] account: adding the invoice.tax.line id in tax_line_move_line_get() to allow inheritance.
Was PR #12576. Courtesy from sbidoul
2016-06-27 10:55:07 +02:00
qdp-odoo 0a6afa1ff1 [FIX] account: grouping tax lines on invoices when no tax account is set on them. Fixes #12561 2016-06-27 10:55:07 +02:00
Martin Trigaux c20ec2ced6 [FIX] website: truncate description at 160
Search engines (or one at least) truncates the description at 160. The advised
size is between 150 and 160 so no need to force a smaller size. It's a good
idea to keep it at 155 but larger sizes should be allowed.
cf: https://moz.com/learn/seo/meta-description

We could fully remove the limit as there is corner cases where a longer size is
used (https://moz.com/blog/i-cant-drive-155-meta-descriptions-in-2015) but this
should be in addition with an indication for the user of the current size and
the advised size.
2016-06-27 09:55:25 +02:00
Christophe Matthieu 15e1f88a73 [FIX] qweb: fix t-if and t-else and add test
Can not use remove with the loop for in (in _compile_directive_content)
2016-06-27 08:08:00 +02:00
Fabien Pinckaers 28656a1a3c [IMP] base: currency: add a form view to hide company 2016-06-25 23:57:58 -07:00
Fabien Pinckaers 770f7dfc6c [IMP] product: Filter on attributes only visible if multi-variants enables 2016-06-25 23:53:03 -07:00
Odoo Translation Bot 3e965ecf58 [I18N] Update translation terms from Transifex 2016-06-26 02:57:50 +02:00
Odoo Translation Bot 765b3fc6ac [I18N] Update translation terms from Transifex 2016-06-26 00:29:54 +02:00