nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

Scriptrunner scripted field

Reddy_fnd6qb

Hello,

I have a requirement where a custom field displays the sum of other 4 fields values. However if any of those four are blank/null then the total should remain blank/null but instead the field is still displaying the value of other fields which were not null. Below is the groovy script, kindly help.

import com.atlassian.jira.ComponentAccessor
import com.atlassian.jira.component.ComponentAccessor
import com.atlassian.jira.ComponentManager
import com.atlassian.jira.issue.CustomFieldManager
import com.atlassian.jira.issue.fields.CustomField
import com.atlassian.jira.issue.IssueManager
import com.atlassian.jira.issue.Issue
import com.atlassian.jira.event.type.EventDispatchOption
import com.atlassian.jira.issue.util.DefaultIssueChangeHolder;
import com.atlassian.jira.issue.ModifiedValue;
//evaluate(new File("/path/to/your/script/root/HelloWorld.groovy")) // open and evaluate file (change the file path to your helper class)
def score9Field = 0
def score10Field = 0
def score11Field = 0
def score12Field = 0

IssueManager im = ComponentAccessor.getIssueManager()
CustomFieldManager cfm = ComponentAccessor.getCustomFieldManager()

//MJE 04-13-2020These need to stay null if null

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Points"))!=null)
{ score9Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Points"))} else {score9Field = null }

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Points"))!=null)
{ score10Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Points"))} else {score10Field = null }

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Auto Points"))!=null)
{ score11Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Auto Points"))} else {score11Field = null }

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Auto Points"))!=null)
{ score12Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Auto Points"))} else {score12Field = null }

double s9
if(score9Field!=null) { s9 = score9Field}
double s10
if(score10Field!=null) { s10 = score10Field}
double s11
if(score11Field!=null) { s11 = score11Field}
double s12
if(score12Field!=null) { s12 = score12Field}

CustomField lclscoreField = cfm.getCustomFieldObjectByName("LCL Total Points")
double lclsum = 0
if(score9Field!=null && score10Field!=null &&score11Field!=null && score12Field!=null)
{lclsum = s9 + s10 + s11 + s12 }
else
{lclsum = double.NaN}

lclscoreField.updateValue(null, issue, new ModifiedValue(issue.getCustomFieldValue(lclscoreField), lclsum), new DefaultIssueChangeHolder());
return lclsum

Thank you.

Accepted answers

All comments

Nic Brough -Adaptavist-

I would shorten this a bit, to simplify it.

The first line I would change is

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Points"))!=null)
{ score9Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Points"))}

Leave out the "else", as you can just leave the score9Field as a zero for what I'm going to suggest.

Repeat the same for the next three, no else's needed.

Then you can say

if ( (score9Field * score10Field * score11Field * score12Field) == 0 )

{lclsum = 0 }

else

{lclsum = s9 + s10 + s11 + s12 }

If that still doesn't work, at least the "System.out.println (score9Field)" type debugging lines won't have to be used in a large block of code

Reddy_fnd6qb

Hello @Nic Brough -Adaptavist-

Thank you very much for your suggestion. I honestly don't know Groovy scripting. I've changed the code as per your suggestion, but it isn't working as expected. Below is the modified script, kindly correct me if I did it wrong.

IssueManager im = ComponentAccessor.getIssueManager()
CustomFieldManager cfm = ComponentAccessor.getCustomFieldManager()

//MJE 04-13-2020These need to stay null if null //

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Points"))!=null)
{ score9Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Points"))}

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Points"))!=null)
{ score10Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Points"))}

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Auto Points"))!=null)
{ score11Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL RG Auto Points"))}

if(issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Auto Points"))!=null)
{ score12Field = (double)issue.getCustomFieldValue(cfm.getCustomFieldObjectByName("LCL Wi-Fi Auto Points"))}

CustomField lclscoreField = cfm.getCustomFieldObjectByName("LCL Total Points")
double lclsum = 0
if((score9Field * score10Field * score11Field * score12Field) == 0) {lclsum = 0 }
else
{lclsum = s9 + s10 + s11 + s12 }

lclscoreField.updateValue(null, issue, new ModifiedValue(issue.getCustomFieldValue(lclscoreField), lclsum), new DefaultIssueChangeHolder());
return lclsum

Reddy_fnd6qb

Hello,

Could someone please help me with this script?

Thank you.

PD Sheehan

How about compressing this to just a few lines like this:

import com.atlassian.jira.component.ComponentAccessor

def im = ComponentAccessor.getIssueManager()
def cfm = ComponentAccessor.getCustomFieldManager()

def fieldsToSum = ['LCL RG Points','LCL Wi-Fi Points','LCL RG Auto Points','LCL Wi-Fi Auto Points']
def fieldValues = fieldsToSum.collect{ String fieldName ->
   def cfs = cfm.getCustomFieldObjectsByName(fieldName)
   if(!cfs){
      log.warn "$fieldName was not found in custom fields"
      null
   } else {
      if ( cfs.size()>1 ) {
         log.warn "$fieldName matched ${cfs.size()} custom fields. Using the first one."
      }
      issue.getCustomFieldValue(cfs.first())
   }
}

if(fieldValues.any{it == null}){ 
   return null
} else {
   return fieldValues.sum{it.toInteger()}
}

Reddy_fnd6qb

Hello @PD Sheehan ,

Thank you very much for the response!

Attached is the screenshot of the Error when I tried to replace the above code. Kindly suggest.

Thank you. Code Error.PNG

PD Sheehan

Did you try to run it with Preview?

The "Static Type checking" error can often be ignored. This just means that the code editor cannot confirm that the "toInteger() will work.

But if that really bothers you, you can change it to

return fieldValues.sum{it as Integer}

Reddy_fnd6qb

Peter, you're right. The preview worked fine but the actual field in the jira issue isn't displaying 'null', the same issue I had with the actual code. PFA screenshot. Code Error1.PNG

Reddy_fnd6qb

Now I see the sum functionality works perfect in preview and in issue. But, I don't see 'null' value being displayed in issue screen as shown below.

Code Error6.PNG Code Error7.PNG

PD Sheehan

That's normal. a null value shows a blank/empty in forms.

If you want to show the actual word null, then you must set the text value to 'null' as opposed to the logical null value

Reddy_fnd6qb

Okay that was my initial requirement and the limitation of my code. Could you please show me where do I change the text value to 'null' in order for the field 'LCL Total Points' to display it in the jira issue screen.

PD Sheehan

def lclsum
if(fieldValues.any{it == null}){ 
   log.warn "Found at least one null value in $fieldValues"
   lclsum = 'null'
} else {
   lclsum = fieldValues.sum() as String
}

Welcome!

It looks like you're new here. Sign in or register to get started.

Quick Links

Find more posts tagged with

server

scriptrunner

jira

adaptavist

jira-server