Skip to content

l10n: Add words user and because in ShareByMailProvider.php - #26231

Merged
skjnldsv merged 2 commits into
masterfrom
Valdnet-patch-6
Mar 22, 2021
Merged

l10n: Add words user and because in ShareByMailProvider.php#26231
skjnldsv merged 2 commits into
masterfrom
Valdnet-patch-6

Conversation

@Valdnet

Copy link
Copy Markdown
Contributor

Standardizing text strings with other Nextcloud applications.

Standardizing text strings with other Nextcloud applications.
if (!empty($alreadyShared)) {
$message = 'Sharing %1$s failed, this item is already shared with %2$s';
$message = 'Sharing %1$s failed, because this item is already shared with user %2$s';
$message_t = $this->l->t('Sharing %1$s failed, this item is already shared with %2$s', [$share->getNode()->getName(), $shareWith]);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then please also add it here, because that is the string that is translated - the other one is the english one for the debug logs.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An oversight on my part. Thanks 😉.

@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish bug feature: language/translations (l10n/i18n) Localization and translation matters feature: sharing labels Mar 22, 2021
@skjnldsv
skjnldsv merged commit 6b4cfc5 into masterMar 22, 2021
@skjnldsv
skjnldsv deleted the Valdnet-patch-6 branch March 22, 2021 08:42
@MorrisJobkeMorrisJobke added this to the Nextcloud 22 milestone Mar 22, 2021
@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable21

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable20

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable19

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: language/translations (l10n/i18n)Localization and translation mattersfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Valdnet@MorrisJobke@rullzer@skjnldsv