Fix calendar crash when yearmatchgroup regex does not match

[Antra]

Description

Fixes an issue in CalendarUtils.titleTransform() when a title transformation uses yearmatchgroup but the configured regular expression does not match the event title.

Currently, matchAll() returns an empty array when there is no match, but the code subsequently accesses yearmatch[0].length. This results in an exception and causes the calendar module to stop displaying its events.

This can occur when using yearmatchgroup for birthday events, for example, when some matching events have a birth year in the expected format while another event matching the keyword does not.

Change

Check that yearmatch contains at least one match before accessing yearmatch[0].

This is a one-line change and leaves the existing behaviour unchanged when the regular expression matches.

Example

With a transformation such as:

transform: {
    search: "^(.+) \\((\\d{4})\\)$",
    replace: "$1 ($2)",
    yearmatchgroup: 2
}

an event such as:

Test Birthday (1987)

continues to be transformed to:

Test Birthday (39)

while an event matching the custom-event keyword but not the year pattern no longer causes the calendar module to fail.

[KristjanESPERANTO]

Looks good! 👍 Would you like to add a regression test for this?

[Antra]

Looks good! 👍 Would you like to add a regression test for this?

Certainly, there is now also a small regression test for it added.

[KristjanESPERANTO]

Nice! Thank you 😃

The change will be in the October release.