Update von LDAP zu AD mit admin/admin-cli.php

olafderkleine 2021-02-15 10:58:08 UTC #1

Hallo Forum,
eigentlich klappt der Umzug von einem LDAP zu einer Active Directory nun.

Leider bekomme ich bei einer Gruppe folgenden Fehler:
admin/admin-cli.php --change-account-id 'root_admin,xxxx,-xxxxx,-xxxx'
An error happened!: Call to protected method admin_cmd_config::exec() from context 'infolog_customfields'

Diese wird dann auch nicht geändert.
Was kann ich tun, um das Fehlverhalten abzustellen?

olafderkleine 2021-02-15 12:29:52 UTC #2

Ok. Nach etwas suchen im Quelltext habe ich den Fehler gefunden. Mit call_user_func wird versuchet eine protected function aufzurufen. Da dies aus einem anderen Namespace kommt, schlägt das fehl.

RalfBecker 2021-02-16 06:58:55 UTC #3

Welche EGroupware und PHP Version ist das?

Auch in 20.1 wird das admin_cmd_config instanziert und ausgeführt (und nicht per call_user_func aufgerufen):

github.com

EGroupware/egroupware/blob/20.1/infolog/inc/class.infolog_customfields.inc.php#L354


			{
				if (isset($changes[$account_id]))
				{
					$account_id = $changes[$account_id];
					$changed++;
					$needs_save = true;
				}
			}
			if ($needs_save)
			{
				$cmd = new admin_cmd_config('infolog',
						array('group_owners' => $config['group_owners']),
						array('group_owners' => $old)
				);
				$cmd->exec();
			}
		}


		return $changed;
	}
}

Ralf

olafderkleine 2021-03-02 14:48:04 UTC #4

@RalfBecker
Php 7.3, EGroupware 20.1
Folgendes habe ich geändert und dann lief es durch:

index d4d22e9ed0..06206fa1ae 100644
--- a/admin/inc/class.admin_cmd_config.inc.php
+++ b/admin/inc/class.admin_cmd_config.inc.php
@@ -72,23 +72,20 @@ class admin_cmd_config extends admin_cmd
         * @throws Exception(lang('Wrong credentials to access the header.inc.php file!'),2);
         * @throws Exception('header.inc.php not found!');
         */
-       protected function exec($check_only=false)
+       function exec($check_only=false)
        {
                if ($check_only)
                {
                        return true;    // no specific checks exist
                }
-
                $config = new Api\Config($this->store_as_api ? self::STORE_AS_API : $this->app);
                $config->read_repository();
-
                // store the config
                foreach($this->set as $name => $value)
                {
                        $config->value($name, $value);
                }
                $config->save_repository();
-
                return lang('Configuration saved.');
        }

RalfBecker 2021-03-02 15:37:53 UTC #5

Ist jetzt behoben:

Danke für die Info

Ralf