X-Git-Url: https://git.rapsys.eu/.gitweb.cgi/airbundle/blobdiff_plain/62564be28991a32bf3b72f334bd165da5bdb088d..7c929ed199ece6713e03b943415cbf5b3f68e254:/Entity/Location.php?ds=sidebyside

diff --git a/Entity/Location.php b/Entity/Location.php
index 4af1280..f0e1a17 100644
--- a/Entity/Location.php
+++ b/Entity/Location.php
@@ -1,16 +1,17 @@
 <?php declare(strict_types=1);
 
 /*
- * this file is part of the rapsys packbundle package.
+ * This file is part of the Rapsys AirBundle package.
  *
- * (c) raphaël gertz <symfony@rapsys.eu>
+ * (c) Raphaël Gertz <symfony@rapsys.eu>
  *
- * for the full copyright and license information, please view the license
+ * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
 
 namespace Rapsys\AirBundle\Entity;
 
+use Doctrine\Common\Collections\Collection;
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\ORM\Event\PreUpdateEventArgs;
 
@@ -19,74 +20,49 @@ use Doctrine\ORM\Event\PreUpdateEventArgs;
  */
 class Location {
 	/**
-	 * @var integer
+	 * Primary key
 	 */
-	private $id;
+	private ?int $id = null;
 
 	/**
 	 * @var string
 	 */
-	private $title;
+	private ?string $description = null;
 
 	/**
-	 * @var string
-	 */
-	private $address;
-
-	/**
-	 * @var string
+	 * Create datetime
 	 */
-	private $zipcode;
+	private \DateTime $created;
 
 	/**
-	 * @var string
+	 * Update datetime
 	 */
-	private $city;
+	private \DateTime $updated;
 
 	/**
-	 * @var string
+	 * Sessions collection
 	 */
-	private $latitude;
+	private Collection $sessions;
 
 	/**
-	 * @var string
+	 * Snippets collection
 	 */
-	private $longitude;
+	private Collection $snippets;
 
 	/**
-	 * @var boolean
+	 * Users collection
 	 */
-	private $hotspot;
-
-	/**
-	 * @var \DateTime
-	 */
-	private $created;
-
-	/**
-	 * @var \DateTime
-	 */
-	private $updated;
-
-	/**
-	 * @var ArrayCollection
-	 */
-	private $sessions;
-
-	/**
-	 * @var ArrayCollection
-	 */
-	private $snippets;
-
-	/**
-	 * @var ArrayCollection
-	 */
-	private $users;
+	private Collection $users;
 
 	/**
 	 * Constructor
 	 */
-	public function __construct() {
+	public function __construct(private string $title = '', private string $address = '', private string $zipcode = '', private string $city = '', private float $latitude = 0, private float $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();
@@ -97,7 +73,7 @@ class Location {
 	 *
 	 * @return integer
 	 */
-	public function getId(): int {
+	public function getId(): ?int {
 		return $this->id;
 	}
 
@@ -123,6 +99,28 @@ class Location {
 		return $this->title;
 	}
 
+	/**
+	 * Set description
+	 *
+	 * @param string $description
+	 *
+	 * @return Location
+	 */
+	public function setDescription(?string $description): Location {
+		$this->description = $description;
+
+		return $this;
+	}
+
+	/**
+	 * Get description
+	 *
+	 * @return string
+	 */
+	public function getDescription(): ?string {
+		return $this->description;
+	}
+
 	/**
 	 * Set address
 	 *
@@ -233,10 +231,32 @@ class Location {
 		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 boolean $hotspot
+	 * @param bool $hotspot
 	 *
 	 * @return Session
 	 */
@@ -249,7 +269,7 @@ class Location {
 	/**
 	 * Get hotspot
 	 *
-	 * @return boolean
+	 * @return bool
 	 */
 	public function getHotspot(): bool {
 		return $this->hotspot;
@@ -316,7 +336,7 @@ class Location {
 	 * Remove session
 	 *
 	 * @param Session $session
-	 * @return boolean
+	 * @return bool
 	 */
 	public function removeSession(Session $session): bool {
 		return $this->sessions->removeElement($session);
@@ -348,7 +368,7 @@ class Location {
 	 * Remove snippet
 	 *
 	 * @param Snippet $snippet
-	 * @return boolean
+	 * @return bool
 	 */
 	public function removeSnippet(Snippet $snippet): bool {
 		return $this->snippets->removeElement($snippet);
@@ -371,6 +391,9 @@ class Location {
 	 * @return Location
 	 */
 	public function addUser(User $user): Location {
+		//Add from owning side
+		$user->addLocation($this);
+
 		$this->users[] = $user;
 
 		return $this;
@@ -380,9 +403,16 @@ class Location {
 	 * Remove user
 	 *
 	 * @param User $user
-	 * @return boolean
+	 * @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);
 	}
 
@@ -400,7 +430,7 @@ class Location {
 	 */
 	public function preUpdate(PreUpdateEventArgs $eventArgs) {
 		//Check that we have a location instance
-		if (($location = $eventArgs->getEntity()) instanceof Location) {
+		if (($location = $eventArgs->getObject()) instanceof Location) {
 			//Set updated value
 			$location->setUpdated(new \DateTime('now'));
 		}