Create hash on demand if no is available yet

juniwalk · Oct 12, 2022 · f153fef · f153fef
1 parent c04cc69
commit f153fef
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/src/Chronicler.php b/src/Chronicler.php
@@ -74,12 +74,8 @@ public function record(Record $record): void
 
 	public function isRecorded(Record $record, string $period = null): bool
 	{
-		if (!$hash = $record->getHash()) {
-			return false;
-		}
-
-		$result = $this->recordRepository->findOneBy(function($qb) use ($hash, $period) {
-			$qb->andWhere('e.hash = :hash')->setParameter('hash', $hash);
+		$result = $this->recordRepository->findOneBy(function($qb) use ($record, $period) {
+			$qb->andWhere('e.hash = :hash')->setParameter('hash', $record->getHash());
 
 			if (is_null($period)) {
 				return $qb;

diff --git a/src/Entity/Record.php b/src/Entity/Record.php
@@ -191,9 +191,9 @@ public function getNote(): ?string
 	}
 
 
-	public function getHash(): ?string
+	public function getHash(): string
 	{
-		return $this->hash;
+		return $this->hash ?: $this->createUniqueHash();
 	}