Problem/Motivation

In Drupal CMS, we are getting schema errors in our tests because, although none of our config sets the weight property on fields which use the leaflet_formatter_default field formatter, it gets added anyway because it's in LeafletSettingsElementSettingsTrait::getDefaultSettings(), but is not defined in config schema.

Proposed resolution

Either remove the property from getDefaultSettings() (if it shouldn't exist), or add it to config schema.

Comments

phenaproxima created an issue. See original summary.

itamair’s picture

Thanks @phenaproxima ...
Let me have another look to this ("dear") weight property ...

itamair’s picture

Status: Needs work » Needs review
StatusFileSize
new450 bytes

@phenaproxima could you test/QA if the simple attached patch fixes this issue?

phenaproxima’s picture

Status: Needs review » Reviewed & tested by the community

Can confirm this worked when I patched Leaflet with it in the context of Drupal CMS.

  • itamair committed bf163442 on 10.2.x
    Issue #3498186 by itamair, phenaproxima:...
itamair’s picture

Status: Reviewed & tested by the community » Fixed

Thanks @phenaproxima
leaflet 10.2.33 release just deployed with this fix

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.