bee icon indicating copy to clipboard operation
bee copied to clipboard

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

Open herbdool opened this issue 1 year ago • 1 comments

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 |

herbdool avatar Jan 18 '24 04:01 herbdool

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 avatar May 06 '24 15:05 yorkshire-pudding

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?

yorkshire-pudding avatar Sep 03 '24 18:09 yorkshire-pudding

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

herbdool avatar Sep 06 '24 13:09 herbdool