Skip to content
Snippets Groups Projects
  1. Aug 20, 2019
    • Martin Trigaux's avatar
      [FIX] website: proper override of auth_signup · cb9f48d3
      Martin Trigaux authored
      
      Add the dependency between website and auth_signup
      auth_signup is in auto-install but there was no strict dependency
      
      Before this commit, installing website transformed the field
      auth_signup_uninvited from a static list to a computed list (due to related).
      
      This fixes the following bug:
      1. install auth_signup
      -> create selections
      2. install webstite
      -> remove selections and turn it into a computed field with dynamic
      selections
      3. update auth_signup
      -> recreate the selections when updating the module
      4. reload the registry (automatic at the end of update)
      -> try to remove the selections -> error as not in install/uninstall
      mode
      
      Instead, keep the list static and turn it into a computed field with
      inverse
      
      closes odoo/odoo#35861
      
      Signed-off-by: default avatarMartin Trigaux (mat) <mat@odoo.com>
      cb9f48d3
    • Arnaud Baes's avatar
      [FIX] stock: write sequence on picking type · 8adf660e
      Arnaud Baes authored
      
      Fix an issue when the sequence was updated even if the form was discarded
      
      closes odoo/odoo#35844
      
      Signed-off-by: default avatarSimon Lejeune (sle) <sle@openerp.com>
      8adf660e
  2. Aug 19, 2019
  3. Aug 20, 2019
  4. Aug 15, 2019
  5. Jul 31, 2019
  6. Aug 20, 2019
    • Sébastien Theys's avatar
      [IMP] website, website_slides: adapt get_base_url for website · 1e8b9254
      Sébastien Theys authored
      
      On `website` itself there is no `website_id` so the domain has to be read from
      `self` directly.
      
      Also oversight of forward-port 039e6bae:
      On `slide.channel` the override that was done in an earlier version is doing the
      same as the generic override that was introduced later, so it can be removed.
      
      closes odoo/odoo#35839
      
      Signed-off-by: default avatarJérémy Kersten (jke) <jke@openerp.com>
      1e8b9254
    • Xavier Morel's avatar
      [FIX] web: race condition in export widget · df1aa689
      Xavier Morel authored
      
      Looks like the race condition dates back to the introduction of
      "default" export fields but #32339 made it into a hard error (rather
      than silently not load the fields): setting / selecting the default
      fields waited in the exports lists to be loaded, but the exports lists
      are just predefined lists of fields, that's got no relation to the
      loaded or default fields, which are loaded (somewhat implicitly) by
      _onChangeCompatibleInput.
      
      As a result, if the lookup of the ir.exports was faster than fetching
      the model's fields, we'd execute "select the fields from the list
      view" before the exports widget knew anyting about the model's own
      fields.
      
      Because the old code iterated on the loaded fields and checked if they
      were to enable it would just do nothing (as it would iterate an empty
      object). Because the new code iterates on the defaults fields and
      looks them up in the records map, it gets an undefined and triggers an
      error.
      
      Fix by waiting on the proper RPC call.
      
      Reported-by: default avatarAndreas Stauder <(@BT-astauder)>
      
      closes odoo/odoo#35837
      
      Signed-off-by: default avatarXavier Morel (xmo) <xmo@odoo.com>
      df1aa689
    • Raphael Collet's avatar
      [IMP] models: ORM speedup · 9920f20e
      Raphael Collet authored
      
      This branch is the combination of several optimizations in the ORM:
      
      * store field values once in the cache: the cache reflects more
      faithfully the database, only fields that explicitly depend on the
      context have an extra indirection in the cache;
      
      * delay recomputations by default: use method `recompute` to explicitly
      flush out pending recomputations;
      
      * delay updates in method `write`: updates are stored in a data
      structure that can be flushed efficiently to the database with method
      `flush` (which also flush out recomputations);
      
      * make method `modified` take advantage of inverse fields to inverse
      dependencies;
      
      * filter records by evaluating a domain on records in Python;
      
      * a computed field with `readonly=False` behaves like a normal field
      with an onchange method;
      
      * computed fields are computed in superuser mode by default.
      
      Work done by Toufik Ben Jaa, Raphael Collet, Denis Ledoux and Fabien
      Pinckaers.
      
      closes odoo/odoo#35659
      
      Signed-off-by: default avatarDenis Ledoux <beledouxdenis@users.noreply.github.com>
      9920f20e
    • Hiral Bhavsar's avatar
      [IMP] crm: merge lead and opportunity form views · e28d3aa4
      Hiral Bhavsar authored
      
      When redirecting to record, the wrong 'form' view will sometimes be
      loaded. A 'hack' has been done in crm_lead.py to redirect the user to
      the relevant 'form' view based on the value of the 'type' field.
      
      However, this solution is not clean as it does not solve all cases
      (multiple crm.lead records loaded from the activity systray, reloading
      the page after redirection, etc.)
      
      Goal of this task is to solve those issues by merging crm.lead.form.lead and
      crm.lead.form.opportunity form views. That way multi records actions will
      always display the relevant information according to record type.
      
      Note: Here some fields used as a twice in the form view just to stay
      close to the current design. Still few things which are not possible
      here for 'lead' and 'opportunity', that is
      * A dynamic 'placeholder' on 'name' field.
      * A 'domain' on 'partner_id' field.
      * A 'widget' on 'probability' field.
      
      Task 47118
      
      closes odoo/odoo#32248
      
      Signed-off-by: default avatarThibault Delavallee (tde) <tde@openerp.com>
      e28d3aa4
    • Hiral Bhavsar's avatar
      [IMP] crm: remove useless code · b63b8e0a
      Hiral Bhavsar authored
      In this commit useless code is removed
      
        * method 'close_dialog' and 'edit_dialog' from crm.lead;
        * actions: crm_opportunity_report_action_graph,
          create_opportunity_simplified, crm_lead_action_activities,
      	merge_opportunity_act;
        * views: view_create_opportunity_simplified
      
      Indeed this is dead code and can be removed to clean a bit module.
      
      Task 47118
      b63b8e0a
    • David Tran's avatar
      [IMP] sale: split _create_analytic_account() for potential inheritance · df0cff26
      David Tran authored
      
      analytic account data preparation is now a hook method
      This allows to modify the creation method without unecessary updates
      
      closes odoo/odoo#35784
      
      Signed-off-by: default avatarMartin Trigaux (mat) <mat@odoo.com>
      df0cff26
    • Hetashree Chauhan's avatar
      [IMP] stock,_account: usability changes · c0d3d7e9
      Hetashree Chauhan authored
      This commit will change,
      1. tooltip for stock valuation accounts and costing method
      2. change message when on-hand quantity is zero.
      task-2006467
      
      closes : https://github.com/odoo/odoo/pull/35328
      
      
      
      closes odoo/odoo#35328
      
      Signed-off-by: default avatarSimon Lejeune (sle) <sle@openerp.com>
      c0d3d7e9
  7. Aug 19, 2019
    • wan's avatar
      [IMP] account: improve the onboarding · 646a6d5a
      wan authored
      Task 2031686
      
      * Change some labels on the dashboard
      * Move the action "Take money In/Out" to a button because it was also dsplayed in bank journals and we only want it for cash journals
      * Add reconciliation param on the first account config list, as well as a button to go to the form (it is an editable list)
      * Add an action to duplicate accounts in batch
      * Smartly detect when a cash statement can be extended instead of creating a new one, making it more obvious to the user that he may need to validate his previous statement(s)
      
      closes odoo/odoo#35137
      
      Signed-off-by: default avatarQuentin De Paoli (qdp) <qdp@openerp.com>
      646a6d5a
  8. Aug 20, 2019
    • Jérome Maes's avatar
      [FIX] mail: context to read follower · dc066a54
      Jérome Maes authored
      
      As the 'allow_company_ids' are now depending on the context and impact
      the access to some data, we need to provide them to some controller
      (route). Json route auto-pop the context when given (0322f772)
      into the environment. In the 'read_follower' route, the `check_access_rules`
      raises when you want to access a document of another company (even if
      this one is in your allowed companies), simply because their is no
      'allowed_company_ids' key in the context. This context key is automatically
      added when calling the `rpc` method in JS (as explain in session.js
      of a5b6f31c).
      To fix the wrong access error on followers reading, we simly have to give
      an empty context when calling `_rpc` in the follower widget.
      
      Task-1999686
      
      closes odoo/odoo#35705
      
      Signed-off-by: default avatarJérome Maes (jem) <jem@openerp.com>
      dc066a54
    • Jérome Maes's avatar
      [IMP] project: default company of a task · 07017465
      Jérome Maes authored
      In Project app, we now ensure the company of a task. Indeed, since that
      model can be billed, and is important in some business, we need to ensure
      its link to a company. A task can live outside a project, so the company
      of a task can not be related. Task company can be set by user, but must
      be the same as the project (is set), as the task can be billed depending
      on project properties.
      
      Task-1999686
      07017465
    • Jérome Maes's avatar
      [FIX] sale_timesheet: created task company depends on its project · 0c5d6498
      Jérome Maes authored
      When confirming a SO creates a task, the task is in a project. The
      company from the task should be the one from the project, as we will
      (in the next commit) restrict the company of a task: it will only
      allow a project to have task in the same company as its.
      
      Task-1999686
      0c5d6498
    • Nicolas Martinelli's avatar
      [FIX] account: UK country code · ddeb7b7c
      Nicolas Martinelli authored
      
      The country code for United Kingdom is GB.
      
      closes odoo/odoo#35828
      
      Signed-off-by: default avatarNicolas Martinelli (nim) <nim@odoo.com>
      ddeb7b7c
  9. Aug 19, 2019
    • Gert Pellin's avatar
      [IMP] point_of_sale, pos_restaurant: multi-user sessions · 118190f3
      Gert Pellin authored
      PURPOSE
      =========
      
      When working in restaurants, there are often more than one waiter working on the same floor.
      But Odoo doesn't support that.
      There can only be one person working on one session for the same tables.
      The purpose of this task to allow several people to work on the same session (while synchronizing sessions).
      
      SPECIFICATIONS
      ===============
      
      - MultiUser is active by default.
      - Allow multiple waiters to login on one PoS Config.
      - All waiters login to the same session, the waiter that does the paiment is the responsible for the order (same as now).
      - Synchonize the information across the multiple instances running on same floors.
      
      Syncronisation will take place at switch of table.
      (close tables automaticly if no activity, to force sync)
      
      - Syncro cannot be done if no internet : how do we manage it ? We should inform the user the internet connection has been lost and that synchro cannot be done. What impacts? To check
      
      Useful links
      https://github.com/it-projects-llc/pos-addons
      https://apps.odoo.com/apps/modules/11.0/pos_bus_restaurant/
      
      
      
      Related PR:
      odoo#32789
      
      TASK-ID: 1891130
      
      closes odoo/odoo#33973
      
      Signed-off-by: default avatarpimodoo <pimodoo@users.noreply.github.com>
      118190f3
    • Pierre Masereel's avatar
      [FIX] pos_restaurant: fix typo · 761f695e
      Pierre Masereel authored
      
      Forgot to remove unused function with pudb in a248171d
      
      closes odoo/odoo#35822
      
      Signed-off-by: default avatarpimodoo <pimodoo@users.noreply.github.com>
      761f695e
    • Martin Trigaux's avatar
      [IMP] base: better default for translation overwrite · 30ab0d3a
      Martin Trigaux authored
      
      Following 43d355c2, same logic
      Hide it but make it checked by default
      
      closes odoo/odoo#35819
      
      Signed-off-by: default avatarMartin Trigaux (mat) <mat@odoo.com>
      30ab0d3a
    • Robot Odoo's avatar
      [REF] mrp_subcontracting · 9b6a5398
      Robot Odoo authored
      
      This PR modify the subcontracting process:
      - Support extra quantity
      - No more interaction with the subcontract order. Everything is directly accessible on the operation
      - Dropshipping with subcontracting
      - Bypass reservation process
      - The partner type subcontractor is removed and every partner could be set on the BoM
      - Update the initial demand will trigger the rules.
      - Demo data
      - Subcontractor is required on the BoM (no more BoM for all partners of type subcontractor)
      
      Visually:
      - Remove the stat button for linked operation.
      - Register Components is only available when a subcontract component is tracked
      - Button on operations in order to correct tracked components
      - Show Operation button will ask to record components until the total quantity is provided(correction on finish lines are allowed after)
      
      More details available in each commit
      Task 1962187
      
      closes odoo/odoo#35738
      
      Signed-off-by: default avatarSimon Lejeune (sle) <sle@openerp.com>
      9b6a5398
    • Arnold Moyaux's avatar
      [IMP] mrp_subcontracting: bypass reservation · 40552938
      Arnold Moyaux authored
      A subcontracting receipt should be always assigned even
      if the components are not deliver. It depends on the partner
      and not our company.
      40552938
    • Arnold Moyaux's avatar
      [IMP] stock: Hook for reservation · c7b94736
      Arnold Moyaux authored
      Before a modification on a quant, stock.move and stock.move.line
      check if they satisfy the condition to impact the quant.
      However in a new module this condition is hardly editable.
      The purpose of this commit is to easily add condition to bypass
      reservation.
      c7b94736
    • Arnold Moyaux's avatar
      [FIX] mrp_subcontracting: return to subcontract location · 673450df
      Arnold Moyaux authored
      Before this commit a returns would select the classic
      partner location. However in the case of a subcontractor return
      it should be returned to the subcontract location
      673450df
    • Arnold Moyaux's avatar
    • Arnold Moyaux's avatar
      [IMP] mrp_subcontracting: subcontractors required · a684372f
      Arnold Moyaux authored
      Force to set subcontractors on subcontracting BoM since
      it's always used in order to determine if a subcontracting
      should be apply on a receipt or not. Also there is not magical
      behavior has 'if there is no subcontractor on the BoM then it
      means all partners...'
      a684372f
    • Arnold Moyaux's avatar
      [IMP] mrp_subcontracting: cancel subcontract order · 39c5d0f3
      Arnold Moyaux authored
      A subcontracting receipt cancellation should cancel the
      subcontracting order and the associated document.
      39c5d0f3
    • Arnold Moyaux's avatar
      [FIX] mrp: do not base default on active_id · a3aeabd4
      Arnold Moyaux authored
      Since active_id do not works allows to use a context
      key that will not be modified magically.
      a3aeabd4
    • Arnold Moyaux's avatar
      [REF] mrp_subcontracting: views v2 · d1513739
      Arnold Moyaux authored
      - nothing tracked
      The "register components" button shouldn't appear, the receipt looks normal.
      - finished products tracked
      The "register components" button shouldn't appear, the receipt looks normal.
      - finished products not tracked and components tracked
      The "register components" button appear and the burger redirects to "Produce" wizard. The user should record all the productions before being able to access SML through the burger wizard. He cannot force any SML for subcontracted products with tracked components before having recorded all the productions.
      - finished products and components tracked
      The "register components" button appear and the burger redirects to "Produce" wizard. The user should record all the productions before being able to access SML through the burger wizard. He cannot force any SML for subcontracted products with tracked components before having recorded all the productions.
      d1513739
    • Arnold Moyaux's avatar
      [IMP] mrp: inline view to ir.ui.view · 18612003
      Arnold Moyaux authored
      The purpose is to use it independently from the MO
      18612003
    • Arnold Moyaux's avatar
      [IMP] mrp_subcontracting: modify initial demand · 5b526e87
      Arnold Moyaux authored
      Modify the initial demand for a subcontract receipt should
      update the delivery to the subcontractor.
      5b526e87
    • Arnold Moyaux's avatar
      [IMP] stock: allow to skip next activity · a6fb45d4
      Arnold Moyaux authored
      Due to subcontracting. Some documents are hidden from the users since
      they are there to ease technicaly. However they are linked to useful
      documents and we want to avoid to log an activity on them.
      a6fb45d4
    • Arnold Moyaux's avatar
      [FIX] mrp_subcontracting: access error on purchase · cbea919a
      Arnold Moyaux authored
      Usecase to reproduce:
      - Validate a purchase_order with a user without group inventory user or manufacturing user.
      
      AccessError on operation read for BoM. A purchase user should be able
      to validate an order independently of the documents that will be
      generate.
      cbea919a
    • Arnold Moyaux's avatar
      [ADD] mrp_subcontracting_dropshipping · 4c4a7879
      Arnold Moyaux authored
      Allow to do a dropshipping with a subcontractor.
      If the product uses the routes drophipping and have
      a subcontracting BoM. Then a PO for the subcontractors
      will genrate a need in the subcontracting location for
      the required components.
      
      A module is needed because the picking_type_id for dropshipping
      do not have a warehouse_id. The warehouse_id is needed in order to
      know from which warehouse the subcontracting components should be
      deliver.
      4c4a7879
    • Arnold Moyaux's avatar
      [IMP] mrp_subcontracting: common test file · a29d3e52
      Arnold Moyaux authored
      The purpose is to use the global setup in external
      module.
      a29d3e52
    • Arnold Moyaux's avatar
      [ADD] mrp_subcontracting: MTO, not ressuply flow · d8495215
      Arnold Moyaux authored
      Test the flow when a product is set as resupply on subcontract
      and not MTO, it should be trigger by a reordering rule and not
      by a subcontracting receipt.
      d8495215
Loading