Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting `image: false` in pages/files field within structure field throws error #1891

Closed
texnixe opened this issue Jul 1, 2019 · 1 comment

Comments

2 participants
@texnixe
Copy link

commented Jul 1, 2019

Describe the bug
When setting the image property to false in a files or pages field within a structure field, the Panel throws the error

Invalid value for "image"

Example blueprint field definition:

    audio:
        label: Audiobeispiele
        type: structure
        fields:
            titel:
                label: Titel
                type: text
                width: 1/3
            desc:
                label: Beschreibung
                type: text
                width: 1/3
            audio:
                label: Audio-Datei
                type: pages
                image: false
                query: page.children
                multiple: false
                width: 1/3

To Reproduce
Steps to reproduce the behavior:

  1. Create a blueprint with the above field definition
  2. Create/open a page with that field definition and try to add a structure item
  3. See error

Expected behavior
The image property should work inside a structure field context the same way as outside of this context.

Kirby Version
3.2.0

Additional context
If the image property is not set, the Panel uses the file url as image source no matter what file type is used, which doesn't make sense. I'll create a new issue for that.

@bastianallgeier bastianallgeier added this to the 3.2.1 milestone Jul 1, 2019

bastianallgeier added a commit that referenced this issue Jul 1, 2019

@bastianallgeier

This comment has been minimized.

Copy link
Contributor

commented Jul 1, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.