Skip to content

Start refactoring of ObjectFactory#6162

Draft
alxbilger wants to merge 1 commit into
sofa-framework:masterfrom
alxbilger:refactorobjectfactory
Draft

Start refactoring of ObjectFactory#6162
alxbilger wants to merge 1 commit into
sofa-framework:masterfrom
alxbilger:refactorobjectfactory

Conversation

@alxbilger

Copy link
Copy Markdown
Contributor

Early stage of a simplification of the object factory (now ComponentFactory).

Two unanswered questions:

  1. How to treat the template? (related to [Core] Improve UX when defining template of a Component #6161
  2. How to select the template (canCreate)? And how to get rid of create?

runSofa compiles, but no guarantee for others.

Current changes allow:

<Node name="root">
    <RequiredPlugin pluginName="Sofa.Component.StateContainer"/>
    <MechanicalObject/>
</Node>
<Node name="root">
    <Sofa.Component.StateContainer.MechanicalObject/>
</Node>

By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

  • it builds with SUCCESS for all platforms on the CI.
  • it does not generate new warnings.
  • it does not generate new unit test failures.
  • it does not generate new scene test failures.
  • it does not break API compatibility.
  • it is more than 1 week old (or has fast-merge label).

@alxbilger alxbilger added the pr: status to review To notify reviewers to review this pull-request label Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: status to review To notify reviewers to review this pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant