diff options
| author | Lester Caine <lester@lsces.co.uk> | 2026-06-05 12:08:55 +0100 |
|---|---|---|
| committer | Lester Caine <lester@lsces.co.uk> | 2026-06-05 12:08:55 +0100 |
| commit | 9b8641f687e050548020bfa484fa36b5955d5f21 (patch) | |
| tree | 6b261d92245dd1ebf52688c1371b0d89debee7c3 | |
| parent | 1f161be58a0417d4b45aab744cfa1f273181e37e (diff) | |
| download | users-9b8641f687e050548020bfa484fa36b5955d5f21.tar.gz users-9b8641f687e050548020bfa484fa36b5955d5f21.tar.bz2 users-9b8641f687e050548020bfa484fa36b5955d5f21.zip | |
Fix countries_inc.php scope in RoleUser::load() — require not require_once
Same issue as preferences/register: require_once inside a method is globally
tracked, so a second user object load won't re-execute the file, leaving
$bwIso3166 undefined and flag unset.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
| -rwxr-xr-x | includes/classes/RoleUser.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/includes/classes/RoleUser.php b/includes/classes/RoleUser.php index 7803413..655404e 100755 --- a/includes/classes/RoleUser.php +++ b/includes/classes/RoleUser.php @@ -210,7 +210,7 @@ class RoleUser extends \Bitweaver\Liberty\LibertyMime { // Load attachments \Bitweaver\Liberty\LibertyMime::load(); if( $code = $this->getPreference( 'users_country' ) ) { - require_once( USERS_PKG_INCLUDE_PATH.'countries_inc.php' ); + require( USERS_PKG_INCLUDE_PATH.'countries_inc.php' ); $this->setPreference( 'users_country_code', $code ); $this->setPreference( 'flag', $bwIso3166[$code] ?? '' ); $this->setPreference( 'users_country', $bwCountries[$code] ?? $code ); |