1. 03 Jun, 2019 1 commit
    • Jérome Perrin's avatar
      testInvalidationBug: update after CMFActivity's node families · ef9fe5da
      Jérome Perrin authored
      Test was failing because family mapping is loaded while connection is
      already closed.
      
      ```
      Traceback (most recent call last):
        File "erp5/Products/ERP5/tests/testInvalidationBug.py", line 156, in testLateInvalidationFromZEO
          activity_tool.tic()
        File "erp5/product/ERP5Type/tests/ProcessingNodeTestCase.py", line 96, in tic
          self._orig_tic(processing_node, force)
        File "erp5/product/CMFActivity/ActivityTool.py", line 1326, in tic
          node_family_id_set = self.getCurrentNodeFamilyIdSet()
        File "erp5/product/CMFActivity/ActivityTool.py", line 921, in getCurrentNodeFamilyIdSet
          return self._getNodeFamilyIdDict().get(getCurrentNode(), ())
        File "ZODB/Connection.py", line 909, in setstate
          self._setstate(obj, oid)
        File "ZODB/Connection.py", line 953, in _setstate
          p, serial = self._storage.load(oid, '')
        File "ZEO/ClientStorage.py", line 825, in load
          result = self.loadBefore(oid, m64)
        File "ZEO/ClientStorage.py", line 839, in loadBefore
          raise ClientDisconnected()
      ClientDisconnected
      ```
      
      Load the object explicitly before so that it's in cache.
      
      /reviewed-on !881
      ef9fe5da
  2. 31 May, 2019 3 commits
  3. 29 May, 2019 6 commits
  4. 28 May, 2019 2 commits
  5. 24 May, 2019 6 commits
    • Romain Courteaud's avatar
      [erp5_hal_json_style] Fix listbox pagination · 30b4f89e
      Romain Courteaud authored
      The pagination was not working when using the limit parameter, and when
      the total number of documents is equal to the pagination limit + 1.
      30b4f89e
    • Tatuya Kamada's avatar
      simulation: Test the divergence on Float Equivalence Tester with the epsilon-span by default · 6d17f0a1
      Tatuya Kamada authored
      Before:
         Float Equivalence Tester does not test the divergence even if Divergence Provider is ON
          
      After:
           Float Equivalence Tester test the divergence with the epsilon-span if Divergence Provider is ON
          
      Before, we needed the following condition to test the divergence at Float Equivalence Tester:
          
             "Divergence Provider is ON"  AND
             ("Absolute Tolerance has some value"  OR
              "Relative Tolerance has some value" )
          
      However this default behavior was not consistent with other testers, and
      difficult to notice from the user interface.
      Thus test the divergence by default with the epsilon span which is defined in 
      FloatEquivalenceTester class.
          
      The epsilon-span is the same amount that when we put 0.0 into the ranges.
      The 0.0 has been the configuration of erp5_configurator_standard_trade_template.
      
      /reviewed-on nexedi/erp5!865
      6d17f0a1
    • Vincent Pelletier's avatar
    • Vincent Pelletier's avatar
    • Vincent Pelletier's avatar
      ZSQLCatalog: Fix security indexation when indexing the same object multiple times · 048c4414
      Vincent Pelletier authored
      When an object is being indexed, it get wrapped so it has magic properties,
      like security definitions as they will be accessed by indexation methods.
      One such property is used to trigger the insertion of rows in
      roles_and_users table. That property has the special feature that it is
      only produced on the first object needing a new set of rows added to
      roles_and_users, other objects will get a None value instead for that
      property so that no further lines are inserted.
      Then, SQCatalog iterates over that wrapper list to compute all indexation
      method parameters. It does so via a cache (LazyIndexationParameterList) so
      that unused parameters are not computed. The cache key is wrapped object's
      uid.
      All this together means that if an object is indexed multiple times in a
      single call (which happens at site creation, for example) and it needs a
      new set of rows added in roles_and_users, only one of produced wrappers
      will get the appropriate value. If that object is not the first on which
      that parameter is evaluated (which gets more likely with every copy of the
      same object being indexed), it will be cached as None and shared between
      all copies, causing no row to be added to roles_and_users.
      
      Full investigation done by by Yusei Tahara.
      wrappers will get the
      048c4414
    • Nicolas Wavrant's avatar
      erp5_interface_post: do not pass potentially huge data as acitvity parameters · 51c065f4
      Nicolas Wavrant authored
      As we may reach database limits, and to avoid them to be logged.
      51c065f4
  6. 23 May, 2019 4 commits
  7. 22 May, 2019 12 commits
  8. 21 May, 2019 6 commits