nazar-pc / CleverStyle-Framework

modules/HybridAuth/functions.php

<?php
/**
 * @package  HybridAuth
 * @category modules
 * @author   Nazar Mokrynskyi <nazar@mokrynskyi.com>
 * @license  0BSD
 */
namespace cs\modules\HybridAuth;
use
	Hybrid_Auth,
	cs\Config,
	cs\Request,
	cs\Session,
	cs\User;

/**
 * Get HybridAuth instance with current configuration. Strongly recommended instead of direct usage
 *
 * @param null|string $provider
 * @param null|string $base_url
 *
 * @return Hybrid_Auth
 */
function get_hybridauth_instance ($provider = null, $base_url = null) {
	require_once __DIR__.'/Hybrid/Auth.php';
	$Config     = Config::instance();
	$User       = User::instance();
	$HybridAuth = new Hybrid_Auth(
		[
			'base_url'  => $base_url ?: $Config->base_url()."/HybridAuth/$provider/endpoint/".md5($provider.Session::instance()->get_id()),

Are you sure cs\Session::instance()->get_id() of type false|string can be used in concatenation?

			'providers' => $Config->module('HybridAuth')->providers

The property providers does not exist on cs\Config\Module_Properties. Since you implemented __get, consider adding a @property annotation.

		]
	);
	if ($User->user() && Request::instance()->current_module != 'HybridAuth') {
		$HybridAuth->restoreSessionData(serialize($User->get_data('HybridAuth_session')));
	}
	return $HybridAuth;
}