tonyair/silverstripe-subsites
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-subsites synced 2024-10-22 09:05:55 +00:00
Code Issues Projects Releases Wiki Activity

FIX Catch database exceptions in middleware when DB is not ready, set session ID after delegation

Browse Source
This commit is contained in:
Robbie Averill 2017-09-05 12:07:49 +12:00
parent 1ac6e78bb3
commit 65f85faff6
4 changed files with 37 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
code/Middleware/InitStateMiddleware.php
View File

@ -7,6 +7,7 @@ use SilverStripe\Control\HTTPRequest;
use SilverStripe\Control\Middleware\HTTPMiddleware;
use SilverStripe\Core\Config\Configurable;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\ORM\Connect\DatabaseException;
use SilverStripe\Subsites\Model\Subsite;
use SilverStripe\Subsites\State\SubsiteState;
@ -27,6 +28,7 @@ class InitStateMiddleware implements HTTPMiddleware
public function process(HTTPRequest $request, callable $delegate)
{
try {
// Initialise and register the State
$state = SubsiteState::create();
Injector::inst()->registerService($state);
@ -41,14 +43,20 @@ class InitStateMiddleware implements HTTPMiddleware
// Persist to the session if using the CMS
if ($state->getUseSessions()) {
$original = $request->getSession()->get('SubsiteID');
$request->getSession()->set('SubsiteID', $subsiteId);
// Track session changes
$state->setSessionWasChanged($subsiteId === $original);
$originalSubsiteId = $request->getSession()->get('SubsiteID');
}
return $delegate($request);
} catch (DatabaseException $ex) {
// No-op, database is not ready
} finally {
if ($state->getUseSessions()) {
// Track session changes
$request->getSession()->set('SubsiteID', $subsiteId);
$state->setSessionWasChanged($subsiteId === $originalSubsiteId);
}
}
}
/**

2
code/extensions/ControllerSubsites.php
View File

@ -15,7 +15,7 @@ class ControllerSubsites extends Extension
{
if ($subsite = Subsite::currentSubsite()) {
if ($theme = $subsite->Theme) {
SSViewer::add_themes([$theme]);
SSViewer::set_themes([$theme, SSViewer::DEFAULT_THEME]);
}
}
}

13
code/extensions/LeftAndMainSubsites.php
View File

@ -267,13 +267,18 @@ class LeftAndMainSubsites extends LeftAndMainExtension
if ($request->getVar('SubsiteID') !== null) {
// Clear current page when subsite changes (or is set for the first time)
if ($state->getSessionWasChanged()) {
$session->clear($this->owner->sessionNamespace() . '.currentPage');
// sessionNamespace() is protected - see for info
$override = $this->owner->config()->get('session_namespace');
$sessionNamespace = $override ? $override : get_class($this->owner);
$session->clear($sessionNamespace . '.currentPage');
}
// Subsite ID has already been set to the state via InitStateMiddleware
if ($this->owner->canView()) {
// Redirect to clear the current page
return $this->owner->redirect($this->owner->Link());
// Redirect to clear the current page, retaining the current URL parameters
return $this->owner->redirect(
Controller::join_links($this->owner->Link(), ...array_values($this->owner->getURLParams()))
);
}
// Redirect to the default CMS section
return $this->owner->redirect(AdminRootController::config()->get('url_base') . '/');
@ -304,7 +309,7 @@ class LeftAndMainSubsites extends LeftAndMainExtension
if ($canViewElsewhere) {
// Redirect to clear the current page
return $this->owner->redirect(
Controller::join_links($this->owner->Link(), $record->ID, '?SubsiteID=' . $record->SubsiteID)
Controller::join_links($this->owner->Link('show'), $record->ID, '?SubsiteID=' . $record->SubsiteID)
);
}
// Redirect to the default CMS section

4
tests/php/SubsiteAdminFunctionalTest.php
View File

@ -85,7 +85,7 @@ class SubsiteAdminFunctionalTest extends FunctionalTest
$this->assertEquals(302, $response->getStatusCode());
$this->assertContains(
'admin/pages/edit/' . $subsite1Home->ID . '?SubsiteID=' . $subsite1Home->SubsiteID,
'admin/pages/edit/show/' . $subsite1Home->ID . '?SubsiteID=' . $subsite1Home->SubsiteID,
$response->getHeader('Location')
);
@ -95,7 +95,7 @@ class SubsiteAdminFunctionalTest extends FunctionalTest
$response = $this->get("admin/pages/edit/show/$subsite1Home->ID");
$this->assertEquals(302, $response->getStatusCode());
$this->assertContains(
'admin/pages/edit/' . $subsite1Home->ID . '?SubsiteID=' . $subsite1Home->SubsiteID,
'admin/pages/edit/show/' . $subsite1Home->ID . '?SubsiteID=' . $subsite1Home->SubsiteID,
$response->getHeader('Location')
);