Default value for useComputedState is undefined but treated as true value.

[sculpt0r]

Type of report

Bug

Provide detailed reproduction steps (if any)

Documentation says that default value for useComputedState is true. However, there is no place in the code that assigns true to this config option.

This option is used in justify plugin, which interprets the undefined value as true:

ckeditor4/plugins/justify/plugin.js

Line 12 in 728e192

useComputedState = useComputedState === undefined || useComputedState;

This option is used in bidi plugin, which interprets the undefined value as true:

ckeditor4/plugins/bidi/plugin.js

Line 18 in 728e192

useComputedState = useComputedState === undefined || useComputedState;

This option is used in list plugin, which interprets the undefined value as true:

ckeditor4/plugins/list/plugin.js

Line 367 in 728e192

useComputedState = useComputedState === undefined || useComputedState;

Please note, that there are few other lines in those plugins, that use this 'trick'.

Code logic makes the value of config.useComputedState interpreted the same as the documentation says: as true value. However, If the user tries to use config.useComputedState, he will get undefined instead of true.

Expected result

What is the expected result of the above steps?
Plain config.useComputedState config option is equal to true.

Actual result

What is the actual result of the above steps?

Plain config.useComputedState config option is equal to undefined and interpreted as true.

Other details

• Browser: -
• OS: -
• CKEditor version: 4.16.2 (it works like that since the option was introduced)
• Installed CKEditor plugins: -

[CKEditorBot]

Closed in #4940