
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-21-2019 02:31 PM
i have this below piece of code in a business rule on sc_task
business rule conditions are as below
the business rule intends to copy work notes of one task into all adjacent tasks of the same request item.
the code however doesn't work and i do not get any error. when i printed different messages, it seems all the code works fine except for the part where i am trying to update the work notes. i de-activated all other business rules on task and sc_task and it still doesn't work.
the execution reaches till line 16.
Solved! Go to Solution.
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-22-2019 12:45 PM
Hi Ravish,
I had a similar requirement to copy customer comments from a sc_task to sc_req_item. I had to clean up the comments and compare in order to prevent an infinite loop. I've modified the code below which should work for your use case:
(function executeRule(current, previous /*null when async*/) {
var taskSysId = current.getUniqueValue();
//gs.info(gs.getMessage("BLS: taskSysIdk: {0}",[taskSysId]));
var trgt = new GlideRecord("sc_task");
trgt.addQuery("parent",current.parent);
trgt.addQuery("sys_id","!=",taskSysId);
trgt.addActiveQuery();
trgt.query();
while (trgt.next()) {
gs.info("BLS: trgt: " + trgt.number);
// Before we update the target task, we need to prevent an infinite loop by ensuring the work_notes that is causing this business rule to run is not the same text that was in the last work_notes on the target task.
//return the last work_notes journal entry for the current task and extract the journal text below the line break.
var workNotes = cleanWorkNotes(current.work_notes.getJournalEntry(1));
//return the last work_notes journal entry for the Catalog Task and extract the journal text below the line break.
var trgtWorkNotes = cleanWorkNotes(trgt.work_notes.getJournalEntry(1));
gs.info(gs.getMessage("BLS: Current Task: {0} >>> Current Task Work Notes: {1} - Target Task: {2} >>> trgtWorkNotes: {3} indexOf: {4}",[current.number,workNotes,trgt.number, trgtWorkNotes,trgtWorkNotes.indexOf(workNotes)]));
/***** NOTE: If the same work_notes are entered twice, the second journal entry will not be copied to the target task as it will match the first work_notes *****/
if (trgtWorkNotes.indexOf(workNotes) < 0) {
// OnAfter business rules cannot process comments so we need to pull from the journal
trgt.work_notes = workNotes;
trgt.update();
}
}
/**
* Description: Cleans up the journal entry to remove timestamp, audit information and trim any white space.
* Parameters:
* @PARM1 - journalEntry (String) - Journal entry by index number i.e. current.work_notes.getJournalEntry(1)
* Returns: note (String) - cleaned up journal entry with just the work_notes information.
*/
function cleanWorkNotes(journalEntry){
var note = "";
if (journalEntry) {
//We need to clean-up the journal entry
//return all text after the first line break in the journal entry (first line contains timestamp and audit information).
note = journalEntry.substr(journalEntry.indexOf('\n')+1);
note = note.trim(note); //remove any whitespace before and after string
}
return note;
}
})(current, previous);
Let me know if it worked for you.
Brent
P.S. If my suggestion helped then please mark as helpful and/or correct so other community members can benefit from this information.

- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-21-2019 03:27 PM
I tried writing a code similar to yours in my PDI and it seems to be working for me. Here's a screenshot-
On all tasks other than the task just updated, I see my work note-
Try doing a <your-sn-url>/cache.do on your instance and see if this helps.
P.S. I notice one difference- I don't think line #1 of your code is correct because when you create a BR, an IIFE as under is created.

- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-21-2019 03:39 PM
Yea, please pay attention to what this person just mentioned about your beginning and ending line. I think that's the problem. You need to replace that onAfter with what is generated there by the system (like....make a new BR and see what's already there before you go pasting in stuff).
Please mark reply as Helpful/Correct, if applicable. Thanks!
Please consider marking my reply as Helpful and/or Accept Solution, if applicable. Thanks!
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-21-2019 04:46 PM
Yeah, I agree with Priyanka and Allen. It should be an IIFE and start with code like the following:
(function executeRule(current, previous /*null when async*/) {
// Add your code here
})(current, previous);
Brent

- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-22-2019 08:06 AM

- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
01-22-2019 09:00 AM
Not sure where the encoded query info message is coming from so perhaps you have some different script that what we see here, but either way, it appears the while loop is working and outputting the different tasks associated.
So if the work notes aren't being set, try removing all of your statement there and just do
gr.work_notes = "Test";
Is that even working?
If that works...then it's something to do with your current line of information that's there.
Are you getting anything in your syslog?
Unfortunately, you aren't really giving us anything to go off of, but saying didn't work.
Thanks
Please consider marking my reply as Helpful and/or Accept Solution, if applicable. Thanks!