At line 16 and 97, the schema.yml-file for the module has entries like this one:

    export:
      type: mapping
      label: 'Export'
      mapping:
        folder:
          type: string
          label: 'Folder'

I suggest removing them, as they seem not connected to the options available on the admin page, and they are also included twice, which obviously is wrong.

CommentFileSizeAuthor
#3 eu-cookie-compliance-mapping-folder-2900105-3.patch876 bytessvenryen

Comments

svenryen created an issue. See original summary.

svenryen’s picture

svenryen commented
Issue summary: View changes
svenryen’s picture

svenryen commented
StatusFileSize
new eu-cookie-compliance-mapping-folder-2900105-3.patch876 bytes

The attached patch fixes the issue.

svenryen’s picture

svenryen commented
Status: Active » Needs review
killua99’s picture

killua99 commented
Status: Needs review » Needs work

I've no clue that it does. We should have a hook_update removing them from DB as well.

svenryen’s picture

svenryen commented

Does it really matter a lot if these short strings made it into somebody's database? The module is still in beta, so by definition, those who installed the module should be prepared to wipe their data and reinstall the module if necessary. I think we'll be just fine just removing these strings from the .yml without any update hook to remove the database entries. I would agree with you that we need a hook if the module was in RC or final.

  • svenryen committed f3b0d03 on 8.x-1.x 
    Issue #2900105 by svenryen: A mapping for string labelled 'Folder' is...
svenryen’s picture

svenryen commented
Status: Needs work » Fixed
svenryen’s picture

svenryen commented
Status: Fixed » Closed (fixed)