Skip to content
Snippets Groups Projects
  1. Jun 08, 2018
    • Christophe Monniez's avatar
      [REM] packaging: add sassc to the rpm package · 0e117c87
      Christophe Monniez authored
      Since 565277ac sassc is used as a
      fallback when the python libsass is not available.
      
      With this commit, the saasc package dependency is added to the rpm to
      avoid installing libsass with pip.
      Unverified
      0e117c87
    • oco-odoo's avatar
      [IMP] web: add decoration support for all field widgets (#25160) · d4cd8be4
      oco-odoo authored
      It is sometimes useful to be able to customize slightly the color of a
      field depending on some conditions on the current record.
      
      For example, imagine that we want to show a VAT number field in green/red, if
      it was validated or not. To do that, we can add a field with an
      onchange, and simply add decoration-danger and decoration-success
      attributes to the field.
      Unverified
      d4cd8be4
    • RomainLibert's avatar
      [IMP] hr_holidays: add accrual allocations support · a8aa0bcb
      RomainLibert authored
      We need to support the accrual allocation for leaves.
      This is a special type of allocations that adds some leaves to the
      allocation every X amount of time worked.
      
      This kind of allocations is especially used in the USA, but can also be
      used in other contexts (eg: In Belgium we could allocate 20 days per
      year for the legal leaves)
      
      These allocations have support for :
      
        * A max balance limit
        * Going in negative (if set on the leave_type and the manager
          approves the leaves)
        * Removing the unpaid leaves from the quantity of days worked
        * Setting an end date after which it does not allocate anymore leaves
      
      Task #46114
      Closes #22292
      a8aa0bcb
    • Laurent Smet's avatar
      [IMP] account: compute tax lines on account.move with onchange · d49ab568
      Laurent Smet authored
      Allow the user to set taxes on lines when creating manually a new journal entry, and tax lines will be computed automatically thanks to an onchange based on new unstored fields.
      
      Was task: 33802
      Was PR #19029
      d49ab568
    • Unknown's avatar
      [IMP] hr_recruitment: Add kanban state to recruitment pipeline · d6d30819
      Unknown authored
      Purpose
      =======
      
      recruitment pipeline lacks a kanban state management
      
      Specifications
      ==============
      
      Add kanban state management like what is already done in project:
      - A kanban state was added to the recruitment pipeline. The kanban colored dot is visible in the kanban view and in the form view.
      - A search filter on the kanban state was also added.
      
      Project : RD for Dummies
      Task : 1819164
      d6d30819
    • Géry Debongnie's avatar
      [FIX] web: properly apply css in mobile search view · e1f2e723
      Géry Debongnie authored
      In some cases, if the device is in a mobile mode, but does not support
      touch, then the mobile search view is shown, as expected, but the css
      is not applied.
      
      Maybe this is not a big deal in practice, since the only way to notice this is
      to use odoo on a browser with a small window and no touch emulation, but
      this is still annoying and wrong: if the mobile search view is
      displayed, the its css should be applied in all cases.
      
      We simply fix the css rule to make it work like expected.
      e1f2e723
  2. Jun 07, 2018
    • David Monjoie's avatar
    • Jérome Maes's avatar
      [IMP] purchase: add access rule for report · 3c1e886c
      Jérome Maes authored
      When splitting purchase and stock, the access
      rules of the report was wrongly moved to
      purchase_stock. This commit restores balance
      in the universe.
      3c1e886c
    • RomainLibert's avatar
      [IMP] hr_expense: add draft state to expense lines · 631891dc
      RomainLibert authored
      We need to be able to change an expense report before submitting it, for
      this we add a new step in the expense process (a draft state for the
      expense sheets).
      
      Task #41700
      Closes #24806
      631891dc
    • Siddarth Gajjar's avatar
      [IMP] calendar: generic improvements · d4b5668c
      Siddarth Gajjar authored
      We need the calendar view to be consistent in the usage of time formats
      currently the week column (on the far left) is not consistent with what
      we display in the meeting time slot.
      
      It is currently not ovious that you can add and view anybody's calendar
      so we need to add a placeholder.
      
      In order to create a meeting you have to click on create which is not
      explicit.
      
      Task #1838108
      Closes #24675
      d4b5668c
    • XavierDo's avatar
      [ADD] mail, mass_mailing: allow a user to resend a mail with failures · 133eeb1b
      XavierDo authored
      When a mail is in failure state, a red envelope appears next to the message
      in a thread but it was difficult to send the mail again.
      This tasks will allow users to send mail again easily, or mark notification
      as cancelled if the user want to ignore this failure.
      A notification will appear in sender systray while mail are in failure.
      
      Task: #46158
      PR: #24628
      133eeb1b
    • XavierDo's avatar
      [IMP] mail: add warning when partner email is invalid · aa7fb8c7
      XavierDo authored
      Change the border and cross color of a recipient in mail
      composer when the email is invalid (doesn't contains "@").
      
      When sending an mail from composer the user will see easily
      if an email adress is invalid and will cause a NO_VALID_RECIPIENT
      error.
      
      Task: #46158
      PR: #24628
      aa7fb8c7
    • Christophe Simonis's avatar
    • Quentin De Paoli's avatar
      [FIX] purchase: fix test_main_flows. · e0591346
      Quentin De Paoli authored
      The action 'account.action_invoice_tree2' has been changed into a server action, so we want to open the tree view of invoices by modifying some values of the action we should now use the xml reference 'account.action_vendor_bill_template'.
      
      Additionally, the help tooltip is now completed as would do the server action
      e0591346
    • Olivier Colson's avatar
      [FIX] purchase: anglosaxon reconciliation test: delete rate demo data at the beginning of the test · c00df146
      Olivier Colson authored
      This is mandatory to ensure consistency of the data generated by the test, as stock pickings created from a PO are by design always created for the current date (there is no way forcing this), meaning that they always use today's exchange rate for their valuation.
      
      Despite the fact we rewrite the date of the valuation moves artificially, we cannot correct the debit and credit values, since the anglosaxon entries get automatically reconciled (and you cannot modify a reconciled entry).
      
      So, we have to make sure that "today"'s rate will always be the last rate we created in order to ensure complete control of the test. This deletion will have no further impact, as the transaction will be rollbacked at the end of the test.
      c00df146
  3. Jun 06, 2018
    • RomainLibert's avatar
      [IMP] hr_holidays: add support for taking leaves differently · b6642f1b
      RomainLibert authored
      We would like to be able to take leaves using different units, currently
      we only support taking leaves by hours because we use datetimes.
      
      We need to support three units:
       * days
       * half days
       * hours
      
      These units will be defined on the leave type.
      
      Task #40995
      Closes #21760
      b6642f1b
    • Prakash Prajapati's avatar
      [FIX] mail: Add a mark as read option on systray message dropdown · d8a5141c
      Prakash Prajapati authored
      - 'Mark as Read' icon will be shown by moving the mouse over the message
      - if it's channel then make it Mark as Seen
      - if it's inbox notification then mark all message as read
      - The inbox notification icon has been changed
      
      This commit is related to task ID 1817646
      d8a5141c
    • Mathieu Duckaerts-Antoine's avatar
      [IMP] mail: add moderation on channels · 299ebb2c
      Mathieu Duckaerts-Antoine authored
      Purpose of this commit is to allow moderation on incoming messages in
      discussion channels. On some channels on which moderation is required
      messages should be in a pending moderation stage. Moderators can accept
      or refuse messages as well as always allow or ban messages coming from
      a given set of emails.
      
      Channels now have an option to be moderated. Moderators can be added on
      channels. They have access to a specific UI in Discuss to see and take
      action on messages waiting for moderation.
      
      Concerning mail.thread message that are pending moderation are not notified.
      It means nobody receives a notification about them. Moderation process calls
      the notification once the message is validated.
      
      Various features included in this commit :
      
       * a model is added to store the decision about emails, allow or ban;
       * access rights are updated so that only moderators can modify moderation
         fields on message;
       * specific bus notifications are send to moderated people as well as to
         moderators on incoming emails as well as when a decision is taken;
       * options are added on channels to send explanations to moderated emails;
       * options are added on channels to write and send guidelines explaining
         why and how moderation is performed;
       * a reminder is send daily to moderators with remaining messages to moderate;
       * discuss UI is adapted and a new channel is added below Inbox and Starred
         giving access to moderation tools;
       * chanenl UI is adapted allowing to moderate directly inside channels;
      
      This commit is linked to task ID 29521. Closes #21921.
      299ebb2c
    • Thibault Delavallée's avatar
      eeab18d7
    • Thibault Delavallée's avatar
      [IMP] test_mail: add an assert method for bus notifications content · de0b9e51
      Thibault Delavallée authored
      Purpose is to ease the check of bus notifications content by having a tool
      method doing it.
      de0b9e51
    • Christophe Simonis's avatar
    • Olivier Colson's avatar
    • Olivier Colson's avatar
      [FIX] purchase: anglosaxon reconciliation test: set proper date to generated credit note · 6e51f4cd
      Olivier Colson authored
      The date of the invoice is used for currency conversion in anglosaxon stock valuation. Before this fix, the tests crashed between june 6th and december 31th, as the demo data define a currency rate for USD on june 6th. Using the proper date on the credit note ensures the good rate (the one in application when making the credit) is always applied.
      6e51f4cd
    • Christophe Simonis's avatar
      c834c0f5
    • Sanjay Jamod's avatar
      [IMP] account: vendor bill creation upon email reception · a4df9f8c
      Sanjay Jamod authored
      This allows to set up a mail alias per purchase journal. Then share that email address to your supplier, or use it internally to forward the vendor bills received by mail, to automatically create an empty vendor bill with the mail attachments linked, and the partner might be filled if the source email matches a supplier.
      
      Thanks to the document preview on the side, it's now super easy and super fast to copy the vendor bill info from the received PDF into the account.invoice object. This would eventually be improved later on (IAP).
      
      Was task: 37703
      Was PR #22158
      a4df9f8c
    • Quentin De Paoli's avatar
      [REF] mail_thread: refactoring of _find_partner_from_emails() · a6b75d40
      Quentin De Paoli authored
      More flexibility equals more powa
      a6b75d40
    • Nicolas Martinelli's avatar
      [FIX] purchase: fix test · 10da9045
      Nicolas Martinelli authored
      The `price_unit` on `stock.move` is not rounded since it is a technical
      field. Therefore, the comparison might fail depending on the rate (which
      happens after June 6th because of `base.rateUSDbis`).
      10da9045
    • Christophe Simonis's avatar
      116826e0
    • Olivier Dony's avatar
      [IMP] mail: make connection test attempt to send an email · 86ae4b90
      Olivier Dony authored
      The "connection test" was only attempting to establish a socket to the
      SMTP server and then trying to login, if credentials were setup.
      
      This would catch basic config errors but would fail to detect setups
      were the SMTP server would forbid relaying, because of missing
      credentials, missing STARTSSL mode, and other more complex cases.
      
      By actually beginning a real SMTP session and sending the MAIL FROM,
      RCPT TO and DATA command, we make sure that the server appeas willing to
      relay emails from the current user address to an arbitrary recipient
      address. Then we close the session before actually sending the message.
      
      In order to have a valid destination domain with a valid MX server, we
      use an odoo.com recipient address for the test, which should prove that
      the destination server will relay to arbitrary addresses (unless the
      destination server is an Odoo.com MX server ;-))
      Unverified
      86ae4b90
    • Olivier Dony's avatar
      [FIX] mail: do not shadow SMTP errors with QUIT · c72eec46
      Olivier Dony authored
      When an exception is raised during the course of a normal SMTP session,
      smtplib's `sendmail()` method will close the channel automatically,
      so any further attempt at sending SMTP commands will fail with an
      SMTPServerDisconnected error, even just calling `smtp.quit()`.
      This will shadow the original error, and make debugging harder.
      
      Fixes #1493
      Unverified
      c72eec46
    • Olivier Dony's avatar
      [REM] core: remove deprecated v7 onchange support · 6ef27bb9
      Olivier Dony authored
      Old API support has been dropped as of 11.0, and the old onchange system
      should have been discontinued along with it.
      
      New-API onchange mechanism is automatically triggered without requiring
      any extra markup in view declarations, as explained in the
      documentation:
      - https://www.odoo.com/documentation/11.0/reference/orm.html#porting-from-the-old-api-to-the-new-api
      - https://www.odoo.com/documentation/11.0/reference/orm.html#onchange-updating-ui-on-the-fly
      Unverified
      6ef27bb9
    • Olivier Dony's avatar
      [REV] Revert "[IMP] base: select countries & state based on exact code,... · 025e65a9
      Olivier Dony authored
      [REV] Revert "[IMP] base: select countries & state based on exact code, (useful for import and fast many2one)"
      
      This reverts commit 09a22e3b, which was
      attempting to fix a use case that was actually working.
      It also created a duplicate `name_search()` method for res.country, in
      addition to the one aliased to `location_name_search()`
      
      It is indeed supported to import countries and states, using the
      country code or state code as the imported value. These will be matched
      uniquely thanks to the special `name_search()` performed for import,
      forcing the `operator`.
      And this is done while still allowing proper auto-completing for normal
      UI case, where the "exact code match" will only promote an extra result
      at the top of the suggested matches, rather than limit the results to a
      single match.
      
      The confusion apparently stemmed from the test case using "UK" as the
      country code. UK is not a valid ISO country code, so it would never
      match as expected.
      Unverified
      025e65a9
    • Martin Trigaux's avatar
      [FIX] calendar: shows the event failing · e7aff377
      Martin Trigaux authored
      When importing many events (e.g. google synchronisation), it is not clear which
      one is failing
      
      opw-1855471
      Unverified
      e7aff377
    • Lucas Perais (lpe)'s avatar
      [FIX] event: events smartbutton on partner display if user has rights · 087dd851
      Lucas Perais (lpe) authored
      With a user that do not have access to the the Event module,
      go into Contacts > open a contact
      
      Before this commit, a ACL error was thrown. This was because of the presence of
      a smartbutton refering to events in the form view of the partner
      
      After this commit, the form view of the partner displays correctly
      
      OPW 1851783
      closes #24931
      087dd851
    • Ivan Yelizariev's avatar
      [IMP] base: correct link in comment · f81963ad
      Ivan Yelizariev authored
      To describe the date format on babel website
      Closes #25085
      f81963ad
    • Goffin Simon's avatar
      [FIX] web: Button delete on SO line unclickable · 9d943f7a
      Goffin Simon authored
      Due to ef540eb0, the button delete was not clickable on a SO line
      on safari.
      
      opw:1848652
      9d943f7a
    • Tomas Alvarez's avatar
      [FIX] mail, web_editor: allow tel links in email templates (#24926) · 1db6c8d0
      Tomas Alvarez authored
      Before this commit: links like `tel:+1-555-555-5555` were being replaced
      both in javascript (prefixing `http`, when using the summernote editor)
      and python (prefixing the base url, when they were added directly to the template).
      
      After this commit: the replacement is avoided on links with `tel` scheme.
      
      This is related to commit: 8b97a399
      1db6c8d0
    • Lucas Perais (lpe)'s avatar
      [FIX] web: new record with o2m also with new records prevent name_get · 654ee788
      Lucas Perais (lpe) authored
      Have a model A with a o2m to B.
      In the list have a field  B > m2o > C
      
      Also have a sequence field (widget handle) on B
      
      Then Create a record A
      Arrange yourself to have two new records B popping right away (through an onchange)
      in the o2m. Their C field have to be empty
      
      Resequence those two B lines with the handle
      
      Before this commit, there was a traceback when trying to do an name_get on the C fields
      Since both of them are empty, there is no record in the localData
      Hence, no model. And also no records anyway, so no need to do a name_get
      
      After this commit, there is no traceback, the name_get is avoided when no data is to be fetched
      
      OPW 1853088
      closes #24988
      654ee788
  4. Jun 05, 2018
Loading