fix

symfony · ronfroy · Sep 13, 2018 · Sep 13, 2018 · Sep 17, 2018 · Sep 17, 2018
commit f9413e09265e837868f9b2ebc7cbb3c34b59e7a5
diff --git a/src/Symfony/Component/Ldap/Adapter/ExtLdap/Query.php b/src/Symfony/Component/Ldap/Adapter/ExtLdap/Query.php
@@ -69,7 +69,7 @@ public function execute()
                QueryInterface::SCOPE_SUB => 'ldap_search',
            );

-            if (\in_array($this->options['scope'], $funcMap, true)) {
+            if (!\array_key_exists($this->options['scope'], $funcMap)) {
                throw new LdapException(sprintf('Could not search in scope "%s".', $this->options['scope']));
            }
-Original file line number
+Diff line change
@@ -69,7 +69,7 @@ public function execute()
                     QueryInterface::SCOPE_SUB => 'ldap_search',
                 );
-                if (\in_array($this->options['scope'], $funcMap, true)) {
+                if (!\array_key_exists($this->options['scope'], $funcMap)) {
             Copy link

  
      
    
  

  
      

  
  Member


      

  

  
    
      

      
            nicolas-grekas
  

      

      

      


        Sep 17, 2018


      
    

  


        
      
  
  
  
    

    There was a problem hiding this comment.


  

 
  
    

    Choose a reason for hiding this comment

    
      The reason will be displayed to describe this comment to others. Learn more.
    

    
      
      


  


  
    
      "isset" would do it
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
      
        
            
    All reactions
                     throw new LdapException(sprintf('Could not search in scope "%s".', $this->options['scope']));
                 }
-          Expand Down