Notice: Undefined property: Drupal\ldap_servers\Entity\Server::$connection in Drupal\ldap_servers\Entity\Server->search() (line 494 of modules/contrib/ldap/ldap_servers/src/Entity/Server.php).

CommentFileSizeAuthor
#3 2727647-server.patch465 byteslarowlan
#2 undefined_property-2727647-2.patch847 bytesgrahl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grahl created an issue. See original summary.

grahl’s picture

grahl
he/him
Location Zürich
CreditAttribution: grahl at Namics commented
Status: Active » Needs review
FileSize
undefined_property-2727647-2.patch847 bytes

I kept the check to !$this->connection in case that gets instantiated in some places but could return false (haven't checked).

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext for Department of Justice & Community Safety, Victoria commented
FileSize
2727647-server.patch465 bytes

I think this is a better approach, default value is NULL.

  • larowlan committed 1bdd8c5 on 8.x-3.x 
    Issue #2727647 by grahl, larowlan: Undefined property connection
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext for Department of Justice & Community Safety, Victoria commented
Status: Needs review » Fixed

Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.