iTop icon indicating copy to clipboard operation
iTop copied to clipboard

N°7570 - feat(Typology): Add useful tabs to certain typology classes

Open Hipska opened this issue 2 months ago • 3 comments

Base information

Question Answer
Related to a SourceForge thead / Another PR / Combodo ticket? N/A
Type of change? Enhancement

Objective

When clicking on a Typology item from a linked object, it is sometimes handy to see which other objects are also linking to the same Typology object.

Proposed solution

Add tabs to the Typology objects to see which other objects are using this Typology.

Checklist before requesting a review

  • [x] I have performed a self-review of my code
  • [ ] I have tested all changes I made on an iTop instance
  • [ ] I have added a unit test, otherwise I have explained why I couldn't
  • [x] Is the PR clear and detailed enough so anyone can understand digging in the code?

Checklist of things to do before PR is ready to merge

  • [ ] Add dictionary items when the additional fields are accepted

Hipska avatar Oct 02 '25 14:10 Hipska

Hello Hipska, Have you tested the use case, of adding or removing ospatches (or any other sub-objects)? Add means moving them from one OS version to another. When do you need to do this ? It is highly subject to errors. And if you really do, the bulk modify is much better to control your changes. Remove means empty the osversion of an ospatch (or multiple), why would you do that? You should rather replace the osversion if it was wrong by the correct one using the bulk modify of ospatches.

As a conclusion, our policy for Typology is to propose "add_only", nothing else, which means allow sub-object creation. For the "edit_when" option, the policy is more open, if there are no dependencies, it can be omitted, which means you can create a sub-object in read or edition of the host. (This "edit_mode" "add_only" should have been named "create_only")

In some case like "model" and "brand", because of dependencies and class restriction, even this "add_only" mode is nearly unusable. So me might prevent even the creation in the standard model.

Translation are missing.

v-dumas avatar Nov 28 '25 15:11 v-dumas

@jf-cbd @v-dumas You were right. It's indeed not possible to remove linked items using the add_remove mode. I was confused and actually meant to use the in_place mode.

I tried all new lists and was able to create and remove items this way:

  • Brand
    • Model
    • IOSVersion
  • OSVersion
    • OSPatch
  • OSFamily
    • OSVersion

Translations are indeed missing. Are we sure these new lists would be accepted to be added in the datamodel? Then I will make the translations for them..

Hipska avatar Dec 02 '25 11:12 Hipska

@jf-cbd @v-dumas You were right. It's indeed not possible to remove linked items using the add_remove mode. I was confused and actually meant to use the in_place mode.

I tried all new lists and was able to create and remove items this way:

* Brand
  
  * Model
  * IOSVersion

* OSVersion
  
  * OSPatch

* OSFamily
  
  * OSVersion

Translations are indeed missing. Are we sure these new lists would be accepted to be added in the datamodel? Then I will make the translations for them..

Yes it has been functionally accepted so it will be integrated in iTop :)

jf-cbd avatar Dec 05 '25 14:12 jf-cbd