IntlNumberConverter does not respect "signDisplay" property

[psommer-hri]

Hello Oracle JET Team,

I have the following issue with Jet Version 9.2.0:

Current Behaviour:
The oj.IntlNumberConverter does not respect the "signDisplay" property when it is specified in its' options. E.g. when "signDisplay" is set to "always", the sign is not always displayed.

Expected Behaviour:
According to ECMA-402, same formating as with Intl.NumberFormat.

Setps to Reproduce:

1. Open the Converter Demo in the Cookbook.
2. Add signDisplay: 'always' to the options and apply changes. Note, that no sign (i.e. "+") is displayed.
3. Change this.converter from new NumberConverter.IntlNumberConverter(options) to new Intl.NumberFormat('en', options) and apply changes. Note, that the sign is now displayed.

Kind Regards,
Philip

[peppertech]

Thanks for the heads up Philip, I'll get it written up internally.

…

On Tue, Oct 27, 2020 at 6:48 AM psommer-hri ***@***.***> wrote: Hello Oracle JET Team, I have the following issue with Jet Version 9.2.0: *Current Behaviour:* The oj.IntlNumberConverter <https://www.oracle.com/webfolder/technetwork/jet/jsdocs/oj.IntlNumberConverter.html> does not respect the "signDisplay" property when it is specified in its' options. E.g. when "signDisplay" is set to "always", the sign is not always displayed. *Expected Behaviour:* According to ECMA-402 <https://tc39.es/ecma402/#conformance>, same formating as with Intl.NumberFormat <https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat/NumberFormat> . *Setps to Reproduce:* 1. Open the Converter Demo in the Cookbook <https://www.oracle.com/webfolder/technetwork/jet/jetCookbook.html?component=textInput&demo=converter> . 2. Add signDisplay: 'always' to the options and apply changes. Note, that no sign (i.e. "+") is displayed. 3. Change this.converter from new NumberConverter.IntlNumberConverter(options) to new Intl.NumberFormat('en', options) and apply changes. Note, that the sign is now displayed. Kind Regards, Philip — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#70>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJR2QRBAXI2V6BXL6OIG7DSM3FRHANCNFSM4TA3IXPA> .