Problems with user commands ..!!

Hello everyone. As usual I am in facing a strange problem. I have been playing with user commands all the time and I thought I knew all. However a problem occurs when a user command is used on a onmessage trigger. The code which I wrote on the trigger works perfectly on the trigger, but if I use the same code in a user command called "onmessage" flexsim detects the msgsendingobject incorrectly.

For more details the code on the trigger is:
/**IDENTIFY NURSES AND DOCS*/
treenode current = ownerobject(c);
treenode XX = msgsendingobject();
int ID = getlabelnum(msgsendingobject,"ID");
if (msgparam(2) == 1) setlabelnum(current,"Nurse Num",ID);
if (msgparam(2) == 2) setlabelnum(current,"Doc Num",ID);
if (msgparam(2) == 2) setlabelnum(current,"WTBS",time - getlabelnum(current,"Arrival Time"));
if (msgparam(1) == 1) closeinput(current);
if (msgparam(1) == 2) openinput(current);

and the code in the onmessage user command is:
/**IDENTIFY NURSES AND DOCS*/
treenode bed = parnode(1);
treenode XX = msgsendingobject();
int ID = getlabelnum(XX,"ID");
if (msgparam(2) == 1) setlabelnum(bed,"Nurse Num",ID);
if (msgparam(2) == 2) setlabelnum(bed,"Doc Num",ID);
if (msgparam(2) == 2) setlabelnum(bed,"WTBS",time - getlabelnum(bed,"Arrival Time"));
if (msgparam(1) == 1) closeinput(bed);
if (msgparam(1) == 2) openinput(bed);

The result for node XX comes different when the code is written inside user command. Please help

If you read the user manual you will find that msgsendingobject() and msgparam() are only used in the OnMessage trigger. So in your usercommand you cannot use these functions and you have to use parval, parnode and parstr.

Got it.. Thanks a ton.. Also posting the solution if someone else faces the same issue.
So instead of using msgsendingobject() command inside the user command (ie. onmessage(bed,sender) I declared
treenode bed = parnode(1);
treenode sender = parnode(2);
and wrote the commands with reference to bed and sender and on the on message trigger of the multiprocessor, I declared
treenode current = ownerobject(c);
treenode X = msgsendingobject();
then I wrote onmessage(current,X); and bingo.. it worked.

Originally Posted by Steven Hamoen

If you read the user manual you will find that msgsendingobject() and msgparam() are only used in the OnMessage trigger. So in your usercommand you cannot use these functions and you have to use parval, parnode and parstr.

Another quick question. Is there a way to call the item that is currently on the multiprocessor in the on message trigger as it only begins with treenode current = ownerobject(c). I tried using treenode item = parnode(1) inside on message trigger but it points to the msgsendingobject instead of the item.
I also tried using treenode item = getiteminvloved(current); but it returns NULL

Hi,

have you tried :

if(Content(current)==1)
{
treenode item = first(current);
}

Yes it almost worked. For me it worked as
treenode item;
if(content(current) == 1)
{
item = first(current);
}
Nevertheless thanks a ton for such a quick reply