Wolsztyn, 🇵🇱
Account created on 31 January 2019, over 6 years ago
#

Merge Requests

More

Recent comments

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

Issues were fixed

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

The request notification contains the wrong user name; it should display the name of the administrator instead of the name of the person who requested it.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I found two cases where the notification contains incorrect data; see the merge request (MR).

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

Issue: If I approve a rejected request, the system will not send a notification. i expect that system sends "Approve notification"

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I am fine with it

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I was able to use "private://..." as path for keys, tested on the latest v6.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

## Performance impact

**Before fix:** Route lookups (`pathValidator->getUrlIfValidWithoutAccessCheck()` + `routeProvider->getRouteByName()`) occur on EVERY request, regardless of whether OAuth is being used.

**After fix:** Route lookups only occur when a Bearer token is present (typically <1% of requests on most sites). For non-OAuth requests, the method returns FALSE immediately after a simple header check.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

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

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

For some reason, 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.”

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

Approved and merged

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I performed manual tests—import and export—and did not notice any issues.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue. See original summary .

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

The module is not yet stable, so I assume that we do not need an update script.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

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

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc changed the visibility of the branch 11.x to hidden.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc changed the visibility of the branch 1.2.x to hidden.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc changed the visibility of the branch 3547582-link-widget-placeholderurl to hidden.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱
🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

babel_update_8001 removes duplication
Duplicated are not longer created

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I performed manual tests: hook removes OG menus. I uninstalled and reinstalled the module a few times in order to test the batch process. I also updated a few menu items. It looks fine, thanks.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

alorenc created an issue.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I can see warning messages during the import if any occur.

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

Is there any reason why the README.md file was not updated as well?

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

I can still see many references to the "4.32.2" library version.

Production build 0.71.5 2024