Lisbon
Account created on 5 March 2012, over 13 years ago
#

Merge Requests

More

Recent comments

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

I cannot set credits: you need to be a maintainer of the module to make changes. Leaving it as “Reviewed and tested by the community.”

🇵🇹Portugal saidatom Lisbon

Approved and merged

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon

Changes:

babel.services.yml

  • Removed needs_destruction tag from LocaleSubscriber service definition

src/EventSubscriber/LocaleSubscriber.php

  • Added check to return early if $event->getLids() is empty
  • Prevents attempting to query with an empty array in the IN condition, which can cause database errors
🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon
🇵🇹Portugal saidatom Lisbon

saidatom created an issue.

🇵🇹Portugal saidatom Lisbon

Looks good, moving to RTBC.

🇵🇹Portugal saidatom Lisbon

Looks good, moving to RTBC.

🇵🇹Portugal saidatom Lisbon

Looks good

🇵🇹Portugal saidatom Lisbon

saidatom created an issue.

🇵🇹Portugal saidatom Lisbon

Duplicate of #3546222

🇵🇹Portugal saidatom Lisbon

saidatom created an issue.

🇵🇹Portugal saidatom Lisbon
1. OauthClientScopeHelper::getScopes() → OauthClientScopeHelper::getCurrentUserScopes()
  - Updated in /src/Service/OauthClientScopeHelper.php:35
  - Updated internal call in /src/Service/OauthClientScopeHelper.php:85
  - Updated external calls in /src/Controller/OauthClientUserController.php:171
  - Updated external calls in /src/Form/OauthClientAddForm.php:157
2. OauthClientPermissions::requestPermission() → OauthClientPermissions::getUserPermssion()
  - Updated in /src/Permission/OauthClientPermissions.php:61
  - Updated internal call in /src/Permission/OauthClientPermissions.php:46
  - Updated external calls in /src/Service/OauthClientScopeHelper.php:56
  - Updated external calls in /src/Form/OauthClientAddForm.php:171
🇵🇹Portugal saidatom Lisbon

saidatom created an issue.

🇵🇹Portugal saidatom Lisbon

Accidentally left out README.md from the commit.

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

Thank you for catching that detail.

🇵🇹Portugal saidatom Lisbon

saidatom changed the visibility of the branch 3542871-fatal-error-and to hidden.

🇵🇹Portugal saidatom Lisbon

Great! Looks good, moving to RTBC.

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

Great! Looks good, moving to RTBC.

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

Great! Looks good, moving to RTBC.

🇵🇹Portugal saidatom Lisbon

@yaqbick One last note: when translation is disabled, the Save button should be disabled as well. Otherwise, it results in errors.

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

Looks good move into RTBC.

🇵🇹Portugal saidatom Lisbon

saidatom made their first commit to this issue’s fork.

🇵🇹Portugal saidatom Lisbon

saidatom changed the visibility of the branch project-update-bot-only to hidden.

🇵🇹Portugal saidatom Lisbon

hook_configurable_language_insert|update() return $parsed NULL in DrupalOrgPluralFormula::updatePluralFormula()

Production build 0.71.5 2024