Skip to content
Snippets Groups Projects
  1. May 28, 2018
    • Lucas Perais (lpe)'s avatar
      [FIX] web: o2m editable decorated, paged, ordered supports adds/cancel · 9654f6b8
      Lucas Perais (lpe) authored
      Have a decorated (conditional on field) editable list, with a default_order.
      Have more records than the pager allows.
      
      Add an item, and hit escape to cancel addition.
      
      Before this commit, the cancellation crashed because the first record on the other page
      was trying to evaluate the decoration for its row, which is wrong in the first place.
      This was because creating a record increases the limit of the list, to be able to see the record
      being created.
      At cancellation though, that limit was never decreased.
      
      After this commit, the whole flow happens without problem.
      
      OPW 1844495
      closes #24682
      9654f6b8
  2. May 27, 2018
  3. May 25, 2018
    • Fabien Meghazi's avatar
      [ADD] http: addons preload opt-out for socket activation · 32c5392f
      Fabien Meghazi authored
      Before this patch, early connections made to a 11.0 Odoo server running
      on Python 3 and deployed in threaded mode with socket activation would
      generate invalid registries.
      With this patch it is now possible, when this deployment mode is used,
      to opt-out addons preload by setting the following environment variable:
      
          ODOO_PRELOAD_ADDONS=no
      
      Note: this environment variable is only available for v11.0 as later
      versions does not preload anymore (cf: 1a39c9b)
      32c5392f
    • Nicolas Lempereur's avatar
      [FIX] mass_mailing: have unsubscribe in multi lang · 7703f1fb
      Nicolas Lempereur authored
      
      Since 953a693d link not corresponding to a route are tought multilang
      by default.
      
      Thus in the mass mailing, if the user had not the default language on
      the website he would have eg. a /fr_FR/unsubscribe_from_list placeholder
      in the mail.
      
      The placeholder is replaced by a real link pinpointing to the items the
      given person would unsubscribe to, but having /fr_FR/ part prevented
      that.
      
      With this PR, there is a dummy route so /unsubscribe_from_list is
      explicitely known as not multilang.
      
      opw-1850696
      closes #24912
      
      Co-authored-by: default avatarGoffin Simon <sig@odoo.com>
      7703f1fb
    • Christophe Simonis's avatar
    • Aaron Bohy's avatar
      [FIX] web: AbstractController: handle concurrent reloads · 04ba755f
      Aaron Bohy authored
      Let's assume the following scenario in a Kanban view with a default
      filter. The user removes the filter, and quickly adds a new one.
      On a slow network, it may happen that the second reload request
      returns before the first one. On odoo.com, this is easy to
      reproduce on the tasks Kanban view.
      
      Before this rev., when this occured, the result of the first
      request was displayed, whereas it should have been ignored as
      another request was done later on.
      04ba755f
  4. May 24, 2018
  5. May 23, 2018
    • Joren Van Onder's avatar
      [FIX] website_quote: install sale_payment together with website_quote · 1e21b79f
      Joren Van Onder authored
      Stable version of 65a589eb (reverted
      at 935043dd) that doesn't cause the
      following errors:
      
      odoo.modules.graph: module website_quote: Unmet dependencies:
      sale_payment
      
      on existing databases without sale_payment installed.
      
      A downside of this approach is that sale_payment can be uninstalled
      without uninstalling website_quote. Adding an uninstall_hook to
      sale_payment could solve that, but it wouldn't warn the user
      website_quote would be removed in the "Uninstall module"
      wizard. Keeping in mind this is only a temporary hack for stable,
      keeping sale_payment installed in this case shouldn't cause any
      issues.
      
      opw-1847977
      Unverified
      1e21b79f
    • Christophe Simonis's avatar
    • Christophe Simonis's avatar
    • Christophe Simonis's avatar
      373a1c11
    • Christophe Simonis's avatar
      a496ca98
    • Josse Colpaert's avatar
      [FIX] stock: non-tracked products should follow FIFO removal strategy too · 73e213ed
      Josse Colpaert authored
      In a warehouse, mostly most of the products have no lot.  It stays
      important however to avoid having old products in stock, certainly if
      we can distinguish them by pack or location.
      
      As now in stable, in_date is not put on the quant for non-tracked products,
      databases might mix quants with and without in_date of the same product.
      We should see those without in_date as the oldest.
      That is why we need to be able to apply a ASC NULLS FIRST (ASC is NULLS LAST by default) for FIFO
      or DESC NULLS LAST for LIFO.
      
      In order to do that, we copied the code from the _search method of the ORM to
      be able to do this special order as the orm itself is restrictive as it
      supposes the order itself has no spaces.
      
      We adapted and added some tests to check it handles the combination of NULL/NOT NULL well.
      73e213ed
    • Nicolas Martinelli's avatar
      [FIX] website_sale_delivery: selection at first rendering · 0b412c25
      Nicolas Martinelli authored
      - Set up as a shipper with the highest priority one which will cause an
        error (e.g. service not available for customer address)
      - Order a product on eCommerce, go to Payment page
      
      Nothing prevents to click on the 'Pay Now' button, although the shipper
      rating has failed. Moreover, the error is not displayed in the label of
      the shipper.
      
      Workaround: we force a click on the selected default shipper to trigger
      the update. The good solution would probably be to have a completely
      dynamic shipper rating integration, instead of some processing at first
      rendering and then extra dynamic processing.
      
      opw-1838765
      0b412c25
    • Goffin Simon's avatar
      [FIX] account: wrong view from button "Invoices" from a payment · 04dd6880
      Goffin Simon authored
      When clicking on button "Invoices" from a payment, it always displayed
      the supplier views instead of taking care about the type of the payment.
      
      opw:1849397
      04dd6880
    • Alexandre Kühn's avatar
      [FIX] web: fetch name_gets on list only when there are data · 8dbebe02
      Alexandre Kühn authored
      Revision on 8ac98fc7
      
      The commit above does not check whether the list contains data or not.
      As a result, if there are no data in the list, there is no reason to
      fetch `name_gets`.
      
      In fact, there could be a traceback because some parameters of this rpc
      are computed from data in the list. So if there are no data in the list,
      the parameters of the rpc are undefined. This was the case for a test
      in enterprise.
      8dbebe02
    • Alexandre Kühn's avatar
      [FIX] web: only abandon records on discard without savepoint · c587c002
      Alexandre Kühn authored
      Revision on https://github.com/odoo/odoo/commit/9a9da5763b38a3bce54e9c4870af0032e38cacc3
      
      The intent of the commit above was to no abandon records in a list on
      discard when these records have been created from a `default_get`.
      
      Also, the `default_get` may also perform an `onchange` that could create
      more records. The records from this `onchange` must also be included in
      the "do not abandon" rule.
      
      However, the rule was too relax, so that any following `onchanges` from
      a user interaction was also considered into this rule. As a result, it was
      not possible to abandon such records.
      
      With this fix, we restrict the rule to `default_get` (already the case) and
      only the `onchange` right after a `default_get`. Any record created on an
      `onchange` from a user interaction after a `default_get` should be abandoned
      on discard.
      
      Representation of creating the list:
      
                                     savepoint
         |----------|-----------|-------||--------|------->
       create   default_get  onchange1         onchange2
      
      record in the list is created by default_get or onchange1
          => do not abandon on discard
      record in the list is created by onchange2
          => abandon on discard
      c587c002
    • len-odoo's avatar
      [FIX] mrp: display info text on work orders view · cf82fbab
      len-odoo authored
      When seeing the work orders view, it would display an empty black box instead of
      the proper message.
      
      The message is not displaed since creation is deactivated.
      
      opw 1839001
      cf82fbab
    • Lucas Perais (lpe)'s avatar
      [FIX] stock: portal user can see pack operations · d1df278f
      Lucas Perais (lpe) authored
      Have a delivery for a client, with pack operations
      Connect on this user's portal
      
      Go on the SO and then on the delivery
      
      Before this commit, there was an Error 500
      because access to fields of pack_operations was forbidden
      
      After this commit, the report downloads well
      
      OPW 1849520
      closes #24849
      d1df278f
    • David's avatar
      [FIX] website_form: set meta field · 682ae1cc
      David authored
      Before this commit, if you enable website_form_enable_metadata, that
      will crash with a "KeyError: 'meta'"
      
      This commit closes #24848
      682ae1cc
    • len-odoo's avatar
      [FIX] calendar: do not create events past year 9999 · 4e57f94f
      len-odoo authored
      datetime.date(year, month, day).weekday() is undefined for year > 9999.
      When creating a google recurrent event that is set to never end,
      Odoo tries to create 100 individual events.
      If in Google calendar the user sets a recurring event to a custom interval,
      of say a century, then the hundredth event is in 100x100 years, so in 12018.
      Since 12018 > 9999, we get a traceback and synchronisation never happens.
      
      opw 1845311
      4e57f94f
    • Adrian Torres's avatar
      Revert "[FIX] iap: missing dependency (#24816)" · 79343891
      Adrian Torres authored
      No dependency change in stable version, instead, an installation hook
      should be used.
      
      This reverts commit e9e0f4af.
      Unverified
      79343891
    • Ivan Yelizariev's avatar
      [FIX] iap: missing dependency (#24816) · e9e0f4af
      Ivan Yelizariev authored
      The module uses web_settings_dashboard, but doesn't have it in dependency
      e9e0f4af
    • Josse Colpaert's avatar
      [FIX] mrp: workorders of sub-BOMs · 8b328665
      Josse Colpaert authored
      - Create the following BOM structure:
      ```
        Prod 1 (Manufacture)
        |
        --- Prod 2
            Prod 3 (Kit)
            |
            --- Prod 4
      	  Prod 5
      ```
      - The BOMs for Prod 1 & 3 must have a routing set, to generate
        workorders
      - Create a MO for 1 unit of Prod 1, process to the end
      
      2 units of Prod 1 are created instead of 1.
      
      This is due to the WO for Prod 3 not having a `next_work_order_id`.
      Therefore, `record_production` adds an extra quantity at:
      ```
      production_move.quantity_done += self.qty_producing
      ```
      
      To prevent this, we set as `next_work_order_id` of the BOM of Prod 3 the
      WO of Prod 1.
      
      Closes #19527
      opw-1817398
      8b328665
  6. May 22, 2018
Loading