Fix location latitude and longitude field type

[airbundle] / Entity / Location.php

diff --git a/Entity/Location.php b/Entity/Location.php
index 43605ed6142ff70fb260a5af6857283f3a19d0c1..62ef9fc60bdae7ec8686cd4310a3021f2b79d43b 100644 (file)
--- a/Entity/Location.php
+++ b/Entity/Location.php
@@ -1,71 +1,71 @@
-<?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;
 
 
 namespace Rapsys\AirBundle\Entity;
 

+use Doctrine\Common\Collections\Collection;
+use Doctrine\Common\Collections\ArrayCollection;
+use Doctrine\ORM\Event\PreUpdateEventArgs;
+

 /**
  * Location
  */
 class Location {
        /**
 /**
  * Location
  */
 class Location {
        /**

-        * @var integer
-        */
-       private $id;
-
-       /**
-        * @var string
+        * Primary key

         */
         */

-       private $title;
+       private ?int $id = null;

 
        /**
         * @var string
         */
 
        /**
         * @var string
         */

-       private $short;
+       private ?string $description = null;

 
        /**
 
        /**

-        * @var string
+        * Create datetime

         */
         */

-       private $address;
+       private \DateTime $created;

 
        /**
 
        /**

-        * @var string
+        * Update datetime

         */
         */

-       private $zipcode;
+       private \DateTime $updated;

 
        /**
 
        /**

-        * @var string
+        * Sessions collection

         */
         */

-       private $city;
+       private Collection $sessions;

 
        /**
 
        /**

-        * @var string
+        * Snippets collection

         */
         */

-       private $latitude;
+       private Collection $snippets;

 
        /**
 
        /**

-        * @var string
+        * Users collection

         */
         */

-       private $longitude;
-
-       /**
-        * @var \DateTime
-        */
-       private $created;
-
-       /**
-        * @var \DateTime
-        */
-       private $updated;
-
-       /**
-        * @var \Doctrine\Common\Collections\Collection
-        */
-       private $sessions;
+       private Collection $users;

 
        /**
         * Constructor
         */
 
        /**
         * Constructor
         */

-       public function __construct() {
-               $this->sessions = new \Doctrine\Common\Collections\ArrayCollection();
+       public function __construct(private string $title = '', private string $address = '', private string $zipcode = '', private string $city = '', private string $latitude = '0', private string $longitude = '0', private bool $hotspot = false, private bool $indoor = false) {
+               //Set defaults
+               $this->created = new \DateTime('now');
+               $this->updated = new \DateTime('now');
+
+               //Set collections
+               $this->sessions = new ArrayCollection();
+               $this->snippets = new ArrayCollection();
+               $this->users = new ArrayCollection();

        }
 
        /**
        }
 
        /**


@@ -73,7 +73,7 @@ class Location {
         *
         * @return integer
         */
         *
         * @return integer
         */

-       public function getId() {
+       public function getId(): ?int {

                return $this->id;
        }
 
                return $this->id;
        }
 


@@ -84,7 +84,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setTitle($title) {
+       public function setTitle(string $title): Location {

                $this->title = $title;
 
                return $this;
                $this->title = $title;
 
                return $this;


@@ -95,30 +95,30 @@ class Location {
         *
         * @return string
         */
         *
         * @return string
         */

-       public function getTitle() {
+       public function getTitle(): string {

                return $this->title;
        }
 
        /**
                return $this->title;
        }
 
        /**

-        * Set short
+        * Set description

         *
         *

-        * @param string $short
+        * @param string $description

         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setShort($short) {
-               $this->short = $short;
+       public function setDescription(?string $description): Location {
+               $this->description = $description;

 
                return $this;
        }
 
        /**
 
                return $this;
        }
 
        /**

-        * Get short
+        * Get description

         *
         * @return string
         */
         *
         * @return string
         */

-       public function getShort() {
-               return $this->short;
+       public function getDescription(): ?string {
+               return $this->description;

        }
 
        /**
        }
 
        /**


@@ -128,7 +128,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setAddress($address) {
+       public function setAddress(string $address): Location {

                $this->address = $address;
 
                return $this;
                $this->address = $address;
 
                return $this;


@@ -139,7 +139,7 @@ class Location {
         *
         * @return string
         */
         *
         * @return string
         */

-       public function getAddress() {
+       public function getAddress(): string {

                return $this->address;
        }
 
                return $this->address;
        }
 


@@ -150,7 +150,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setZipcode($zipcode) {
+       public function setZipcode(string $zipcode): Location {

                $this->zipcode = $zipcode;
 
                return $this;
                $this->zipcode = $zipcode;
 
                return $this;


@@ -161,7 +161,7 @@ class Location {
         *
         * @return string
         */
         *
         * @return string
         */

-       public function getZipcode() {
+       public function getZipcode(): string {

                return $this->zipcode;
        }
 
                return $this->zipcode;
        }
 


@@ -172,7 +172,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setCity($city) {
+       public function setCity(string $city): Location {

                $this->city = $city;
 
                return $this;
                $this->city = $city;
 
                return $this;


@@ -183,7 +183,7 @@ class Location {
         *
         * @return string
         */
         *
         * @return string
         */

-       public function getCity() {
+       public function getCity(): string {

                return $this->city;
        }
 
                return $this->city;
        }
 


@@ -194,7 +194,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setLatitude($latitude) {
+       public function setLatitude(string $latitude): Location {

                $this->latitude = $latitude;
 
                return $this;
                $this->latitude = $latitude;
 
                return $this;


@@ -205,7 +205,7 @@ class Location {
         *
         * @return string
         */
         *
         * @return string
         */

-       public function getLatitude() {
+       public function getLatitude(): string {

                return $this->latitude;
        }
 
                return $this->latitude;
        }
 


@@ -216,7 +216,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setLongitude($longitude) {
+       public function setLongitude(string $longitude): Location {

                $this->longitude = $longitude;
 
                return $this;
                $this->longitude = $longitude;
 
                return $this;


@@ -227,10 +227,54 @@ class Location {
         *
         * @return string
         */
         *
         * @return string
         */

-       public function getLongitude() {
+       public function getLongitude(): string {

                return $this->longitude;
        }
 
                return $this->longitude;
        }
 

+       /**
+        * Set indoor
+        *
+        * @param bool $indoor
+        *
+        * @return Session
+        */
+       public function setIndoor(bool $indoor): Location {
+               $this->indoor = $indoor;
+
+               return $this;
+       }
+
+       /**
+        * Get indoor
+        *
+        * @return bool
+        */
+       public function getIndoor(): bool {
+               return $this->indoor;
+       }
+
+       /**
+        * Set hotspot
+        *
+        * @param bool $hotspot
+        *
+        * @return Session
+        */
+       public function setHotspot(bool $hotspot): Location {
+               $this->hotspot = $hotspot;
+
+               return $this;
+       }
+
+       /**
+        * Get hotspot
+        *
+        * @return bool
+        */
+       public function getHotspot(): bool {
+               return $this->hotspot;
+       }
+

        /**
         * Set created
         *
        /**
         * Set created
         *


@@ -238,7 +282,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setCreated($created) {
+       public function setCreated(\DateTime $created): Location {

                $this->created = $created;
 
                return $this;
                $this->created = $created;
 
                return $this;


@@ -249,7 +293,7 @@ class Location {
         *
         * @return \DateTime
         */
         *
         * @return \DateTime
         */

-       public function getCreated() {
+       public function getCreated(): \DateTime {

                return $this->created;
        }
 
                return $this->created;
        }
 


@@ -260,7 +304,7 @@ class Location {
         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function setUpdated($updated) {
+       public function setUpdated(\DateTime $updated): Location {

                $this->updated = $updated;
 
                return $this;
                $this->updated = $updated;
 
                return $this;


@@ -271,18 +315,18 @@ class Location {
         *
         * @return \DateTime
         */
         *
         * @return \DateTime
         */

-       public function getUpdated() {
+       public function getUpdated(): \DateTime {

                return $this->updated;
        }
 
        /**
         * Add session
         *
                return $this->updated;
        }
 
        /**
         * Add session
         *

-        * @param \Rapsys\AirBundle\Entity\Session $session
+        * @param Session $session

         *
         * @return Location
         */
         *
         * @return Location
         */

-       public function addSession(\Rapsys\AirBundle\Entity\Session $session) {
+       public function addSession(Session $session): Location {

                $this->sessions[] = $session;
 
                return $this;
                $this->sessions[] = $session;
 
                return $this;


@@ -291,21 +335,107 @@ class Location {
        /**
         * Remove session
         *
        /**
         * Remove session
         *

-        * @param \Rapsys\AirBundle\Entity\Session $session
+        * @param Session $session
+        * @return bool

         */
         */

-       public function removeSession(\Rapsys\AirBundle\Entity\Session $session) {
-               $this->sessions->removeElement($session);
+       public function removeSession(Session $session): bool {
+               return $this->sessions->removeElement($session);

        }
 
        /**
         * Get sessions
         *
        }
 
        /**
         * Get sessions
         *

-        * @return \Doctrine\Common\Collections\Collection
+        * @return ArrayCollection

         */
         */

-       public function getSessions() {
+       public function getSessions(): ArrayCollection {

                return $this->sessions;
        }
 
                return $this->sessions;
        }
 

+       /**
+        * Add snippet
+        *
+        * @param Snippet $snippet
+        *
+        * @return Location
+        */
+       public function addSnippet(Snippet $snippet): Location {
+               $this->snippets[] = $snippet;
+
+               return $this;
+       }
+
+       /**
+        * Remove snippet
+        *
+        * @param Snippet $snippet
+        * @return bool
+        */
+       public function removeSnippet(Snippet $snippet): bool {
+               return $this->snippets->removeElement($snippet);
+       }
+
+       /**
+        * Get snippets
+        *
+        * @return ArrayCollection
+        */
+       public function getSnippets(): ArrayCollection {
+               return $this->snippets;
+       }
+
+       /**
+        * Add user
+        *
+        * @param User $user
+        *
+        * @return Location
+        */
+       public function addUser(User $user): Location {
+               //Add from owning side
+               $user->addLocation($this);
+
+               $this->users[] = $user;
+
+               return $this;
+       }
+
+       /**
+        * Remove user
+        *
+        * @param User $user
+        * @return bool
+        */
+       public function removeUser(User $user): bool {
+               if (!$this->locations->contains($user)) {
+                       return true;
+               }
+
+               //Remove from owning side
+               $user->removeLocation($this);
+
+               return $this->users->removeElement($user);
+       }
+
+       /**
+        * Get users
+        *
+        * @return ArrayCollection
+        */
+       public function getUsers(): ArrayCollection {
+               return $this->users;
+       }
+
+       /**
+        * {@inheritdoc}
+        */
+       public function preUpdate(PreUpdateEventArgs $eventArgs) {
+               //Check that we have a location instance
+               if (($location = $eventArgs->getObject()) instanceof Location) {
+                       //Set updated value
+                       $location->setUpdated(new \DateTime('now'));
+               }
+       }
+

        /**
         * Returns a string representation of the location
         *
        /**
         * Returns a string representation of the location
         *