Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1076

Folder: fix checkConsistency on folder containing non erp5 objects

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Jérome Perrin requested to merge fix/checkConsistency_recursive into master Mar 12, 2020
  • Overview 3
  • Commits 1
  • Pipelines 1
  • Changes 2

contentValues() nowadays return also non ERP5-ish objects such as python scripts , OFS.File etc. Because of that, if an ERP5 Folder contained a non ERP5 object, which does not have fixConsistency or checkConsistency methods, Folder.checkConsistency was causing an infinite loop because it was acquiring checkConsistency method from self.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/checkConsistency_recursive
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7