Skip to content
Snippets Groups Projects
  1. Oct 12, 2016
    • Alexandre Fayolle's avatar
      [FIX] context mutations (#10368) · 65c685ad
      Alexandre Fayolle authored
      when extending these methods with the new api, the context is a frozendict
      so we need to copy before mutating.
      
      this patch was made by searching for key addition to context and calls to the
      update() method on the 8.0 addons, and checking if a copy was made before in
      the method.
      65c685ad
  2. Oct 11, 2016
  3. Oct 10, 2016
    • Nicolas Martinelli's avatar
      [FIX] account: check exchange rate · a1d6c2d6
      Nicolas Martinelli authored
      An issue occurs in the following situation:
      - Define a currency exchange rate at day 1 and day 2
      - Create an invoice at day 1, and calculate the taxes. Do not set an
        invoice date!
      - Validate the invoice at day 2
      
      The exchange rate for taxes is the rate at day 1, while the exchange
      rate for other amounts is the rate at day 2.
      
      There is actually no way to know what was the rate applied for the
      taxes at invoice validation. There are two solutions:
      - recompute the taxes at validation
      - force the user to set an invoice date and recompute manually the
        taxes
      
      The first solution might have unexpected effects, therefore the second
      solution is applied.
      
      Fixes #13473
      opw-688517
      a1d6c2d6
  4. Oct 09, 2016
  5. Oct 06, 2016
    • Denis Ledoux's avatar
      [FIX] purchase_double_validation: from waiting approval to cancel · 964f42fb
      Denis Ledoux authored
      It wasn't possible to cancel a purchase order
      within the state "Waiting for Approval", despite
      the fact the button cancel is displayed.
      
      Clicking on the cancel button just did nothing.
      
      This is now properly working, thanks to the additional
      path added by this revision, between the workflow states
      `CheckForApproval` and `cancel`.
      
      opw-688685
      964f42fb
  6. Oct 03, 2016
  7. Oct 02, 2016
  8. Sep 30, 2016
    • Manuel Vázquez Acosta's avatar
      [FIX] mail: Ignore case for mail aliases. · 5b692df1
      Manuel Vázquez Acosta authored
      When receiving new mail (not replies) to an alias we should not take
      case into account.
      
      This also homogenize the treatment of local parts.  For instance, lines
      967 and 980 convert the local part to lower case to avoid
      case-sensitivity issues.
      
      Also `mail_alias` normalizes alias names by lowering case and finding,
      if necessary to make it unique, a suffix to alias name provided.
      
      From RFC 5321, section 2.4:
      
        > Exploiting the case sensitivity of mailbox local-parts impedes
        > interoperability and is discouraged
      
      Closes #334
      Closes #13037
      5b692df1
  9. Sep 28, 2016
    • Jairo Llopis's avatar
      [FIX] tools: cherry-pick of 0529a7f9 · 295f6274
      Jairo Llopis authored
      extract terms in correct folder
      
      If two addons path have a common part in the folder name (e.g. `/home/alice/dev`
      and `/home/alice/devodoo`), the `get_module_from_path` method may match the
      wrong folder.
      A file `/home/alice/devodoo/bob/models.py` would wrongly match `/home/alice/dev`
      path (due to the lack of separator) and the returned module would be `odoo`
      (`"odoo/bob/models.py".split('/')[0]`).
      
      In such scenario, the translations of files (code, static folder, report) would
      not be included in the exported translation file.
      
      Force the module path to ends with a folder separator to avoid wrong matching.
      
      Closes #13363
      Unverified
      295f6274
    • Alexandre Fayolle's avatar
      [FIX] auth_ldap: fix authentication with special chars · 94f58d64
      Alexandre Fayolle authored
      encode the filter in utf-8
      
      This prevents a UnicodeDecode error in python-ldap when the
      filter contains non ascii characters.
      
      opw-682783
      closes #10899
      closes #12710
      94f58d64
  10. Sep 26, 2016
  11. Sep 25, 2016
  12. Sep 22, 2016
  13. Sep 21, 2016
    • Jeremy Kersten's avatar
      [FIX] web_graph: force row title to be string · 8b41933a
      Jeremy Kersten authored
      If the title was 'true' or 'false', the export failed, because we are tying to
      concat bool and str. TypeError: cannot concatenate 'str' and 'bool' objects
      
      How to reproduce, install sale, be sure to have one order with delivered = True
      In Reporting / Sale analysis, add a group by 'shipped' filed to have 'true' as
      title.
      
      This commit closes issue odoo/odoo#13425
      8b41933a
  14. Sep 18, 2016
  15. Sep 15, 2016
  16. Sep 14, 2016
    • Nicolas Lempereur's avatar
      [FIX] web: keep m2m value in list view · 1da79b87
      Nicolas Lempereur authored
      Currently, when rendering a list view cell with a many2many we would
      empty the list of ids, and fill it again once a name_get is resolved.
      
      But in some instance, the code could use the data when it has been
      emptied out.
      
      For example, if we set the tax_id field (inside the order_line list view
      inside the sale.order form view) as requred, if we modify the order line
      and save directly (without clicking outside of the list view) we can get
      an incorrect error saying that the "Order Line" is not valid.
      
      It has been reproduced when saving with CTRL + SHIFT + S on google
      chrome and firefox, and there have been reports that for some
      configuration it also happen when clicking on the "Save" button.
      
      This commit change the behaviour so the value is kept whilst the name_get
      is ongoing, and just use a default "false" value for the name during this
      interval.
      
      closes #13478
      opw-668067
      1da79b87
  17. Sep 12, 2016
    • Damien Bouvy's avatar
      [FIX] payment_paypal: accept GET requests for Paypal DPN · 3e4206b4
      Damien Bouvy authored
      Although we have been reluctant to perform this change, a specific
      use case can cause customers to be redirect to the Odoo DPN url
      with a GET request.
      
      This happens when a Paypal Merchant account has the feature Guest
      Checkout active; in that case, a customer can pay without having
      a Paypal account (using only his credit card) and will *not* be
      subjected to auto-return; as detailed here:
      https://www.sandbox.paypal.com/be/cgi-bin/webscr?cmd=p/pop/help-account-optional
      Request coming from that payment flow will always trigger a GET
      request, causing the customer to be welcomed by a
      405 - Method Not allowed
      error on the Odoo server. The payment is normally correctly processed
      through IPN, so this does not normally causes loss of data; however
      this is not a nice way to welcome back your customer right after
      they pay you.
      Unverified
      3e4206b4
  18. Sep 11, 2016
  19. Sep 09, 2016
  20. Sep 06, 2016
  21. Sep 04, 2016
  22. Sep 02, 2016
    • Quentin De Paoli's avatar
      [FIX] anglo_saxon_dropshipping: fix dropship use case in anglo saxon with perpetual valuation. · 7bdd4de8
      Quentin De Paoli authored
      OPW: 684742
      When using dropship+anglo-saxon+perpetual valuation, there is no journal move for the delivery to debit outgoing inventory (since the goods don't transit by an internal stock) but the sale does credit it so there was a build up in the holding account that has to be moved out manually. This was also reported in #12687.
      
      The solution implemented is to check if the invoice line is related to sale order lines having one of its procurement_ids with a purchase_line_id set. If yes, it means that it is a confirmed dropship and in that case we don't call to super (we don't create the cost of sale line).
      
      That means that:
      * If the procurement is in exception at the customer invoice time, the behavior will be as it is currently, but it's fine as you don't know how the procurement will be solved, and it'll be only at the beginning (once the config is done it shouldn't go in exception anymore). People will have to manually fix those amounts with a miscellaneous operation.
      * users in anglo saxon mode should not use the 'stock interim account (received)' on supplier invoices for dropshipped goods, but rather use the COGS directly (sounds to me logical, and that's actually why I wouldn't go for the solution to create the stock move entries every time even for the dropshipped goods. That, and the fact that it would pollute the accounting with useless moves)
      7bdd4de8
    • Nicolas Martinelli's avatar
      [FIX] web: statusbar click · 2559b2cf
      Nicolas Martinelli authored
      When the statusbar is clicked, a `debounce` function prevents a
      doucle-click, and therefore making several `write` calls. On some status
      bars, clicking doesn't work anymore.
      
      The reason is because, in some mysterious cases, the event is propagated
      to the parent. The `currentTarget` is not the `li` element, but the
      parent `ul`. By setting the `immediate` argument to `true` (execute the
      first function instead of the last), this solves the issue.
      2559b2cf
  23. Aug 31, 2016
  24. Aug 29, 2016
    • Jeremy Kersten's avatar
      [FIX] website_sale: fix order for the pager in front end. · f1a33185
      Jeremy Kersten authored
      In psql, use LIMIT and OFFSET together without a fully specified and uniq sort order
      will generate unexpected behaviour.
      
      Eg:
      > id   id_dept  name
      > -------------------
      > 1    1        Tom
      > 2    1        Mike
      > 3    2        Meggie
      > 4    2        Marge
      > 5    3        Bart
      > 6    3        Lisa
      
      > using LIMITed selects like:
      
      >    SELECT * FROM employee ORDER BY id_dept LIMIT 3
      >    SELECT * FROM employee ORDER BY id_dept LIMIT 3 OFFSET 3
      >    SELECT * FROM employee ORDER BY id_dept LIMIT 3 OFFSET 6
      
      > You can have some result missings from the 3 requests, and others duplicated.
      > Because id_dept is not a uniq order.
      
      opw-686639
      
      note: backport of saas-12 4dce8616
      f1a33185
    • Odoo Translation Bot's avatar
  25. Aug 26, 2016
    • Simon Lejeune's avatar
      [FIX] packaging: Backport of efe37469 to 8.0 · c55c15ae
      Simon Lejeune authored
      [FIX] packaging: debian: update-python-module is no more
      
      The command is not installed by default. It was available in the
      python-support library which we didn't depend on and now this library
      is no more in debian jessie/ubuntu xenial.
      
      Backported for #13302
      Unverified
      c55c15ae
  26. Aug 25, 2016
Loading