Merge pull request #66 from openaustralia/fix/constructors-return

Br3nda · web-flow · commit d6cfd7924222 · 2026-04-05T11:36:05.000+12:00
constructors shouldn't return values
diff --git a/www/includes/easyparliament/searchengine.php b/www/includes/easyparliament/searchengine.php
@@ -66,7 +66,7 @@ class SEARCHENGINE {
      */
     public function __construct($query) {
         if (!defined('XAPIANDB') || !XAPIANDB) {
-            return NULL;
+            return;
         }
 
         $this->query = $query;

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ class SEARCHENGINE {`
`66`	`66`	`*/`
`67`	`67`	`public function __construct($query) {`
`68`	`68`	`if (!defined('XAPIANDB') \|\| !XAPIANDB) {`
`69`		`- return NULL;`
	`69`	`+ return;`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`$this->query = $query;`