backdrop-contrib/bee

Importing field config with file names too long: Warning - Trying to access array offset on value of type bool in bee_get_remaining_rows()

herbdool opened this issue · 3 comments

herbdool commented

I'm seeing this when importing config - it doesn't seem to impact the actual command. It seems to only happen after rows where the filename exceeds 68 characters.

| field.instance.paragraphs_item.short_term_goals.field_goal          | update |
| field.instance.paragraphs_item.short_term_goals.field_goal_start_da | update |

 [!] Warning: Trying to access array offset on value of type bool
in bee_get_remaining_rows() (line 373 of /usr/local/bin/bee/includes/render.inc).

| te                                                                  |        |
| field.instance.paragraphs_item.tb_screening.field_chest_x_ray_date  | update |
| field.instance.paragraphs_item.tb_screening.field_chest_x_ray_resul | update |

 [!] Warning: Trying to access array offset on value of type bool
in bee_get_remaining_rows() (line 373 of /usr/local/bin/bee/includes/render.inc).

| ts                                                                  |        |
| field.instance.paragraphs_item.tb_screening.field_tb_date_read      | update |
| field.instance.paragraphs_item.tb_screening.field_tb_results        | update |
| field.instance.paragraphs_item.tb_screening.field_tb_screen_date    | update |
| field.instance.paragraphs_item.weight_history.field_weight_date     | update |
| field.instance.paragraphs_item.weight_history.field_weight_dec      | update |
yorkshire-pudding commented

Hi @herbdool - I tried to reproduce this but so far haven't been able to.
image

I wonder what I'm missing? This file name was 97 characters long.

Any additional steps you can provide? Thanks

yorkshire-pudding commented

Hi @herbdool - there is a PR for this issue and for #430
As the OP of #430 is using the phar file from the release, they may struggle to test a patch. Are you able to test this?

herbdool commented

Sorry I didn't get to this sooner @yorkshire-pudding

  • 👍1