No, I understand why they were together initially, but I think it actually makes more sense to group variable declarations by type. In this example, it's getting two variables from the input - that's one group - and then it creates a new variable that will later hold the users - another group. In this example, I would actually rather format it like this:

```php
    function acl_autocomplete()
    {
        $this->load_config();

        $search = rcube_utils::get_input_value('_search', rcube_utils::INPUT_GPC, true);
        $sid    = rcube_utils::get_input_value('_id', rcube_utils::INPUT_GPC);

        $users = array();
        if ($this->init_ldap()) {
            $max  = (int) $this->rc->config->get('autocomplete_max', 15);
            $mode = (int) $this->rc->config->get('addressbook_search_mode');

            $this->ldap->set_pagesize($max);

            $result = $this->ldap->search('*', $search, $mode);

            foreach ($result->records as $record) {
                $user = $record['uid'];
[...]
                $users[] = $user;
[...]
```