Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
erp5 erp5
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Labels
    • Labels
  • Merge requests 139
    • Merge requests 139
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • erp5erp5
  • Merge requests
  • !1556

Merged
Created Feb 17, 2022 by Vincent Pelletier@vpelletierOwner

ERP5{,Type.Core}.InteractionWorkflow: Deprecate activeScript.

  • Overview 6
  • Commits 1
  • Changes 3

Replace it with a method on ERP5Type.Base so these activities get found by CopySupport.unindexObject and flushed, rather than remaining on the interaction workflow's context and failing when run. It seems a lot more likely for a document to be deleted while interactions are being spawned than an interaction workflow itself. So this should be a net benefit in activity stability.

The reason behind this change is that untitests fail en masse in beforeTearDown when erp5_local_transaction is installed: it has an interaction which spawns an activeScript activity, whose context is the interation workflow. Then, beforeTearDown deletes the object before this activity could be run. Because the activity context is not the document, it does not get flushed by CopySupport.unindexObject, so when beforeTearDown then calls self.tic() it fails because the activity cannot run (cannot traverse to the now-deleted document).

/cc @arnau @jerome @jm @romain (not sure who else have ideas & feedback on this topic)

Edited Feb 17, 2022 by Vincent Pelletier
Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: depreate_activeScript
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7