Do not change the session-stored subsite, if session is not enabled.

This causes issues with Security::findAnAdmistrator which incorrectly
forces the current session-stored subsite to 0 - it uses
Subsite::currentSubsiteID before the session support is enabled, and
hence obtains wrong value.
This commit is contained in:
Mateusz Uzdowski 2013-11-26 12:56:59 +13:00
parent c04208ed79
commit a771e2239b

View File

@ -123,11 +123,15 @@ class Subsite extends DataObject implements PermissionProvider {
* @param int|Subsite $subsite Either the ID of the subsite, or the subsite object itself
*/
public static function changeSubsite($subsite) {
// Session subsite change only meaningful if the session is active.
// Otherwise we risk setting it to wrong value, e.g. if we rely on currentSubsiteID.
if (!Subsite::$use_session_subsiteid) return;
if(is_object($subsite)) $subsiteID = $subsite->ID;
else $subsiteID = $subsite;
Session::set('SubsiteID', (int)$subsiteID);
// Set locale
if (is_object($subsite) && $subsite->Language != '') {
$locale = i18n::get_locale_from_lang($subsite->Language);
@ -135,8 +139,8 @@ class Subsite extends DataObject implements PermissionProvider {
i18n::set_locale($locale);
}
}
Permission::flush_permission_cache();
Permission::flush_permission_cache();
}
/**