Optional fields for record only work with variables but not expressions

[TimonFlick]

Using optional type for records with expression instead of variable does not work.

This is working:

let setDefaultOptions = (~newOptions: options) => {
  let weekStartOn = newOptions.weekStartOn->Option.map(Weekday.toInt)
  let options: options_ = {
    weekStartOn: ?weekStartOn,
  }
  setDefaultOptions_(~newOptions=options)
}

This is not working:

let setDefaultOptions = (~newOptions: options) => {
  let options: options_ = {
    weekStartOn: ?newOptions.weekStartOn->Option.map(Weekday.toInt),
  }
  setDefaultOptions_(~newOptions=options)
}

Thank you for filing! Check list:

• Is it a bug? Usage questions should often be asked in the forum instead.
• Concise, focused, friendly issue title & description.
• A minimal, reproducible example.
• OS and browser versions, if relevant.
• Is it already fixed in master?

[cristianoc]

Have you tried putting it inside (parens) ?

[mununki]

?Option.map(newOptions.weekStartOn, Weekday.toInt),

It would work, I guess.

[vikfroberg]

Have you tried putting it inside (parens) ?

Yes we tried that. And for one, the formatter removed it. But we turned off the formatter and it still didn't work.

[cristianoc]

Would you try to install rescript@next? It might be this: https://github.com/rescript-lang/rescript-compiler/pull/5581/files

In which case, as a temporary workaround you can use the variable or avoid using ->.

[TimonFlick]

?Option.map(newOptions.weekStartOn, Weekday.toInt),

It would work, I guess.

this works yes