- Dec 10, 2015
-
-
Christophe Simonis authored
-
- Dec 09, 2015
-
-
Christophe Simonis authored
-
Thibault Delavallée authored
-
Thibault Delavallée authored
Do not restrict followers-based alias to members of non public channels. Otherwise you may have issues defining a public mailing list with a subscription step. People still wouldn't be able to post on a public channel using a restricted alias.
-
Thibault Delavallée authored
When using a template adding the signature, do not re-add it when posting the message on a document. For example the Sale Order - Send by Email templates generated two signature in the email. This issue has been introduced when migrating mail to the new API.
-
Nicolas Martinelli authored
When the delimiter '@' is used in the composer, a regular expression is built in order to suggest followers. However, if this regular expression is incorrect, it causes a crash since RegExp will throw a syntax error. Since a new regular expression is built for each new character the user types, this situation can occur very easily. For example, typing '@(' will cause a crash. This fix replaces characters which may easily lead to a syntax error by a space.
-
Martin Trigaux authored
-
- Dec 08, 2015
-
-
Jeremy Kersten authored
When no update has been done on website, we don't have this default-lang data/attribute. So we use an hack to find the default lang which one is the shortest because no lang is present in the url.
-
Jeremy Kersten authored
-
Christophe Simonis authored
Method expect a recordset.
-
Denis Ledoux authored
Most occurences of the fullscreen feature have been removed in the below revision: eebdbb677fdee1163c16ec2bb967f37e1e4b69e Only the field has been left. This isn't allowed to go fullscreen without a direct user click by most browsers, for security purposes. opw-658838
-
Thibault Delavallée authored
message_get_email_values is declared as receiving cr, uid, id. However this method is an override of a multi method. A decorator has been added to tell that id is actually a list of ids and avoid list encapsulation issues with the result. This way the method parameters are not changed. [CLEAN] mail: removed some unused imports, by the way
-
Thibault Delavallée authored
This computed field allows to know whether the current user is a channel member or not. This is necessary to fix the join / leave action button on channels. It was still based on the followers and has not been updated with the new v9 model of channel members.
-
Thibault Delavallée authored
Now that channels can follow documents, restricted aliases are a bit too restricted. We consider that followers-based aliases accept emails from the direct followers (message_partner_ids) or member of non public channels (message_channel_ids, filetered public field). This way we avoid having public channels added by mistake, leading to information leak.
-
Thibault Delavallée authored
It is not interesting to be able to add chat channels to document.
-
Thibault Delavallée authored
Not being able to add / remove members of a channel through its form view is quite limitating. Also added the followers widget in the channel form view in technical mode. Indeed it is sometimes necessary to be able to check a channel followers.
-
Aaron Bohy authored
Concerns only chat messages.
-
Goffin Simon authored
When making several expenses for the same analytic account, it can be possible that these expenses have the same product but different price unit(e.g.:this is the case for plane ticket with a vairiable price according to the time). Then the function "_get_sale_order_line" must check for all the existing confirmed SO lines if there is one which matches with the analytic account, the product and the price before creating a new SO line. opw:658383
-
Nicolas Martinelli authored
Time is now localized in the discussion.
-
Christophe Simonis authored
-
Nicolas Martinelli authored
In a multi-currency environment, the difference amount computed will be affected by the exchange rate change. In other words, if the exchange rate changes between the invoice date and the payment date, this difference will be reflected in the difference amount. For example: - Company currency is USD, invoice currency is EUR - At invoice date, 1 USD = 1 EUR - At payment date, 1 USD = 0.9 EUR - An invoice of 1000 EUR is created - A payment of 1000 EUR is done Without this fix, a payment difference of -100 EUR is shown. The reason is that at invoice date, we expect a payment corresponding to 1000 USD, which is equal to 900 EUR at payment date. This fix doesn't do any currency conversion if the payment currency is the same than the invoice currency, to prevent any exchange rate difference shown. Indeed, if an invoice is 1000 EUR, a user expects that a payment of 1000 EUR will not show any payment difference. Note that this doesn't fix the payment difference amount when an invoice in a given currency is paid in another currency. opw-658177
-
Nicolas Martinelli authored
On a payment form, the company currency is obtained from the journal. This means that as long as no journal has been selected, no company currency is defined and the payment difference is wrongly calculated. This fix uses as company currency the currency of the user's company if no journal is chosen, which should be a safe choice and lead to a correct calculation of the payment difference. opw-658177
-
Christophe Simonis authored
-
Quentin De Paoli authored
-
Christophe Simonis authored
-
Christophe Simonis authored
This parameter is not part of the spec [1] and may not be supported by all OAuth providers. [1] http://tools.ietf.org/html/rfc6749#section-4.2.1
-
Christophe Simonis authored
-
Quentin De Paoli authored
-
Martin Trigaux authored
Messages about join and leaving channel are displayed in /groups/
-
Quentin De Paoli authored
[FIX] hr_expense: accounting entries created now have the employee name as label, which was a usability lack
-
Jeremy Kersten authored
Add 'base' as a requirement to be sur that dom is ready before to bind the popover on the cart. Without it, the popver is just never displayed. Bug introduced during ref 71732dce.
-
Nicolas Martinelli authored
Commit fa1da9dc removes this part of the code, but it should be kept. Now with a clearer comment.
-
Aaron Bohy authored
The read_more button displayed in long messages is an <a> inside a <span>, both having classname oe_mail_expand, and an handler in bound on that classname since rev. 169f819a. This handler was executed twice when clicking on that button, toggling twice the message's bodies long and short, thus doing nothing.
-
Christophe Matthieu authored
-
Goffin Simon authored
Equipments must only be visible for HR Equipment Managers, HR Officers and HR Managers. opw:658458
-
Nicolas Martinelli authored
When a user records an activity or a timesheet entry on an analytic account not linked to a SO, the employee's cost is not taken into account. This is due to the fact that the search of the timesheet cost is only performed if there is a SO line. This fix uncouples the SO line search with the timesheet cost search, since there is no reason to link them. opw-657899
-
Nicolas Martinelli authored
Some code cleaning here, because this is really ugly.
-
- Dec 07, 2015
-
-
Cédric Snauwaert authored
Problem was that if we don't have a sale_order_line, it might crash trying to compute between a currency and an empty recordset
-
Thibault Delavallée authored
When changing the user of a lead, correctly compute its sales team. Moreover get rid of references to a now removed multi salesteam group.
-
Thibault Delavallée authored
-