setup_cmd_header::exec()

Complexity

Conditions	24
Paths	210

Size

Total Lines	83
Code Lines	43

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	protected function exec($check_only=false)
	{
		if ($check_only && $this->remote_id)

The expression $this->remote_id of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

		{
			return true;	// can only check locally
		}
		if (!file_exists($this->header_path) || filesize($this->header_path) < 200)	// redirect header in rpms is ~150 byte
		{
			if ($this->sub_command != 'create')

The property sub_command does not exist on setup_cmd_header. Since you implemented __get, consider adding a @property annotation.

			{
				throw new Api\Exception\WrongUserinput(lang('EGroupware configuration file (header.inc.php) does NOT exist.')."\n".lang('Use --create-header to create the configuration file (--usage gives more options).'),1);
			}
			$this->defaults(false);

The method defaults() does not exist on setup_cmd_header. Since you implemented __call, consider adding a @method annotation.

		}
		else
		{
			if ($this->sub_command == 'create')
			{
				throw new Api\Exception\WrongUserinput(
					lang('EGroupware configuration file header.inc.php already exists, you need to use --edit-header or delete it first!'),20);
			}
			if ($this->arguments)

The property arguments does not exist on setup_cmd_header. Since you implemented __get, consider adding a @property annotation.

			{
				list($this->header_admin_password,$this->header_admin_user) = explode(',',$this->arguments[1]);

The property header_admin_password does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

The property header_admin_user does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

			}
			$this->check_setup_auth($this->header_admin_user,$this->header_admin_password);	// no domain, we require header access!

			$GLOBALS['egw_info']['server']['server_root'] = EGW_SERVER_ROOT;
			$GLOBALS['egw_info']['server']['include_root'] = EGW_INCLUDE_ROOT;
		}

		if ($this->arguments)	// we have command line arguments
		{
			$this->_parse_cli_arguments();
		}
		elseif ($this->sub_command == 'delete')
		{
			self::_delete_domain($this->domain);

The property domain does not exist on setup_cmd_header. Since you implemented __get, consider adding a @property annotation.

		}
		else
		{
			$this->_parse_properties();
		}
		if (($errors = $this->validation_errors($GLOBALS['egw_info']['server']['server_root'],

The method validation_errors() does not exist on setup_cmd_header. Since you implemented __call, consider adding a @method annotation.

			$GLOBALS['egw_info']['server']['include_root'])))
		{
			if ($this->arguments)
			{
				unset($GLOBALS['egw_info']['flags']);
				echo '$GLOBALS[egw_info] = '; print_r($GLOBALS['egw_info']);
				echo '$GLOBALS[egw_domain] = '; print_r($GLOBALS['egw_domain']);
			}
			throw new Api\Exception\WrongUserinput(lang('Configuration errors:')."\n- ".implode("\n- ",$errors)."\n".lang("You need to fix the above errors, before the configuration file header.inc.php can be written!"),23);
		}
		if ($check_only)
		{
			return true;
		}
		// check if php has persistent mysql connections disabled --> disable it in header, to not fill the log with warnings
		if ($GLOBALS['egw_info']['server']['db_persistent'])
		{
			$GLOBALS['egw_info']['server']['db_persistent'] = $this->check_db_persistent($GLOBALS['egw_domain']);

The method check_db_persistent() does not exist on setup_cmd_header. Since you implemented __call, consider adding a @method annotation.

		}
		$header = $this->generate($GLOBALS['egw_info'],$GLOBALS['egw_domain']);

The method generate() does not exist on setup_cmd_header. Since you implemented __call, consider adding a @method annotation.

		if ($this->arguments)
		{
			echo $header;	// for cli, we echo the header
		}
		if (file_exists($this->header_path) && is_writable($this->header_path) || is_writable(dirname($this->header_path)) ||
			function_exists('posix_getuid') && !posix_getuid())	// root has all rights
		{
			if (file_exists($this->header_path) && !is_writable($this->header_path))
			{
				unlink($this->header_path);
			}
			if (($f = fopen($this->header_path,'wb')) && fwrite($f,$header))
			{
				fclose($f);
				return lang('header.inc.php successful written.');
			}
		}
		throw new Api\Exception\NoPermission(lang("Failed writing configuration file header.inc.php, check the permissions !!!"),24);
	}