Skip to content

Rename NodeFactory to PreBasis

Carsten Gräser requested to merge feature/rename-nodefactories into master

We discussed and decided the renaming on the last meeting. Reasons for renaming:

  • The name NodeFactory does not fit because it creates more than just nodes
  • The FactoryTag/FactoryBuilder classes are really factories that create a NodeFactory, but we would not like to use NodeFactoryFactory
  • PreBasis is more understandable: It's almost a basis.

I renamed all occurrences in dune-functions. If you implemented your own node-factory, you only need to take care that the concept name was changed from NodeFactory to PreBasis. The actual naming of your class is an implementation detail. However, for consistency you may want to adapt this also.

Merge request reports