Vervang var->prop === null door !isset(var->prop)

M2WZ · M2WZ · commit da89b99cbe14 · 2025-01-03T15:25:09.000+01:00
diff --git a/htdocs/bar/controller/Barsysteem.class.php b/htdocs/bar/controller/Barsysteem.class.php
@@ -36,7 +36,7 @@ function isBeheer() {
 	}
 
 	function getCsrfToken() {
-		if ($this->csrfToken === null) {
+		if (!isset($this->csrfToken)) {
 			if ($this->isBeheer()) {
 				$this->csrfToken = md5('Barsysteem CSRF-token C.S.R. Delft' . $_COOKIE['barsysteembeheer']);
 			} else {
diff --git a/lib/common/Doctrine/Type/Serializer/SafeJsonSerializer.php b/lib/common/Doctrine/Type/Serializer/SafeJsonSerializer.php
@@ -70,7 +70,7 @@ protected function unserializeObject($value)
 	 */
 	protected function classAllowed($className)
 	{
-		return $this->allowedClasses === null ||
+		return !isset($this->allowedClasses) ||
 			in_array($className, $this->allowedClasses);
 	}
 }
diff --git a/lib/controller/ProfielController.php b/lib/controller/ProfielController.php
@@ -282,7 +282,7 @@ private function profielBewerken(Profiel $profiel, $alleenFormulier = false)
 				$this->addFlash(FlashType::INFO, 'Geen wijzigingen');
 			} else {
 				$nieuw =
-					$profiel->uid === null ||
+					!isset($profiel->uid) ||
 					$this->profielRepository->find($profiel->uid) == null;
 				$changeEntry = ProfielRepository::changelog($diff, $this->getUid());
 				foreach ($diff as $change) {
diff --git a/lib/controller/fiscaat/PinTransactieController.php b/lib/controller/fiscaat/PinTransactieController.php
@@ -153,11 +153,11 @@ public function aanmaken()
 				PinTransactieMatch::class
 			);
 
-			if ($pinTransactieMatch->bestelling !== null) {
+			if (isset($pinTransactieMatch->bestelling)) {
 				throw new CsrGebruikerException('Er bestaat al een bestelling.');
 			}
 
-			if ($pinTransactieMatch->transactie === null) {
+			if (!isset($pinTransactieMatch->transactie)) {
 				throw new CsrGebruikerException(
 					'Geen transactie gevonden om een bestelling voor aan te maken'
 				);
@@ -249,11 +249,11 @@ public function ontkoppel()
 				PinTransactieMatch::class
 			);
 
-			if ($pinTransactieMatch->bestelling === null) {
+			if (!isset($pinTransactieMatch->bestelling)) {
 				throw new CsrGebruikerException(
 					'Ontoppelen niet mogelijk, geen bestelling gevonden.'
 				);
-			} elseif ($pinTransactieMatch->transactie === null) {
+			} elseif (!isset($pinTransactieMatch->transactie)) {
 				throw new CsrGebruikerException(
 					'Ontkoppelen niet mogelijk, geen transactie gevonden.'
 				);
@@ -318,16 +318,16 @@ public function koppel()
 				$pinTransactieMatch2
 			) {
 				if (
-					$pinTransactieMatch1->bestelling === null &&
-					$pinTransactieMatch2->transactie === null
+					!isset($pinTransactieMatch1->bestelling) &&
+					!isset($pinTransactieMatch2->transactie)
 				) {
 					$nieuwePinTransactieMatch = $this->koppelMatches(
 						$pinTransactieMatch2,
 						$pinTransactieMatch1
 					);
 				} elseif (
-					$pinTransactieMatch2->bestelling === null &&
-					$pinTransactieMatch1->transactie === null
+					!isset($pinTransactieMatch2->bestelling) &&
+					!isset($pinTransactieMatch1->transactie)
 				) {
 					$nieuwePinTransactieMatch = $this->koppelMatches(
 						$pinTransactieMatch1,
@@ -408,11 +408,11 @@ public function crediteer()
 			$form = new PinBestellingCrediterenForm($oudePinTransactieMatch);
 			$values = $form->getValues();
 
-			if ($oudePinTransactieMatch->transactie !== null) {
+			if (isset($oudePinTransactieMatch->transactie)) {
 				throw new CsrGebruikerException('Er bestaat wel een transactie.');
 			}
 
-			if ($oudePinTransactieMatch->bestelling === null) {
+			if (!isset($oudePinTransactieMatch->bestelling)) {
 				throw new CsrGebruikerException(
 					'Geen bestelling gevonden om een creditbestelling voor aan te maken'
 				);
@@ -511,13 +511,13 @@ public function update()
 			$form = new PinBestellingVeranderenForm($oudePinTransactieMatch);
 			$values = $form->getValues();
 
-			if ($oudePinTransactieMatch->transactie === null) {
+			if (!isset($oudePinTransactieMatch->transactie)) {
 				throw new CsrGebruikerException(
 					'Geen transactie gevonden voor verkeerd bedrag'
 				);
 			}
 
-			if ($oudePinTransactieMatch->bestelling === null) {
+			if (!isset($oudePinTransactieMatch->bestelling)) {
 				throw new CsrGebruikerException(
 					'Geen bestelling gevonden voor verkeerd bedrag'
 				);
diff --git a/lib/entity/corvee/CorveeTaak.php b/lib/entity/corvee/CorveeTaak.php
@@ -153,7 +153,7 @@ public function getPuntenPrognose()
 
 	public function getLaatstGemaildDate()
 	{
-		if ($this->wanneer_gemaild === null) {
+		if (!isset($this->wanneer_gemaild)) {
 			return null;
 		}
 		$pos = strpos($this->wanneer_gemaild, '&#013;');
@@ -170,7 +170,7 @@ public function getLaatstGemaildDate()
 	 */
 	public function getAantalKeerGemaild()
 	{
-		if ($this->wanneer_gemaild === null) {
+		if (!isset($this->wanneer_gemaild)) {
 			return 0;
 		}
 		return substr_count($this->wanneer_gemaild, '&#013;');
diff --git a/lib/entity/declaratie/Declaratie.php b/lib/entity/declaratie/Declaratie.php
@@ -395,7 +395,7 @@ public function valideer(): array
 		if (empty($this->omschrijving)) {
 			$fouten[] = 'Geef een duidelijke omschrijving van de declaratie';
 		}
-		if ($this->csrPas === null) {
+		if (!isset($this->csrPas)) {
 			$fouten[] = 'Selecteer de betaalwijze';
 		} elseif ($this->csrPas === true) {
 			if (empty($this->naam)) {
@@ -490,7 +490,7 @@ public function naarObject(UrlGeneratorInterface $generator): array
 	{
 		$eigenRekening =
 			$this->csrPas ||
-			$this->csrPas === null ||
+			!isset($this->csrPas) ||
 			($this->rekening === $this->indiener->bankrekening &&
 				$this->naam === $this->indiener->getNaam('voorletters'));
 		return [
diff --git a/lib/entity/declaratie/DeclaratieRegel.php b/lib/entity/declaratie/DeclaratieRegel.php
@@ -177,7 +177,7 @@ public function valideer($bonIndex, $regelIndex): array
 		if (empty($this->bedrag)) {
 			$toFill[] = 'het bedrag';
 		}
-		if ($this->inclBtw === null) {
+		if (!isset($this->inclBtw)) {
 			$toFill[] = 'het btw-percentage';
 		}
 
diff --git a/lib/entity/pin/PinTransactieMatch.php b/lib/entity/pin/PinTransactieMatch.php
@@ -353,9 +353,9 @@ public function getVerschil()
 	 */
 	public function logischeStatus()
 	{
-		if ($this->bestelling === null) {
+		if (!isset($this->bestelling)) {
 			return PinTransactieMatchStatusEnum::STATUS_MISSENDE_BESTELLING;
-		} elseif ($this->transactie === null) {
+		} elseif (!isset($this->transactie)) {
 			return PinTransactieMatchStatusEnum::STATUS_MISSENDE_TRANSACTIE;
 		} else {
 			$bestellingInhoud = $this->bestelling->getProduct(
diff --git a/lib/model/entity/profiel/ProfielCreateLogGroup.php b/lib/model/entity/profiel/ProfielCreateLogGroup.php
@@ -24,7 +24,7 @@ public function toHtml()
 	{
 		return "<div class='ProfielLogEntry'>Aangemaakt door " .
 			ProfielRepository::getLink($this->editor) .
-			($this->timestamp === null
+			(!isset($this->timestamp)
 				? '?'
 				: DateUtil::reldate($this->timestamp->format('Y-m-d H:i:s'))) .
 			'</div>';
diff --git a/lib/model/entity/profiel/ProfielUpdateLogGroup.php b/lib/model/entity/profiel/ProfielUpdateLogGroup.php
@@ -43,7 +43,7 @@ public function toHtml()
 			<div class='metadata'>Gewijzigd door " .
 			ProfielRepository::getLink($this->editor, 'civitas') .
 			' ' .
-			($this->timestamp === null
+			(!isset($this->timestamp)
 				? '?'
 				: DateUtil::reldate($this->timestamp->format('Y-m-d H:i:s'))) .
 			"</div>
diff --git a/lib/repository/corvee/CorveeTakenRepository.php b/lib/repository/corvee/CorveeTakenRepository.php
@@ -258,7 +258,7 @@ public function getKomendeTakenVoorLid(Profiel $profiel)
 	public function saveTaak(CorveeTaak $taak)
 	{
 		return $this->_em->transactional(function () use ($taak) {
-			if ($taak->taak_id === null) {
+			if (!isset($taak->taak_id)) {
 				$taak = $this->newTaak($taak);
 			} else {
 				$oldTaak = $this->getEntityManager()
diff --git a/lib/repository/forum/ForumDradenRepository.php b/lib/repository/forum/ForumDradenRepository.php
@@ -175,7 +175,7 @@ public function getPaginaVoorDraad(ForumDraad $draad)
 		if ($draad->plakkerig) {
 			return 1;
 		}
-		if ($this->aantal_plakkerig === null) {
+		if (!isset($this->aantal_plakkerig)) {
 			$qb = $this->createQueryBuilder('d');
 			$qb->select('count(d.draad_id)');
 			$qb->where(
diff --git a/lib/service/LidZoekerService.php b/lib/service/LidZoekerService.php
@@ -246,7 +246,7 @@ public function getSortableVelden()
 
 	public function count()
 	{
-		if ($this->result === null) {
+		if (!isset($this->result)) {
 			$this->search();
 		}
 		return count($this->result);
@@ -491,7 +491,7 @@ public function searched()
 
 	public function getLeden()
 	{
-		if ($this->result === null) {
+		if (!isset($this->result)) {
 			$this->search();
 		}
 		return $this->result;
diff --git a/lib/service/Roodschopper.php b/lib/service/Roodschopper.php
@@ -63,7 +63,7 @@ public static function getDefaults()
 	 */
 	public function getLeden()
 	{
-		if ($this->teschoppen === null) {
+		if (!isset($this->teschoppen)) {
 			$this->generateMails();
 		}
 		$leden = [];
@@ -155,7 +155,7 @@ public function replace($invoer, $profiel, $saldo)
 	 */
 	public function preview()
 	{
-		if ($this->teschoppen === null) {
+		if (!isset($this->teschoppen)) {
 			$this->generateMails();
 		}
 		foreach ($this->teschoppen as $uid => $bericht) {
@@ -172,7 +172,7 @@ public function preview()
 	 */
 	public function sendMails()
 	{
-		if ($this->teschoppen === null) {
+		if (!isset($this->teschoppen)) {
 			$this->generateMails();
 		}
 
diff --git a/lib/view/formulier/CsrfField.php b/lib/view/formulier/CsrfField.php
@@ -47,7 +47,7 @@ public function getType()
 
 	public function getHtml()
 	{
-		if ($this->token === null) {
+		if (!isset($this->token)) {
 			return '';
 		}
 
diff --git a/lib/view/formulier/invoervelden/InputField.php b/lib/view/formulier/invoervelden/InputField.php
@@ -90,7 +90,7 @@ public function __construct(
 			static::class
 		);
 
-		if ($this->description === null) {
+		if (!isset($this->description)) {
 			$this->labelClassName .= ' d-none';
 			$this->fieldClassName = str_replace(
 				'col-9',
diff --git a/lib/view/maalcie/forms/CorveeRepetitieForm.php b/lib/view/maalcie/forms/CorveeRepetitieForm.php
@@ -55,7 +55,7 @@ public function __construct(CorveeRepetitie $repetitie)
 				']=' .
 				$functie->standaard_punten .
 				';';
-			if ($repetitie->standaard_punten === null) {
+			if (!isset($repetitie->standaard_punten)) {
 				$repetitie->standaard_punten = $functie->standaard_punten;
 			}
 		}
diff --git a/lib/view/maalcie/forms/MaaltijdRepetitieForm.php b/lib/view/maalcie/forms/MaaltijdRepetitieForm.php
@@ -39,7 +39,7 @@ public function __construct($model, $verplaats = null)
 			'/maaltijden/repetities/opslaan/' . $model->mlt_repetitie_id
 		);
 
-		if ($model->mlt_repetitie_id === null) {
+		if (!isset($model->mlt_repetitie_id)) {
 			$this->titel = 'Maaltijdrepetitie aanmaken';
 		} else {
 			$this->titel = 'Maaltijdrepetitie wijzigen';
diff --git a/lib/view/maalcie/forms/TaakForm.php b/lib/view/maalcie/forms/TaakForm.php
@@ -48,7 +48,7 @@ public function __construct(CorveeTaak $taak, $action)
 			];
 			$functiePunten .=
 				'punten["' . $functie->naam . '"]=' . $functie->standaard_punten . ';';
-			if ($taak->punten === null) {
+			if (!isset($taak->punten)) {
 				$taak->punten = $functie->standaard_punten;
 			}
 		}
diff --git a/lib/view/maalcie/forms/VrijstellingForm.php b/lib/view/maalcie/forms/VrijstellingForm.php
@@ -25,10 +25,10 @@ public function __construct(CorveeVrijstelling $vrijstelling)
 		parent::__construct(
 			$vrijstelling,
 			'/corvee/vrijstellingen/opslaan' .
-				($vrijstelling->uid === null ? '' : '/' . $vrijstelling->uid)
+				(isset($vrijstelling->uid) ? '/' . $vrijstelling->uid : '')
 		);
 
-		if ($vrijstelling->uid === null) {
+		if (!isset($vrijstelling->uid)) {
 			$this->titel = 'Vrijstelling aanmaken';
 		} else {
 			$this->titel = 'Vrijstelling wijzigen';

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ function isBeheer() {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`function getCsrfToken() {`
`39`		`- if ($this->csrfToken === null) {`
	`39`	`+ if (!isset($this->csrfToken)) {`
`40`	`40`	`if ($this->isBeheer()) {`
`41`	`41`	`$this->csrfToken = md5('Barsysteem CSRF-token C.S.R. Delft' . $_COOKIE['barsysteembeheer']);`
`42`	`42`	`} else {`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ protected function unserializeObject($value)`
`70`	`70`	`*/`
`71`	`71`	`protected function classAllowed($className)`
`72`	`72`	`{`
`73`		`- return $this->allowedClasses === null \|\|`
	`73`	`+ return !isset($this->allowedClasses) \|\|`
`74`	`74`	`in_array($className, $this->allowedClasses);`
`75`	`75`	`}`
`76`	`76`	`}`
Original file line number	Diff line number	Diff line change
`@@ -153,7 +153,7 @@ public function getPuntenPrognose()`
`153`	`153`
`154`	`154`	`public function getLaatstGemaildDate()`
`155`	`155`	`{`
`156`		`- if ($this->wanneer_gemaild === null) {`
	`156`	`+ if (!isset($this->wanneer_gemaild)) {`
`157`	`157`	`return null;`
`158`	`158`	`}`
`159`	`159`	`$pos = strpos($this->wanneer_gemaild, ' ');`
`@@ -170,7 +170,7 @@ public function getLaatstGemaildDate()`
`170`	`170`	`*/`
`171`	`171`	`public function getAantalKeerGemaild()`
`172`	`172`	`{`
`173`		`- if ($this->wanneer_gemaild === null) {`
	`173`	`+ if (!isset($this->wanneer_gemaild)) {`
`174`	`174`	`return 0;`
`175`	`175`	`}`
`176`	`176`	`return substr_count($this->wanneer_gemaild, ' ');`
Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ public function valideer($bonIndex, $regelIndex): array`
`177`	`177`	`if (empty($this->bedrag)) {`
`178`	`178`	`$toFill[] = 'het bedrag';`
`179`	`179`	`}`
`180`		`- if ($this->inclBtw === null) {`
	`180`	`+ if (!isset($this->inclBtw)) {`
`181`	`181`	`$toFill[] = 'het btw-percentage';`
`182`	`182`	`}`
`183`	`183`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public function toHtml()`
`24`	`24`	`{`
`25`	`25`	`return "<div class='ProfielLogEntry'>Aangemaakt door " .`
`26`	`26`	`ProfielRepository::getLink($this->editor) .`
`27`		`- ($this->timestamp === null`
	`27`	`+ (!isset($this->timestamp)`
`28`	`28`	`? '?'`
`29`	`29`	`: DateUtil::reldate($this->timestamp->format('Y-m-d H:i:s'))) .`
`30`	`30`	`'</div>';`
Original file line number	Diff line number	Diff line change
`@@ -258,7 +258,7 @@ public function getKomendeTakenVoorLid(Profiel $profiel)`
`258`	`258`	`public function saveTaak(CorveeTaak $taak)`
`259`	`259`	`{`
`260`	`260`	`return $this->_em->transactional(function () use ($taak) {`
`261`		`- if ($taak->taak_id === null) {`
	`261`	`+ if (!isset($taak->taak_id)) {`
`262`	`262`	`$taak = $this->newTaak($taak);`
`263`	`263`	`} else {`
`264`	`264`	`$oldTaak = $this->getEntityManager()`
Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ public function getPaginaVoorDraad(ForumDraad $draad)`
`175`	`175`	`if ($draad->plakkerig) {`
`176`	`176`	`return 1;`
`177`	`177`	`}`
`178`		`- if ($this->aantal_plakkerig === null) {`
	`178`	`+ if (!isset($this->aantal_plakkerig)) {`
`179`	`179`	`$qb = $this->createQueryBuilder('d');`
`180`	`180`	`$qb->select('count(d.draad_id)');`
`181`	`181`	`$qb->where(`
Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ public function getSortableVelden()`
`246`	`246`
`247`	`247`	`public function count()`
`248`	`248`	`{`
`249`		`- if ($this->result === null) {`
	`249`	`+ if (!isset($this->result)) {`
`250`	`250`	`$this->search();`
`251`	`251`	`}`
`252`	`252`	`return count($this->result);`
`@@ -491,7 +491,7 @@ public function searched()`
`491`	`491`
`492`	`492`	`public function getLeden()`
`493`	`493`	`{`
`494`		`- if ($this->result === null) {`
	`494`	`+ if (!isset($this->result)) {`
`495`	`495`	`$this->search();`
`496`	`496`	`}`
`497`	`497`	`return $this->result;`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ public static function getDefaults()`
`63`	`63`	`*/`
`64`	`64`	`public function getLeden()`
`65`	`65`	`{`
`66`		`- if ($this->teschoppen === null) {`
	`66`	`+ if (!isset($this->teschoppen)) {`
`67`	`67`	`$this->generateMails();`
`68`	`68`	`}`
`69`	`69`	`$leden = [];`
`@@ -155,7 +155,7 @@ public function replace($invoer, $profiel, $saldo)`
`155`	`155`	`*/`
`156`	`156`	`public function preview()`
`157`	`157`	`{`
`158`		`- if ($this->teschoppen === null) {`
	`158`	`+ if (!isset($this->teschoppen)) {`
`159`	`159`	`$this->generateMails();`
`160`	`160`	`}`
`161`	`161`	`foreach ($this->teschoppen as $uid => $bericht) {`
`@@ -172,7 +172,7 @@ public function preview()`
`172`	`172`	`*/`
`173`	`173`	`public function sendMails()`
`174`	`174`	`{`
`175`		`- if ($this->teschoppen === null) {`
	`175`	`+ if (!isset($this->teschoppen)) {`
`176`	`176`	`$this->generateMails();`
`177`	`177`	`}`
`178`	`178`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function getType()`
`47`	`47`
`48`	`48`	`public function getHtml()`
`49`	`49`	`{`
`50`		`- if ($this->token === null) {`
	`50`	`+ if (!isset($this->token)) {`
`51`	`51`	`return '';`
`52`	`52`	`}`
`53`	`53`