Open silopolis opened 1 year ago
cc @Pytal
This would need a mapping between the property name and readable string for all properties passed to queueUpdate()
https://github.com/search?q=repo%3Anextcloud%2Fserver+queueUpdate&type=code
And yes since there may be multiple properties (introduced in https://github.com/nextcloud/server/pull/20938) we'd have to show each one
⚠️ This issue respects the following points: ⚠️
Bug description
The queueUpdate() success message uses the share's raw property name, which is not that English users friendly and not translatable.
Plus, my shamy PHP non-skills make me believe that it doesn't return all properties when several properties are passed to queueUpdate().
https://github.com/nextcloud/server/blob/master/apps/files_sharing/src/mixins/SharesMixin.js#L280
Steps to reproduce
Update share properties and get a success message
Expected behavior
Have user messages in plain English and translatable, whether one or several properties are passed to queueUpdate().
Installation method
None
Nextcloud Server version
master
Operating system
None
PHP engine version
None
Web server
None
Database engine version
None
Is this bug present after an update or on a fresh install?
None
Are you using the Nextcloud Server Encryption module?
None
What user-backends are you using?
Configuration report
No response
List of activated Apps
Nextcloud Signing status
No response
Nextcloud Logs
No response
Additional info
No response