CombineHostPortLdapUriRector doesn't handle concatenated URI parameters #8964
Description
Bug Report
| Subject | Details |
|---|---|
| Rector version | Rector 1.2.10 |
The CombineHostPortLdapUriRector rule incorrectly handles concatenated URI parameters
Minimal PHP Code Causing Issue
- $authConnection = ldap_connect ($this->protocol . $this->server, $this->port);
+ $authConnection = ldap_connect ("{$this->protocol . $this->server}:{$this->port}");
Expected Behaviour
- $authConnection = ldap_connect ($this->protocol . $this->server, $this->port);
+ $authConnection = ldap_connect ($this->protocol . $this->server . ":" . $this->port);