An error occurred fetching the project authors.
  1. 14 Mar, 2019 9 commits
    • Vincent Pelletier's avatar
      DCWorkflowDefinition_listObjectActions: Always return a vector. · e793b164
      Vincent Pelletier authored
      Simplifies callers, as they then do not have to test for None.
      Also, use sort(key=...), which removes the need for internal 2-tuple
      wrapping & unwrapping of all result entries.
      Also, coding style (helps line-based profiling readability) & better
      naming.
      e793b164
    • Vincent Pelletier's avatar
      WorkflowTool: Accelerate action generation. · bba23043
      Vincent Pelletier authored
      Include worklist parameter generation in the scope of existing cache.
      Otherwise, it will be generated in pure loss if it is followed by a cache
      hit. Most of WorkflowTool change is just indentation change.
      
      Also, do some minor optimisations/simplifications in patchess.DCWorkflow.
      Some comments:
      - Guard_checkWithoutRoles return value is evaluated as a boolean, so no
        need to cast to int before returning based on boolean evaluation...
      - DCWorkflowDefinition.worklists is always true, even when empty.
      - Listing portal types per workflow requires checking all workflows, so
        build the whole mapping and cache it instead of caching for each workflow
        type individually (many more cache hits, fewer redundant computations)
      - getVarMatch is expensive just for a fallback and a wrap, bypass it to
        reduce redundant work.
      bba23043
    • Vincent Pelletier's avatar
      ERP5Type.Core.Folder: Optimise _getFolderHandlerData. · 43dd2515
      Vincent Pelletier authored
      Backward-compatibility path does not need to be fast. Optimise for sane
      folders.
      43dd2515
    • Vincent Pelletier's avatar
      ERP5Type.Core.Folder: Do not try to access subobjects with invalid names. · 735d4748
      Vincent Pelletier authored
      This improves performance when unset protected properties are being
      accessed.
      735d4748
    • Arnaud Fontaine's avatar
    • Arnaud Fontaine's avatar
      erp5_base: Remove ParentDeliveryCategoryMovementGroup broken since its introduction apparently. · 1051b18f
      Arnaud Fontaine authored
      In contrary to the ParentDeliveryPropertyMovementGroup for property, it has
      never actually checked the parent Delivery but the current one.
      1051b18f
    • Arnaud Fontaine's avatar
      ui_dump_test: No need to track Activities. · 2c8fcf67
      Arnaud Fontaine authored
      This also avoids KeyError exceptions when Activities have already been processed
      in the meantime.
      2c8fcf67
    • Jérome Perrin's avatar
      ui_test_core: remove dead code · fcd4c826
      Jérome Perrin authored
      These extensions does not seem to be used.
      
      /reviewed-on nexedi/erp5!845
      fcd4c826
    • Jérome Perrin's avatar
      Repair testFunctionalStandaloneUserTutorial · ddd3e26c
      Jérome Perrin authored
      Test was never ending processing activities:
      
      ```
      2019-03-11T03:09:20 WARNING CMFActivity invalid after_path_and_method_id value: ('/erp5/organisation_module/2', 'immediateRecusriveReindexObject', 'immediateReindexObject')
      Traceback (most recent call last):
        File "product/CMFActivity/Activity/SQLBase.py", line 319, in getValidationSQL
          validate_list.append(' AND '.join(method(v, quote)))
        File "product/CMFActivity/Activity/SQLBase.py", line 345, in _validate_after_path_and_method_id
          path, method_id = value
      ValueError: too many values to unpack
      2019-03-11T03:09:20 WARNING CMFActivity invalid after_path_and_method_id value: ('/erp5/organisation_module/2', 'immediateRecusriveReindexObject', 'immediateReindexObject')
      Traceback (most recent call last):
        File "product/CMFActivity/Activity/SQLBase.py", line 319, in getValidationSQL
          validate_list.append(' AND '.join(method(v, quote)))
        File "product/CMFActivity/Activity/SQLBase.py", line 345, in _validate_after_path_and_method_id
          path, method_id = value
      ValueError: too many values to unpack
      2019-03-11T03:09:20 WARNING CMFActivity invalid after_path_and_method_id value: ('/erp5/organisation_module/2', 'immediateRecusriveReindexObject', 'immediateReindexObject')
      Traceback (most recent call last):
        File "product/CMFActivity/Activity/SQLBase.py", line 319, in getValidationSQL
          validate_list.append(' AND '.join(method(v, quote)))
        File "product/CMFActivity/Activity/SQLBase.py", line 345, in _validate_after_path_and_method_id
          path, method_id = value
      ValueError: too many values to unpack
      2019-03-11T03:09:20 ERROR CMFActivity message uid 7256619472683296857L has a circular dependency
       ```
      
      Fix this failing test by using the proper `after_path_and_method_id` syntax and also some other small changes to make the test fail earlier in case of problems.
      
      /reviewed-on nexedi/erp5!844
      ddd3e26c
  2. 13 Mar, 2019 4 commits
  3. 12 Mar, 2019 1 commit
  4. 11 Mar, 2019 4 commits
  5. 08 Mar, 2019 1 commit
    • Tristan Cavelier's avatar
      [erp5_web_renderjs_ui] fix changing relation field input state with empty value_text · 82c5ecae
      Tristan Cavelier authored
      Update relation field input value only on demand,
      changing the state of a relation_input gadget with value_text = ""
      now updates the field.
      
      You can reproduce by doing :
      - open an object in tab "View" with a relation field (eg my_successor_title) filled
      - open a new tab, and edit the field to make it empty
      - from the first tab, refresh the view by click on the left panel "View" tab
      - the relation field should be emptied.
      82c5ecae
  6. 07 Mar, 2019 9 commits
  7. 06 Mar, 2019 3 commits
  8. 05 Mar, 2019 2 commits
    • Sebastien Robin's avatar
      erp5.util: Release 0.4.58 · 59f55bfb
      Sebastien Robin authored
      59f55bfb
    • Sebastien Robin's avatar
      testnode: give time to supervisord to kill sub processes · 32c5eebf
      Sebastien Robin authored
      supervirord when stopping might need several seconds to kill
      properly sub processes. So give him some time before killing
      more brutaly with killall.
      
      This should solve issue with functional tests are not working
      due to apache not starting. Apache when killed to brutally
      let unclosed semaphore, and after some time, we have too many
      semaphore to start new apache, making functional tests failing.
      32c5eebf
  9. 04 Mar, 2019 1 commit
    • Julien Muchembled's avatar
      CMFActivity: new activate() parameter to prefer executing on the same node · 1bbd5e6e
      Julien Muchembled authored
      This implements a special case of node specialization, to make better
      use of the ZODB Storage cache. By default, a non-grouped message is
      marked to be executed by the same node that created it, if the object
      is not a tool and if it was not activated by path. This can be
      overridden (either forced or prevented) using a new 'node' activate()
      parameter. See message of the first merged commits for details, and
      also ActiveObject.activate() docstring. For SQLDict & SQLQueue only.
      
      In the future, the new 'node' argument could accept any other string
      value that refers to a group of nodes. Groups would be defined on the
      activity tool, and be assigned negative integers. Contrary to what is
      implemented here, such specialization would be strict, in that a node
      would never process a message for a group it does not belong.
      
      /reviewed-on nexedi/erp5!836
      1bbd5e6e
  10. 01 Mar, 2019 1 commit
    • Georgios Dagkakis's avatar
      erp5_core: Ignore non-existent transitions in Module_listWorkflowTransitionItemList · 14940e3f
      Georgios Dagkakis authored
      instead of crashing.
      
      This can happen in workflow if we have:
      - transition_x is declared as possible in state_x
      - then transition_x gets deleted
      In this case in state_x in manage_properties nothing is visible,
      which creates an inconsistent in its data, yet functional generally workflow
      
      Since it is easy to create such cases, just ignore them in Module_listWorkflowTransitionItemList
      so that mass change state can work
      14940e3f
  11. 27 Feb, 2019 1 commit
  12. 26 Feb, 2019 4 commits