fix(BuildTicketCreationDataAction): deduplicate emulators

[wescopeland]

Resolves #3234.

Root Cause
I have never been able to repro this issue locally, but I can see how it would happen. Because emulator names are not normalized in addInactiveEmulators, subtle nuances in the decoded client string could plausibly cause a duplicate emulator to be inserted into the $emulators collection.

We could alternatively solve this issue in the front-end, but that feels too far downstream when it's technically a logic error in the back-end action.

[Jamiras]

I am able to reproduce this by changing some data in the DB and then trying to create a ticket.

update player_sessions set user_id=[me] where game_id=22308 and user_id=13448;
update player_games set user_id=[me] where game_id=22308 and user_id=13448;

http://localhost:64000/achievement/285575/tickets/create?type=2

The problem is the contains check on line 82 is case sensitive. The DB contains "Bizhawk" as an emulator, but the User-Agent is "BizHawk". We should probably update the DB to be "BizHawk", but there's going to be a secondary issue with the new "EmuHawk" user agent. I think the correct solution is to double check the collection after resolving the emulator:

if (!$emulators->contains('name', $decoded['client'])) {
    $emulatorUserAgent = EmulatorUserAgent::firstWhere('client', $decoded['client']);
    if (!$emulatorUserAgent) {
        $needsOther = true;
    } else if (!$emulators->contains('name', $emulatorUserAgent->emulator->name)) { // <---
        $emulators->add(EmulatorData::fromEmulator($emulatorUserAgent->emulator));
    }
}