Commit Graph
109831 Commits
Author SHA1 Message Date
Xavier Morel e5ef1c5917 [FIX] P3: don't resize the value False
Yields really strange errors in P3.
2017-05-22 13:30:50 +02:00
Laurent Smet 915aa9e4db [IMP] account: improve usability on journals.
This commit add computed fields that allow to easily change the next number that will be given by the journal's sequence.
Was PR #15163
2017-05-22 09:29:17 +02:00
Xavier Morel ee894b1384 [FIX] provide items method on lru so shim works 2017-05-19 16:46:42 +02:00
xmo-odoo 05abc1faed [FIX] test_pylint: pylint error output to failure
Sometimes PyLint itself blows up which fails the test but does not
provide any indication as to why, which is inconvenient.
2017-05-19 16:46:42 +02:00
Deep Patel b6660433c6 [IMP] mail : Display Log instead of Send in full mail composer
Purpose
=======

Improve label in full mail composer.

Specification
=============

In full mail composer if it's log than send button string should be "Log".
2017-05-19 15:50:00 +02:00
Nicolas Martinelli f57f133047 [IMP] account, l10n_*: remove dependency on base_vat
Localizations depend on the module "base_vat". It is therefore
impossible to remove base_vat without the localization. This causes
issues for users who simply want to deactivate the VAT check.

opw-743260
2017-05-18 17:05:22 +02:00
Yannick Tivisse 707e5ea861 [FIX] sale: Don't hide CRM menuitem when installing ecommerce 2017-05-18 16:51:53 +02:00
Yannick Tivisse 5dfd342d08 [ADD] sale_management: Separate sales menu items from sale app
Purpose
=======

Don't display sales app icon in Odoo homepage when installing ecommerce (only website should be visible)
it's confusing to have sales icon  -> where do I manage my sales? In sales or in website? -> in website

Specfication
============

We need one new modules to add Sales menus to the homepage. Currently they come with "sale". So when installing "website_sale", we get them (because sale and account are dependencies of website_sale). We don't want that anymore. Along with that you need to replace modules in Apps menu:
sales menu items must be in the new sale_management module, no longer in sale

- create new app "sale_management"
- Currently 3 app icons when installing ecommerce (Sales, Invoicing, Website) -> too many icons, where to start? -> GOAL: Only 2 icons > Website, Invoicing
- new module to display Sales Icon (like account_accountant): "sale_management"
- move menu items to sale_management
- sale, sale_management: move app description, rewrite the manifest file
- sale, sale_management: move tour from sale > sale_management
- Changes in some dependency module "sale" > "sale_management" modules: event_sale, mrp_repair, pos_sale, report_intrastat, sale_crm, sale_expense, sale_stock, sale_timesheet, website_quote

Menu items to move 'sale' --> 'sale_management'
===============================================

Sales
	Quotations
        Sales Orders
Invoicing
	Orders to invoice
        Orders to Upsell
Catalog
	Products
        Product Variants
        Pricelists
Reporting
	sales
Configuration
	Settings
        Products -Attributes, Attribute Values, Internal Categories
2017-05-18 14:41:48 +02:00
Yannick Tivisse 0f92eed8a0 Revert "[ADD] sale_management: Separate sales menu items from sale app"
This reverts commit 9ab421d99a.

We should use the active field on the ir.ui.menu with the root menuitem instead of moving all the menuitems into a new module
2017-05-18 14:41:48 +02:00
Raphael Collet afef71d6b9 [FIX] Always round monetary values in database (#17010)
Let `create` and `write` round monetary field values before sending them to the
database. Pass the values to be written to `field.convert_to_column`, so that
the currency can be retrieved from the values, and the value be rounded.
2017-05-18 14:15:07 +02:00
Fabien Pinckaers 440177f2fa [IMP] MRP: managers should be able to fix eroneous times 2017-05-17 11:42:17 -07:00
Pragnesh Mistry 1b8050a30a [IMP] sale: Move the Margin field below Total on a sales order
move margin field from the bottom left corner of sales order to below the total_amount field
2017-05-17 16:52:09 +02:00
Mahendra Barad 0982047101 [IMP] mail: better navigation when expanding chat windows
Before this commit, expanding a chat window would open the discuss application, which means that the current navigation state (breadcrumbs) is lost.  With this commit, it simply opens a new action in the breadcrumb, and does not lose the current action. (PR 16725, closes task 31767)

Note that this implies that it is possible to have two discuss client actions open at once.
2017-05-17 16:08:46 +02:00
qdp-odoo 6db235385a [IMP] account: replace bank_acc_number by bank_account_id on bank journal form view.
Was PR#16105
2017-05-17 15:35:06 +02:00
Jitendra Prajapati 6cd0f5c44c [MERGE] payment, website_sale: improve view and use of acquirers
Purpose is to ease user flow and understanding when configuring and
managing payment methods. UX of acquirers (form, kanban) and their
integration in ecommerce is improved. Please refer to individual
commits for more details.

This merges 'dev/master-clean-payment-views-jpr-tde'.
2017-05-17 14:45:30 +02:00
Pragnesh Mistry 22e0b857bb [IMP] hr_expense: Show expense date instead of creation date in reports
Purpose
=======

In Expenses > Reporting > All Expenses, the X-axis of this pivot by default is the creation date of expense report. It should be the expense date instead.
make the same changes in graph view.

Specification
=============

In "All Expenses" report by default "Expense Date" instead of "Create Date"  in pivot
Also rename "Date" to "Expnese Date"
Changes should be made in to graph view.
2017-05-17 14:42:37 +02:00
Jitendra Prajapati 9a9f9e1ab7 [IMP] website_sale: perform small UX improvements
* Display 'order confirmed' only if it is actually the case;
 * Display uom only if those are managed;
2017-05-17 14:41:00 +02:00
Jitendra Prajapati b6d636617c [FIX] payment: perform small code improvements
* add a constraints about authorized state for tx and authorize feature
   available on acquirer;
 * order acquirers first by publish, then sequence then name;
 * add a sequence on some provider data and use a default to have some
   popular acquirers first;
 * use update instead of write in an onchange;
2017-05-17 14:41:00 +02:00
Jitendra Prajapati de5433f415 [IMP] payment: improve usability of tx views
* add provider, related on acquirer_id.provider, in order to display
   certain piece of information on tx depending on their provider;
 * better tx form view and partner address display;
 * remove unnecessary notebook on tx form view as it is not used;
 * rename Payment Acquirers menu into Payment Methods, easier to
   understand;
 * acquirer kanban view improved to install / activate / configure them
   depending on their state;
 * be able to publish / unpublish acquirers without having to wait for
   website_payment to add a button;
2017-05-17 14:40:35 +02:00
Dhaval Panchal db41366b67 [IMP] discuss: action buttons always visible on touch screens
We add the o_touch_device css class to the body (it was done in enterprise only), and various other small changes to make sure that the action buttons (in discuss, next to the message title) are always visible (not only on hover) when we are on a touch device. (pr (#15459, closes task 31208)
2017-05-17 14:39:09 +02:00
Jitendra Prajapati 327be89d0b [IMP] payment: improve labelling and description of various acquirers
Be less ecommerce centric and try to explain a bit better the various
payment options. Purpose is to ease user understanding.
2017-05-17 13:14:54 +02:00
Deepak Geriani 6ea3b8de38 [IMP] mail: close chat window with Esc key
Closes task 32316
2017-05-17 12:48:44 +02:00
Olivier Dony 5f4db9df66 [MERGE] Forward-port saas-16 up to 5afe894f44 2017-05-16 18:23:15 +02:00
Olivier Dony 5afe894f44 [MERGE] Forward-port saas-15 up to 878fbc75ff 2017-05-16 17:09:45 +02:00
Thibault Delavallée cbca7e1313 [MERGE] payment, website_sale(_delivery): simplify payment configuration
This commit proposes to simplify payment acquirers configuration in
backend and frontend. Includes mainly

 * configuration boxes for website admins at checkout process;
 * better and fixed display of transaction status at checkout;
 * hide unnecessary fields for transfer provider;

This commit merges 'dev/master-payment-tde'.
2017-05-16 15:41:26 +02:00
Laurent Smet 259c5d9145 [IMP] report: pass pdfreport_path to postprocess_pdf_report method
Restore a feature lost due to refactoring in https://github.com/odoo/odoo/commit/e80238042c9d93d492ea8b06b0041aced0d81dcd
that pass the path to the report instead of the attachment content in order to perform changes to be part of the final result.

see https://github.com/odoo/odoo/pull/16936#issuecomment-300827614
2017-05-16 15:37:23 +02:00
Laurent Smet d5fa3232fe [FIX] base, report: fix some issues due to removed 'report' module.
fix related to task: https://www.odoo.com/web#id=31625&view_type=form&model=project.task&action=333&active_id=967&menu_id=4720

The 'report' module no longer exists but a dependency was still there in l10n_ca.

- fix dependency in l10n_ca
- fix broken company layout
- move layout templates from base to web
2017-05-16 15:37:15 +02:00
Thibault Delavallée dac90e79f7 [FIX] payment_transfer: hack the broken hack
Put bank data back in transfer providers. Otherwise users are very very
sad to not have a default message containing bank accounts data.
2017-05-16 15:35:11 +02:00
Thibault Delavallée f0abe5b0ed [IMP] payment_transfer: hide unnecessary fields for transfer provider
Indeed as transfer transactions are always pending and independant of
any callback, it is not necessary to see credentials, auto confirmation
settings and error / cancel / done messages.
2017-05-16 15:35:11 +02:00
Thibault Delavallée a995e28554 [FIX] payment: set enviroment back in its own environment 2017-05-16 15:35:11 +02:00
Thibault Delavallée 24d996c534 [IMP] website_sale: simplify code and improve display of tx status after checkout
Currently displaying tx status after checkout is done using two templates
and two separate pieces of code, aka "order_state_message" and
"payment_confirmation_status". This leads to checkout information
being displayed twice in the page. It is displayed once before the
order details and once after payment information.

This commit simplifies the code to display only once the transaction
status after payment information. This one is the "polled" one aka
refreshed automatically several times to try to catch responses from
payment providers.

From the confirmation message it is also now possible to directly jump
in the backend for website administrators in order to edit the displayed
messages.
2017-05-16 15:35:11 +02:00
Thibault Delavallée 6c2535b9b6 [IMP] website_sale(_delivery): allow to choose a payment / delivery methods in frontend
When choosing a payment and/or delivery provider in website_sale admin
now see a link to install and configure them.
2017-05-16 15:35:11 +02:00
Thibault Delavallée f9a8c53e48 [REF] website_sale(_delivery): correctly set data processing methods for payment in controllers instead of model
Currently /shop/payment controller perform some data processing. It calls
model methods _get_errors and _get_website_data that are defined on sale
order model only to be inherited in website_sale_delivery.

This commit proposes instead to have a data processing method in sale
controllers that is inherited in website_sale_delivery. There is no need
to have model methods only to format data.
2017-05-16 15:23:14 +02:00
Sanjay Jamod 8dac9cfc93 [FIX] web: website tour and website seo are using ajax_service
so load ajax_service service into assets_common instead of assets_backend (#16993)
2017-05-16 15:05:16 +02:00
Géry Debongnie c3ca877d81 [IMP] web: better error logging for qunit tests
Before this commit, when a test in the qunit test suite failed, the
error displayed by the runbot was not really interesting.  It just said
that test_01_js failed.  With this commit, each assertion error is now
logged as an error, which means that they are displayed in the short
error log by the runbot.
2017-05-16 14:41:03 +02:00
Géry Debongnie 15a227375a [ADD] web: add benchmark support
With this commit, we introduce a benchmarking infrastructure: a new
controller, accessible at the route /web/benchmarks which will render a
new template (web.benchmark_suite).

This template uses benchmark.js and qunit.js to display a list of
benchmarking informations.  For example, the number of op/s for
instantiating and destroying a list view.

I hope that this is the start of the beginning of taking the habit to
check our JS code performance sometimes, and making sure we do not have
large regression without a good reason.
2017-05-16 14:41:00 +02:00
Yannick Tivisse 0a523a16cd [IMP] delivery: Define _get_shipping_price_from_so for a single order
These functions are always called with a single order. We don't need to make a big pâté in multi
2017-05-16 14:37:23 +02:00
jpr-odoo bc705c0fce [IMP] (website_sale_)delivery: Clean carrier form view, enlarge carrier box on payment 2017-05-16 14:37:23 +02:00
Aaron Bohy 0c8d1f6089 [FIX] web: Form: col width of x2many lists inside groups
The form renderer defines inline style on the groups it renders to
set the width of their children according to the col and colspan
attributes specified in the view.

However, before this rev., this impacted inner list views (for
x2many fields), as their columns get an harcoded width as well,
whereas they shouldn't.

This was for example the case in the stock.location.route form
view, for the push rules and procurement rules o2m fields.
2017-05-16 14:07:49 +02:00
Olivier Dony 878fbc75ff [MERGE] Forward-port saas-14 up to a6e2b3929f 2017-05-16 13:27:08 +02:00
Olivier Dony a6e2b3929f [FIX] website: remove extra % char in forward-port 2017-05-16 13:25:56 +02:00
Olivier Dony 61c2b8efd5 [MERGE] Forward-port of saas-14 up to 9b3ca1af23 2017-05-16 12:47:44 +02:00
Olivier Dony 9b3ca1af23 [MERGE] Forward-port 10.0 up to 1545995b39 2017-05-16 12:12:30 +02:00
qsm-odoo 1545995b39 [FIX] web_tour: tips were overlapping the chat windows
Before this commit, the tour tips were overlapping the chat windows.
This was due to the fact they always had a z-index of 1051 wherever
they were. If a tip was placed in a location where none of its
ancestors created a new stacking context, the closed tip was then
positioned above everything else, including the chat windows.

A simple solution would have been the one proposed by this commit:
https://github.com/odoo/odoo/commit/856e78b35a47a8732e5bd4da1cce247df70e5414
However, as it explained, this would not be entirely correct. In the
website, elements are using z-indexes and would thus hide closed
tooltips when it is not wanted if those have no z-index at all.

The solution this commit is making is to force the creation of a
stacking context for tooltips' parent. The z-index will still exist
on all tips (closed and not closed) to force them above every other
elements, but now only elements of the same parent. This is indeed
more correct (but maybe still not entirely): website tooltip are
often placed in the body (as it is the element which scrolls) and
are thus still placed above elements like the navbar as their
z-index is left high. However, when placed in the appswitcher, they
will now be placed above appswitcher elements only (and thus not
above outside elements which are themself above the appswitcher
(e.g. the chat windows)).

opw-743330
2017-05-16 11:17:13 +02:00
Aaron Bohy b0e51ffade [FIX] mail: Chatter: crash when mentionning non followers
Caused by a stupid programming error.

Also refactor a bit the code that performs mention RPCs (that are
throttled) to allow to override the throttling value for the tests.
2017-05-16 11:16:03 +02:00
Thibault Delavallée 877d4b60e7 [IMP] website_quote: define default order quote template coming in settings
Using a real ir.values default that user can customize in settings
is better for user experience than an hardcoded template using an
xml_id. Betcha.
2017-05-16 10:48:58 +02:00
Olivier Dony ab8c81a96f [MERGE] Forward-port saas-11 up to 0570033ae7 2017-05-16 10:30:23 +02:00
Olivier Dony 0570033ae7 [MERGE] Forward-port 9.0 up to 2c4512c263 2017-05-16 10:06:36 +02:00
Olivier Dony 2c4512c263 [MERGE] Forward-port saas-6 up to 7dc2ed94c3 2017-05-16 09:22:27 +02:00
Aaron Bohy ac092e9ccd [FIX] web: List: force grouped lists to be non editable
Editable grouped list views isn't a supported feature. Before this
rev., grouped lists weren't automatically set as non editable, so
it could cause crashes when the user clicked on rows of grouped
lists with editable="top" or "bottom".

This was for example the case in Timesheets > All timesheets ->
go to list view.
2017-05-16 08:49:26 +02:00