Brewtarget / brewtarget

Main brewtarget source code repository.
GNU General Public License v3.0
312 stars 135 forks source link

Bugs/constructors #555

Closed mikfire closed 3 years ago

mikfire commented 3 years ago

Side discussions made me aware that I had horribly broken the model in the primitive constructors by making each of them aware of the database column names. This is my first pass at fixing it.

I was able to remove a lot of unused constructors, one slightly redundant template method and generally clean the code up. It comes with a price tag of a slightly more complex NamedEntity constructor, and that one of the newNamedEntity methods has to search for the record it just created.