miguel456/athenahr
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: set username based on game license requirement while both slack notifs and licence req are active

Browse Source 
Fixes a problem where sign ups would fail due to a misconfigured UUID middleware that would only trigger if slack notifications were active

Signed-off-by: Miguel Nogueira <me@nogueira.codes>
This commit is contained in:
Miguel Nogueira
2025-08-07 23:09:50 +01:00
parent 2b44fe35a8
commit 9e77205820
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/Http/Middleware/UsernameUUID.php
View File

@@ -22,6 +22,8 @@
namespace App\Http\Middleware; namespace App\Http\Middleware;
use App\Facades\UUID; use App\Facades\UUID;
use App\Options;
use Closure; use Closure;
class UsernameUUID class UsernameUUID
@@ -35,6 +37,10 @@ class UsernameUUID
*/ */
public function handle($request, Closure $next) public function handle($request, Closure $next)
{ {
if (!\App\Facades\Options::getOption('requireGameLicense')) {
return $next($request);
}
$input = $request->all(); $input = $request->all();
if (isset($input['uuid'])) { if (isset($input['uuid'])) {
try { try {

2
app/Notifications/NewUser.php
View File

@@ -82,7 +82,7 @@ class NewUser extends Notification implements ShouldQueue
$user['name'] = $this->user->name; $user['name'] = $this->user->name;
$user['email'] = $this->user->email; $user['email'] = $this->user->email;
$user['username'] = UUID::toUsername($this->user->uuid); $user['username'] = Options::getOption('requireGameLicense') ? UUID::toUsername($this->user->uuid) : $this->user->username;
$date = \Carbon\Carbon::parse($this->user->created_at); $date = \Carbon\Carbon::parse($this->user->created_at);
$user['created_at'] = $date->englishMonth.' '.$date->day.' '.$date->year; $user['created_at'] = $date->englishMonth.' '.$date->day.' '.$date->year;