Joseph,
The for...in syntax is not optimal, since GlideRecord objects can have methods and properties that aren't accessible that way. I've written up a couple of options for you try, which should get the query encoded in the way you expect.
The first block should work if your action is part of the global scope.
//not in scoped app - use this section if you are in global scope
var fieldsToIgnore = ["sys_id", "u_rescheduled_entry", "schedule", "u_change_request", "start_date_time", "end_date_time"];
var recForQuery = new GlideRecord(current.getTableName());
var au = new ArrayUtil();
var name, elem, list = current.getFields();
for (var i = 0; i < list.size(); ++i) {
elem = list.get(i);
name = elem.getName();
if(name && !au.contains(fieldsToIgnore, name))
if(current.getValue(name))
recForQuery.addQuery(name,current.getValue(name));
}
action.setRedirectURL('/cmn_schedule_span.do?sys_id=-1&sysparm_query=' + recForQuery.getEncodedQuery());
If your action is part of a scoped app, the script is a little bit different:
//in scoped app - use this section if you are in a scoped app
var fieldsToIgnore = ["sys_id", "u_rescheduled_entry", "schedule", "u_change_request", "start_date_time", "end_date_time"];
var recForQuery = new GlideRecord(current.getTableName());
var au = new global.ArrayUtil();
var name, elem, list = current.getElements();
for (var i = 0; i < list.length; i++) {
elem = list[i];
try {
name = elem.getName();
if(name && !au.contains(fieldsToIgnore, name))
if(current.getValue(name))
recForQuery.addQuery(name,current.getValue(name));
} catch (e) {
//sometimes the element can't be accessed in scoped app
//skip it
}
}
action.setRedirectURL('/cmn_schedule_span.do?sys_id=-1&sysparm_query=' + recForQuery.getEncodedQuery());
You'll see that we build the query using a GlideRecord object- this allows us to take advantage of the getEncodedQuery() method, since the standard URL-params don't pre-fill the fields as you expect them to.
You will want to go through the actual fields that typically get submitted and add more to the list of exclusions- or pick-and-choose the specific fields you want, and reverse the 'contains' condition, thereby limiting the pre-filled fields to ones you specifically choose.
The scoped version can't use getFields, which actually opens it up to errors when the particular GlideElement can't be accessed in JavaScript. This happens infrequently, but will throw an error. I've wrapped a try/catch around it, which isn't ideal. However, unless this UI Action is being used hundreds of times a minute, this should work fine.
