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
  • !1231

BusinessTemplate: update tools, except legacy tools

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Jérome Perrin requested to merge fix/business-template-update-tool into master Aug 03, 2020
  • Overview 6
  • Commits 1
  • Pipelines 4
  • Changes 2

Since 248f59e5 (Business Template: Likewise ERP5Site.addERP5Tool(), do not re-create Tool if it already exists., 2020-06-22) we no longer update tools, because of the problem that business template does not currently handle updating objects with lots of sub-objects.

But we realized that we really need to update tools when they contain configuration as object attributes, like mimetypes_registry, where the problem was observed.

Instead of unconditionnally skipping any tool during update, we only skip the ones that were initially managed by ERP5Generator and are moved to business templates.

Edited Aug 05, 2020 by Jérome Perrin
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/business-template-update-tool
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7