Custom Script Post-Function [ScriptRunner]

Almu

Hi everyone,

I would appreciate if someone could shed light on this, please.

I am a groovy learner, and I need to fulfill this requirement:

There are 2 checkbox custom fields: cf1, cf2 used in several issuetypes

In Create transition, there is a postfunction that fills in cf1, depending on other custom field, so each issue has different cf1 values. 

The thing is that cf2 must be filled in with some values of cf1 that apply to that issue. That is:

when creating issue1, cf1 takes values: a,b,c,e. Then, cf2 checks a list, and takes b,c, because b,c are in that list.

when creating issue2, cf1 takes values: a,c,d. Then, cf2 checks a list, and takes c,d, because c,d are in that list.

The list depends on the issuetype, that is why I have decided to add a postfuntion in each workflow of each issuetype. This postfunction is placed after the postfunction to fill in cf1, and this is the code: 

 

import com.atlassian.jira.component.ComponentAccessor
def customFieldManager = ComponentAccessor.customFieldManager
def optionsManager = ComponentAccessor.optionsManager
def issueManager = ComponentAccessor.issueManager

def cf2 = customFieldManager.getCustomFieldObjectsByName("custom field 2")[0]
def cf1 = customFieldManager.getCustomFieldObjectsByName("custom field 1")[0]
def fieldConfig2 = cf2.getRelevantConfig(issue)
def options2 = optionsManager.getOptions(fieldConfig2)
def fieldConfig1 = cf1.getRelevantConfig(issue)
def options1 = optionsManager.getOptions(fieldConfig1)

for (x in options1) {
if (x == "b")
{
def optionsToSet = options2.findAll { it.value in ["b"] }
issue.setCustomFieldValue(cf2, optionsToSet)
}
if (x == "c")
{
def optionsToSet = options2.findAll { it.value in ["c"] }
issue.setCustomFieldValue(cf2, optionsToSet)

}
if (x == "d")
{
def optionsToSet = options2.findAll { it.value in ["d"] }
issue.setCustomFieldValue(cf2, optionsToSet)

}}

 

As I do not know how to compare options1 with a list, I have used "if" statements.

But it does not work. Any idea, please?

 

Thanks in advance.

Cheers,

Almu

 

Nic Brough -Adaptavist-

Welcome to the Atlassian Community!

You say "In Create transition, there is a postfunction that fills in cf1, depending on other custom field, [...] cf2 must be filled in with some values of cf1 that apply to that issue.".  This seems like a clear specification to me, but I just want to say that I am not 100% sure I am reading it right, so if anything in the rest of my answer seems to be looking at the wrong thing or answering the wrong question, please do tell me!

I'm going to imagine that you are aiming for:

• Post-function 1: runs a script that does something simple like "set cf1 to 'badgers'"
• Post-function 2: runs a script that looks at cf1 and if it sees 'badgers', sets cf2 to 'mushroom', and if cf1 is not 'badgers', sets cf2 to 'snake'
• cf1 and cf2 are select-lists

The exact logic of what you are reading and setting is not too important, your script shows you know what you're doing with that.  I've just constructed a simplified scenario I can refer back to

Problem 1:

I think the problem here is, as you say, the "if":

for (x in options1) {
if (x == "b")
{

The for/in part is fine, options1 will contain a list of options, and x a single option but the x == "badgers" is going to false every time.  This is because an option is an option object, not a string.

I think what you probably want to do here is compare the name of the option, not the whole thing.  For my example, this would mean

for (x in options1) {
if ( "badgers".equals( x.getName() ) )
{

Note that I've used equals through force of habit, I'm more Java than Groovy and the "use .equals. not == when comparing strings" is a hard habit to break

Problem 2:

Depending on what is run and where, cf1 may not contain 'badgers' when you come to ask whether cf2 should be 'mushroom' or 'snake'

There's a long and complicated story about the timing cycles in here, but the actual fix is to think slightly differently when coding the second script.

Instead of 

• Read cf1 and set cf2 from it

You code "defensively", assume cf1 is not set correctly, and do:

• Work out what cf1 should be set to, and set cf2 from that