Sheet: Missing updates when pasting more rows than available in the current sheet about primefaces-extensions

[melloware]

You are correct. The model of the backing bean has 10 rows. So the Sheet Updates get aligned with those backing bean rows. If 5 new rows are created they don't exist in your backing bean model. Not sure how i will solve this since I can't create new instances of the objects for you I don't think. Have to think about this one.

[blutorange]

I can create the objects myself, but that requires I know about the additional rows. Perhaps just creating a SheetUpdate with a null rowkey might work (and SheetUpdate#isNew), but I don't know if a null rowkey would cause issues somewhere else. Alternatively, perhaps a Sheet#getNewlyCreatedRows?

But take your time, I'm sure you can come up with a better way.

[blutorange]

This seems to be fixed now. I am now receiving the SheetUpdates for the newly added rows. Looking at the commit history, I suppose this is due to the update to HandsonTable 5.0 (#590).

[melloware]

Awesome I will close for now!

[blutorange]

Hmm, I'm not sure what I've been seeing, but unfortunately, it seems the updates are still not sent to the server. Would be great if you could take a look at it when you've got time.

If it helps, the use case is a simple generic CSV editor where the user can copy&paste data into the sheet.

[melloware]

I reopened the issue. Will investigate when I can.

[melloware]

I believe this Stack Overflow user has the same issue: https://stackoverflow.com/questions/60191691/how-to-access-pasted-rows-in-primefaces-extensions-sheet

[melloware]

This one is definitely messy. I took a deeper dive and without RowKey it then fails the validate step etc. I think this would have to be fixed in many many places for it to work and pass down into SheetUpdates. Was hoping it was going to be simple.

[blutorange]

Yeah, though I think the real issue for us is that (as far as I heard) the sheet component was developed as an alternative for DataTable with a mostly fixed column schema. We're kinda using the sheet as a general excel / CSV editor and it really shows the sheet wasn't developed with that usage in mind.

[github-actions]

This issue is marked as stale because there was no activity on it for the last 2 years. Remove stale label or comment or this will be closed in 30 days