EloquentExchangeLogger::log() - Code Metrics - Inspection of "Add exchanges history" - Hackathonners/swap - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( dbe277...ea1e27 )

by Francisco

created 2017-08-26 16:04 UTC

EloquentExchangeLogger::log() A

↳ Parent: EloquentExchangeLogger

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	6
nc	1
nop	2
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0

<?php

namespace App\Judite\Logger;

use App\Judite\Models\Enrollment;
use App\Judite\Models\LogExchange;
use App\Judite\Contracts\ExchangeLogger;

class EloquentExchangeLogger implements ExchangeLogger
{
    /**
     * {@inheritdoc}
     */
    public function log(Enrollment $fromEnrollment, Enrollment $toEnrollment)
    {
        $logExchange = LogExchange::make();
        $logExchange->fromShift()->associate($fromEnrollment->shift);
        $logExchange->toShift()->associate($toEnrollment->shift);
        $logExchange->fromStudent()->associate($fromEnrollment->student);
        $logExchange->toStudent()->associate($toEnrollment->student);
        $logExchange->save();
    }

    /**
     * {@inheritdoc}
     */
    public function history()
    {
        return LogExchange::latest('id')->paginate();
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
    }
}


1			<?php
2
3			namespace App\Judite\Logger;
4
5			use App\Judite\Models\Enrollment;
6			use App\Judite\Models\LogExchange;
7			use App\Judite\Contracts\ExchangeLogger;
8
9			class EloquentExchangeLogger implements ExchangeLogger
10			{
11			/**
12			* {@inheritdoc}
13			*/
14			public function log(Enrollment $fromEnrollment, Enrollment $toEnrollment)
15			{
16			$logExchange = LogExchange::make();
17			$logExchange->fromShift()->associate($fromEnrollment->shift);
18			$logExchange->toShift()->associate($toEnrollment->shift);
19			$logExchange->fromStudent()->associate($fromEnrollment->student);
20			$logExchange->toStudent()->associate($toEnrollment->student);
21			$logExchange->save();
22			}
23
24			/**
25			* {@inheritdoc}
26			*/
27			public function history()
28			{
29			return LogExchange::latest('id')->paginate();
			0 ignored issues – show Bug Best Practice introduced 2017-08-26 16:06 UTC by Report Bug Copy Issue Report The expression `return App\Judite\Models...atest('id')->paginate()` returns the type `Illuminate\Pagination\LengthAwarePaginator` which is incompatible with the return type mandated by `App\Judite\Contracts\ExchangeLogger::history()` of `Illuminate\Contracts\Pagination`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
30			}
31			}
32

Hackathonners / swap

Push — develop ( dbe277...ea1e27 )

EloquentExchangeLogger::log() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like