Skip to content

Commit

Permalink
Merge pull request #137 from silverstripe-terraformers/feature/publis…
Browse files Browse the repository at this point in the history
…her-extensibility

NEW: Extension points for FilesystemPublisher.
  • Loading branch information
GuySartorelli committed Dec 22, 2023
2 parents 622672b + 50519cd commit d7dcffe
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/Publisher.php
Expand Up @@ -10,6 +10,7 @@
use SilverStripe\Core\Config\Configurable;
use SilverStripe\Core\CoreKernel;
use SilverStripe\Core\Environment;
use SilverStripe\Core\Extensible;
use SilverStripe\Core\Injector\Injectable;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\FieldType\DBDatetime;
Expand All @@ -22,6 +23,7 @@

abstract class Publisher implements StaticPublisher
{
use Extensible;
use Injectable;
use Configurable;

Expand Down
4 changes: 4 additions & 0 deletions src/Publisher/FilesystemPublisher.php
Expand Up @@ -109,7 +109,11 @@ public function publishURL($url, $forcePublish = false)
return;
}
$success = false;

$this->extend('onBeforeGeneratePageResponse', $url);
$response = $this->generatePageResponse($url);
$this->extend('onAfterGeneratePageResponse', $url, $response);

$statusCode = $response->getStatusCode();
$doPublish = ($forcePublish && $this->getFileExtension() === 'php') || $statusCode < 400;

Expand Down

0 comments on commit d7dcffe

Please sign in to comment.