Skip to content
Snippets Groups Projects
  1. Nov 29, 2017
  2. Nov 28, 2017
    • Nicolas Martinelli's avatar
      [FIX] stock_account: lock date · e56d91be
      Nicolas Martinelli authored
      `_check_lock_date` is defined on a `account.move`, while in this case
      `move` refers to a `stock.move`.
      
      We re-implement a logic similar to the original commit 287b2932.
      
      Fixes #20913
      opw-782395
      e56d91be
    • flyingL123's avatar
      [FIX] stock: Add return value · c7cc59e2
      flyingL123 authored
      Fixes #21264
      Closes #21266
      Unverified
      c7cc59e2
    • flyingL123's avatar
      [CLA] signature for flyingL123 · c650e63d
      flyingL123 authored
      Done at #21266
      Unverified
      c650e63d
    • Rémi Rahir's avatar
      [FIX] website_portal: edition of company name · 7a379039
      Rémi Rahir authored
      Before this commit, one could edit its company_name (char) at will but this has no effect on the actual company name (commercial_partner_id that is a company). Thus once the partner is linked to a company, the information entered in the 'details' form will not be in synch with the details info on the '/my/home' page.
      
      After this commit, we only allow edition of the 'Company name' field while the partner doesn't have a company set and display the company name once it is set.
      7a379039
    • Florent de Labarre's avatar
      [FIX] account: Access error when opening a partner · f7e80442
      Florent de Labarre authored
      If a user can read account.analytic.account but not the account.move.line,
      when he tried to open a partner, there were an raise AccessDenied errors.
      
      opw:782961
      f7e80442
    • Nicolas Martinelli's avatar
      [FIX] mail: auto subscription to channel · 8b854ab6
      Nicolas Martinelli authored
      1. Install mail and hr
      2. Create a new mail channel:
         name: test
         Who can follow: selected group of user
         Authorized group: Employee / Manager
         Auto subscription: Employee / Manager
      3. Edit the Demo user, and add him to group Employee Managers
      
      The Demo user is not subscribed to the channel (only Admin is)
      
      In this case, the web client sends a `sel_groups_` key, which is not
      handled by the `write` method.
      
      Fixes #20897
      opw-782301
      8b854ab6
    • Christophe Simonis's avatar
      77545387
    • Christopher Tribbeck's avatar
      [FIX] stock: remaining quantity · f20df81c
      Christopher Tribbeck authored
      Uses `float_compare` instead of comparing floats directly, which is
      anyway something that leads to precision errors.
      
      See related PR for detailed use case.
      
      Closes #20770
      opw-781384
      f20df81c
    • Lucas Perais (lpe)'s avatar
      [FIX] web_editor: Fieldtexthtml wait until iframe loaded to save · cae18851
      Lucas Perais (lpe) authored
      In the case where:
      - you are too fast editing a mass mailing, trigger an onchange (e.g. mailing_model) and then save
      You can use the edit/save shortcuts to make yourself believe you are Flash Gordon
      - your internet is too slow (Down: 4Mb/s, Up: 500kb/s, Latency: 20ms)
      
      Before ths commit, the onchange will return *before* the field (the iframe actually) is loaded.
      This onchange contains a signal *in* the body_html of the field to allow communication with the iframe.
      If you save before the iframe is loaded, this body_html keyword will be saved.
      
      After this commit we deactivate the save button until the iframe is loaded.
      We could have gone for a python user error on the value of the body_html too.
      But it would have added a supplementary click and potential reloading
      
      OPW 783616
      cae18851
    • Christophe Simonis's avatar
      ad6f3079
    • qsm-odoo's avatar
      [FIX] website: do not duplicate iframes at each edition · 6d2bf44f
      qsm-odoo authored
      Before this commit, when an user added a video in his page (an <iframe>)
      it was then duplicated each time the snippet in which the video is was
      edited and saved again, slowing the page loading.
      
      As the code responsible for that is quite outdated, it should be
      reviewed and refactored in master. This commit only fixes the problem
      with more code. This commit will also fix existing databases, if the
      user edit the video snippet once again.
      
      (Note: even if this commit's code will be even more strange in master,
      it *should* be forward-ported anyway).
      6d2bf44f
    • Joren Van Onder's avatar
      [FIX] point_of_sale: search customers with full search query · 11ae103e
      Joren Van Onder authored
      When the keypress handler is called the pressed key has not yet been
      added to this.value. This leads to the search being performed with
      e.g. 'Agr' when the user has typed 'Agro'.
      
      Fix this by getting this value when the timeout expires. At this point
      the value will be updated.
      
      Fixes #21129
      Unverified
      11ae103e
  3. Nov 27, 2017
    • qsm-odoo's avatar
      [FIX] website: declare correct mimetype for data images · 948c429e
      qsm-odoo authored
      Some website default images were declared with the wrong filename
      extension, inducing the wrong mimetype. As a result, some browsers
      (IE...) were unable to render these images.
      948c429e
    • Martin Geubelle's avatar
      [FIX] web: display selection label in graph · ccaced92
      Martin Geubelle authored
      When grouping a graph view by a selection field, the labels on x axis
      were the selection values themselves (not their corresponding label).
      
      This issue was only present if there was only one group by field. In
      this case, the attribute `grouped_on` is not an array but a string
      containing the field name. This case wasn't correctly handled.
      
      Note that this issue is not present in the new views (from saas-16).
      
      Closes #19790
      ccaced92
    • Stefan Rijnhart's avatar
      [FIX] mail: handle emails with a single non-text part · 10de33a5
      Stefan Rijnhart authored
      SAP sends out emails that only contain a PDF part, without a text or
      html body, e.g.:
      
          X-Mailer: SAP NetWeaver 751
          Content-Type: application/pdf; name="Afleveringsbewijs.pdf"
          Content-Transfer-Encoding: base64
          Content-Description: Afleveringsbewijs
      
      Before this patch, this single non-text part was parsed into the body of
      the final message, as if it was a text part.
      
      Making the check for the main part more restrictive solves the issue, as
      there is no reason to assume that a non-multipart content is
      necessarily of text/* type. This PR also includes a regression test.
      
      Closes #21102
      Unverified
      10de33a5
    • Nicolas Martinelli's avatar
      [FIX] account_dashboard : rounding and zero negative · 3246f3b3
      Nicolas Martinelli authored
      During the generation of data in kanban view, there are no rounding when
      computing the amount differences. It might lead to amount differences
      lower than the currency rounding, which should not be displayed.
      
      It also fixes the case where the amount is -0.0.
      
      opw-782962
      Closes #17840
      3246f3b3
    • Nicolas Martinelli's avatar
      [FIX] hr_expense: report in various currencies · 3643cc0a
      Nicolas Martinelli authored
      Displays the total amount of an expense report as displayed in the PDF
      report.
      
      opw-783620
      3643cc0a
    • Florent de Labarre's avatar
    • Nicolas Martinelli's avatar
      [FIX] point_of_sale: Pin and Barcode · a1e30265
      Nicolas Martinelli authored
      - Create a user B with 'Barcode' and 'Security PIN' set.
      - Log in the POS with user A
      - Scan the barcode
      
      The PIN is not asked and the user is switched.
      
      Closes #20760
      opw-781360
      a1e30265
    • Damien Bouvy's avatar
      [FIX] payment_authorize: stop raising on 'Error' authorize status · fc69b37c
      Damien Bouvy authored
      It seems that Authorize.net returns an 'Error' status when transactions
      fail (note that it can also return an Ok status for different types of
      failures).
      
      This is incoherent with the way the authorize requests are processed,
      since an Error resultCode will raise a ValidationError, rollbacking the
      complete cursor transaction. This makes little sense, since there should
      be a trace of the payment.transaction in the backend (preferably with an
      explanation for the failure).
      
      This fix adds an error processing step to the response handling of
      authorize.net - while before, it was assumed that an 'error' result
      meant an issue in communication/setup, now it is treated as an error
      in payment and the payment.transaction status is set accordingly.
      
      I've added a test for this and it was green (pinky promise), but this
      particular test suite is deactivated (or perhaps only on nightly, I need
      to check the nightly process with @nim-odoo).
      fc69b37c
  4. Nov 26, 2017
  5. Nov 24, 2017
    • Nicolas Martinelli's avatar
      [FIX] event_sale: canceled registrations · ac31252d
      Nicolas Martinelli authored
      - Open an event registration page
      - Fill form for 2 persons => redirection to /shop/checkout
      - Click Review Order => open /shop/cart
      - Change quantity from 2 to 1
      - Finish checkout process
      - Login to backend, confirm SO
      
      Two tickets are confirmed instead of one.
      
      opw-783945
      ac31252d
  6. Nov 23, 2017
  7. Nov 22, 2017
  8. Nov 21, 2017
  9. Nov 20, 2017
Loading