Script include to check if a record that meets condition exists in a table

[JJG]

Hello,

I have an OnLoad client script that uses a script include to check if a table has a record that meets specific conditions. if there is a record the client script should do nothing. if there is no record it should set the pay_period_status field to "Not Submitted". My code is not working, is there an error?

OnLoad client script:

function onLoad() {
var payPeriod = g_form.this_pay_period;
var nameOfEmployee = g_form.name_of_employee;

var gaMatchingRecord = new GlideAjax('timeManagement');
gaMatchingRecord .addParam('sysparm_name', 'check_payAndName');
gaMatchingRecord .addParam('sysparm_payPeriod', payPeriod);
gaMatchingRecord .addParam('sysparm_nameOfEmployee', nameOfEmployee);
gaMatchingRecord .getXMLAnswer(_handleResponse);

function _handleResponse(response) {
var answer = response;
     if (answer == true) {
     return;
     }

     else if(answer == false){
     g_form.pay_period_status = "Not Submitted";
     g_form.save();;
}}}

Script include:

var timeManagement = Class.create();
timeManagement.prototype = Object.extendsObject(global.AbstractAjaxProcessor, {

check_payAndName: function() {
     var grPayPeriod = new GlideRecord('x_utsll_time_manag_pay_period_submit');
     if (grPayPeriod.get(this.getParameter('sysparm_payPeriod')&& this.getParameter('sysparm_nameOfEmployee'))) {
          return true;
     }
     else{
          return false;
     }
},

type: 'timeManagement'
});

[Tanaji Patil]

Why are you using a client script for this?

You should use a display business rule in this case.

According to ServiceNow best practise you should put your logic as much as possible on server side otherwise your browser becomes heavy and takes lots of time to load forms full.

-Tanaji
Please mark response correct/helpful if applicable

[Tanaji Patil]

Here is how your display BR code should look like-

var grPayPeriod = new GlideRecord('x_utsll_time_manag_pay_period_submit');
grPayPeriod.addQuery("pay_period", current.this_pay_period);
grPayPeriod.addQuery("name_of_employee", current.name_of_employee);
grPayPeriod.query();
if(!grPayPeriod.hasNext()) {
current.pay_period_status = "Not Submitted";
}

Please replace the bold field names with your actual field names.

-Tanaji

Please mark response correct/helpful if applicable.

[Michael Ritchie]

@JJG I cannot agree more with @Tanaji Patil that a display business rule is a much better way to go.  While GlideAjax can work, it is more "expensive" on the user performance than a display business rule.  Plus it is less to maintain.

[Prateek kumar]

Try below:

function onLoad() {
var payPeriod = g_form.getValue('this_pay_period');
var nameOfEmployee = g_form.getValue('name_of_employee');

var gaMatchingRecord = new GlideAjax('timeManagement');
gaMatchingRecord .addParam('sysparm_name', 'check_payAndName');
gaMatchingRecord .addParam('sysparm_payPeriod', payPeriod);
gaMatchingRecord .addParam('sysparm_nameOfEmployee', nameOfEmployee);
gaMatchingRecord .getXMLAnswer(_handleResponse);

function _handleResponse(response) {
var answer = response;
     if (answer == true) {
     return;
     }

     else if(answer == false){
     g_form.setValue('pay_period_status',"Not Submitted");
     g_form.save();;
}}}