Skip to content

Commit 3236e75

Browse files
author
Mike Edinger
committed
Fix bug in tier mapper
1 parent 4bcaa05 commit 3236e75

2 files changed

Lines changed: 9 additions & 5 deletions

File tree

src/Mappers/Tiers/TierMapper.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,15 @@ class TierMapper
1414
* @param $data
1515
* @return Tier
1616
*/
17-
public function map($data): Tier
17+
public function map($data): ?Tier
1818
{
19+
if (!$data->name) {
20+
return null;
21+
}
22+
1923
return new Tier(
2024
$data->name,
21-
$data->position,
25+
$data->position ?? 0,
2226
$data->uuid ?? null,
2327
$data->description ?? null,
2428
$data->media ? get_object_vars($data->media) : null

src/Resources/OAuth/Tiers/TiersResource.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,15 @@ public function list(array $params = []): array
3535

3636
/**
3737
* @param string $contactUuid
38-
* @return Tier
38+
* @return ?Tier
3939
* @throws PiggyRequestException
4040
*/
41-
public function getTierForContact(string $contactUuid): Tier
41+
public function getTierForContact(string $contactUuid): ?Tier
4242
{
4343
$resourceUri = "/api/v3/oauth/clients/contacts";
4444

4545
$response = $this->client->get("$resourceUri/$contactUuid/tier");
46-
46+
4747
$mapper = new TierMapper();
4848

4949
return $mapper->map($response->getData());

0 commit comments

Comments
 (0)