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 140
    • Merge requests 140
  • 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
  • !1231

Merged
Created Aug 03, 2020 by Jérome Perrin@jeromeOwner

BusinessTemplate: update tools, except legacy tools

  • 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
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: fix/business-template-update-tool
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7