Skip to content

fix(core): Fix mandatory fields error when creating an item with an assigned technician - #1332

Open
MyvTsv wants to merge 3 commits into
11.0/bugfixesfrom
ticket47460
Open

MyvTsv wants to merge 3 commits into
11.0/bugfixesfrom
ticket47460

Conversation

@MyvTsv

@MyvTsv MyvTsv commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor
  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !46829 & !47460 & !46413
  • Here is a brief description of what this PR does

When an item with a filled mandatory field is created with an assigned technician, the Some mandatory fields are empty error is displayed although the value is saved, and the ticket stays in the New status instead of Processing (assigned).

Steps to reproduce

  1. Go to Setup > Additional fields and create a block:
    • Type: Insertion in the form
    • Associated item type: Ticket
  2. In this block, add a Text field named Tel and check Mandatory field.
  3. Go to Assistance > Tickets > Create ticket.
  4. Fill the title, the description and the Tel field.
  5. In the Actors section, add a technician in Assigned to.
  6. Click Add.

@MyvTsv MyvTsv self-assigned this Nov 5, 2026
@MyvTsv
MyvTsv marked this pull request as ready for review November 6, 2026 09:05
@MyvTsv
MyvTsv requested review from a team, Lainow, Mary-Clb and stonebuzz and removed request for a team November 6, 2026 09:06
Comment thread inc/container.class.php Resolved
@Mary-Clb

Mary-Clb commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

@MyvTsv I did the same fix on #1337 and #1338 : I close both in favor of yours because you did this one first.

@MyvTsv
MyvTsv requested a review from stonebuzz November 9, 2026 10:37
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