Script Include Instantiation in Business Rule Returns Error "[object Object] is not a function."

[Corey Farmer]

Hello,

I have a Business Rule created on a custom table configured to trigger off of two conditions:

  1. Field 'u_status' is 'Refresh'

or 

  2. Field 'u_last_webhook_dt' changes

Condition 1 is triggered from a UI Action button labeled 'Refresh', while Condition 2 is triggered from a Scripted REST API script.

When triggering using the 'Refresh' UI Action the Business Rule errors when attempting to instantiate my global script include; however, when triggering from the Scripted REST API or from a background script the Business Rule instantiates the object successfully and completes without error.  

Ultimately, my question is: why am I getting the `[object Object] is not a function` error when instantiating the Script Include through the UI Action, but not getting the error when using the Scripted REST API or background script?

Things I have verified:

• All scripts are in `Global` scope
• 'Isolate Script' is turned off for all relevant scripts
• Verified the user clicking the UI Action has all necessary roles 

The Business Rule code and error thrown are shown below:

try {
    var DealHubCPQHelper	= new global.DealHubCPQHelper( ),     // <-- ERROR THROWN HERE
	quotedetails			= DealHubCPQHelper.getDealHubQuoteDetails( current ),
	nil;

} catch ( err ) {
	err.func = "Business Rule: DealHub API Get Quote Details";
	gs.error( [ err.message, JSON.stringify( err ) ].join( ), "DealHubCPQ" );
}

Error Message:

[object Object] is not a function.,
{"message":"[object Object] is not a function.","fileName":"sys_script.609271651b0d42509c9b0e12604bcb93.script","sourceName":"sys_script.609271651b0d42509c9b0e12604bcb93.script","lineNumber":9,"name":"TypeError","stack":"\tat sys_script.609271651b0d42509c9b0e12604bcb93.script:9 (executeRule)\n\tat sys_script.609271651b0d42509c9b0e12604bcb93.script:40\n\tat sys_script_include.00087ee91b74c61062638556cc4bcbc4.script:333 (cleanseGlideRecord)\n\tat sys_script_include.00087ee91b74c61062638556cc4bcbc4.script:307\n\tat sys_ui_action.c3b9e62d1bcd42509c9b0e12604bcb75.script:24\n","rhinoException":{},"func":"Business Rule: DealHub API Get Quote Details"}

 So, starting from the bottom of the stack, we see the 'sys_ui_action' execute, followed by the script call within the UI Action to 'cleanseGlideRecord' which completes successfully and provides the record `.update()` function which triggers the Business Rule 'DealHub API Get Quote Details' to execute.

The UI Action script is as follows:

function getQuoteDetails ( ) {
	var answer 	= confirm( "Are you sure you want to refresh details using the DealHub API Engine?\n\nThis will clear, retrieve, and parse new quote data from DealHub for the selected record(s)."),
		nil;

	// Interact with the form button using g_form to allow use of confirm dialog
	if ( answer == true )
		gsftSubmit( null, g_form.getFormElement( ), 'dealhub_get_detail' );
	else 
		return false;
}

try { 
	if ( typeof window == "undefined" ) {

		current.setValue( "u_status", 					"Refresh" );
		
		var DealHubCPQHelper	= new global.DealHubCPQHelper( ),
			GrDealHubCPQQuote	= DealHubCPQHelper.cleanseQuoteDetails( null, current ),
			nil; 	

		action.setRedirectURL( current );		
	}
} catch ( err ) {
	err.func = "UI Action.Refresh";
	gs.error( JSON.stringify( [ err, err.message ].join( ) ), "DealHubCPQ" );
}

[SanjivMeher]

I dont know if it is a silly question or you already have it corrected, but why do you have comma after each line instead of ;

try {
    var DealHubCPQHelper	= new global.DealHubCPQHelper( ),     // <-- ERROR THROWN HERE
	quotedetails			= DealHubCPQHelper.getDealHubQuoteDetails( current ),
	nil;

[Corey Farmer]

Hi, not a silly question - just programming preference I suppose.  The `var` for variable instantiation works by chaining as well, so it doesn't need to be called for each variable if you don't want to (Javascript trick).

var myVar1 = "value1";
var myVar2 = "value2";

// Is the same as:

var myVar1 = "value1", myVar2 = "value2";

// Is the same as:

var myVar1  = "value1",
    myVar2  = "value2";

Arguably, there may be performance gains from not calling `var` so often; but, I've never done the analysis and it's likely too minute to impact execution time at all - so, "just programming preference I suppose."

[Neel Patel]

Hi Corey,

Is DealHubCPQHelper a script include? 

[Corey Farmer]

Hi @Neel Patel ,

Thank you for your contribution!

Yes, `DealHubCPQHelper` is a Script Include within the Global scope (the Scripted REST API, Business Rule, and UI Action are also within Global), with Accessibility set to "All application scopes", and Protection Policy is "None"

[Neel Patel]

Just a thought, can you try this: (Invoking function with the SC object)

var my_sc = DealHubCPQHelper.getDealHubQuoteDetails(param);