Skip to content

Commit

Permalink
Fix pint issues
Browse files Browse the repository at this point in the history
  • Loading branch information
Bubka committed Nov 27, 2024
1 parent e7ad0df commit 1fdeddd
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 12 deletions.
2 changes: 1 addition & 1 deletion app/Api/v1/Controllers/TwoFAccountController.php
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ public function update(TwoFAccountUpdateRequest $request, TwoFAccount $twofaccou
$this->authorize('update', $twofaccount);

$validated = $request->validated();

$twofaccount->fillWithOtpParameters($validated, $twofaccount->icon && is_null(Arr::get($validated, 'icon', null)));
$request->user()->twofaccounts()->save($twofaccount);

Expand Down
3 changes: 0 additions & 3 deletions app/Services/IconStoreService.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,6 @@ class IconStoreService
*/
protected Sanitizer $svgSanitizer;

/**
*
*/
public function __construct(Sanitizer $svgSanitizer)
{
$this->usesDatabase = Settings::get('storeIconsInDatabase');
Expand Down
3 changes: 0 additions & 3 deletions app/Services/LogoService.php
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,6 @@ class LogoService
*/
const TFA_IMG_URL = 'https://raw.githubusercontent.com/2factorauth/twofactorauth/master/img/';

/**
*
*/
public function __construct()
{
$this->setTfaCollection();
Expand Down
4 changes: 2 additions & 2 deletions tests/Api/v1/Controllers/TwoFAccountControllerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -829,12 +829,12 @@ public function test_update_with_removed_icon_prevents_official_logo_fetching()
'legacy_uri' => OtpTestData::TOTP_SHORT_URI,
'icon' => 'icon.png',
]);
$twofaccount = TwoFAccount::factory()->for($this->user)->create($attributes);
$twofaccount = TwoFAccount::factory()->for($this->user)->create($attributes);
$attributes['icon'] = '';

$response = $this->actingAs($this->user, 'api-guard')
->json('PUT', '/api/v1/twofaccounts/' . $twofaccount->id, $attributes);

$this->assertNull($response->json('icon'));
}

Expand Down
3 changes: 0 additions & 3 deletions tests/Feature/Services/IconServiceTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,6 @@ class IconServiceTest extends FeatureTestCase
{
use WithoutMiddleware;

/**
*
*/
protected IconService $iconService;

public function setUp() : void
Expand Down

0 comments on commit 1fdeddd

Please sign in to comment.