tables icon indicating copy to clipboard operation
tables copied to clipboard
verified publisher icon nextcloud

`data` field empty at GET `/index.php/apps/tables/api/1/rows/{rowId}`

Open Pilzinsel64 opened this issue 1 year ago • 6 comments

Steps to reproduce

  1. Create a new table, one view and add at least one row.
  2. Get the row id by listing all rows via /index.php/apps/tables/api/1/views/{viewId}/rows.
  3. Query a row via /index.php/apps/tables/api/1/rows/{rowId} with the id you found out before.

Expected behavior

/index.php/apps/tables/api/1/rows/{rowId} should return the cell values in the data field. grafik

Actual behavior

/index.php/apps/tables/api/1/rows/{rowId} returns all row data but leaves the data field wich holds the cell values empty.

Tables app version

0.7.1

Browser

Firefox 125.0.3

Client operating system

Windows

Operating system

Ubuntu Server 22.04 LTS

Web server

Apache

PHP engine version

PHP 8.2

Database

MariaDB

Additional info

This started happening with Nextcloud 29.0.0 (at least I didn't regonized it before the update). I'm using tables as small pseudo database within a public software and just published the public beta before I've seen that the data field keeps empty.

This error appeared once (but not reproduceable): grafik

Pilzinsel64 avatar May 02 '24 17:05 Pilzinsel64

I reproduce the bug when i use large request in the table with /index.php/apps/tables/api/1/tables/{tableId}/rows then i have the fied "data" correctly populated by json, but when i use /index.php/apps/tables/api/1/rows/{rowId}, then the field "data" is empty

MyPsyOnline avatar May 06 '24 07:05 MyPsyOnline

The only workaround is polling all rows each time I need a data from one row. This is absolutely imperformant. There is no real workaround for that.

Pilzinsel64 avatar Jun 02 '24 08:06 Pilzinsel64

I have the same bug in Tables app version 0.7.3. Please fix it.

ISC-PRO avatar Jun 27 '24 13:06 ISC-PRO

I can confirm this bug is still present as of version 0.8.0.

yshsy avatar Sep 27 '24 14:09 yshsy

Good morning. Can you tell me what the further planning looks like?

ISC-PRO avatar Jan 06 '25 10:01 ISC-PRO

Needs investigation (debugging), if anyone is up for this.

Entrypoints should be:

  • for a single row: https://github.com/nextcloud/tables/blob/e64ceb4d549e426a4b212f28a795b26a9febff92/lib/Controller/Api1Controller.php#L1215
  • for all rows https://github.com/nextcloud/tables/blob/e64ceb4d549e426a4b212f28a795b26a9febff92/lib/Controller/Api1Controller.php#L1067

blizzz avatar Jan 06 '25 15:01 blizzz