Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Products should have relationship to ProductFamily #42

Closed
montanajava opened this issue Feb 2, 2024 · 5 comments
Closed

Products should have relationship to ProductFamily #42

montanajava opened this issue Feb 2, 2024 · 5 comments
Labels
enhancement New feature or request jira Tag to activate JIRA integration v2 A major release Vs. 2

Comments

@montanajava
Copy link
Collaborator

In working group discussions it was decided that there should be no relationship of a respective product to the ProductFamily enumeration. The reason for that was because it was in once sense redundant: the name of the class of the product revealed the family.

That said, "software" has no way of knowing that -- without some form of intelligence.

This proposal is to add that relationship for all products (ManureRecycing, PlantProtection, and so on).

@montanajava montanajava added v2 A major release Vs. 2 enhancement New feature or request labels Feb 2, 2024
@montanajava montanajava changed the title Products have no relationship to ProductFamily Products shoud have relationship to ProductFamily Feb 2, 2024
@montanajava montanajava changed the title Products shoud have relationship to ProductFamily Products should have relationship to ProductFamily Feb 2, 2024
@AFoletti AFoletti added the jira Tag to activate JIRA integration label Aug 14, 2024
@AFoletti
Copy link
Member

@montanajava could you please better specify what you meant (with an example if possible). Otherwise, this will be closed as "won't fix"

@montanajava
Copy link
Collaborator Author

The rationale for the change request is so that systems can explicitly, not implicitly, know to what productFamily a product belongs. This allows for filtering. Use cases include UIs allowing users to filter products by family, and systems that want to filter out certain families of products from consideration in any number of contexts. A branch https://github.com/blw-ofag-ufag/eCH-0263/tree/42-referenceProductFamily has been created as an example of how it could be implemented.

@AFoletti
Copy link
Member

Thanks for the explanatin and the example. I now understand better what you mean.
However, this means mixing two different modeling choices: "productType" as discinct classes (one for each Type, as it is today) and a general class, where the productType is given thanks to a codelist.
Mixing the two approaches is in my opinion wrong and redundant.
As you say in the initial post, the type of product is given by the class of said product. Being an instance of a class is the most clear way to say "I am exactly this thing" and every application should be able to handle that (or stop trying to handle structured data).

However...
You are right, "substanceType" does not make any reference to actual instances of the various "ProductType", but only to an enumerated type.
A better way to solve this would be, in my opinion, to get rid of the "productFamilyType" as enumerated type and transform the productType relationship to a proper reference (via ID) to an instance of one of the many defined "productType" classes.

@montanajava
Copy link
Collaborator Author

Agreed. This issue was created under a false premise of lists of products of multiple families being unified into one list. At least in the case of one well-known active BLW project, that has turned out to be not the case. In that project within the GUI, products of divergent families are, in fact, separated into separate tabs. I withdraw this proposal and support your position of closing this with "won't fix". Sorry for the run-around.

As to your second paragraph, I would recommend a new issue for that.

@AFoletti
Copy link
Member

All right! Thanks for the discussion. Even if we go in another direction, you raised an important issue.

I will close this one and delete your branch.
As you suggested, there is a new ticket up for discussion: #61

@AFoletti AFoletti closed this as not planned Won't fix, can't repro, duplicate, stale Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request jira Tag to activate JIRA integration v2 A major release Vs. 2
Projects
None yet
Development

No branches or pull requests

2 participants