Migration to_registry fails if coming from collective.ckeditor < 4.10.0

[gbastien]

Hi @gotcha

Property "disallowedContent" does not exist and locally, to properties "bodyClass" and "width" were set to None instead an empty string.

Suggested changes:

    # if coming from collective.ckeditor < 4.10.0, disallowedContent does not exist
    if hasattr(props, "disallowedContent"):
        api.portal.set_registry_record(
            "collective.ckeditor.browser.ckeditorsettings.ICKEditorSchema.disallowedContent",
            safe_unicode(props.disallowedContent)
        )
    # do not fail if bodyClass is None
    api.portal.set_registry_record(
        "collective.ckeditor.browser.ckeditorsettings.ICKEditorSchema.bodyClass",
        props.bodyClass or ''
    )
    # do not fail if width is None
    api.portal.set_registry_record(
        "collective.ckeditor.browser.ckeditorsettings.ICKEditorSchema.width",
        props.width or ''
    )

Thank you,
Gauthier

[gotcha]

The failure to migrate disallowedContent is due to bad setup of the property to registry upgrade step.
It should be from 4350 instead of from *.

[gotcha]

What do you mean by "locally, some properties were set to None" ?