BristolSU\Module\StaticPage\CompletionConditions\HasClickedSubmit

Complexity

Total Complexity

7

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	alias()	0	3	1
A	percentage()	0	6	2
A	name()	0	3	1
A	options()	0	3	1
A	description()	0	3	1
A	isComplete()	0	3	1

<?php

Missing file doc comment

namespace BristolSU\Module\StaticPage\CompletionConditions;


use BristolSU\Module\StaticPage\Models\ButtonClick;
use BristolSU\Module\StaticPage\Models\PageView;
use BristolSU\Support\ActivityInstance\ActivityInstance;
use BristolSU\Support\Completion\Contracts\CompletionCondition;
use BristolSU\Support\ModuleInstance\Contracts\ModuleInstance;
use FormSchema\Schema\Form;

class HasClickedSubmit extends CompletionCondition

Missing doc comment for class HasClickedSubmit

{

    /**
     * Is the condition fully complete?
     *
     * @param array $settings Settings of the completion condition

Expected 12 spaces after parameter type; 1 found

Expected 9 spaces after parameter name; 1 found

Tag value for @param tag indented incorrectly; expected 2 spaces but found 1

     * @param ActivityInstance $activityInstance Activity instance to test

Tag value for @param tag indented incorrectly; expected 2 spaces but found 1

     * @param ModuleInstance $moduleInstance Module instance to test

Expected 3 spaces after parameter type; 1 found

Expected 3 spaces after parameter name; 1 found

Tag value for @param tag indented incorrectly; expected 2 spaces but found 1

     * @return bool If the condition is complete

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    public function isComplete($settings, ActivityInstance $activityInstance, ModuleInstance $moduleInstance): bool
    {
        return ButtonClick::forResource($activityInstance->id, $moduleInstance->id)->count() > 0;

Accessing id on the interface BristolSU\Support\ModuleInstance\Contracts\ModuleInstance suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    public function percentage($settings, ActivityInstance $activityInstance, ModuleInstance $moduleInstance): int

Missing doc comment for function percentage()

    {
        if($this->isComplete($settings, $activityInstance, $moduleInstance)) {

Expected "if (...) {\n"; found "if(...) {\n"

            return 100;
        }
        return 0;
    }

    /**
     * Options required by the completion condition.
     *
     * Any settings requested in here will be passed into the percentage or isComplete methods.
     *
     * @return Form
     * @throws \Exception
     */
    public function options(): Form
    {
        return \FormSchema\Generator\Form::make()->getSchema();
    }

    /**
     * A name for the completion condition
     *
     * @return string
     */
    public function name(): string
    {
        return 'Has clicked the submit button';
    }

    /**
     * A description of the completion condition
     *
     * @return string
     */
    public function description(): string
    {
        return 'Marked as complete when the user/group/role has clicked the submit button.';
    }

    /**
     * The alias of the completion condition
     *
     * @return string
     */
    public function alias(): string
    {
        return 'static_page_has_submitted';
    }
}