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

sap.m.UploadSet removeIncompleteItem() does not remove the last items #4029

Open
T0bbes opened this issue Apr 22, 2024 · 2 comments
Open

sap.m.UploadSet removeIncompleteItem() does not remove the last items #4029

T0bbes opened this issue Apr 22, 2024 · 2 comments

Comments

@T0bbes
Copy link

T0bbes commented Apr 22, 2024

Hi Guys,

We use the Document Upload to upload N documents in the "IncompleteItems" aggregation and process them in sequence. When a document has been edited, it should be deleted from the aggregation. As we have problems with the two aggregations "items" and "incompleteItems", we use both "remove" methods as a precaution.

All but the last item are deleted. This remains despite calling the two remove methods.

OpenUI5 version: 1.120.8

Browser/version (+device/version): Chrome Version 124.0.6367.61 (Official Build) (arm64)

Steps to reproduce the problem:

  1. Upload multiple document as incomplete item
  2. Remove items with oUploadSet.removeIncompleteItem(oSelectedItem.item); and oUploadSet.removeItem(oSelectedItem.item);
  3. Last items remains

What is the expected result?
Last item should be removed

What happens instead?
Last item remains in the list

Any other information? (attach screenshot if possible)
I think the reason is that the getItems only fetches the "completedItems" and does not find the incompletedItems. We set the items to completed, but that doesn't change anything, as the items still remain incomplete. see #4027
SCR-20240422-nrrh

@NHristov-sap
Copy link
Contributor

Hi @T0bbes ,

It seems that the reported behavior is caused by the issue reported by #4027 (as you mention). So I will add this issue to the internal issue created for #4027 (DINC0129133).

Best Regards,
Nikolay Hristov
UI5 GitHub Dispatcher

@chnaahmad
Copy link

Any update regarding this issue, we are also facing the same problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants