Set field required based on value of two other fields.

[Fabian Duft]

Hi,

I just started using Scriptrunner and already have a problem I can't solve for some while now.

I have 3 fields. A, B and C. 
If A or B have the value "Yes" then C should be required. Else it should be optional.

I tried the following code in a field behavior and it works for one field (the one I put the code into) but not for both at the same time.

def c = getFieldByName("C")
def a = getFieldByName("A")
def b = getFieldByName("B")

def aOption = a.getValue() as String
def bOption = b.getValue() as String

def isA = aOption == "Yes"
def isB = bOption == "Yes"

if (isA || isB){
c.setRequired(true)
} else {
c.setRequired(false)
}

Can somebody tell me how I can make this work for both fields at the same time?

Thank you.

[Nic Brough -Adaptavist- · Accepted solution]

The words in your code imply that you've made a fundamentally incorrect assumption.

I'm looking at:

def aOption = a.getValue() as String

Your use of the word "option" in there suggests that field a is a select-list of some sort (might also be multi-select, radio button, check box etc).  This type of field's data is "options".

Not strings. 

When you cast an option to a String, Java can convert it, but the string representation of an object is often the entire object, not just one of the attributes.

What you'll probably need to do is get the attribute of the option that you really want to work with.  That is going to be the name of the option.

So, try

def aOptionValue = a.getValue()

def aOption = aOptionValue.getValue().getName()

Note, if your field is a multi-select, the getValue will return an array of options, not a single one.  In some places, a select list can also return an array with a single option in it,

[Fabian Duft]

Hi Nic,

Thank you for fast response.
I tried your solution as well but I get the following error: Cannot find matching method java.lang.Object#getValue()

What I still don't understand is the following:
My code works for just a or just b and only in the corresponding field.

So putting the code into a and then selection a = yes works. (makes c required)
But putting the code into b and then selecting b = yes does not work.

[Nic Brough -Adaptavist-]

Ah, sorry, I've got the new line utterly wrong.

The first line should simply get the object from the field value.  The second line should get the name of the option that should be in there.  I'd double pasted the getValue.  I am really sorry to have wasted your time!

def aOptionValue = a.getValue()

def aOption = aOptionValue.getName()

[Fabian Duft]

Hi Nic you are definetely not wasting my time, I appreciate every help.
I tried this yesterday (and now) but got the same error:

"Cannot find matching method java.lang.Object#getName()"

I am not sure if this is really neccessary, because a tutorial from Scriptrunner has almost the same setup as mine, but just needs one field. (see: https://scriptrunner.adaptavist.com/5.3.0/jira/recipes/behaviours/select-list-other.html)

I checked what happens in the console and do believe the problem is that I somehow can't use the value of b in the behaviour field of a and vice versa. 

Everything works fine in the upper one, but not in the one below. (The action does not get triggered for customfield_10343 = c)

 

[Ram Kumar Aravindakshan _Adaptavist_]

Hi Fabian,

Thank you for your patience.

After reviewing your code, the approach that you are using will not work.

Instead of using the || i.e. or condition to differentiate between the selected list options, it would be better to create 2 separate Server Side behaviour i.e. one for List A and one for List B.

Below are two sample Server Side Behaviour codes for your reference:-

1. For List A

def listA = getFieldById(fieldChanged)
def listB = getFieldByName("List B")
def textC = getFieldByName("Text C")

def listAValue = listA.value.toString()
def listBValue = listB.value.toString()

textC.setRequired(false)

if(listAValue == "Yes" && listBValue == "Yes") {
    textC.setRequired(true)
}

2. For List B

def listB = getFieldById(fieldChanged)
def listA = getFieldByName("List A")
def textC = getFieldByName("Text C")

def listAValue = listA.value.toString()
def listBValue = listB.value.toString()

textC.setRequired(false)

if(listBValue == "Yes" && listAValue == "Yes") {
    textC.setRequired(true)
} 

Below is a print screen of the Behaviour configuration:-

Hope this helps solve your question :)

[Fabian Duft]

Hi Ram,

your proposed solution works :)

Thank you

[Mathis Hellensberg]

Hi @Fabian Duft 

What kind of field are A, B and C? Assuming they are of type Select List this should work :)

def aField = getFieldByName("A")
def bField = getFieldByName("B")
def textField = getFieldByName("textField")

def aFieldValue = aField.getValue()
def bFieldValue = bField.getValue()

if (aFieldValue.equals("Yes") || bFieldValue.equals("Yes")) {
   textField.setRequired(true)
} else {
   textField.setRequired(false)
}

[Fabian Duft]

Hi @Mathis Hellensberg

Yes sorry those are Select List (single choice) fields. :)
I tried your solution but still this does not work.

Maybe I am doing something fundamentally wrong by placing the code in the wrong spot.

I put it in a Behaviour inside the aField.

Could this be the problem?