indexOf not working as expected

[Community Alums]

Hey Everyone, 

I need an ui action to display only if the user is in the L2 assignment group and not the team lead 1, team lead 2, or manager. 

Everything works until I try to use indexOf. It isn't working for the L2 assignment groups. Below is my script include code. 

What am I doing wrong? This ui action only needs to display if the current assignment group starts with L2 and the user isn't a team lead 1, team lead 2, or manager. 

var showChangePriority = Class.create();
showChangePriority.prototype = Object.extendsObject(AbstractAjaxProcessor, {
  
    checkCondition: function(current) {
        if ( gs.getUser().isMemberOf(current.assignment_group).toString().indexOf("L2") > -1 && (!gs.getUser().isMemberOf(current.assignment_group.manager) || !gs.getUser().isMemberOf(current.assignment_group.u_team_lead_1) || !gs.getUser().isMemberOf(current.assignment_group.u_team_lead_2))) {
            return true;
        } else {
            return false;
        }

    },
    type: 'showChangePriority'
});

 This is the condition field for UI action. 

new showChangePriority().checkCondition(current);

[Robbie · Accepted solution]

Hi @Community Alums,

I see a couple of issues with the logic here (Assuming I understand what you're trying to achieve correctly)

The 'isMemberOf' function returns a boolean, as in true or false. You therefore don't need to use an additional indexOf check.

Use the following tweaks to your script to achieve the same (based on my understanding) which I've tried and tested.

I would also consider only calling this script from your UI action if the group starts with 'L2' in the first place, but the following will work

To help others (or for me to help you more directly), please mark this response correct by clicking on Accept as Solution and/or Helpful.

Thanks, Robbie

 

var showChangePriority = Class.create();
showChangePriority.prototype = Object.extendsObject(AbstractAjaxProcessor, {
  
    checkCondition: function(current) {
        if (current.assignment_group.name.startsWith('L2')
        && gs.getUser().isMemberOf(current.assignment_group) 
	&& gs.getUser() != current.assignment_group.manager
	&& gs.getUser() != current.assignment_group.u_team_lead_1
	&& gs.getUser() != current.assignment_group.u_team_lead_2
     ) {
            return true;
        } else {
            return false;
        }

    },
    type: 'showChangePriority'
});

Based on your feedback, you can also try the below:

var showChangePriority = Class.create();
showChangePriority.prototype = Object.extendsObject(AbstractAjaxProcessor, {
    checkCondition: function(current) {
        // Check if the user is in the L2 assignment group
        if (gs.getUser().isMemberOf(current.assignment_group) && current.assignment_group.name.startsWith("L2") &&
            gs.getUserID() != current.assignment_group.manager && 
			gs.getUserID() != current.assignment_group.u_team_lead_1 &&
			gs.getUserID() != current.assignment_group.u_team_lead_2


        ) {
            // Check if the user is not a team lead 1, team lead 2, or manager

            return true; // User meets the conditions
        } else {
            return false;
        } // User does not meet the conditions
    },
    type: 'showChangePriority'
});

[Sonam_Tiwari]

Hi @Community Alums ,

current.assignment_group

This might be the sys_id which is why indexOf isn't working for you. Try dot walking to name or getDisplayValue() and then do indexOf().

[Community Alums]

gs.getUser().isMemberOf(current.assignment_group).getDisplayValue().indexOf("L2") > -1

I tried both ways, but the same issue. 

[Sonam_Tiwari]

Looks like @Sohithanjan G's 1st point  holds valid as isMemberOf will return a boolean and we are trying to find L2 from a true or false value which is not correct.

gs.info(gs.getUser().isMemberOf('ITSM App-Dev')&& current.assignment_group.name.indexOf("L2") > -1); should work

[chetanb]

Please check with search() method instead of indexOf