ViewProviderVersionedObject::getArchiveFieldClass() - Code Metrics - Inspection of "Merge branch '1'" - silverstripe/silverstripe-versioned-admin - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b101db...a45b24 )

by Robbie

created 2018-08-13 06:29 UTC

getArchiveFieldClass() A

↳ Parent: ViewProviderVersionedObject

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

namespace SilverStripe\VersionedAdmin\Tests\ArchiveAdminTest;

use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
use SilverStripe\Versioned\Versioned;
use SilverStripe\VersionedAdmin\ArchiveViewProvider;

class ViewProviderVersionedObject extends DataObject implements ArchiveViewProvider
{
    private static $table_name = 'Test_ViewProviderVersionedObject';


    private static $extensions = [

        Versioned::class,
    ];

    public function getArchiveFieldClass()
    {
        return ViewProviderVersionedObject::class;
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.
    }
}
    }
    public function getArchiveField()
    {
        return null;
    }
    public function isArchiveFieldEnabled()
    {
        return true;
    }
}


1			<?php
2
3			namespace SilverStripe\VersionedAdmin\Tests\ArchiveAdminTest;
4
5			use SilverStripe\Dev\TestOnly;
6			use SilverStripe\ORM\DataObject;
7			use SilverStripe\Versioned\Versioned;
8			use SilverStripe\VersionedAdmin\ArchiveViewProvider;
9
10			class ViewProviderVersionedObject extends DataObject implements ArchiveViewProvider
11			{
12			private static $table_name = 'Test_ViewProviderVersionedObject';
			0 ignored issues – show introduced 2018-08-13 06:34 UTC by Report Bug Copy Issue Report The private property `$table_name` is not used, and could be removed. Loading history...
13
14			private static $extensions = [
			0 ignored issues – show introduced 2018-08-13 06:34 UTC by Report Bug Copy Issue Report The private property `$extensions` is not used, and could be removed. Loading history...
15			Versioned::class,
16			];
17
18			public function getArchiveFieldClass()
19			{
20			return ViewProviderVersionedObject::class;
			0 ignored issues – show Bug Best Practice introduced 2018-08-13 06:34 UTC by Report Bug Copy Issue Report The expression `return SilverStripe\Vers...rVersionedObject::class` returns the type `string` which is incompatible with the return type mandated by `SilverStripe\VersionedAd...:getArchiveFieldClass()` of `array`. 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...
21			}
22			public function getArchiveField()
23			{
24			return null;
25			}
26			public function isArchiveFieldEnabled()
27			{
28			return true;
29			}
30			}
31

silverstripe / silverstripe-versioned-admin

Push — master ( b101db...a45b24 )

getArchiveFieldClass() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like