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 136
    • Merge requests 136
  • 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
  • !132

Closed
Created Jun 22, 2016 by Sebastien Robin@sebContributor
  • Report abuse
Report abuse

simulation: identify the trade model path used for generating a simulation movement

  • Overview 2
  • Commits 1
  • Changes 1

A business process can define several trade model path for a single trade phase. This will generate several simulation movement per rule input simulation movement.

When reexpanding a simulation tree, we have to do matching between generated movement and existing simulation movements. Up to now, we do indirect matching by looking at property that would be updated by trade model path (like source or destination). But this indirect way is inconvenient and does not work as expected when one of the matching property is changed.

It would be more convenient and more reliable to allow checking directly from which trade model path comes a simulation movement. So add property trade_model_path_reference on simulation movements, and then we are free to do matching on it if we want.

As an example, this is particular useful in the case of accounting, where in the invoice transaction rule we usually generate an "accounting+" and an "accounting-" movement through two distinct trade model path.

@jm @jerome @kazuhiko, can you please let me know your though on such change. This will add a property "trade_model_path_reference" on all simulation movements. But this should not affect existing simulation since usually we will just not look at this property. Do you see better ways ?

I will update erp5_configurator_standard_invoicing_template and add unit test before merging once solution is agreed.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: seb_project
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7