Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 26, 2025

Backport of #51049

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added bug 3. to review Waiting for reviews feature: language/translations (l10n/i18n) Localization and translation matters labels Feb 26, 2025
@backportbot backportbot bot added this to the Nextcloud 30.0.7 milestone Feb 26, 2025
- No leading/trailing whitespace
- Use asci single quote

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the backport/51049/stable30 branch from 60a6113 to a5e6f46 Compare February 26, 2025 14:06
@nickvergessen nickvergessen marked this pull request as ready for review February 26, 2025 14:07
@nickvergessen
Copy link
Member

/compile /

@$this->invokeLDAPMethod('modReplace', $userDN, $password);
} catch (ConstraintViolationException $e) {
throw new HintException('Password change rejected.', \OCP\Util::getL10N('user_ldap')->t('Password change rejected. Hint: ') . $e->getMessage(), (int)$e->getCode());
throw new HintException('Password change rejected.', \OCP\Util::getL10N('user_ldap')->t('Password change rejected. Hint: %s', $e->getMessage()), (int)$e->getCode());

Check notice

Code scanning / Psalm

RedundantCast Note

Redundant cast to int
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nickvergessen nickvergessen merged commit 812a4d5 into stable30 Feb 26, 2025
174 of 176 checks passed
@nickvergessen nickvergessen deleted the backport/51049/stable30 branch February 26, 2025 15:20
@blizzz blizzz mentioned this pull request Mar 4, 2025
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug feature: language/translations (l10n/i18n) Localization and translation matters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants