Delete application does not work

[oculos]

Steps to reproduce

The newly introduced function to delete an application does not work. I get a "Cannot delete application: unknown error" or something like that.

Expected behavior

An app should be deleted

Actual behavior

It doesn't get deleted.

Tables app version

0.7.2

Browser

Safari

Client operating system

macOS

Operating system

RHEL 9

Web server

Nginx

PHP engine version

PHP 8.0

Database

SQLite

Additional info

No response

[blizzz]

Thanks for your report. Could you please paste the nextcloud.log with the relevant entry?

[oculos]

Here you go:

{"reqId":"6LlPhMN0Zix0B1ezrGIN","level":3,"time":"2024-05-10T10:28:47+00:00","remoteAddr":"129.240.130.7","user":"franciaa","app":"no app in context","method":"DELETE","url":"/ocs/v2.php/apps/tables/api/2/contexts/1","message":"OCA\\Tables\\Db\\PageContentMapper::deleteByPageId(): Argument #1 ($pageId) must be of type int, string given, called in /var/www/nextcloud/apps/tables/lib/Service/ContextService.php on line 245 in file '/var/www/nextcloud/apps/tables/lib/Db/PageContentMapper.php' line 62","userAgent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.3.1 Safari/605.1.15","version":"29.0.0.19","exception":{"Exception":"Exception","Message":"OCA\\Tables\\Db\\PageContentMapper::deleteByPageId(): Argument #1 ($pageId) must be of type int, string given, called in /var/www/nextcloud/apps/tables/lib/Service/ContextService.php on line 245 in file '/var/www/nextcloud/apps/tables/lib/Db/PageContentMapper.php' line 62","Code":0,"Trace":[{"file":"/var/www/nextcloud/lib/private/AppFramework/App.php","line":184,"function":"dispatch","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->"},{"file":"/var/www/nextcloud/lib/private/Route/Router.php","line":338,"function":"main","class":"OC\\AppFramework\\App","type":"::"},{"file":"/var/www/nextcloud/ocs/v1.php","line":66,"function":"match","class":"OC\\Route\\Router","type":"->"},{"file":"/var/www/nextcloud/ocs/v2.php","line":23,"args":["/var/www/nextcloud/ocs/v1.php"],"function":"require_once"}],"File":"/var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php","Line":170,"Previous":{"Exception":"TypeError","Message":"OCA\\Tables\\Db\\PageContentMapper::deleteByPageId(): Argument #1 ($pageId) must be of type int, string given, called in /var/www/nextcloud/apps/tables/lib/Service/ContextService.php on line 245","Code":0,"Trace":[{"file":"/var/www/nextcloud/apps/tables/lib/Service/ContextService.php","line":245,"function":"deleteByPageId","class":"OCA\\Tables\\Db\\PageContentMapper","type":"->"},{"file":"/var/www/nextcloud/lib/public/AppFramework/Db/TTransactional.php","line":63,"function":"OCA\\Tables\\Service\\{closure}","class":"OCA\\Tables\\Service\\ContextService","type":"->","args":["*** sensitive parameters replaced ***"]},{"file":"/var/www/nextcloud/apps/tables/lib/Service/ContextService.php","line":249,"function":"atomic","class":"OCA\\Tables\\Service\\ContextService","type":"->"},{"file":"/var/www/nextcloud/apps/tables/lib/Controller/ContextController.php","line":162,"function":"delete","class":"OCA\\Tables\\Service\\ContextService","type":"->"},{"file":"/var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php","line":232,"function":"destroy","class":"OCA\\Tables\\Controller\\ContextController","type":"->"},{"file":"/var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php","line":138,"function":"executeController","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->"},{"file":"/var/www/nextcloud/lib/private/AppFramework/App.php","line":184,"function":"dispatch","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->"},{"file":"/var/www/nextcloud/lib/private/Route/Router.php","line":338,"function":"main","class":"OC\\AppFramework\\App","type":"::"},{"file":"/var/www/nextcloud/ocs/v1.php","line":66,"function":"match","class":"OC\\Route\\Router","type":"->"},{"file":"/var/www/nextcloud/ocs/v2.php","line":23,"args":["/var/www/nextcloud/ocs/v1.php"],"function":"require_once"}],"File":"/var/www/nextcloud/apps/tables/lib/Db/PageContentMapper.php","Line":62},"message":"OCA\\Tables\\Db\\PageContentMapper::deleteByPageId(): Argument #1 ($pageId) must be of type int, string given, called in /var/www/nextcloud/apps/tables/lib/Service/ContextService.php on line 245 in file '/var/www/nextcloud/apps/tables/lib/Db/PageContentMapper.php' line 62","exception":{},"CustomMessage":"OCA\\Tables\\Db\\PageContentMapper::deleteByPageId(): Argument #1 ($pageId) must be of type int, string given, called in /var/www/nextcloud/apps/tables/lib/Service/ContextService.php on line 245 in file '/var/www/nextcloud/apps/tables/lib/Db/PageContentMapper.php' line 62”}}

On 10 May 2024, at 11:35, Arthur Schiwon @.***> wrote:

Thanks for your report. Could you please paste the nextcloud.log with the relevant entry?

— Reply to this email directly, view it on GitHub https://github.com/nextcloud/tables/issues/1081#issuecomment-2104283561, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABT2F43RRZLK7DVF4I2KKOLZBSIF7AVCNFSM6AAAAABHOQMTFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBUGI4DGNJWGE. You are receiving this because you authored the thread.

[blizzz]

You may try to apply https://github.com/nextcloud/tables/pull/1083, it should fix that issue.