AttachmentContainingDBElement::__construct() - Code Metrics - Inspection of "Allow to create new parts." - Part-DB/Part-DB-symfony - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 813e7d...76abef )

by Jan

created 2019-08-19 21:42 UTC

AttachmentContainingDBElement::__construct() A

↳ Parent: AttachmentContainingDBElement

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php
/**
 *
 * part-db version 0.1
 * Copyright (C) 2005 Christoph Lechner
 * http://www.cl-projects.de/
 *
 * part-db version 0.2+
 * Copyright (C) 2009 K. Jacobs and others (see authors.php)
 * http://code.google.com/p/part-db/
 *
 * Part-DB Version 0.4+
 * Copyright (C) 2016 - 2019 Jan Böhmer
 * https://github.com/jbtronics
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
 *
 */

declare(strict_types=1);
/**
 * Part-DB Version 0.4+ "nextgen"
 * Copyright (C) 2016 - 2019 Jan Böhmer
 * https://github.com/jbtronics.
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
 */

namespace App\Entity\Attachments;

use App\Entity\Base\NamedDBElement;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\ORM\Mapping as ORM;
use Doctrine\ORM\PersistentCollection;
use Exception;

/**
 * @ORM\MappedSuperclass()
 */
abstract class AttachmentContainingDBElement extends NamedDBElement
{
    /**
     * @var Attachment[]
     * //TODO
     * //@ORM\OneToMany(targetEntity="Attachment", mappedBy="element")
     *
     * Mapping is done in sub classes like part
     */
    protected $attachments;

    //TODO
    protected $attachmentTypes;

    public function __construct()
    {
        $this->attachments = new ArrayCollection();

    }

    /********************************************************************************
     *
     *   Getters
     *
     *********************************************************************************/

    /**
     * Get all different attachement types of the attachements of this element.
     *
     * @return AttachmentType[] the attachement types as a one-dimensional array of AttachementType objects,
     *                          sorted by their names
     *
     * @throws Exception if there was an error
     */
    public function getAttachmentTypes(): ?array
    {
        return $this->attachmentTypes;
    }

    /**
     * Get all attachements of this element / Get the element's attachements with a specific type.
     *
     * @param int  $type_id                 * if NULL, all attachements of this element will be returned
     *                                      * if this is a number > 0, only attachements with this type ID will be returned
     * @param bool $only_table_attachements if true, only attachements with "show_in_table == true"
     *
     * @return Attachment[] the attachements as a one-dimensional array of Attachement objects
     *
     * @throws Exception if there was an error
     */
    public function getAttachments($type_id = null, bool $only_table_attachements = false) : Collection
    {
        if ($only_table_attachements || $type_id) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $attachements = $this->attachments;

            foreach ($attachements as $key => $attachement) {
                if (($only_table_attachements && (!$attachement->getShowInTable()))
                    || ($type_id && ($attachement->getType()->getID() !== $type_id))) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
                    unset($attachements[$key]);
                }
            }

            return $attachements;

        }

        return $this->attachments;

    }
}


1			<?php
2			/**
3			*
4			* part-db version 0.1
5			* Copyright (C) 2005 Christoph Lechner
6			* http://www.cl-projects.de/
7			*
8			* part-db version 0.2+
9			* Copyright (C) 2009 K. Jacobs and others (see authors.php)
10			* http://code.google.com/p/part-db/
11			*
12			* Part-DB Version 0.4+
13			* Copyright (C) 2016 - 2019 Jan Böhmer
14			* https://github.com/jbtronics
15			*
16			* This program is free software; you can redistribute it and/or
17			* modify it under the terms of the GNU General Public License
18			* as published by the Free Software Foundation; either version 2
19			* of the License, or (at your option) any later version.
20			*
21			* This program is distributed in the hope that it will be useful,
22			* but WITHOUT ANY WARRANTY; without even the implied warranty of
23			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
24			* GNU General Public License for more details.
25			*
26			* You should have received a copy of the GNU General Public License
27			* along with this program; if not, write to the Free Software
28			* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
29			*
30			*/
31
32			declare(strict_types=1);
33			/**
34			* Part-DB Version 0.4+ "nextgen"
35			* Copyright (C) 2016 - 2019 Jan Böhmer
36			* https://github.com/jbtronics.
37			*
38			* This program is free software; you can redistribute it and/or
39			* modify it under the terms of the GNU General Public License
40			* as published by the Free Software Foundation; either version 2
41			* of the License, or (at your option) any later version.
42			*
43			* This program is distributed in the hope that it will be useful,
44			* but WITHOUT ANY WARRANTY; without even the implied warranty of
45			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
46			* GNU General Public License for more details.
47			*
48			* You should have received a copy of the GNU General Public License
49			* along with this program; if not, write to the Free Software
50			* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
51			*/
52
53			namespace App\Entity\Attachments;
54
55			use App\Entity\Base\NamedDBElement;
56			use Doctrine\Common\Collections\ArrayCollection;
57			use Doctrine\Common\Collections\Collection;
58			use Doctrine\ORM\Mapping as ORM;
59			use Doctrine\ORM\PersistentCollection;
60			use Exception;
61
62			/**
63			* @ORM\MappedSuperclass()
64			*/
65			abstract class AttachmentContainingDBElement extends NamedDBElement
66			{
67			/**
68			* @var Attachment[]
69			* //TODO
70			* //@ORM\OneToMany(targetEntity="Attachment", mappedBy="element")
71			*
72			* Mapping is done in sub classes like part
73			*/
74			protected $attachments;
75
76			//TODO
77			protected $attachmentTypes;
78
79			public function __construct()
80			{
81			$this->attachments = new ArrayCollection();
			0 ignored issues – show Documentation Bug introduced 2019-08-19 21:46 UTC by Report Bug Copy Issue Report It seems like `new Doctrine\Common\Collections\ArrayCollection()` of type `Doctrine\Common\Collections\ArrayCollection` is incompatible with the declared type `App\Entity\Attachments\Attachment[]` of property `$attachments`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
82			}
83
84			/********************************************************************************
85			*
86			* Getters
87			*
88			*********************************************************************************/
89
90			/**
91			* Get all different attachement types of the attachements of this element.
92			*
93			* @return AttachmentType[] the attachement types as a one-dimensional array of AttachementType objects,
94			* sorted by their names
95			*
96			* @throws Exception if there was an error
97			*/
98			public function getAttachmentTypes(): ?array
99			{
100			return $this->attachmentTypes;
101			}
102
103			/**
104			* Get all attachements of this element / Get the element's attachements with a specific type.
105			*
106			* @param int $type_id * if NULL, all attachements of this element will be returned
107			* * if this is a number > 0, only attachements with this type ID will be returned
108			* @param bool $only_table_attachements if true, only attachements with "show_in_table == true"
109			*
110			* @return Attachment[] the attachements as a one-dimensional array of Attachement objects
111			*
112			* @throws Exception if there was an error
113			*/
114			public function getAttachments($type_id = null, bool $only_table_attachements = false) : Collection
115			{
116			if ($only_table_attachements \|\| $type_id) {
			0 ignored issues – show Bug Best Practice introduced 2019-03-09 19:53 UTC by Report Bug Copy Issue Report The expression `$type_id` of type `integer\|null` is loosely compared to `true`; this is ambiguous if the integer can be 0. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
117			$attachements = $this->attachments;
118
119			foreach ($attachements as $key => $attachement) {
120			if (($only_table_attachements && (!$attachement->getShowInTable()))
121			\|\| ($type_id && ($attachement->getType()->getID() !== $type_id))) {
			0 ignored issues – show Bug Best Practice introduced 2019-03-09 19:53 UTC by Report Bug Copy Issue Report The expression `$type_id` of type `integer\|null` is loosely compared to `true`; this is ambiguous if the integer can be 0. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
122			unset($attachements[$key]);
123			}
124			}
125
126			return $attachements;
			0 ignored issues – show Bug Best Practice introduced 2019-08-12 16:08 UTC by Report Bug Copy Issue Report The expression `return $attachements` returns the type `App\Entity\Attachments\Attachment[]` which is incompatible with the type-hinted return `Doctrine\Common\Collections\Collection`. Loading history...
127			}
128
129			return $this->attachments;
			0 ignored issues – show Bug Best Practice introduced 2019-08-12 16:08 UTC by Report Bug Copy Issue Report The expression `return $this->attachments` returns the type `App\Entity\Attachments\Attachment[]` which is incompatible with the type-hinted return `Doctrine\Common\Collections\Collection`. Loading history...
130			}
131			}
132

Part-DB / Part-DB-symfony

Push — master ( 813e7d...76abef )

AttachmentContainingDBElement::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like