Argument #3 ($currentTranslationOnly) must be of type bool, null given

Here is a patch that works around the problem. The requested value is still 'stored' as NULL for some reason, but this fallback substitutes the default FALSE in that case.

Whichever way a person selects the option to do it only for "selected translations" will be ignored, and it will be done for all translations of the selected content.

This is good enough for me, and maybe there should be different actions for "all translations" and "selected translations" to remove the annoying and not working anyway second selection page.

Thank you!

Thanks but what i did was definitely a workaround!! Not a fix! Also was coming back to post a patch that covers publish and unpublish. Still a workaround, does not address the configuration not being saved properly in intermediate step for use in final step.

This line:

$config = \Drupal::config('moderated_content_bulk_publish.settings');

would seem to be what's wrong?

The setting was done on the bulk operations form, not from the settings form (which does have a related setting that would be the appropriate fallback i guess, not my 'false', of current_translation_only_default_value).

Again the easiest solution i can think of that might also improve user experience is to have unpublish_all_translations and unpublish_selected_translations as separate options.

ok I'll revert it as soon as the git repo comes back.

@mlncn, if you figure this issue out, please let us know.

Seems like it must be related to configuration options and translation settings? Otherwise others would likely have raised an issue sooner?