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

Missing remoteshare name in activity notification mail #5125

Closed
patschi opened this issue May 26, 2017 · 4 comments
Closed

Missing remoteshare name in activity notification mail #5125

patschi opened this issue May 26, 2017 · 4 comments

Comments

@patschi
Copy link
Member

patschi commented May 26, 2017

Steps to reproduce

  1. Getting informed about any activity by mail

Expected behaviour

To know which remote share of the activity mail.

Actual behaviour

Missing remote share name in mail, seen here:
image

Additionally showing the image fails as well. It seems it tries to load the SVG image over the internet, IMHO it should be directly integrated within the mail using base64?

Server configuration

Operating system: Ubuntu 16.04 LTS
Web server: nginx
PHP version: 7.0
Nextcloud version: Nextcloud 12.0.0.29
Updated from an older Nextcloud/ownCloud or fresh install: Fresh

@patschi
Copy link
Member Author

patschi commented May 31, 2017

I've found the line of the origin translation, everything seems to be right there. Something failed when retrieving the file-Array-Key by getParsedParameters() of the IEvent $event: /stable12/apps/files_sharing/lib/Activity/Providers/RemoteShares.php#L94

@mxschmitt
Copy link

I can reproduce the issue with the following system:

Operating system: Debian 8.7
Web server: nginx
PHP version: 7.1
Nextcloud version: Nextcloud 12.0.0.29
Updated from an older Nextcloud/ownCloud or fresh install: Fresh

@MorrisJobke
Copy link
Member

cc @nickvergessen

@nickvergessen
Copy link
Member

Should be solved since a long time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants