Javascript to pull list of all active users

[timnardoni]

Hello, all. This script is meant to produce a list of active users but skipped ~1,500. What am I doing wrong?

Thank you in advance!

-Tim

var users = new GlideRecord('sys_user');
users.addQuery('active', true); //Active users only
users.query();
    
while (users.next()) {
        users.next();
	gs.info(users.user_name);
    }

[Nishant8]

Hello @timnardoni , can you please try following:

var users = new GlideRecord('sys_user');

users.addQuery('active', true); //Active users only

users.query();

while (users.hasNext()) {

    users.next();

    gs.info(users.user_name);

    }

Regards,

Nishant

[Ankur Bawiskar]

@timnardoni 

did you print the row count?

Try this and skip the query business rule and see what count it gives

var users = new GlideRecord('sys_user');
users.addActiveQuery(); //Active users only
users.setWorkflow(false); // skip query business rule
users.query();
gs.info('row count' + users.getRowCount());
while (users.next()) {
	gs.info(users.user_name);
    }

If my response helped please mark it correct and close the thread so that it benefits future readers.

[Robert H · Accepted solution]

Hello @timnardoni ,

You're using the "next()" method twice per loop. That's why every second user is skipped.

Please remove the second one, so that it looks like this:

var users = new GlideRecord('sys_user');
users.addQuery('active', true); //Active users only
users.query();
    
while (users.next()) {
    gs.info(users.user_name);
}

Regards,

Robert 

[timnardoni]

Perfect, thank you, @Robert H and @Nishant8! 

@Ankur Bawiskar, that row count trick is a good; I will use it again! 😁