openui5 icon indicating copy to clipboard operation
openui5 copied to clipboard

EngineCustomFilters

Open TawillAvi opened this issue 8 months ago • 2 comments

What is the issue and how can we reproduce it?

  1. Open the sample https://sdk.openui5.org/entity/sap.m.p13n.Engine/sample/sap.m.sample.p13n.EngineCustomFilters/code
  2. Click on Settings button
  3. Choose filter by 'First name'
  4. Type Maria and chose from the list, then type john then chose from the list.
  5. Click on Ok to close personalization window -> table will be filtered according to the filter as expected.
  6. Click again on Settings button
  7. Delete 'Maria' tag
  8. Click on Ok to close personalization window.

Expected result - only John apprears in the table. Actual result - only Maria - which has been delete from the filter appears.

Debugging this issue I seems the problem is in: sap/m/p13n/modules/xConfigAPI.js - line 434.

It takes the first item that it can find only using the key.

BR Avi.

Which OpenUI5 version is your project targeting?

Latest - 1.135.0

Is the issue device- or browser-specific?

I dont think so, I reproduce it on Chrome on PC, Chrome on adroid and Edge.

Confirmation

  • [x] I have searched the existing issues and reviewed the relevant documentation as well as the API reference.
  • [x] I am not disclosing any internal or sensitive information.

TawillAvi avatar Apr 30 '25 15:04 TawillAvi

Hello @TawillAvi , Thank you for your pull request. I've created an internal record DINC0490163. The status of the issue will be updated here in GitHub. Regards, Tsanislav

tsanislavgatev avatar May 02 '25 10:05 tsanislavgatev

Hi @TawillAvi, thanks for reaching out! just wanted to let you know that we're working on your issue. I'll get back to you as soon as we have a solution for it.

cuvar avatar Jun 05 '25 07:06 cuvar

Hi @cuvar thank you,

Is there any news? I see a new version has been released for this class but the issue remain.

TawillAvi avatar Sep 21 '25 21:09 TawillAvi

Hi @TawillAvi,

the referenced commit should fix the issue. It should be available in the next version.

DonkeyCo avatar Sep 23 '25 16:09 DonkeyCo