Sausin\Signere\Http\Controllers\Admin\ExternalSignController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	99
Duplicated Lines	9.09 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	index()	0	6	1
A	show()	0	6	1
A	store()	9	47	2

<?php

namespace Sausin\Signere\Http\Controllers\Admin;

use Illuminate\Http\Request;
use Sausin\Signere\ExternalSign;
use Illuminate\Support\Facades\Config;
use Sausin\Signere\Http\Controllers\Controller;

class ExternalSignController extends Controller
{
    /** @var \Sausin\Signere\ExternalSign */
    protected $extSign;

    /**
     * Create a new controller instance.
     *
     * @param  \Sausin\Signere\ExternalSign $extSign
     */
    public function __construct(ExternalSign $extSign)
    {
        parent::__construct();

        $this->extSign = $extSign;
    }

    /**
     * Get the URLs to sign the Document.
     *
     * @param  string $documentId
     * @return \Illuminate\Http\Response
     */
    public function index(string $documentId)
    {
        return $this->extSign->getUrlForSign($documentId)
                ->getBody()
                ->getContents();
    }

    /**
     * Get the URLs to view a viewerapplet in a iFrame on your site.
     *
     * @param  string $documentId
     * @param  string $domain
     * @param  string $language
     * @return \Illuminate\Http\Response
     */
    public function show(string $documentId, string $domain, string $language)
    {
        return $this->extSign->getUrlForApplet($documentId, ['Domain' => $domain, 'Language' => $language])
                ->getBody()
                ->getContents();
    }

    /**
     * Creates an externalsign request.
     *
     * @param  Request $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        $this->validate($request, [
            'description' => 'required|string|min:1|max:255',
            'ext_doc_id' => 'required|string|min:1|max:255',
            'file_content' => 'required|string',
            'filename' => 'required|string|min:1|max:255',
            // unique ref is nothing but the signee ref returned
            // by signere when a receiver is created
            'signee_refs.*.unique_ref' => 'required|string|size:36',
            'signee_refs.*.first_name' => 'required|string|min:1|max:255',
            'signee_refs.*.last_name' => 'required|string|min:1|max:255',
            'signee_refs.*.email' => 'required|email|min:1|max:255',
            'title' => 'required|string|min:1|max:255',
        ]);

        $body = [];

        // this is used to only set the keys which have been sent in
        $body['Description'] = $request->description;
        $body['ExternalDocumentId'] = $request->ext_doc_id;
        $body['FileContent'] = $request->file_content;

The property file_content does not seem to exist. Did you mean content?

        $body['Filename'] = $request->filename;
        $body['Title'] = $request->title;

        $body['SigneeRefs'] = [];

        // populate the signee references
        foreach ($request->signee_refs as $signee) {

This code seems to be duplicated across your project.

            // append this to the body
            $body['SigneeRefs'][] = [
                'UniqueRef' => $signee['unique_ref'],
                'FirstName' => $signee['first_name'],
                'LastName' => $signee['last_name'],
                'Email' => $signee['email'],
            ];
        }

        // populate the callback URLs
        $body['ReturnUrlError'] = Config::get('signere.sign_error_url');
        $body['ReturnUrlSuccess'] = Config::get('signere.sign_success_url');
        $body['ReturnUrlUserAbort'] = Config::get('signere.sign_cancel_url');

        return $this->extSign->createRequest($body)
                ->getBody()
                ->getContents();
    }
}