Skip to content

Fix: merging tickets triggers unwanted followup/task notifications - #25077

Merged
trasher merged 3 commits into
glpi-project:11.0/bugfixesfrom
Lainow:fix-ticket-fusion
Aug 6, 2026
Merged

Fix: merging tickets triggers unwanted followup/task notifications#25077
trasher merged 3 commits into
glpi-project:11.0/bugfixesfrom
Lainow:fix-ticket-fusion

Conversation

@Lainow

@Lainow Lainow commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • This change requires a documentation update.

Description

  • It fixes !45665
  • Here is a brief description of what this PR does
    Ticket::merge() copies the source ticket's content, followups and tasks onto the target ticket without disabling notifications, causing "New followup"/"New task" notifications to fire for content that is only being duplicated as part of the merge, not newly authored.

Screenshots (if appropriate):

@Lainow Lainow self-assigned this Aug 4, 2026
@Lainow
Lainow requested a review from Rom1-B August 4, 2026 10:21
Ticket::merge() copies the source ticket's content, followups and
tasks onto the target ticket without disabling notifications,
causing "New followup"/"New task" notifications to fire for content
that is only being duplicated as part of the merge, not newly
authored.
@Lainow
Lainow force-pushed the fix-ticket-fusion branch from 2f4a547 to 5c521f6 Compare August 4, 2026 10:51
Comment thread tests/functional/TicketTest.php
@Lainow
Lainow requested a review from Rom1-B August 4, 2026 14:36
Comment thread src/Ticket.php
@Lainow
Lainow requested a review from Rom1-B August 6, 2026 07:46
@Rom1-B
Rom1-B requested a review from trasher August 6, 2026 08:33
@trasher
trasher merged commit 55260a6 into glpi-project:11.0/bugfixes Aug 6, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants