Sign upLogin

nilsabegg/sesame

View on GitHub
Star
src/Sesame/Model/Article.php

Summary

Maintainability
A
0 mins
Test Coverage

Article has 30 functions (exceeds 20 allowed). Consider refactoring.
Wontfix

class Article
{

    /**
     * @var string
Severity: Minor
Found in src/Sesame/Model/Article.php - About 3 hrs to fix

    Class "Article" has 30 methods, which is greater than 20 authorized. Split it into smaller classes.
    Open

    class Article
    Severity: Major
    Found in src/Sesame/Model/Article.php by sonar-php

    A class that grows too much tends to aggregate too many responsibilities and inevitably becomes harder to understand and therefore to maintain. Above a specific threshold, it is strongly advised to refactor the class into smaller ones which focus on well defined topics.

    Assigning string to property but \Sesame\Model\Article->sellerFeedback is float
    Open

            $this->sellerFeedback = $sellerFeedback;
    Severity: Minor
    Found in src/Sesame/Model/Article.php by phan

    Returning type float but getPriceCurrency() is declared to return string
    Open

            return $this->priceCurrency;
    Severity: Minor
    Found in src/Sesame/Model/Article.php by phan

    Returning type float but getRating() is declared to return int
    Open

            return $this->rating;
    Severity: Minor
    Found in src/Sesame/Model/Article.php by phan

    Doc-block of $sellerFeedback in setSellerFeedback contains phpdoc param type float which is incompatible with the param type string declared in the signature
    Open

         * @param float $sellerFeedback
    Severity: Minor
    Found in src/Sesame/Model/Article.php by phan

    Assigning string to property but \Sesame\Model\Article->priceCurrency is float
    Open

            $this->priceCurrency = $priceCurrency;
    Severity: Minor
    Found in src/Sesame/Model/Article.php by phan

    There are no issues that match your filters.

    Category
    Status