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

Drop `_recorded_property_dict` from XML on components

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Jérome Perrin requested to merge feat/no_recorded_property_dict into master Apr 25, 2022
  • Overview 3
  • Commits 3
  • Pipelines 3
  • Changes 361

When a validated component is edited, mixin.component.RecordablePropertyMetaClass records the text_content property. This recorded property will be cleared afterwards if no errors are found, but this causes differences in business template XML for each component the first time they are edited.

This change is about removing the _recorded_property_dict when there are no property recorded, which leads to slightly smaller pickles (and business template XML, because they are also pickles).

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