Merge pull request #224 from DigitalTimK/bugfix/bbb_constructor

GhaziTriki · web-flow · commit 568ae13b7248 · 2024-04-01T06:58:12.000+01:00
Bugfix check in BBB-constructor
diff --git a/src/BigBlueButton.php b/src/BigBlueButton.php
@@ -78,7 +78,7 @@ class BigBlueButton
     public function __construct(?string $baseUrl = null, ?string $secret = null, ?array $opts = [])
     {
         // Provide an early error message if configuration is wrong
-        if (is_null($secret) && false === getenv('BBB_SERVER_BASE_URL')) {
+        if (is_null($baseUrl) && false === getenv('BBB_SERVER_BASE_URL')) {
             throw new \RuntimeException('No BBB-Server-Url found! Please provide it either in constructor ' .
                 "(1st argument) or by environment variable 'BBB_SERVER_BASE_URL'!");
         }

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ class BigBlueButton`
`78`	`78`	`public function __construct(?string $baseUrl = null, ?string $secret = null, ?array $opts = [])`
`79`	`79`	`{`
`80`	`80`	`// Provide an early error message if configuration is wrong`
`81`		`- if (is_null($secret) && false === getenv('BBB_SERVER_BASE_URL')) {`
	`81`	`+ if (is_null($baseUrl) && false === getenv('BBB_SERVER_BASE_URL')) {`
`82`	`82`	`throw new \RuntimeException('No BBB-Server-Url found! Please provide it either in constructor ' .`
`83`	`83`	`"(1st argument) or by environment variable 'BBB_SERVER_BASE_URL'!");`
`84`	`84`	`}`