edi icon indicating copy to clipboard operation
edi copied to clipboard

[15.0][FIX] edi_oca: fix search logic

Open ThiagoMForgeFlow opened this issue 8 months ago • 5 comments

Maintaining this part of code can occur a visual issue. If the result variable has 80 records (default by Odoo) and the orig_ids is less than result, Odoo only shows the orig_ids, and we lost the previous work finding next records.

@ForgeFlow

ThiagoMForgeFlow avatar Apr 22 '25 11:04 ThiagoMForgeFlow

Hi @simahawk, @etobella, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Apr 22 '25 11:04 OCA-git-bot

Hi @simahawk

We saw that you specifically added this change here https://github.com/OCA/edi/commit/2324d212a23249f809fcffa0f346e3470e6f8a1e

do you have some more insights?

LoisRForgeFlow avatar May 19 '25 15:05 LoisRForgeFlow

@simahawk a kind reminder here

@etobella maybe you have some comments?

LoisRForgeFlow avatar Jul 04 '25 07:07 LoisRForgeFlow

I'm not sure if the issues is clear, but it is quite problamatic. In a multicompany environment, if you only enable one company you won't see all the records that you should. Real example:

  1. With only one company active you don't see all the records of that company.

image

  1. You can only access all the records activiating all the companies.

image

LoisRForgeFlow avatar Jul 04 '25 07:07 LoisRForgeFlow

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Nov 02 '25 12:11 github-actions[bot]