CreateAnswerForm::callbackSubmit() - Code Metrics - Nicklas766/Comment - Measure and Improve Code Quality continuously with Scrutinizer

CreateAnswerForm::callbackSubmit() B
last analyzed 2017-10-24 13:43 UTC

↳ Parent: CreateAnswerForm

Complexity

Conditions	3
Paths	3

Size

Total Lines	29
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
dl	0
loc	29
ccs	0
cts	18
cp	0
rs	8.8571
c	0
b	0
f	0
cc	3
eloc	18
nc	3
nop	0
crap	12

<?php

namespace Nicklas\Comment\HTMLForm\Comment;

use \Anax\HTMLForm\FormModel;
use \Anax\DI\DIInterface;
use \Nicklas\Comment\Modules\Post;

/**
 * Example of FormModel implementation.
 */
class CreateAnswerForm extends FormModel
{
    /**
     * Constructor injects with DI container.
     *
     * @param Anax\DI\DIInterface $di a service container
     */
    public function __construct(DIInterface $di, $questionId)
    {
        $this->questionId = $questionId;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        parent::__construct($di);
        $this->form->create(
            [
                "id" => __CLASS__,
            ],
            [
                "text" => [
                    "type"        => "textarea",
                    "label" => "Här kan du skriva ditt svar",
                    "placeholder" => "Ditt svar"
                ],

                "submit" => [
                    "type" => "submit",
                    "value" => "Svara",
                    "callback" => [$this, "callbackSubmit"]
                ],
            ]
        );
    }



    /**
     * Callback for submit-button which should return true if it could
     * carry out its work and false if something failed.
     *
     * @return boolean true if okey, false if something went wrong.
     */
    public function callbackSubmit()
    {
        // Get values from the submitted form
        $text       = $this->form->value("text");

        if (!$this->di->get('session')->has("user")) {
            $this->form->addOutput("Du behöver logga in för att kommentera.");
            return false;
        }

        if ($text == "") {
            $this->form->addOutput("Du skrev aldrig något. Skriv gärna något.");
            return false;
        }


        $user = $this->di->get('session')->get("user"); # get user name

        $post     = new Post($this->di->get("db"));
        $post->questionId = $this->questionId;
        $post->user       = $user;
        $post->text       = $text;
        $post->type       = "answer";
        $post->accepted   = "no";
        $post->save();

        $this->form->addOutput("Du skapade ett svar!");
        return true;
    }
}


1			<?php
2
3			namespace Nicklas\Comment\HTMLForm\Comment;
4
5			use \Anax\HTMLForm\FormModel;
6			use \Anax\DI\DIInterface;
7			use \Nicklas\Comment\Modules\Post;
8
9			/**
10			* Example of FormModel implementation.
11			*/
12			class CreateAnswerForm extends FormModel
13			{
14			/**
15			* Constructor injects with DI container.
16			*
17			* @param Anax\DI\DIInterface $di a service container
18			*/
19			public function __construct(DIInterface $di, $questionId)
20			{
21			$this->questionId = $questionId;
			0 ignored issues – show Bug introduced 2017-10-18 17:30 UTC by Report Bug Copy Issue Report The property `questionId` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
22			parent::__construct($di);
23			$this->form->create(
24			[
25			"id" => __CLASS__,
26			],
27			[
28			"text" => [
29			"type" => "textarea",
30			"label" => "Här kan du skriva ditt svar",
31			"placeholder" => "Ditt svar"
32			],
33
34			"submit" => [
35			"type" => "submit",
36			"value" => "Svara",
37			"callback" => [$this, "callbackSubmit"]
38			],
39			]
40			);
41			}
42
43
44
45			/**
46			* Callback for submit-button which should return true if it could
47			* carry out its work and false if something failed.
48			*
49			* @return boolean true if okey, false if something went wrong.
50			*/
51			public function callbackSubmit()
52			{
53			// Get values from the submitted form
54			$text = $this->form->value("text");
55
56			if (!$this->di->get('session')->has("user")) {
57			$this->form->addOutput("Du behöver logga in för att kommentera.");
58			return false;
59			}
60
61			if ($text == "") {
62			$this->form->addOutput("Du skrev aldrig något. Skriv gärna något.");
63			return false;
64			}
65
66
67			$user = $this->di->get('session')->get("user"); # get user name
68
69			$post = new Post($this->di->get("db"));
70			$post->questionId = $this->questionId;
71			$post->user = $user;
72			$post->text = $text;
73			$post->type = "answer";
74			$post->accepted = "no";
75			$post->save();
76
77			$this->form->addOutput("Du skapade ett svar!");
78			return true;
79			}
80			}
81

Nicklas766 / Comment

CreateAnswerForm::callbackSubmit() B last analyzed 2017-10-24 13:43 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

CreateAnswerForm::callbackSubmit() B
last analyzed 2017-10-24 13:43 UTC