Obsidian RegistrationEntry Grabs Campus from Random Linkages when a UrlSlug parameter is not present

[tcavaletto]

Description

Reported on behalf of Pinelake Church.

When using the RegistrationEntry block, the block checks for a registration linkage to set the Registration.Campus property (https://github.com/SparkDevNetwork/Rock/blob/develop/Rock.Blocks/Event/RegistrationEntry.cs#L1407-L1412). The problem is that this method does not filter on linkages tied to the individual instance, so if a UrlSlug is not passed in, it pulls a randomly selected linkage with a blank UrlSlug property (https://github.com/SparkDevNetwork/Rock/blob/develop/Rock.Blocks/Event/RegistrationEntry.cs#L1929-L1947).

This causes random Registrations to be tied to, say, Campus A, when the Registration, associated linkages, and families are all linked to Campus B.

Actual Behavior

Registrations are linked to the wrong campus

Expected Behavior

Registrations should be linked to the correct campus

Steps to Reproduce

• Wipe out existing Registration Linkages to force the wrong one to be selected
• Create a Second Campus Registration with a linkage to the second campus and a blank urlslug
• Create a Main Campus Registration with either no linkages or a main campus linkage with a UrlSlug present
• Try to sign up Alisha Admin for the Main Campus registration using only the RegistrationInstanceId parameter (She's tied to Main Campus)

Issue Confirmation

• Perform a search on the Github Issues to see if your bug or enhancement is already reported.
• Reproduced the problem on a fresh install or on the demo site.

Rock Version

16.2 (Demo Site), 16.6 (Client Site)

Client Culture Setting

en-us

[chead4]

@tcavaletto Hi Taylor I haven’t been able to recreate this on v17. To help with our investigation, could you provide any additional details including screenshots of your registration linkage with and without a URLSlug? It would be greatly appreciated!