Abacaphiliac\AwsSdk\ClaimCheck\ClaimCheck

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2
A	getS3BucketName()	0	4	1
A	getS3Key()	0	4	1

<?php

namespace Abacaphiliac\AwsSdk\ClaimCheck;

use Ramsey\Uuid\Uuid;

class ClaimCheck
{
    /** @var  string */
    private $s3BucketName;
    
    /** @var  string */
    private $s3Key;

    /**
     * ClaimCheckMessage constructor.
     * @param string $s3BucketName
     * @param string $s3Key
     */
    public function __construct($s3BucketName, $s3Key = null)
    {
        $this->s3BucketName = $s3BucketName;
        
        if (!$s3Key) {

The expression $s3Key of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $s3Key = Uuid::uuid4()->toString();
        }
        
        $this->s3Key = $s3Key;
    }

    /**
     * @return string
     */
    public function getS3BucketName()
    {
        return $this->s3BucketName;
    }

    /**
     * @return string
     */
    public function getS3Key()
    {
        return $this->s3Key;
    }
}