Feature/8960 compute active users and N°7060 - create ReadOnly profiles

[jf-cbd]

Base information

Question	Answer
Related to a SourceForge thread / Another PR / A GitHub Issue / Combodo ticket?	N°7060
Type of change?	Enhancement

Symptom (bug) / Objective (enhancement)

It also introduces new Read Only profiles.

Proposed solution (bug and enhancement)

• define new profiles
• write tests

Checklist before requesting a review

• 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
• Is the PR clear and detailed enough so anyone can understand without digging in the code?

[greptile-apps]

Greptile Summary

This PR adds user counting support and new read-only profiles. The main changes are:

• New Configuration, Ticket, and Service Catalog read-only profiles.
• A new ITopUserCountingRepository for user category counts.
• Dictionary labels and Composer autoload entries for the repository.
• Unit-test helpers and tests for user rights and counting behavior.

Confidence Score: 2/5

This should be fixed before merging.

• User counts can overlap or omit users in realistic category combinations.
• Calling the read-only count can clear the current user session context.
• The Service Catalog ReadOnly profile grants access to contract data beyond the stated scope.

sources/Users/ITopUserCountingRepository.php and datamodels/2.x/itop-profiles-itil/datamodel.itop-profiles-itil.xml need the main fixes.

Security Review

• Data exposure: datamodels/2.x/itop-profiles-itil/datamodel.itop-profiles-itil.xml grants Service Catalog ReadOnly users access to contract classes through the broad Service group.

Important Files Changed

Filename	Overview
sources/Users/ITopUserCountingRepository.php	Adds the main user counting logic, with category overlap, missing-category, and session side-effect issues.
datamodels/2.x/itop-profiles-itil/datamodel.itop-profiles-itil.xml	Adds read-only profiles, with ticket relation coverage gaps and broad service/contract exposure.

_{Reviews (1): Last reviewed commit: "Remove assertion in test" | Re-trigger Greptile}