Structure field error: Cannot read property 'length' of undefined

Kirby 3 Issues 🐛
1

Hi,

we are about to do the final transition from K2 to K3 (3.3.0) with our project. I just converted all content files with the upgrade script at https://getkirby.com/docs/cookbook/setup/migrate-site#folder-and-file-structure. Most of the content seems to work fine.

However, we use structure fields as participants lists. When I try to open the migrated participants lists in the panel, instead of the structure field it shows a red error box with the message “Cannot read property ‘length’ of undefined”.

This occurs with most of the participants lists but not all (maybe not the more recent ones or the ones with just one or two entries?).

This is what I get in the console:

image
image1250×303 55.5 KB

Can this be related to some broken formatting or invalid characters in the content TXT file? Any hints what to look for? (As I cannot share the original files with names and addresses)

3

Will be fix in 3.3.1:

4

Thanks for your swift reply!

When will 3.3.1 be released?

5

Not sure, but i guess pretty soon. . Does the error go away if you remove sortby?

6

@texnixe yes, this “fixed” it. Do you mean “pretty soon” as in “within the next 7 days” or “within the next 4 weeks”?

7

I don’t know, I would have expected the next rc next Tuesday, but won’t make any guesses because of bt conf.

8

I copied the file edited to fix a bug that will released in 3.3.1 :sweat_smile: I’ll just update when it’s officially released. If you really needed maybe you can check the files edited?

9

When we talked to @bastianallgeier , he said we could take it next week.

1 Like
10

As soon as I add sortBy to the structure field type I get the error “Cannot read property ‘length’ of undefined”. Is this still a bug in 3.3.1?

11

@johan Could you please post your blueprint?

12

Sorry my blueprint is very rough, testing at the moment.

          style:
            label: Style
            sortBy: option desc
            translate: false
            type: structure
            fields:
              option:
                label: Option
                options:
                  alignment: Alignment
                  test: Test
                type: select
              alignment:
                default: align-center
                label: Alignment
                options:
                  align-center: Center
                  align-left: Left
                  align-right: Right
                required: true
                type: select
                when:
                  option: alignment
              test:
                default: align-center
                label: Alignment
                options:
                  align-center: Center
                  align-left: Left
                  align-right: Right
                required: true
                type: select
                when:
                  option: test

56
561754×368 18.7 KB

When I add sortBy: option desc:

57
571766×182 4.69 KB

13

Just now when I posted the blueprint I realized how stupid it is :slight_smile: I’m still using the structure field but removed the option field, and just have (max: 1) one line of input. Won’t need the sortBy at the end. Sorry! But still, maybe this is a bug?

14

Hm, I just added the above field definition in a fresh 3.3.1 Starterkit without any issues.

15

Hm that’s weird :thinking:

16

Which browser and OS are you using?

17

Mac, Mojave, Chrome version 78.

18

A mystery, I’m on Mac OS Mojave myself, checked Chrome, no error.