Skip to content

N°640 - No 'Tickets' tab on FunctionalCIs when no Ticket sub-classes exists#773

Open
v-dumas wants to merge 5 commits intodevelopfrom
feature/640-Team-tickets_list
Open

N°640 - No 'Tickets' tab on FunctionalCIs when no Ticket sub-classes exists#773
v-dumas wants to merge 5 commits intodevelopfrom
feature/640-Team-tickets_list

Conversation

@v-dumas
Copy link
Contributor

@v-dumas v-dumas commented Nov 3, 2025

Base information

Question Answer
Related to a SourceForge thead / Another PR / Combodo ticket?
Type of change? Bug fix / Enhancement / Translations

Symptom (bug) / Objective (enhancement)

Reproduction procedure (bug)

  1. On iTop x.y.z
  2. With PHP x.y.z
  3. First go there
  4. Then do that
  5. ...
  6. Finally, see that...

Cause (bug)

Proposed solution (bug and enhancement)

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 digging in the code?

Checklist of things to do before PR is ready to merge

  • ...
  • ...
  • ...

@CombodoApplicationsAccount CombodoApplicationsAccount added the internal Work made by Combodo label Nov 3, 2025
@v-dumas v-dumas self-assigned this Jan 30, 2026
if (MetaModel::IsValidAttCode(get_class($this), $sTicketListAttCode))
{
if (MetaModel::HasChildrenClasses('Ticket') && MetaModel::IsValidAttCode(get_class($this), $sTicketListAttCode)) {
// Display one list per leaf class (the only way to display the status as of now)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this comment still correct? There now is the operational_status available for all tickets..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Work made by Combodo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants