Skip to content
Snippets Groups Projects
  1. Sep 22, 2016
  2. 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
  3. Sep 18, 2016
  4. Sep 15, 2016
  5. 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
    • Christophe Simonis's avatar
      a5dfe2f0
  6. 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
  7. Sep 11, 2016
  8. Sep 09, 2016
  9. Sep 08, 2016
    • Denis Ledoux's avatar
      [FIX] mrp: fix BOM smart button for product variants · c86201c5
      Denis Ledoux authored
      Do not force the `search_default_product_id`to the current
      product variant, as the domain just above could return a
      result, a BOM which is set for the template of this variant
      but not for the variant istself, that this filter would hide.
      
      This is following the revision
      52cba1b5
      
      Because of this, a product variant smart button could
      display "1 BOM", but when clicking on it, none would be displayed,
      which is not user-friendly at all.
      
      opw-687912
      c86201c5
  10. Sep 06, 2016
  11. Sep 04, 2016
  12. Sep 02, 2016
    • Christophe Simonis's avatar
      ddcd2e77
    • 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
    • Christophe Simonis's avatar
      490f9cfe
    • 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
  13. Sep 01, 2016
  14. Aug 31, 2016
  15. 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
  16. 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
  17. Aug 25, 2016
  18. Aug 24, 2016
    • Jeremy Kersten's avatar
      [FIX] account: remove domain that depend of purchase when not installed · abf17354
      Jeremy Kersten authored
      This commit avoid a traceback when you open the form account_invoice_line
      for a supllier invoice without having installed purchase.
      
      The fields_view_get method add domain with purchase_ok field, but this
      field can be missing on the model product.template because purchase is not
      a dependence of account.
      
      Invalid field 'purchase_ok' in leaf "<osv.ExtendedLeaf:
      ('purchase_ok', '=', True) on product_product
      
      In stable version, the best fix found, is to remove the domain.
      But we need to fix it in master, moving this field from purchase module to
      product module. (odoo/odoo##13271)
      
      This commit closes #13268 and closes #315 for stable version.
      Todo: merge odoo/odoo#13271 in master (@qdp-odoo agreement)
      abf17354
  19. Aug 23, 2016
  20. Aug 22, 2016
    • Jairo Llopis's avatar
      [FIX] website_blog: fix pager when existing filters in url · 4faed0b7
      Jairo Llopis authored
      Go to a URL such as `/blog/our-news-1?date_begin=2015-01-01`
      Now click on page 2.
      
      Without this patch you will go to `/blog/our-news-1?date_begin=2015-01-01/page/2`.
      With this patch, you will go to `/blog/our-news-1/page/2?date_begin=2015-01-01`.
      
      This commit closes #13206
      4faed0b7
  21. Aug 21, 2016
  22. Aug 19, 2016
  23. Aug 18, 2016
    • Goffin Simon's avatar
      [FIX] stock: action_done with moves with pack operation · acc7449d
      Goffin Simon authored
      When action_done is called with moves without pack operation, the pickings
      linked to the computed moves must be in state done and the date_done must
      be set on each picking.
      
      opw:685145
      acc7449d
    • Damien Bouvy's avatar
      [FIX] stock,purchase: stop shortening picking type names · 08ed48e2
      Damien Bouvy authored
      In purchase, a special key is set in the context to simplify the name
      of the picking type: instead of the normal name, only the name of the
      warehouse is used. This is problematic if more than one incoming picking
      type exists for a given warehouse, as it prevents you from being able to
      differentiate them. Asking users to modify the view to remove the
      context key seem a bit too much to ask for something that should be
      simple.
      
      It is my understanding that this was implemented only for cosmetic
      reasons, but I am willing to assume that having to select
      "YourCompany: Delivery Orders" instead of simply
      "YourCompany" for people who only have one picking type should not
      be too disruptive or obscure.
      
      opw-685751
      Unverified
      08ed48e2
Loading