Add default values to class member variables

[airbundle] / Entity / Snippet.php

diff --git a/Entity/Snippet.php b/Entity/Snippet.php
index 999091f6733d220b9f92c72cbf83cf405cdde7c2..dea8897dd0fe3e0b68416b46378d0edec0fb0c38 100644 (file)
--- a/Entity/Snippet.php
+++ b/Entity/Snippet.php
@@ -1,94 +1,93 @@
-<?php
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of the Rapsys AirBundle package.
+ *
+ * (c) Raphaël Gertz <symfony@rapsys.eu>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
 
 namespace Rapsys\AirBundle\Entity;
 
-use Rapsys\AirBundle\Entity\Location;
-use Rapsys\AirBundle\Entity\User;
+use Doctrine\ORM\Event\PreUpdateEventArgs;
 
 /**
  * Snippet
  */
 class Snippet {
        /**
-        * @var integer
+        * Primary key
         */
-       private $id;
+       private ?int $id = null;
 
        /**
         * @var string
         */
-       protected $locale;
+       private ?string $description = null;
 
        /**
         * @var string
         */
-       protected $description;
+       private ?string $class = null;
 
        /**
         * @var string
         */
-       protected $class;
-
-       /**
-        * @var string
-        */
-       protected $short;
+       private ?string $short = null;
 
        /**
         * @var integer
         */
-       protected $rate;
+       private ?int $rate = null;
 
        /**
         * @var bool
         */
-       protected $hat;
+       private ?bool $hat = null;
 
        /**
         * @var string
         */
-       protected $contact;
+       private ?string $contact = null;
 
        /**
         * @var string
         */
-       protected $donate;
+       private ?string $donate = null;
 
        /**
         * @var string
         */
-       protected $link;
+       private ?string $link = null;
 
        /**
         * @var string
         */
-       protected $profile;
-
-       /**
-        * @var \DateTime
-        */
-       protected $created;
+       private ?string $profile = null;
 
        /**
-        * @var \DateTime
+        * Create datetime
         */
-       protected $updated;
+       private \DateTime $created;
 
        /**
-        * @var \Rapsys\UserBundle\Entity\Location
+        * Update datetime
         */
-       protected $location;
-
-       /**
-        * @var \Rapsys\UserBundle\Entity\User
-        */
-       protected $user;
+       private \DateTime $updated;
 
        /**
         * Constructor
+        *
+        * @param string $locale The locale
+        * @param Location $location The location instance
+        * @param User $user The user instance
         */
-       public function __construct() {
-               $this->hat = true;
+       public function __construct(private string $locale, private Location $location, private User $user) {
+               //Set defaults
+               $this->created = new \DateTime('now');
+               $this->updated = new \DateTime('now');
        }
 
        /**
@@ -96,7 +95,7 @@ class Snippet {
         *
         * @return integer
         */
-       public function getId() {
+       public function getId(): ?int {
                return $this->id;
        }
 
@@ -107,7 +106,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setLocale($locale) {
+       public function setLocale(string $locale): Snippet {
                $this->locale = $locale;
 
                return $this;
@@ -118,7 +117,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getLocale() {
+       public function getLocale(): string {
                return $this->locale;
        }
 
@@ -129,7 +128,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setDescription($description) {
+       public function setDescription(?string $description): Snippet {
                $this->description = $description;
 
                return $this;
@@ -140,7 +139,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getDescription() {
+       public function getDescription(): ?string {
                return $this->description;
        }
 
@@ -151,7 +150,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setClass($class) {
+       public function setClass(?string $class): Snippet {
                $this->class = $class;
 
                return $this;
@@ -162,7 +161,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getClass() {
+       public function getClass(): ?string {
                return $this->class;
        }
 
@@ -173,7 +172,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setShort($short) {
+       public function setShort(?string $short): Snippet {
                $this->short = $short;
 
                return $this;
@@ -184,18 +183,18 @@ class Snippet {
         *
         * @return string
         */
-       public function getShort() {
+       public function getShort(): ?string {
                return $this->short;
        }
 
        /**
         * Set rate
         *
-        * @param string $rate
+        * @param int $rate
         *
         * @return Snippet
         */
-       public function setRate($rate) {
+       public function setRate(?int $rate): Snippet {
                $this->rate = $rate;
 
                return $this;
@@ -204,9 +203,9 @@ class Snippet {
        /**
         * Get rate
         *
-        * @return string
+        * @return int
         */
-       public function getRate() {
+       public function getRate(): ?int {
                return $this->rate;
        }
 
@@ -217,7 +216,7 @@ class Snippet {
         *
         * @return User
         */
-       public function setHat(bool $hat) {
+       public function setHat(?bool $hat): Snippet {
                $this->hat = $hat;
 
                return $this;
@@ -228,7 +227,7 @@ class Snippet {
         *
         * @return bool
         */
-       public function getHat(): bool {
+       public function getHat(): ?bool {
                return $this->hat;
        }
        /**
@@ -238,7 +237,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setContact($contact) {
+       public function setContact(?string $contact): Snippet {
                $this->contact = $contact;
 
                return $this;
@@ -249,7 +248,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getContact() {
+       public function getContact(): ?string {
                return $this->contact;
        }
 
@@ -260,7 +259,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setDonate($donate) {
+       public function setDonate(?string $donate): Snippet {
                $this->donate = $donate;
 
                return $this;
@@ -271,7 +270,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getDonate() {
+       public function getDonate(): ?string {
                return $this->donate;
        }
 
@@ -282,7 +281,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setLink($link) {
+       public function setLink(?string $link): Snippet {
                $this->link = $link;
 
                return $this;
@@ -293,7 +292,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getLink() {
+       public function getLink(): ?string {
                return $this->link;
        }
 
@@ -304,7 +303,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setProfile($profile) {
+       public function setProfile(?string $profile): Snippet {
                $this->profile = $profile;
 
                return $this;
@@ -315,7 +314,7 @@ class Snippet {
         *
         * @return string
         */
-       public function getProfile() {
+       public function getProfile(): ?string {
                return $this->profile;
        }
 
@@ -326,7 +325,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setCreated($created) {
+       public function setCreated(\DateTime $created): Snippet {
                $this->created = $created;
 
                return $this;
@@ -337,7 +336,7 @@ class Snippet {
         *
         * @return \DateTime
         */
-       public function getCreated() {
+       public function getCreated(): \DateTime {
                return $this->created;
        }
 
@@ -348,7 +347,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setUpdated($updated) {
+       public function setUpdated(\DateTime $updated): Snippet {
                $this->updated = $updated;
 
                return $this;
@@ -359,7 +358,7 @@ class Snippet {
         *
         * @return \DateTime
         */
-       public function getUpdated() {
+       public function getUpdated(): \DateTime {
                return $this->updated;
        }
 
@@ -370,7 +369,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setLocation(Location $location) {
+       public function setLocation(Location $location): Snippet {
                $this->location = $location;
 
                return $this;
@@ -381,7 +380,7 @@ class Snippet {
         *
         * @return Location
         */
-       public function getLocation() {
+       public function getLocation(): Location {
                return $this->location;
        }
 
@@ -392,7 +391,7 @@ class Snippet {
         *
         * @return Snippet
         */
-       public function setUser(User $user) {
+       public function setUser(User $user): Snippet {
                $this->user = $user;
 
                return $this;
@@ -403,16 +402,16 @@ class Snippet {
         *
         * @return User
         */
-       public function getUser() {
+       public function getUser(): User {
                return $this->user;
        }
 
        /**
         * {@inheritdoc}
         */
-       public function preUpdate(\Doctrine\ORM\Event\PreUpdateEventArgs  $eventArgs) {
+       public function preUpdate(PreUpdateEventArgs $eventArgs) {
                //Check that we have an snippet instance
-               if (($snippet = $eventArgs->getEntity()) instanceof Snippet) {
+               if (($snippet = $eventArgs->getObject()) instanceof Snippet) {
                        //Set updated value
                        $snippet->setUpdated(new \DateTime('now'));
                }