N°9622 - Error in the user story life cycle

[accognet]

Internal

[greptile-apps]

Greptile Summary

This PR changes lifecycle form defaulting for mandatory attributes with a single allowed value. The main changes are:

• Adds special handling for linkset attributes in DisplayStimulusForm().
• Loads the current linkset, adds the only allowed object, and writes it back.
• Leaves scalar defaulting unchanged for non-linkset attributes.

Confidence Score: 3/5

These issues should be fixed before merging.

• Indirect linksets can load the allowed id from the wrong class and add the wrong link.

• The creation form still sends a scalar id through the linkset setter path.

• Both issues affect the single-allowed-value workflow this change is meant to handle.

• application/cmdbabstract.class.inc.php needs the linkset defaulting paths aligned for direct, indirect, lifecycle, and creation forms.

Important Files Changed

Filename	Overview
application/cmdbabstract.class.inc.php	Updates single-allowed-value defaulting in lifecycle forms, but the linkset handling is incomplete for indirect links and creation forms.

Comments Outside Diff (1)

1. application/cmdbabstract.class.inc.php, line 3106-3110 (link)

   Creation form still breaks

   The creation-form path still applies the old scalar defaulting logic to linkset attributes. If a mandatory linkset has exactly one allowed value during object creation, this calls Set() with the scalar id from array_keys(). DBObject::Set() treats non-null linkset values as an object-set iterator and calls UpdateFromCompleteList($value), so the creation form can fail before the user can submit it. This is the same single-allowed-linkset case handled below for lifecycle transitions, but this sibling path remains unchanged.

_{Reviews (1): Last reviewed commit: "9622 - Error in the user story life cycl..." | Re-trigger Greptile}

[greptile-apps]

Indirect linksets misload objects

For an AttributeLinkedSetIndirect, GetAllowedValues_att() returns ids from the remote target class, because AttributeLinkedSet::GetValuesDef() uses LinkSetModel::GetTargetClass(). This branch then loads that id from $oAttDef->GetLinkedClass(), which is the link/intermediate class. When a lifecycle transition has exactly one allowed remote object, the form can throw because no link row has that id, or it can add an unrelated existing link row with the same numeric id. The indirect case needs to create a link object and set its remote external key instead of loading the link class by the remote id.

[odain-cbd]

is there any way $aValues contains more than one single value?

can you add a test that cover DisplayStimulusForm with this new fixed usecase (linkset), pls.

[odain-cbd]

what if GetObject throws an exception when object does not exist? if it could happen, it could be nice for support team to catch the exception, log it and return false (i guess?).

[accognet]

If the attribute is a LinkSet, GetAllowedValues_att returns an ObjectSet from an OQL query. Therefore, I believe this object must exist.

[accognet]

And to answer the first question, it is impossible for $Values ​​to contain more than one value, because the test count($allowedValues) == 1 was performed just before.

[odain-cbd]

ok technically it is not possible. you are right (count==1).

so functionally this PR fixes stimulus form display of current object, by adding first found allowed value to the linkset, right? no all possible allowed values. one is enough...
if there is one value to add, cant it be default one if it exists? maybe it does not make sense?