Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embedded images not loaded inside Notes #2813

Closed
pjft opened this issue Feb 27, 2024 · 2 comments
Closed

Embedded images not loaded inside Notes #2813

pjft opened this issue Feb 27, 2024 · 2 comments
Labels

Comments

@pjft
Copy link

pjft commented Feb 27, 2024

Steps to reproduce

  1. Open note with images

Expected behaviour

Embedded images should render inside the note, at the right location.

Actual behaviour

Image placeholder shows up with error 500.

Screenshots

unnamed (6)

Logs

[index] Error: Exception: Call to a member function getUID() on null in file '/var/www/html/nextcloud/apps/text/lib/Service/AttachmentService.php' line 434 at <<closure>>

0. /var/www/html/nextcloud/lib/private/AppFramework/App.php line 183
   OC\AppFramework\Http\Dispatcher->dispatch()
1. /var/www/html/nextcloud/lib/private/Route/Router.php line 315
   OC\AppFramework\App::main()
2. /var/www/html/nextcloud/lib/base.php line 1068
   OC\Route\Router->match()
3. /var/www/html/nextcloud/index.php line 38
   OC::handleRequest()

Caused by:

Error: Call to a member function getUID() on null at <<closure>>

0. /var/www/html/nextcloud/apps/text/lib/Service/AttachmentService.php line 226
   OCA\Text\Service\AttachmentService->getAttachmentDirectoryForFile()
1. /var/www/html/nextcloud/apps/text/lib/Controller/AttachmentController.php line 101
   OCA\Text\Service\AttachmentService->getAttachmentList()
2. /var/www/html/nextcloud/lib/private/AppFramework/Http/Dispatcher.php line 230
   OCA\Text\Controller\AttachmentController->getAttachmentList()
3. /var/www/html/nextcloud/lib/private/AppFramework/Http/Dispatcher.php line 137
   OC\AppFramework\Http\Dispatcher->executeController()
4. /var/www/html/nextcloud/lib/private/AppFramework/App.php line 183
   OC\AppFramework\Http\Dispatcher->dispatch()
5. /var/www/html/nextcloud/lib/private/Route/Router.php line 315
   OC\AppFramework\App::main()
6. /var/www/html/nextcloud/lib/base.php line 1068
   OC\Route\Router->match()
7. /var/www/html/nextcloud/index.php line 38
   OC::handleRequest()

POST /index.php/apps/text/attachments

Reasoning or why should it be changed/implemented?

Well, it's a bug.

Environment data

iOS version: iOS

Nextcloud iOS app version:

5.2.0.2

Server operating system:

Linux 6.1.66, ARM64

Web server:

Apache

Database:

MySQL

PHP version:

8.2.7

Nextcloud version:

27.1.6

@pjft
Copy link
Author

pjft commented Feb 27, 2024

It seems that the changes from 3 months ago might have impacted this:

https://github.com/nextcloud/text/commits/main/lib/Service/AttachmentService.php

@joshtrichards
Copy link
Member

Duplicate of nextcloud/text#5419

@joshtrichards joshtrichards marked this as a duplicate of nextcloud/text#5419 Aug 2, 2024
@joshtrichards joshtrichards closed this as not planned Won't fix, can't repro, duplicate, stale Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants