Skip to content
Snippets Groups Projects
  1. Sep 02, 2022
    • Demesmaeker's avatar
      [FIX] sale: fallback on the product uom · 83d6ba6e
      Demesmaeker authored
      
      `_compute_discount` from subscription add dependencies that triggered
      the compute before the `product_uom` was set in a sale order line, once
      the uom is set, the `_compute_discount` will be triggered again.
      
      task-2960768
      
      closes odoo/odoo#98780
      
      Related: odoo/enterprise#30721
      Signed-off-by: default avatarVictor Feyens (vfe) <vfe@odoo.com>
      83d6ba6e
    • Odoo's Mergebot's avatar
      [IMP] mrp: back to basics 7 ngtr · 216b1a1b
      Odoo's Mergebot authored
      
      A series of 7 UI improvement or small implementations:
      - Match qty field in Work Order kanban view
      - UI Fix for Lot/serial column in MO form view
      - Improve UI of Manual Consumption field on BOM
      - UI Improvement of review split window field rearrangement
      - Add setup/cleanup time to Work Center Specific Capacities
      - Allow multi-edit Work Order
      
      task-2925474
      
      --
      I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
      
      closes odoo/odoo#97588
      
      Signed-off-by: default avatarTiffany Chang <tic@odoo.com>
      216b1a1b
    • Triet Ngo's avatar
      [IMP] mrp: Allow multi-edit workorder · aa14a802
      Triet Ngo authored
      This commit allows users to multi/mass-edit the Work Order fields
      
      task-2925474
      
      Part-of: odoo/odoo#97588
      aa14a802
    • Triet Ngo's avatar
      [IMP] mrp: Add setup/cleanup time to Workcenter Specific Capacities · d494f598
      Triet Ngo authored
      - Add columns setup/cleanup time to Specific Capatities of Workcenter
      - Take into account these 2 fields when compute expected duration in WO
      
      task-2925474
      
      Part-of: odoo/odoo#97588
      d494f598
    • Triet Ngo's avatar
      [IMP] mrp: UI Improvement of review split window field rearrangement · a968404a
      Triet Ngo authored
      rearrange the fields of the tree from
      Quantity, User, Date to Date, User, Quantity
      
      task_id: 2925474
      
      Part-of: odoo/odoo#97588
      a968404a
    • Triet Ngo's avatar
      [IMP] mrp: Improve UI of Manual Consumption field on BOM · e2d4586a
      Triet Ngo authored
      Auto mark manual_consumption as True as soon as the component is use in
      an operation and make the field read-only
      
      task-2925474
      
      Part-of: odoo/odoo#97588
      e2d4586a
    • Triet Ngo's avatar
      [IMP] mrp: UI Fix for Lot/serial column in MO form view · dae82e7a
      Triet Ngo authored
      The commit modifies the following:
      - add tooltip : 'Displays the consumed Lot/Serial Numbers consumed.'
      - Change the column Serial Numbers to also display lot number
      - Make the column read-only
      
      task-2925474
      
      Part-of: odoo/odoo#97588
      dae82e7a
    • Triet Ngo's avatar
      [IMP] mrp: Match qty field in Work Order kanban view · 562e01eb
      Triet Ngo authored
      Currently, the kanban view show qty_production. However, the Listview
      show the qty_remaining. So this commit match the field for 2 views (both
      should show qty_remaining)
      
      task-2925474
      
      Part-of: odoo/odoo#97588
      562e01eb
    • Triet Ngo's avatar
      [IMP] mrp: Add color to warning multi-lot mass produce · 5f684778
      Triet Ngo authored
      When mass producing from several reserved lots, we display a warning
      message that the lot reservation is 'multiple'. This commit displays
      that message in red.
      
      task-2925474
      
      Part-of: odoo/odoo#97588
      5f684778
    • Nikunj Ladava's avatar
      [IMP] portal_rating, {website_slides}_forum: miscellaneous improvements · 7c66ed56
      Nikunj Ladava authored
      
      PURPOSE
      
      Added some light wording improvements to various screens.
      
      SPECIFICATION
      
      - There was a strange flicker of 'Add a review' and 'Write a review
        while loading the course player because we have different strings
        in XML and js due to which while loading it appears like flickering
        of 'Add a review' and 'Modify Your Review' so, we modified the string
        to 'Review' and also moved the share button from left to right to improve
        the view.
      - This commit will rename the 'New Post' button to 'Forum' and add
        tooltip 'Forum' to it.
      
      task- 2627364
      
      closes odoo/odoo#78139
      
      Related: odoo/upgrade#3444
      Signed-off-by: default avatarWarnon Aurélien (awa) <awa@odoo.com>
      7c66ed56
    • Nikunj Ladava's avatar
      [IMP] website_slides: improve design of share pages in elearning · 7d66dcbf
      Nikunj Ladava authored
      PURPOSE
      
      Globally improve user experience when sharing content from the e-learning app.
      This includes new sharing options, allowing to share to multiple emails, split
      the sharing template from slide and course, etc...
      
      SPECIFICATION
      
      - added two more options 'WhatsApp' and 'Pinterest' to share snippets.
      - we converted the email type of input for 'Share by email' to text,
        so we can enter multiple emails to share and also add validation
        in the controller for this input text input.
      - We improved the design of the share modal by making sure that the full URL
        for sharing link is visible and rearranged the UI fields of
        'Share' nav-tab on non-fullscreen content (if there is
        embed code, the sharing by email and embed code will still be on
        the single row).
      - Improved the share modal of the channel by adding an email field in it to
        Share the channel through the mail.
      - New field name 'share_channel_template_id' for channel template
        in 'slide.channel' modal and also added _send_share_email method
        in it and email template for sharing channel through the mail, and
        also added the dedicated route for sharing the channel.
      - New share button is added on the top of documents in the slide (which
        previously was in the tab below the content).
      
      task- 2627364
      
      Part-of: odoo/odoo#78139
      7d66dcbf
    • Thanh Dodeur's avatar
      [FIX] mail: prevents traceback when downloading rtc call logs · 291e99b7
      Thanh Dodeur authored
      Before this commit, since https://github.com/odoo/odoo/pull/96954
      
       ,
      the `CallActionListView` had its `channel` field renamed into `thread`.
      
      A reference to this old field name was missed which caused the
      traceback, this commit fixes this issue by using the right field name.
      
      closes odoo/odoo#99447
      
      Signed-off-by: default avatarSébastien Theys (seb) <seb@odoo.com>
      291e99b7
    • Lucas Perais's avatar
      [FIX] web: legacy Many2ManyTags in list view: add/remove records · f4a8d5bf
      Lucas Perais authored
      
      Have a list view (new RelationalModel inside a WOWL list view) that has
      a legacy many2many tags field in its arch.
      
      Try to add or remove some records from that many2many.
      
      Before this commit, there was a crash, because the legacy field wrapper
      doesn't use the model the right way for that case.
      
      After this commit, there is no crash.
      
      closes odoo/odoo#99431
      
      Signed-off-by: default avatarGeoris François (fge) <fge@odoo.com>
      f4a8d5bf
    • Nshimiyimana Séna's avatar
      [FIX] account_edi_ubl_cii, l10n_account_edi_ubl_cii_tests: add partner email to factur_x XML · 128b961b
      Nshimiyimana Séna authored
      Currently, when creating an invoice from factur_x XML, the vendor is
      identified by sequentially checking (in this order) the following
      information :
      - VAT number
      - name
      - email
      However, when we generate a factur_x XML, we do not include
      the partner's email address. This means that in some cases, two odoo
      databases are not able to communicate bills/invoices properly.
      This commit brings back a behavior that was unintentionally removed in
      https://github.com/odoo/odoo/commit/d25fdafcd8eb4cfef14c2ef7ed8ad7785dd01b55
      
      
      
      opw-2909408
      
      closes odoo/odoo#99263
      
      X-original-commit: 51609d19
      Signed-off-by: default avatarWilliam André (wan) <wan@odoo.com>
      Signed-off-by: default avatarNshimiyimana Serge Séna (sesn) <sesn@odoo.com>
      128b961b
    • Jorge Pinna Puissant's avatar
      [FIX] web: SettingsFormView - highlight Element with inner html/fields · 285d1c4a
      Jorge Pinna Puissant authored
      
      Before this commit, if a text-muted on a setting (a description of the
      setting) contains fields or HTML tags, the highlight generated wrong
      texts.
      
      Now, we highlight the text in an iterative way, taking care to only
      highlight the text.
      
      closes odoo/odoo#99205
      
      Signed-off-by: default avatarAaron Bohy (aab) <aab@odoo.com>
      285d1c4a
    • Jorge Pinna Puissant's avatar
      [FIX] web: SettingsFormView - avoid searching hidden settings · 129bb263
      Jorge Pinna Puissant authored
      When searching a text of a hidden field (for example, "gate"), the
      setting itself will not be shown, but the group title, and the app
      Search Header will be shown.
      
      This issue arise because, we use an if condition with all the label of
      all the fields (hidden or not) in the group (or app) to decide if the
      group title or the app header will be showed.
      
      Now, we modify this to hide (d-none) the group title or the app header
      if there is not a settings below them.
      
      Part-of: odoo/odoo#99205
      129bb263
    • Aurélien (avd)'s avatar
      [IMP] pos: add some indexes to speedup Tax Report · d6dc795e
      Aurélien (avd) authored
      
      Tax Report _compute_from_amls_grids query calls
      two method overriden in pos_account_reports.
      
      The resulting query contains EXISTS subqueries on pos_session.move_id
      and pos_order.account_move. Because these two fields are not indexed
      by default, the cost of the query can skyrocket in large databases,
      leading to slow tax report generation.
      
      closes odoo/odoo#99095
      
      X-original-commit: 51e607a7
      Signed-off-by: default avatarRémy Voet <ryv@odoo.com>
      d6dc795e
  2. Sep 01, 2022
    • Luis Torres's avatar
      [IMP] l10n_pe: Welcome ISC tax · 82ae8ba0
      Luis Torres authored
      The SUNAT requires than when a ISC tax is used on an invoice, on the
      cbc:TierRange attribute be specified the ISC type.
      
      Now, the 3 options could be:
      - 01: System to value
      - 02: Application of the Fixed Amount
      - 03: Retail Price System
      
      To this, now was added a new field on the tax, with this options.
      
      Extra, the ISC could have many amount options, for now, only is
      generated a tax for 0%, that allows to the customer only change the
      percentage that will to use.
      
      Legal Reference:
      
      Catalog No 08:  https://cpe.sunat.gob.pe/node/88
      
      Guide for the Preparation of Electronic Invoice:
      https://cpe.sunat.gob.pe/sites/default/files/inline-files/gu%C3%ADa%20xml%20para%20factura%20electr%C3%B3nica%20%28ubl%202.0%29_0.pdf
      
      Impuesto selectivo al consumo (ISC):
      https://www.gob.pe/7918-impuesto-selectivo-al-consumo-isc
      
      
      
      closes odoo/odoo#98962
      
      Signed-off-by: default avatarJosse Colpaert <jco@odoo.com>
      82ae8ba0
    • David Beguin's avatar
      [FIX] rating: check model is set before using it as key. · 08561e6d
      David Beguin authored
      
      As message_notify can be done on empty recordset, message.res_id and
      message.model can be Falsy. During filtering on rating messages, we need to
      prevent the code to crash when filtering on those two properties and filter out
      messages that are not linked to any record.
      
      Task-2850521 (Knowledge: trash management)
      
      closes odoo/odoo#99436
      
      Related: odoo/enterprise#30908
      Signed-off-by: default avatarThibault Delavallee (tde) <tde@openerp.com>
      08561e6d
    • Sébastien Theys's avatar
      [IMP] mail, im_livechat, *: add typing for guest and clean code · 545d2388
      Sébastien Theys authored
      
      * = bus, hr_holidays, test_discuss_full
      
      - use channel member instead of partner for typing
      - use channel member instead of partner for all other return values from server
      - remove temporary partner hack in livechat and keep public partner
      - remove some obsolete convert data
      - adapt format methods accordingly
      
      task-2664853
      
      closes odoo/odoo#98923
      
      Related: odoo/enterprise#30760
      Signed-off-by: default avatarAlexandre Kühn (aku) <aku@odoo.com>
      545d2388
    • William Henrotin's avatar
      [FIX] mrp: remove onchange in import hack · 2c91ad3a
      William Henrotin authored
      
      Since d7716f07, the onchanges to create the stock moves in mrp
      production have been replaced by computes.
      There was a hack in the create() method to trigger the onchanges in case
      of import. As those onchanges do not exist anymore, this hack needs to
      be remove too.
      The compute methods are called naturally in imports.
      
      closes odoo/odoo#99434
      
      Signed-off-by: default avatarArnold Moyaux (arm) <arm@odoo.com>
      2c91ad3a
    • Didier (did)'s avatar
      [FIX] mail: avoid partner without name in suggestion · f922f519
      Didier (did) authored
      
      task-2884212
      
      closes odoo/odoo#99432
      
      X-original-commit: 385ba1d062abbc73ca3cdb9a27adb4b9cc7803be
      Signed-off-by: default avatarSébastien Theys (seb) <seb@odoo.com>
      f922f519
    • Florian Damhaut's avatar
      [FIX] calendar: Single char partner create traceback on calendar · 90bf897c
      Florian Damhaut authored
      
      Since 3a145216a6a6fb259c5afef6175669c8a65ba39f, partner_info value
      was changed but not it's usage.
      
      This commit also simplify the code for better understanding.
      
      closes odoo/odoo#99422
      
      Task-id: 2954397
      X-original-commit: a667179e43ab00a8384b75923769782f1836c24b
      Signed-off-by: default avatarArnaud Joset <arj@odoo.com>
      Signed-off-by: default avatarDamhaut Florian (flda) <flda@odoo.com>
      90bf897c
    • Xavier BOL (xbo)'s avatar
      [REM] hr_timesheet: remove QRCode for app store and play store · 2179620a
      Xavier BOL (xbo) authored
      
      This commit removes all custom codes to create a QRCode to get the link
      of the Timesheet Awesome App, the link is sufficient to download the
      mobile app.
      
      closes odoo/odoo#99362
      
      Related: odoo/upgrade#3840
      Related: odoo/enterprise#30942
      Signed-off-by: default avatarLaurent Stukkens (ltu) <ltu@odoo.com>
      2179620a
    • Stanislas Sobieski's avatar
      [FIX] base_import: fix test · d76e28b4
      Stanislas Sobieski authored
      
      Test was skipped on runbot since odf was missing
      
      closes odoo/odoo#99282
      
      Signed-off-by: default avatarChristophe Monniez (moc) <moc@odoo.com>
      d76e28b4
    • Simon Genin (ges)'s avatar
      [IMP] web: char field, password shouldn't be trimmed · 001ec7b2
      Simon Genin (ges) authored
      
      Passwords are something we know that need to be exact. So, the regular
      behaviour where double spaces are changed to single ones and spaces are
      trimmed at the start and end of the string is not a good idea for
      passwords in general. Not that a lot of people put double spaces or
      space in the start or end of their passwords, but it happened and it is
      possible. So, if password is True on the field, it does not make any
      sense to trim it and let's support that in the web client (as the web
      client does the trimming) instead of putting trim=False on all password
       fields in the Python, risking to forget it for the next password field.
      
      closes odoo/odoo#99088
      
      Signed-off-by: default avatarJosse Colpaert <jco@odoo.com>
      001ec7b2
    • nni-odoo's avatar
      [IMP] l10n_sg: Box Mapping for Purchase Tax · 8b28981f
      nni-odoo authored
      
      As required from Singaporean tax regulation, we need to add Box 5 on the tax grid to make sure the amount is correctly mapped in both boxes.
      
      2854163
      
      closes odoo/odoo#98950
      
      X-original-commit: be3c30f3
      Signed-off-by: default avatarOlivier Colson (oco) <oco@odoo.com>
      8b28981f
    • Julien Mougenot's avatar
      [FIX] web: Fix binary and pdf_viewer fields · d3eff197
      Julien Mougenot authored
      
      This commit aims to make the BinaryField and PdfViewerField appear and
      act the same way as they did before their conversion to Owl.
      
      This mostly consists of a few tweaks in the conditional rendering of
      certain elements in the template, and the rest of the changes are meant
      to clean and simplify the values the component is working with.
      
      closes odoo/odoo#98251
      
      Signed-off-by: default avatarBruno Boi (boi) <boi@odoo.com>
      d3eff197
    • Julien Mougenot's avatar
      [IMP] web: editInput handles file inputs · cabed350
      Julien Mougenot authored
      This commit allows the 'editInput' test handler to also handle inputs of
      type "file". To do so, the value must be a file (or a list of files),
      and the handler has been refactored to accept a certain given value only
      if its type allows it.
      
      Part-of: odoo/odoo#98251
      cabed350
    • Victor Feyens's avatar
      [FIX] sale: linting · 638ca2cd
      Victor Feyens authored
      
      * Local variable 'filtered_self' might be referenced before assignment
      * confusing use of `order` from the previous for loop, support multi recordset
      in the method instead.
      
      closes odoo/odoo#98068
      
      Related: odoo/enterprise#30408
      Signed-off-by: default avatarVictor Feyens (vfe) <vfe@odoo.com>
      638ca2cd
    • Victor Feyens's avatar
      [PRF] sale*: avoid useless recomputations · db9dd568
      Victor Feyens authored
      By default, a related field dependencies are the fields in his relation 
      path.
      But for non stored UX fields, the field should only be recomputed when
      values editable in the view are edited.
      
      Example:
      
      On a given model:
      
      product_id = fields.Many2one(...)
      product_type = fields.Char(related='product_id.type')
      
      The product_id field can be changed on the model view, but not the 
      linked product,
      so there is no need for the orm to search records whose product_type 
      must be invalidated
      when the type is changed on a product.
      
      To avoid this 'useless' search, we can manually restrict the field 
      dependencies:
      
      product_type = fields.Char(related='product_id.type', 
      depends=['product_id'])
      
      Part-of: odoo/odoo#98068
      db9dd568
    • Victor Feyens's avatar
      [IMP] sale: privatize portal methods not meant to be called in rpc · bf55e1a7
      Victor Feyens authored
      Part-of: odoo/odoo#98068
      bf55e1a7
    • Victor Feyens's avatar
      [CLN] sale: rename _get_invoice_status · 4e2d7b80
      Victor Feyens authored
      follow code guidelines and use _compute_invoice_status instead.
      
      Part-of: odoo/odoo#98068
      4e2d7b80
    • Victor Feyens's avatar
      [IMP] sale: rename update_prices method · cc5c83e0
      Victor Feyens authored
      make it clear it's expected to be called from the frontend, prefixing it
      with action_.
      Also split the method to have an utilitary method recomputing all prices,
      to use in the tests and/or business code (without the chatter message).
      
      Part-of: odoo/odoo#98068
      cc5c83e0
    • Victor Feyens's avatar
      [IMP] sale: do not store anymore whether the pricelist changed · 7ca41635
      Victor Feyens authored
      As for the show_update_fpos field, do not store this simple UX field.  
      We only want to highlight the pricelist modification when it happens (to 
      allow prices recomputation), no need to store it undefinitely.
      
      Part-of: odoo/odoo#98068
      7ca41635
    • Victor Feyens's avatar
      [CLN] sale: cleanups · 920c1fa0
      Victor Feyens authored
      Methods ordering (after place of call)
      String guidelines (double quoted strings for strings shown to the user)
      
      Part-of: odoo/odoo#98068
      920c1fa0
    • Nicolas (vin)'s avatar
      [IMP] POS: allows to create invoices for a pos order at a later time · 14183883
      Nicolas (vin) authored
      
      Some countries, such as mexico, require a seller to be able to
      generate an invoices from a ticket from a sale done in a shop,
      even after a few days.
      This change aims to allow this, by allowing to create an invoice at
      a later date after the POS has been closed.
      This is done by partially reversing the POS closing entry, and then
      generating an invoice the same way it would be done at the POS closing.
      
      The customer can scan a QR code if enabled in order to request the
      invoice by himself, requiring him to fill a for to give the customer
      information required for the invoicing.
      
      Allows to fill additional fields dependent on the localization that are
      required to be set on the partner or the invoice.
      
      Task id #2946604
      
      closes odoo/odoo#97675
      
      Related: odoo/enterprise#30599
      Signed-off-by: default avatarLaurent Smet <las@odoo.com>
      14183883
    • Antoine Vandevenne (anv)'s avatar
      [FIX] payment_mercado_pago: remove dead code · 7049e4b1
      Antoine Vandevenne (anv) authored
      
      This commit removes code related to refunds that are not implemented in
      this version of the Mercado Pago integration.
      
      closes odoo/odoo#92848
      
      Related: odoo/documentation#2149
      Signed-off-by: default avatarAntoine Vandevenne (anv) <anv@odoo.com>
      7049e4b1
    • Mahendra Barad's avatar
      [ADD] payment_razorpay: integrate with Razorpay's "Payment" gateway · 08a7c779
      Mahendra Barad authored
      
      See README for technical details.
      
      task-2800823
      
      Part-of: odoo/odoo#92848
      Co-authored-by: default avatarAntoine Vandevenne (anv) <anv@odoo.com>
      Co-authored-by: default avatarPrakash Prajapati <ppr@odoo.com>
      08a7c779
    • Ricardo Gomes Rodrigues (rigr)'s avatar
      [IMP] account: improve payment terms with new options · 3dad4b3c
      Ricardo Gomes Rodrigues (rigr) authored
      
      This commit improves the payment terms by:
      - making the payment terms lines editable on-the-fly removing the need for a wizard
      - refactoring the computation of the payment terms
      - adds new demo data for payment terms
      - adds an example section where the user can test and preview the due dates and amount of its payment terms
      - adding a new parameter "Display terms on invoice". If set, the payment deadlines and respective due amounts will be detailed on invoices.
      
      The refactoring:
      - adds a field `months` (months to add after the invoice date)
      - adds a field `end_month` (if True, switch to the end of the month after having added months or days)
      - renames the field `day_of_the_month` to `days_after` (days to be added after the switching to the end of the month, only if `end_month` is set)
      - deletes the field `option` which was too restrictive
      - modifies the calculation of the payments terms taking into account these new fields
      
      This allows some combinations that were not possible before.
      Example: The French payment terms "30 jours fin de mois le 10" (which should be interpreted as "10 days after the end of the next month") can be encoded as
      ```
      months = 1
      days = 0
      end_month = True
      days_after = 10
      ```
      
      Task id 2852814
      
      closes odoo/odoo#91490
      
      Related: odoo/enterprise#29661
      Related: odoo/upgrade#3524
      Signed-off-by: default avatarLaurent Smet <las@odoo.com>
      3dad4b3c
Loading