1. 10 Jan, 2025 1 commit
  2. 09 Jan, 2025 5 commits
  3. 07 Jan, 2025 17 commits
  4. 03 Jan, 2025 1 commit
  5. 02 Jan, 2025 3 commits
    • Nicolas Wavrant's avatar
      erp5_web_renderjs_ui_test: test infinite listboxes · 95b9cfca
      Nicolas Wavrant authored
      Infinite listbox are listboxes for which the "lines" property is set to 0, meaning the listbox shouldn't try
      to paginate results.
      95b9cfca
    • Nicolas Wavrant's avatar
      erp5_web_renderjs_ui: do not ignore a listbox configuration of 0 lines · 913c2abb
      Nicolas Wavrant authored
      This is intentional, and means that no limit should be applied (iow: an infinite
      listbox). In this case, the limit parameter in the call to jIO should be set to 0,
      not undefined, otherwise the parameter will be dropped in the AJAX request,
      and later on be defaulted to 10 by ERP5Document_getHateoas.
      913c2abb
    • Nicolas Wavrant's avatar
      erp5_hal_json_style: do not limit infinite listboxes · 325c5563
      Nicolas Wavrant authored
      If the max number of lines of a listbox was set to 0, meaning "no limit set", then ERP5Document_getHateoas would still return a paginated list of 10 results, 10 being
      the default set in the parameters of ERP5Document_getHateoas.
      
      It looks like there is a beginning of support for such case in ERP5Document_getHateoas,
      with the appearance of code such as "if limit", but the bug comes from the lack of typing in the JS-jIO-JSON-ERP5's HAL interface. In case of a limit set to 0 in the listbox configuration, ERP5
      would receive the string "0" which is a non empty string. It would fail the condition and would
      be handled like there is a limit, or of 0 - so no result - or of a default limit chosen by the code.
      325c5563
  6. 30 Dec, 2024 1 commit
    • Titouan Soulard's avatar
      erp5_xhtml_style: report error messages on clone as errors · 5d240501
      Titouan Soulard authored
      The script `Base_createCloneDocument` was made to be user-friendly, reporting
      nice errors instead of throwing when a problems is detected. These errors are
      displayed using the `portal_status_message` parameter of `Base_redirect`, which
      looks like a warning on XHTML-style, but appears green with RenderJS.
      
      This commit adds an optional parameter `portal_status_level` to all calls to
      `Base_redirect` which are actually errors, so that they also look like a
      warning or error (orange background) on RenderJS. The behaviour in XHTML-style
      says identical.
      
      /cc @romain @vpelletier
      5d240501
  7. 26 Dec, 2024 2 commits
  8. 25 Dec, 2024 1 commit
  9. 23 Dec, 2024 2 commits
    • Titouan Soulard's avatar
      erp5_core: fix `TypeError` on raise in `Base_edit` · a4274276
      Titouan Soulard authored
      String value raised with `ValueError` had an extra `%s` at the end, hence
      raising another error when formatting:
      
      > TypeError: not enough arguments for format string
      
      This commit simply removes the extra formatting character, since I could not
      guess what should have been printed here.
      a4274276
    • Titouan Soulard's avatar
      erp5_trade: fix method `Movement_zGetPackedQuantity` · 4aec00c2
      Titouan Soulard authored
      The ZSQLMethod was using a wrong table `movement` instead of the `stock` table,
      hence throwing errors. This commit makes it use the right table, by adding a
      filter on `node_uid`, taking into account the structure of the new table.
      
      /reviewed-by @jerome
      /reviewed-on !2030
      4aec00c2
  10. 19 Dec, 2024 2 commits
  11. 18 Dec, 2024 4 commits
  12. 17 Dec, 2024 1 commit