Well, APIs in SN 99% of the time are designed to work with "internal" identifiers, not visible/display identifiers. Furthermore of those table and field names are always lower case. This is important because JavaScript is case sensitive. Thus the statement
var sn = g_form.getValue('System');
cannot be correct. System is most likely the label of the field while its name is probably u_system - well if we consult your screenshots, it is for sure. Because all field names are lower case and all custom fields in the global scope have the prefix u_.
@Prasad Pagar showed you above how to get the "internal" name (cmdb_ci) of a field by right-clicking the field's label (Configuration item). Notice that it is entirely possible to set any label to a field, with absolutely no connection between the two.
Thus the correct statement would be:
var sn = g_form.getValue('u_system');
However this is not necessary as the value is readily available in one of the function's parameters - newValue, this Client Script being set to fire when u_system (System) changes.
As mentioned before, 99% of APIs work with internal values, so don't send the system's name with the Ajax request. The only time and place to use (so called) display values, is when showing something to the user. Other than that you should use internal names and values. In your case however the code and naming is misleading, as g_form.getValue() will get you an internal value (the sys_id or the unique value) and not the name - I'm talking about sysparm_system_name. I correct naming would be sysparm_system_id, sysparm_system_sys_id or sysparm_system_unique_value.
Another problem with the Client Script is that the answer received from the server (response) is nowhere written into variable answer. You should convert the string response into the array answer either with the help of JSON or String.prototype.split().
As for the Script Include, you try to refer to it with one name in the Client Script (GetSysName) but you define it with another name (GetRequestedByData). It should be the same name. Also I bet the field name is u_owner (also judging by your screen shots) and not owner. You should also control how the array you are building is serialized (turned into a string). You should use either JSON.stringify or .join(',').
Note that the statement
gn.push(rec.owner);
is not adding the value in field owner (which should be u_owner) to the array, but an object of type GlideElement. To make sure you are adding the value in that field, you need to extract the value from that object and one does that by coercing the object into a string, either by concatenating it with an empty string or calling the .toString() method of that object:
gn.push('' + rec.owner);
// or
gn.push(rec.owner + '');
// or
gn.push(rec.owner.toString());
// or
gn.push(String(rec.owner));
Of course in a final working code using the correct field name - u_owner.
As a general note, for optimization purposes and to obtain a better user experience, as I noted in the other article, you should be sending back to the browser, not only the sys_id of the Owner and the Analyst, but the display values also (the names).
I suppose the corrected final solution would be something like:
- the Script Include:
GetSysName.prototype = Object.extendsObject(AbstractAjaxProcessor, {
getName: function () {
var gn = [];
var sn = this.getParameter('sysparm_system_unique_value');
var rec = new GlideRecord('cmdb_application_product_model');
if (rec.get (sn)) {
gn.push('' + rec.u_owner);
gn.push('' + rec.u_analyst);
}
return gn.join(',');
},
type: 'GetSysName'
});
- the Client Script:
function onChange(control, oldValue, newValue, isLoading, isTemplate) {
if (isLoading || newValue === '') {
return;
}
var ga = new GlideAjax('GetSysName'); // Script Include Name
ga.addParam('sysparm_name', 'getName'); // SI Method
ga.addParam('sysparm_system_unique_value', newValue); // Parameter for SI
ga.getXMLAnswer(function (response) {
alert('response is ' + response);
var answer = response.split(',');
g_form.setValue('u_owner', answer[0]);
g_form.setValue('u_analyst', answer[1]);
});
}
I'm not sure which screen-shot in the end contains the correct field names, so do check those to be correct.
Another more optimized solution, which "saves" two asynchronous calls back to the server (after you set Owner and Analyst) to fetch the names (display values):
- the Script Include (notice I'm declaring gn as an object):
GetSysName.prototype = Object.extendsObject(AbstractAjaxProcessor, {
getName: function () {
var gn = {};
var sn = this.getParameter('sysparm_system_unique_value');
var rec = new GlideRecord('cmdb_application_product_model');
if (rec.get(sn)) {
gn.owner = {
'uniqueValue': '' + rec.u_owner,
'displayValue': rec.u_owner.getDisplayValue(),
};
gn.analyst = {
'uniqueValue': '' + rec.u_analyst,
'displayValue': rec.u_analyst.getDisplayValue(),
};
}
return JSON.stringify(gn);
},
type: 'GetSysName'
});
- the Client Script:
function onChange(control, oldValue, newValue, isLoading, isTemplate) {
if (isLoading || newValue === '') {
return;
}
var ga = new GlideAjax('GetSysName'); // Script Include Name
ga.addParam('sysparm_name', 'getName'); // SI Method
ga.addParam('sysparm_system_unique_value', newValue); // Parameter for SI
ga.getXMLAnswer(function (response) {
alert('response is ' + response);
var answer = JSON.parse(response);
g_form.setValue('u_owner', answer.owner.uniqueValue, answer.owner.displayValue);
g_form.setValue('u_analyst', answer.analyst.uniqueValue, answer.analyst.displayValue);
});
}
Again do check and correct field names where necessary.
