Problem/Motivation

The current implementation throws an exception every time a user tries to access an event with uppercase letters in the event name.

Steps to reproduce

Access a single event page with an uppercase letter in the event name.

Proposed resolution

Change OpenagendaEventConverter::convert() to also convert the event name to lowercase before it is passed as an argument to the controller.

CommentFileSizeAuthor
#2 3395244-1.patch1.13 KBnfabing

Comments

nfabing created an issue. See original summary.

nfabing’s picture

nfabing commented
StatusFileSize
new 3395244-1.patch1.13 KB

Proposed patch

nfabing’s picture

nfabing commented
Status: Active » Needs review
nfabing’s picture

nfabing commented
Title: openagenda.event route throw a exception when viewing a URL with uppercase in event name » openagenda.event route throws an exception when viewing a URL with uppercase in the event name
nfabing’s picture

nfabing commented
mably’s picture

mably commented
Status: Needs review » Needs work

Could we have an MR please?

And why not simply update the route requirement to allow uppercase letters for the event parameter?

mably’s picture

mably commented

I fixed the issue in this MR: https://git.drupalcode.org/project/openagenda/-/merge_requests/25

mably’s picture

mably commented
Version: 3.4.5 » 4.x-dev
Status: Needs work » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.