gmakstutis/cognee
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: Resolve issue with gettings documents ids for user

Browse source
This commit is contained in:
Igor Ilic 2025-05-20 17:25:11 +02:00
parent d7a8b29147
commit 70d49745d9
1 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
cognee/modules/users/permissions/methods/get_document_ids_for_user.py
View file

@ -1,7 +1,9 @@
from uuid import UUID
from cognee.modules.data.methods import get_dataset_data
from sqlalchemy import select
from cognee.infrastructure.databases.relational import get_relational_engine
from cognee.modules.data.models import Dataset, DatasetData, Data
from cognee.modules.data.models import Dataset, DatasetData
from ...models import ACL, Permission
@ -10,9 +12,9 @@ async def get_document_ids_for_user(user_id: UUID, datasets: list[str] = None) -
async with db_engine.get_async_session() as session:
async with session.begin():
document_ids = (
dataset_ids = (
await session.scalars(
select(Data.id)
select(Dataset.id)
.join(ACL.dataset)
.join(ACL.permission)
.where(
@ -22,9 +24,15 @@ async def get_document_ids_for_user(user_id: UUID, datasets: list[str] = None) -
)
).all()
# Get documents from datasets user has read access for
document_ids = []
for dataset_id in dataset_ids:
data_list = await get_dataset_data(dataset_id)
document_ids.extend([str(data.id) for data in data_list])
if datasets:
documents_ids_in_dataset = set()
# If datasets are specified filter out documents that aren't part of the specified datasets
documents_ids_in_dataset = set()
for dataset in datasets:
# Find dataset id for dataset element
dataset_id = (