diff --git a/src/Controller/AdminPages/AttachmentTypeController.php b/src/Controller/AdminPages/AttachmentTypeController.php index 25341828..bd220f99 100644 --- a/src/Controller/AdminPages/AttachmentTypeController.php +++ b/src/Controller/AdminPages/AttachmentTypeController.php @@ -41,7 +41,6 @@ use Doctrine\ORM\EntityManagerInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\Routing\Annotation\Route; -use Symfony\Component\Serializer\SerializerInterface; /** * @Route("/attachment_type") diff --git a/src/Controller/AdminPages/BaseAdminController.php b/src/Controller/AdminPages/BaseAdminController.php index 8a9745ca..caab4012 100644 --- a/src/Controller/AdminPages/BaseAdminController.php +++ b/src/Controller/AdminPages/BaseAdminController.php @@ -108,7 +108,9 @@ abstract class BaseAdminController extends AbstractController $this->addFlash('success', $this->translator->trans('entity.created_flash')); return $this->redirectToRoute($this->route_base . '_edit', ['id' => $new_entity->getID()]); - } elseif ($form->isSubmitted() && ! $form->isValid()) { + } + + if ($form->isSubmitted() && ! $form->isValid()) { $this->addFlash('error', $this->translator->trans('entity.created_flash.invalid')); } diff --git a/src/Controller/AdminPages/CategoryController.php b/src/Controller/AdminPages/CategoryController.php index 468272ee..b3cf9809 100644 --- a/src/Controller/AdminPages/CategoryController.php +++ b/src/Controller/AdminPages/CategoryController.php @@ -32,7 +32,6 @@ namespace App\Controller\AdminPages; -use App\Entity\Attachments\AttachmentType; use App\Entity\Parts\Category; use App\Form\AdminPages\CategoryAdminForm; use App\Services\EntityExporter; @@ -42,7 +41,6 @@ use Doctrine\ORM\EntityManagerInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\Routing\Annotation\Route; -use Symfony\Component\Serializer\SerializerInterface; /** * @Route("/category") diff --git a/src/Controller/AdminPages/CurrencyController.php b/src/Controller/AdminPages/CurrencyController.php index a95b5218..b261d66c 100644 --- a/src/Controller/AdminPages/CurrencyController.php +++ b/src/Controller/AdminPages/CurrencyController.php @@ -32,7 +32,6 @@ namespace App\Controller\AdminPages; -use App\Entity\Attachments\AttachmentType; use App\Entity\PriceInformations\Currency; use App\Form\AdminPages\CurrencyAdminForm; use App\Services\EntityExporter; @@ -42,7 +41,6 @@ use Doctrine\ORM\EntityManagerInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\Routing\Annotation\Route; -use Symfony\Component\Serializer\SerializerInterface; /** * @Route("/currency") diff --git a/src/Controller/AdminPages/DeviceController.php b/src/Controller/AdminPages/DeviceController.php index 02820efd..1b50079d 100644 --- a/src/Controller/AdminPages/DeviceController.php +++ b/src/Controller/AdminPages/DeviceController.php @@ -32,8 +32,6 @@ namespace App\Controller\AdminPages; -use App\Entity\Attachments\AttachmentType; - use App\Entity\Devices\Device; use App\Form\AdminPages\BaseEntityAdminForm; use App\Services\EntityExporter; @@ -43,7 +41,6 @@ use Doctrine\ORM\EntityManagerInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\Routing\Annotation\Route; -use Symfony\Component\Serializer\SerializerInterface; /** * @Route("/device") diff --git a/src/Controller/PartController.php b/src/Controller/PartController.php index 823326ad..0cf7f0ee 100644 --- a/src/Controller/PartController.php +++ b/src/Controller/PartController.php @@ -30,15 +30,11 @@ namespace App\Controller; -use App\Entity\Base\NamedDBElement; -use App\Entity\Base\StructuralDBElement; use App\Entity\Parts\Category; use App\Entity\Parts\Part; -use App\Form\AttachmentFormType; use App\Form\Part\PartBaseType; use App\Services\AttachmentHelper; use App\Services\PricedetailHelper; -use App\Services\StructuralElementRecursionHelper; use Doctrine\ORM\EntityManagerInterface; use Symfony\Bundle\FrameworkBundle\Controller\AbstractController; use Symfony\Component\Form\FormInterface; @@ -177,7 +173,9 @@ class PartController extends AbstractController $this->addFlash('success', $translator->trans('part.created_flash')); return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]); - } elseif ($form->isSubmitted() && ! $form->isValid()) { + } + + if ($form->isSubmitted() && ! $form->isValid()) { $this->addFlash('error', $translator->trans('part.created_flash.invalid')); } diff --git a/src/Controller/PartListsController.php b/src/Controller/PartListsController.php index a703dc0f..45c6db9d 100644 --- a/src/Controller/PartListsController.php +++ b/src/Controller/PartListsController.php @@ -178,7 +178,7 @@ class PartListsController extends AbstractController /** * @Route("/parts/search/{keyword}", name="parts_search") */ - public function showSearch(Request $request, DataTableFactory $dataTable, string $keyword = "") + public function showSearch(Request $request, DataTableFactory $dataTable, string $keyword = '') { $search = $keyword; diff --git a/src/Controller/RedirectController.php b/src/Controller/RedirectController.php index b9d422a8..a5c62d61 100644 --- a/src/Controller/RedirectController.php +++ b/src/Controller/RedirectController.php @@ -34,7 +34,6 @@ namespace App\Controller; use App\Entity\UserSystem\User; use Symfony\Bundle\FrameworkBundle\Controller\AbstractController; use Symfony\Component\HttpFoundation\Request; -use Symfony\Component\HttpKernel\Exception\NotFoundHttpException; class RedirectController extends AbstractController { @@ -52,10 +51,8 @@ class RedirectController extends AbstractController //Check if a user has set a preferred language setting: $user = $this->getUser(); - if ($user instanceof User) { - if(!empty($user->getLanguage())) { - $locale = $user->getLanguage(); - } + if (($user instanceof User) && !empty($user->getLanguage())) { + $locale = $user->getLanguage(); } //$new_url = str_replace($request->getPathInfo(), '/' . $locale . $request->getPathInfo(), $request->getUri()); diff --git a/src/Entity/Attachments/Attachment.php b/src/Entity/Attachments/Attachment.php index 3df48ae5..1f6f3920 100644 --- a/src/Entity/Attachments/Attachment.php +++ b/src/Entity/Attachments/Attachment.php @@ -23,11 +23,9 @@ declare(strict_types=1); namespace App\Entity\Attachments; -use App\Entity\Base\DBElement; use App\Entity\Base\NamedDBElement; use App\Validator\Constraints\Selectable; use Doctrine\ORM\Mapping as ORM; -use Symfony\Component\Validator\Constraints as Assert; /** * Class Attachment. @@ -52,7 +50,7 @@ abstract class Attachment extends NamedDBElement * @var string The filename using the %BASE% variable * @ORM\Column(type="string", name="filename") */ - protected $path = ""; + protected $path = ''; /** * ORM mapping is done in sub classes (like PartAttachment) @@ -101,8 +99,8 @@ abstract class Attachment extends NamedDBElement { //return static::isUrl($this->getPath()); //Treat all pathes without a filepath as external - return (strpos($this->getPath(), "%MEDIA%") === false) - && (strpos($this->getPath(), "%BASE") === false); + return (strpos($this->getPath(), '%MEDIA%') === false) + && (strpos($this->getPath(), '%BASE') === false); } /******************************************************************************** @@ -270,7 +268,7 @@ abstract class Attachment extends NamedDBElement //Only set if the URL is not empty if (!empty($url)) { if (strpos($url, '%BASE%') !== false || strpos($url, '%MEDIA%') !== false) { - throw new \InvalidArgumentException("You can not reference internal files via the url field! But nice try!"); + throw new \InvalidArgumentException('You can not reference internal files via the url field! But nice try!'); } $this->path = $url; diff --git a/src/Entity/Attachments/AttachmentContainingDBElement.php b/src/Entity/Attachments/AttachmentContainingDBElement.php index f8599268..1f696746 100644 --- a/src/Entity/Attachments/AttachmentContainingDBElement.php +++ b/src/Entity/Attachments/AttachmentContainingDBElement.php @@ -56,7 +56,6 @@ use App\Entity\Base\NamedDBElement; use Doctrine\Common\Collections\ArrayCollection; use Doctrine\Common\Collections\Collection; use Doctrine\ORM\Mapping as ORM; -use Exception; /** * @ORM\MappedSuperclass() diff --git a/src/Entity/Attachments/AttachmentType.php b/src/Entity/Attachments/AttachmentType.php index 38934cfb..4c4991f9 100644 --- a/src/Entity/Attachments/AttachmentType.php +++ b/src/Entity/Attachments/AttachmentType.php @@ -56,7 +56,6 @@ use App\Entity\Base\StructuralDBElement; use Doctrine\Common\Collections\ArrayCollection; use Doctrine\Common\Collections\Collection; use Doctrine\ORM\Mapping as ORM; -use s9e\TextFormatter\Configurator\RendererGenerators\PHP\XPathConvertor\Convertors\SingleByteStringManipulation; /** * Class AttachmentType. diff --git a/src/Entity/Attachments/PartAttachment.php b/src/Entity/Attachments/PartAttachment.php index 124980d0..4601372e 100644 --- a/src/Entity/Attachments/PartAttachment.php +++ b/src/Entity/Attachments/PartAttachment.php @@ -52,7 +52,7 @@ class PartAttachment extends Attachment public function setElement(AttachmentContainingDBElement $element): Attachment { if (!$element instanceof Part) { - throw new \InvalidArgumentException("The element associated with a PartAttachment must be a Part!"); + throw new \InvalidArgumentException('The element associated with a PartAttachment must be a Part!'); } $this->element = $element; diff --git a/src/Entity/Base/StructuralDBElement.php b/src/Entity/Base/StructuralDBElement.php index ffe09e12..b924476e 100644 --- a/src/Entity/Base/StructuralDBElement.php +++ b/src/Entity/Base/StructuralDBElement.php @@ -142,7 +142,7 @@ abstract class StructuralDBElement extends AttachmentContainingDBElement * * @throws \InvalidArgumentException if there was an error */ - public function isChildOf(StructuralDBElement $another_element) + public function isChildOf(StructuralDBElement $another_element) : bool { $class_name = \get_class($this); diff --git a/src/Entity/Parts/Part.php b/src/Entity/Parts/Part.php index ddffd7fc..641aa615 100644 --- a/src/Entity/Parts/Part.php +++ b/src/Entity/Parts/Part.php @@ -61,7 +61,6 @@ declare(strict_types=1); namespace App\Entity\Parts; -use App\Entity\Attachments\Attachment; use App\Entity\Attachments\AttachmentContainingDBElement; use App\Entity\Devices\Device; use App\Entity\Parts\PartTraits\AdvancedPropertyTrait; @@ -70,11 +69,8 @@ use App\Entity\Parts\PartTraits\BasicPropertyTrait; use App\Entity\Parts\PartTraits\InstockTrait; use App\Entity\Parts\PartTraits\ManufacturerTrait; use App\Entity\Parts\PartTraits\OrderTrait; -use App\Entity\PriceInformations\Orderdetail; use App\Security\Annotations\ColumnSecurity; -use App\Validator\Constraints\Selectable; use Doctrine\Common\Collections\ArrayCollection; -use Doctrine\Common\Collections\Collection; use Doctrine\ORM\Mapping as ORM; use Symfony\Component\Validator\Constraints as Assert; @@ -116,6 +112,18 @@ class Part extends AttachmentContainingDBElement */ protected $lastModified; + /*************************************************************** + * Overriden properties + * (They are defined here and not in a trait, to avoid conflicts) + ****************************************************************/ + + /** + * @var string The name of this part + * @ORM\Column(type="string") + * @ColumnSecurity(prefix="name") + */ + protected $name = ''; + /** * @ORM\OneToMany(targetEntity="App\Entity\Attachments\PartAttachment", mappedBy="element", cascade={"persist", "remove"}, orphanRemoval=false) * @ColumnSecurity(type="collection", prefix="attachments") diff --git a/src/Entity/Parts/PartTraits/BasicPropertyTrait.php b/src/Entity/Parts/PartTraits/BasicPropertyTrait.php index 295ba6fe..75d962da 100644 --- a/src/Entity/Parts/PartTraits/BasicPropertyTrait.php +++ b/src/Entity/Parts/PartTraits/BasicPropertyTrait.php @@ -40,13 +40,6 @@ use App\Validator\Constraints\Selectable; trait BasicPropertyTrait { - /** - * @var string The name of this part - * @ORM\Column(type="string") - * @ColumnSecurity(prefix="name") - */ - protected $name = ''; - /** * @var string A text describing what this part does * @ORM\Column(type="text") @@ -182,7 +175,7 @@ trait BasicPropertyTrait * @param Category $category The new category of this part * @return self */ - public function setCategory(Category $category): Part + public function setCategory(Category $category): self { $this->category = $category; return $this; @@ -196,7 +189,7 @@ trait BasicPropertyTrait * * @return self */ - public function setFootprint(?Footprint $new_footprint): Part + public function setFootprint(?Footprint $new_footprint): self { $this->footprint = $new_footprint; return $this; diff --git a/src/Entity/Parts/PartTraits/InstockTrait.php b/src/Entity/Parts/PartTraits/InstockTrait.php index 9a728b63..bc423e6f 100644 --- a/src/Entity/Parts/PartTraits/InstockTrait.php +++ b/src/Entity/Parts/PartTraits/InstockTrait.php @@ -33,7 +33,6 @@ namespace App\Entity\Parts\PartTraits; use App\Entity\Parts\MeasurementUnit; -use App\Entity\Parts\Part; use App\Entity\Parts\PartLot; use App\Security\Annotations\ColumnSecurity; use Doctrine\Common\Collections\Collection; @@ -181,7 +180,7 @@ trait InstockTrait /** * Set the minimum amount of parts that have to be instock. * See getPartUnit() for the associated unit. - * @param int $new_minamount the new count of parts which should be in stock at least + * @param float $new_minamount the new count of parts which should be in stock at least * @return self */ public function setMinAmount(float $new_minamount): self diff --git a/src/Entity/Parts/PartTraits/ManufacturerTrait.php b/src/Entity/Parts/PartTraits/ManufacturerTrait.php index 36226957..f8c9e4d7 100644 --- a/src/Entity/Parts/PartTraits/ManufacturerTrait.php +++ b/src/Entity/Parts/PartTraits/ManufacturerTrait.php @@ -73,7 +73,7 @@ trait ManufacturerTrait * @Assert\Choice({"announced", "active", "nrfnd", "eol", "discontinued", ""}) * @ColumnSecurity(type="string", prefix="status", placeholder="") */ - protected $manufacturing_status = ""; + protected $manufacturing_status = ''; /** * Get the link to the website of the article on the manufacturers website diff --git a/src/Entity/Parts/PartTraits/OrderTrait.php b/src/Entity/Parts/PartTraits/OrderTrait.php index acedbf06..8cae9618 100644 --- a/src/Entity/Parts/PartTraits/OrderTrait.php +++ b/src/Entity/Parts/PartTraits/OrderTrait.php @@ -158,7 +158,7 @@ trait OrderTrait * * @param bool $new_manual_order the new "manual_order" attribute * @param int $new_order_quantity the new order quantity - * @param int|null $new_order_orderdetails_id * the ID of the new order orderdetails + * @param Orderdetail|null $new_order_orderdetail * the ID of the new order orderdetails * * or Zero for "no order orderdetails" * * or NULL for automatic order orderdetails * (if the part has exactly one orderdetails, @@ -167,7 +167,7 @@ trait OrderTrait * * @return self */ - public function setManualOrder(bool $new_manual_order, int $new_order_quantity = 1, ?Orderdetail $new_order_orderdetail = null): Part + public function setManualOrder(bool $new_manual_order, int $new_order_quantity = 1, ?Orderdetail $new_order_orderdetail = null): self { //Assert::greaterThan($new_order_quantity, 0, 'The new order quantity must be greater zero. Got %s!'); diff --git a/src/Entity/Parts/Storelocation.php b/src/Entity/Parts/Storelocation.php index 835a2191..9c67ea36 100644 --- a/src/Entity/Parts/Storelocation.php +++ b/src/Entity/Parts/Storelocation.php @@ -62,7 +62,6 @@ declare(strict_types=1); namespace App\Entity\Parts; use App\Entity\Base\PartsContainingDBElement; -use App\Entity\Base\StructuralDBElement; use Doctrine\ORM\Mapping as ORM; /** diff --git a/src/Entity/Parts/Supplier.php b/src/Entity/Parts/Supplier.php index d7effa4f..4c57b0bd 100644 --- a/src/Entity/Parts/Supplier.php +++ b/src/Entity/Parts/Supplier.php @@ -117,14 +117,16 @@ class Supplier extends Company protected $parts; /** + * Gets the currency that should be used by default, when creating a orderdetail with this supplier. * @return ?Currency */ - public function getDefaultCurrency() + public function getDefaultCurrency() : ?Currency { return $this->default_currency; } /** + * Sets the default currency. * @param ?Currency $default_currency * @return Supplier */ @@ -135,16 +137,17 @@ class Supplier extends Company } /** - * @return ?string + * Gets the shipping costs for an order with this supplier, given in base currency. + * @return string|null A bcmath string with the shipping costs */ public function getShippingCosts() : ?string { - dump($this); return $this->shipping_costs; } /** - * @param ?string $shipping_costs + * Sets the shipping costs for an order with this supplier. + * @param string|null $shipping_costs A bcmath string with the shipping costs. * @return Supplier */ public function setShippingCosts(?string $shipping_costs) : Supplier diff --git a/src/Entity/PriceInformations/Currency.php b/src/Entity/PriceInformations/Currency.php index 6953d248..695c8612 100644 --- a/src/Entity/PriceInformations/Currency.php +++ b/src/Entity/PriceInformations/Currency.php @@ -34,7 +34,6 @@ namespace App\Entity\PriceInformations; use App\Entity\Base\StructuralDBElement; use Doctrine\ORM\Mapping as ORM; -use s9e\TextFormatter\Configurator\TemplateNormalizations\AbstractChooseOptimization; use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity; use Symfony\Component\Validator\Constraints as Assert; diff --git a/src/Entity/PriceInformations/Orderdetail.php b/src/Entity/PriceInformations/Orderdetail.php index ff290aab..1eb8fbd1 100644 --- a/src/Entity/PriceInformations/Orderdetail.php +++ b/src/Entity/PriceInformations/Orderdetail.php @@ -68,9 +68,6 @@ use App\Entity\Parts\Supplier; use Doctrine\Common\Collections\ArrayCollection; use Doctrine\Common\Collections\Collection; use Doctrine\ORM\Mapping as ORM; -use Doctrine\ORM\PersistentCollection; -use Exception; -use Symfony\Component\Form\Extension\Core\Type\CollectionType; use Symfony\Component\Validator\Constraints as Assert; /** @@ -110,7 +107,7 @@ class Orderdetail extends DBElement * @var string * @ORM\Column(type="string") */ - protected $supplierpartnr = ""; + protected $supplierpartnr = ''; /** * @var bool @@ -123,7 +120,7 @@ class Orderdetail extends DBElement * @ORM\Column(type="string") * @Assert\Url() */ - protected $supplier_product_url = ""; + protected $supplier_product_url = ''; public function __construct() { @@ -205,8 +202,8 @@ class Orderdetail extends DBElement return $this->supplier_product_url; } - if ($this->supplier === null) { - return ""; + if ($this->getSupplier() === null) { + return ''; } return $this->getSupplier()->getAutoProductUrl($this->supplierpartnr); // maybe an automatic url is available... @@ -282,10 +279,12 @@ class Orderdetail extends DBElement /** * Sets a new part with which this orderdetail is associated * @param Part $part + * @return Orderdetail */ - public function setPart(Part $part) + public function setPart(Part $part) : Orderdetail { $this->part = $part; + return $this; } /** @@ -330,12 +329,11 @@ class Orderdetail extends DBElement * Set this to "", if the function getSupplierProductURL should return the automatic generated URL. * @param $new_url string The new URL for the supplier URL. * @return Orderdetail - * @return Orderdetail */ - public function setSupplierProductUrl(string $new_url) + public function setSupplierProductUrl(string $new_url) : Orderdetail { //Only change the internal URL if it is not the auto generated one - if ($new_url == $this->supplier->getAutoProductUrl($this->getSupplierPartNr())) { + if ($new_url === $this->supplier->getAutoProductUrl($this->getSupplierPartNr())) { return $this; } diff --git a/src/Entity/PriceInformations/Pricedetail.php b/src/Entity/PriceInformations/Pricedetail.php index b08aa920..39c64ffb 100644 --- a/src/Entity/PriceInformations/Pricedetail.php +++ b/src/Entity/PriceInformations/Pricedetail.php @@ -96,7 +96,7 @@ class Pricedetail extends DBElement * @ORM\Column(type="decimal", precision=11, scale=5) * @Assert\Positive() */ - protected $price = "0.0"; + protected $price = '0.0'; /** * @var ?Currency The currency used for the current price information. @@ -210,11 +210,9 @@ class Pricedetail extends DBElement */ public function getPriceRelatedQuantity(): float { - if ($this->orderdetail && $this->orderdetail->getPart()) { - if (!$this->orderdetail->getPart()->useFloatAmount()) { - $tmp = round($this->price_related_quantity); - return $tmp < 1 ? 1 : $tmp; - } + if ($this->orderdetail && $this->orderdetail->getPart() && !$this->orderdetail->getPart()->useFloatAmount()) { + $tmp = round($this->price_related_quantity); + return $tmp < 1 ? 1 : $tmp; } return $this->price_related_quantity; } @@ -227,17 +225,15 @@ class Pricedetail extends DBElement * * The amount is measured in part unit. * - * @return int the minimum discount quantity + * @return float the minimum discount quantity * * @see Pricedetail::setMinDiscountQuantity() */ public function getMinDiscountQuantity(): float { - if ($this->orderdetail && $this->orderdetail->getPart()) { - if (!$this->orderdetail->getPart()->useFloatAmount()) { - $tmp = round($this->min_discount_quantity); - return $tmp < 1 ? 1 : $tmp; - } + if ($this->orderdetail && $this->orderdetail->getPart() && !$this->orderdetail->getPart()->useFloatAmount()) { + $tmp = round($this->min_discount_quantity); + return $tmp < 1 ? 1 : $tmp; } return $this->min_discount_quantity; @@ -264,7 +260,7 @@ class Pricedetail extends DBElement * @param Orderdetail $orderdetail * @return $this */ - public function setOrderdetail(Orderdetail $orderdetail) + public function setOrderdetail(Orderdetail $orderdetail) : self { $this->orderdetail = $orderdetail; return $this; @@ -311,7 +307,7 @@ class Pricedetail extends DBElement * If 100pcs costs 20$, you have to set the price to 20$ and the price related * quantity to 100. The single price (20$/100 = 0.2$) will be calculated automatically. * - * @param int $new_price_related_quantity the price related quantity + * @param float $new_price_related_quantity the price related quantity * * @return self */ @@ -337,7 +333,7 @@ class Pricedetail extends DBElement * (Each of this examples would be an own Pricedetails-object. * So the orderdetails would have three Pricedetails for one supplier.) * - * @param int $new_min_discount_quantity the minimum discount quantity + * @param float $new_min_discount_quantity the minimum discount quantity * * @return self */ diff --git a/src/Entity/UserSystem/User.php b/src/Entity/UserSystem/User.php index ead625dc..25b8d942 100644 --- a/src/Entity/UserSystem/User.php +++ b/src/Entity/UserSystem/User.php @@ -182,22 +182,22 @@ class User extends NamedDBElement implements UserInterface, HasPermissionsInterf * @ORM\ManyToOne(targetEntity="App\Entity\PriceInformations\Currency", fetch="EAGER") * @ORM\JoinColumn(name="currency_id", referencedColumnName="id") */ - protected $currency = ""; + protected $currency = ''; /** * @ORM\Column(type="text", name="config_image_path") */ - protected $image_path = ""; + protected $image_path = ''; /** * @ORM\Column(type="text", name="config_instock_comment_w") */ - protected $instock_comment_w = ""; + protected $instock_comment_w = ''; /** * @ORM\Column(type="text", name="config_instock_comment_a") */ - protected $instock_comment_a = ""; + protected $instock_comment_a = ''; public function __construct() {