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

Grid rows duplicated in Fluid field #3261

Closed
robinsowell opened this issue Apr 11, 2023 · 1 comment · Fixed by #3271
Closed

Grid rows duplicated in Fluid field #3261

robinsowell opened this issue Apr 11, 2023 · 1 comment · Fixed by #3271
Assignees
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. javascript Pull requests that update Javascript code

Comments

@robinsowell
Copy link
Contributor

Really weird one. EE 7.2.14 (but also in some older versions of 7).

I have a channel, only has 1 field- a fluid field.

Fluid field only has 1 field- grid field.

Grid field has 3 columns, a select, a text, a file field.

I start a new entry, add my grid field via fluid, add 1 row- fill out each column. Preview. It all looks good.

I add another grid field (not another row in existing)- repeat above, preview. I get a php error on the preview page. Close preview. Save. Entry saves ok, but there is an extra empty row added to my second grid field.

Error I saw:

Screenshot 2023-04-11 at 1 46 50 PM

Saved entry- not the extra empty grid row:

Screenshot 2023-04-11 at 2 16 15 PM

It's definitely quirky. When I first tried replicating locally, I just had a text and a file field in my grid and I couldn't replicate. The user reporting also had a manually populated select field in their grid, so I added that and I replicated! Though- user didn't report the php error I was seeing (but that could have been error settings).

Thinking it was something specific to selects, I tried replicating with a grid field that ONLY had 1 select field in it, but I didn't replicate with that.

Not sure what's up. Was weird. But I can reproduce it.

@robinsowell robinsowell added the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label Apr 11, 2023
@intoeetive intoeetive changed the title Fluid live preview bug Grid rows duplicated in Fluid field Apr 12, 2023
@intoeetive intoeetive self-assigned this Apr 12, 2023
@intoeetive
Copy link
Contributor

intoeetive commented Apr 12, 2023

I was able to replicate it without LivePreview (looks like PHP errors there though are related to malformed data posted, as shown on video)

Only happens if you have both Select and File in the Grid - and this appears to be related to JS stuff, where disabled attribute is mistakenly removed. I'm attaching video https://www.dropbox.com/s/1ek308o6sairng4/Rec_15.29.52%2020230412.mp4?dl=0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. javascript Pull requests that update Javascript code
Projects
None yet
3 participants