Skip to content

Commit

Permalink
Cleanup some warnings from phpcompatibility
Browse files Browse the repository at this point in the history
  • Loading branch information
effgarces committed Sep 14, 2021
1 parent 01a309b commit bf1b367
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@
"./tools/phpunit",
"@lint"
],
"sniffer:php8": "phpcs -p ./ --standard=vendor/phpcompatibility/php-compatibility/PHPCompatibility --report-full=./php8-report.log --ignore=./vendor/*,./tools/*,./.git/*,./tpl_c/*,./build/*,./.phpdoc/*,./var/* --runtime-set testVersion 8.0"
"sniffer:php8": "phpcs -p ./ --standard=vendor/phpcompatibility/php-compatibility/PHPCompatibility --report-full=./php8-report.log --ignore=./vendor/*,./tools/*,./.git/*,./tpl_c/*,./build/*,./.phpdoc/*,./var/*,./Web/scripts/*,./Web/css/* --runtime-set testVersion 8.0"
}
}
2 changes: 1 addition & 1 deletion plugins/Authentication/Ldap/LDAP2.php
Original file line number Diff line number Diff line change
Expand Up @@ -1642,7 +1642,7 @@ public function registerSchemaCache($cache)
*/
public static function checkLDAPExtension()
{
if (!extension_loaded('ldap') && !@dl('ldap.' . PHP_SHLIB_SUFFIX)) {
if (!extension_loaded('ldap')) {
return new Net_LDAP2_Error("It seems that you do not have the ldap-extension installed. Please install it before using the Net_LDAP2 package.");
} else {
return true;
Expand Down
2 changes: 1 addition & 1 deletion tests/fakes/EmailFakes.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ class FakeMailer extends PHPMailer
public $sendWasCalled = false;
public $isHtml = true;

public function FakeMailer()
public function __construct()
{
}

Expand Down
2 changes: 1 addition & 1 deletion tests/fakes/FakeReminder.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

class FakeReminder extends Reminder
{
public function FakeReminder()
public function __construct()
{
$day = date('d');
$mon = date('m');
Expand Down

0 comments on commit bf1b367

Please sign in to comment.