BackendVerificationJob::process() - Code Metrics - Inspection of "NEW: Scaffolded queuedjob to ping backend for a ch..." - phptek/silverstripe-verifiable - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 7f61ea...38db0f )

by Russell

created 2018-06-04 10:25 UTC

BackendVerificationJob::process() A

↳ Parent: BackendVerificationJob

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

/**
 * @author  Russell Michell 2018 <[email protected]>
 * @package silverstripe-verifiable
 */

namespace PhpTek\Verifiable\Job;

use Symbiote\QueuedJobs\Services\AbstractQueuedJob;
filter:
    dependency_paths: ["lib/*"]

/**
 * Simple job to verify a hash against a backend and on success, receive and
 * save a chainpoint proof.
 */
class BackendVerificationJob extends AbstractQueuedJob
{
    /**
     * @return string
     */
    public function getSignature() : string
    {
        return $this->verifiableService->hash($model->normaliseData());

    }

    /**
     * Do the work to ping the remote backend.
     *
     * @return void
     */
    public function process()
    {
        $body = $this->verifiableService->read($this->getSignature());

        // TODO
        // What does a valid/invalid response look like?
        if ($isValid) {

            $this->getObject()->setField('Proof', $body)->write();
        }
    }

}


1			<?php
2
3			/**
4			* @author Russell Michell 2018 <[email protected]>
5			* @package silverstripe-verifiable
6			*/
7
8			namespace PhpTek\Verifiable\Job;
9
10			use Symbiote\QueuedJobs\Services\AbstractQueuedJob;
			0 ignored issues – show Bug introduced 2018-06-04 10:37 UTC by Report Bug Copy Issue Report The type `Symbiote\QueuedJobs\Services\AbstractQueuedJob` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
11
12			/**
13			* Simple job to verify a hash against a backend and on success, receive and
14			* save a chainpoint proof.
15			*/
16			class BackendVerificationJob extends AbstractQueuedJob
17			{
18			/**
19			* @return string
20			*/
21			public function getSignature() : string
22			{
23			return $this->verifiableService->hash($model->normaliseData());
			0 ignored issues – show Comprehensibility Best Practice introduced 2018-06-04 10:37 UTC by Report Bug Copy Issue Report The variable `$model` seems to be never defined. Loading history...
24			}
25
26			/**
27			* Do the work to ping the remote backend.
28			*
29			* @return void
30			*/
31			public function process()
32			{
33			$body = $this->verifiableService->read($this->getSignature());
34
35			// TODO
36			// What does a valid/invalid response look like?
37			if ($isValid) {
			0 ignored issues – show Comprehensibility Best Practice introduced 2018-06-04 10:37 UTC by Report Bug Copy Issue Report The variable `$isValid` seems to be never defined. Loading history...
38			$this->getObject()->setField('Proof', $body)->write();
39			}
40			}
41
42			}
43

phptek / silverstripe-verifiable

Push — master ( 7f61ea...38db0f )

BackendVerificationJob::process() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like