V8-archive: Collection index page shows wrong(?) item count

Created on 31 May 2019  路  5Comments  路  Source: directus/v8-archive

Bug Report

Steps to Reproduce

  • create a collection
  • create a new role (making things only readable to "me")
    Screenshot 2019-05-31 at 10 34 36
  • create new entry as admin, and one as user with the new role

Expected Behavior

when logged in as non-admin user, the search field should say "1 item"

Actual Behavior

it shows "2 items" (also counting the item created by the admin user)
Screenshot 2019-05-31 at 10 33 49
not sure if this is intented or not.

Other Context & Screenshots

Technical Details

docker: directus/api:latest + directus/app:latest

bug enhancement

All 5 comments

ref https://github.com/directus/api/pull/946

@philleepflorence I guess that filtered_count is even more important than we thought :wink:

@bjgajjar @benhaynes I had some trouble deciding where to put this / how to label it. It's an enhancement on the API (adding the filtered_count that gives accurate results) but also a bug on the app cause it feels broken on that side.

Despite the App and API being decoupled, they are both part of the suite. This bug on the App needs to be fixed in the API, so I think calling it a bug/enhancement on the API repo makes most sense.

Hopefully we can close this once #946 is done.

I will get this updated and released online tomorrow!
It is important after all.

It should be result_count instead of total_count over there. API returns the expected response but this change should be done at APP side.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mimamuh picture mimamuh  路  25Comments

carolin-skiply picture carolin-skiply  路  51Comments

rijkvanzanten picture rijkvanzanten  路  55Comments

sebj54 picture sebj54  路  25Comments

zeusstl picture zeusstl  路  26Comments