BookmartController::add() - Code Metrics - Elbar/InvestMe - Measure and Improve Code Quality continuously with Scrutinizer

BookmartController::add() A
last analyzed 2016-05-23 13:40 UTC

↳ Parent: BookmartController

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	13
rs	9.4285
cc	1
eloc	9
nc	1
nop	1

<?php

namespace App\Http\Controllers;

use App\Bookmark;
use App\Project;
use App\Http\Requests;
use Illuminate\Support\Facades\Auth;

class BookmartController extends Controller
{
    public function add($id)
    {
        $id_user = Auth::user()->id;
        $project = Project::find($id);
        $model = new Bookmark();
        $model -> title = $project ->title;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $model -> user_id = $id_user;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $model -> project_id = $project -> id;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $model ->save();

        return redirect('/show/'.$id);

    }

    public function delete($id)
    {
        $model = Bookmark::find($id);
        $model->delete();
    }

}


1			<?php
2
3			namespace App\Http\Controllers;
4
5			use App\Bookmark;
6			use App\Project;
7			use App\Http\Requests;
8			use Illuminate\Support\Facades\Auth;
9
10			class BookmartController extends Controller
11			{
12			public function add($id)
13			{
14			$id_user = Auth::user()->id;
15			$project = Project::find($id);
16			$model = new Bookmark();
17			$model -> title = $project ->title;
			0 ignored issues – show Documentation introduced 2016-05-19 05:29 UTC by Report Bug Copy Issue Report The property `title` does not exist on `object<App\Bookmark>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
18			$model -> user_id = $id_user;
			0 ignored issues – show Documentation introduced 2016-05-19 05:29 UTC by Report Bug Copy Issue Report The property `user_id` does not exist on `object<App\Bookmark>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
19			$model -> project_id = $project -> id;
			0 ignored issues – show Documentation introduced 2016-05-19 05:29 UTC by Report Bug Copy Issue Report The property `project_id` does not exist on `object<App\Bookmark>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
20			$model ->save();
21
22			return redirect('/show/'.$id);
23
24			}
25
26			public function delete($id)
27			{
28			$model = Bookmark::find($id);
29			$model->delete();
30			}
31
32			}
33

Elbar / InvestMe

BookmartController::add() A last analyzed 2016-05-23 13:40 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

BookmartController::add() A
last analyzed 2016-05-23 13:40 UTC