Fix update ticket status when ticket task is de-scheduled

[RomainLvr]

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets	!32381

When a task is de-scheduled, the ticket status remains In progress (Scheduled) instead of being changed back to In progress (Assigned).

[trasher]

Please add a test

[Rom1-B]

There is already a function updateParentStatus, which you could use

[cedric-anne]

I change the target to main branch. This can be considered as a new feature and may have unexpected side effects.

[cedric-anne]

@RomainLvr I rebased your branch to fix conflicts.

[cedric-anne]

I am not sure of this, but I think we should not rely on the current profile statuses matrix to filter available status. Indeed, I think that when a task begin date is updated, the ticket status should be changed to PLANNED, even if the technician is not supposed to be able to do it by itself.

@orthagh What do you think of this?

Suggested change

	$parentitem::isAllowedStatus($parentitem->fields["status"], CommonITILObject::ASSIGNED)
	$parentitem->isStatusExists(CommonITILObject::ASSIGNED)

[orthagh]

Yes, previously, we just checked the planned status really exists for the ITILobject (Changes doesn't have this status).
The right of a technician to change the status to planned should not be checked.
This is for the UI and prevents manually changing the status field.
All shortcuts, like "adding a tech actor -> status move to in ASSIGN", "remove all attributed actor -> return to INCOMING" and the new one to move to PLANNED should be done by the framework automatically and do not impact the status field directly.

[cedric-anne]

Same remark here.

Suggested change

	$parentitem::isAllowedStatus($parentitem->fields["status"], CommonITILObject::INCOMING)
	$parentitem->isStatusExists(CommonITILObject::INCOMING)