diff --git a/.github/workflows/pull-requests.yml b/.github/workflows/pull-requests.yml index 2b479da..54ab047 100644 --- a/.github/workflows/pull-requests.yml +++ b/.github/workflows/pull-requests.yml @@ -3,7 +3,7 @@ name: Pull Requests on: pull_request: branches: - - main + - master jobs: runner: diff --git a/Building/Building.php b/Building/Building.php index f89bd8d..9a98fd6 100644 --- a/Building/Building.php +++ b/Building/Building.php @@ -102,31 +102,31 @@ public static function init() SiteViewFactory::addPath('Building/views'); SiteViewFactory::registerView( 'building-block-audio', - 'BuildingBlockAudioView' + BuildingBlockAudioView::class ); SiteViewFactory::registerView( 'building-block-video', - 'BuildingBlockVideoView' + BuildingBlockVideoView::class ); SiteViewFactory::registerView( 'building-block-image', - 'BuildingBlockImageView' + BuildingBlockImageView::class ); SiteViewFactory::registerView( 'building-block-xhtml', - 'BuildingBlockXHTMLView' + BuildingBlockXHTMLView::class ); SiteViewFactory::registerView( 'building-block-attachment', - 'BuildingBlockAttachmentView' + BuildingBlockAttachmentView::class ); SiteViewFactory::registerView( 'building-block', - 'BuildingBlockCompositeView' + BuildingBlockCompositeView::class ); SiteViewFactory::registerView( 'building-block-admin', - 'BuildingBlockAdminCompositeView' + BuildingBlockAdminCompositeView::class ); SwatUI::mapClassPrefixToPath('Building', 'Building'); diff --git a/Building/BuildingBlockViewFactory.php b/Building/BuildingBlockViewFactory.php index a80f036..fbd6445 100644 --- a/Building/BuildingBlockViewFactory.php +++ b/Building/BuildingBlockViewFactory.php @@ -36,18 +36,19 @@ public static function getBlockView( */ public static function getViewType(BuildingBlock $block) { - $type = 'building-block-xhtml'; - if ($block->media instanceof SiteAudioMedia) { - $type = 'building-block-audio'; - } elseif ($block->media instanceof SiteVideoMedia) { - $type = 'building-block-video'; - } elseif ($block->image instanceof SiteImage) { - $type = 'building-block-image'; - } elseif ($block->attachment instanceof SiteAttachment) { - $type = 'building-block-attachment'; + return 'building-block-audio'; + } + if ($block->media instanceof SiteVideoMedia) { + return 'building-block-video'; + } + if ($block->image instanceof SiteImage) { + return 'building-block-image'; + } + if ($block->attachment instanceof SiteAttachment) { + return 'building-block-attachment'; } - return $type; + return 'building-block-xhtml'; } } diff --git a/Building/admin/components/Block/AttachmentEdit.php b/Building/admin/components/Block/AttachmentEdit.php index e475e80..cf02e41 100644 --- a/Building/admin/components/Block/AttachmentEdit.php +++ b/Building/admin/components/Block/AttachmentEdit.php @@ -28,8 +28,7 @@ protected function getAttachmentSet() $this->attachment_set = $this->getObject()->attachment->attachment_set; } else { - $class_name = SwatDBClassMap::get('SiteAttachmentSet'); - $this->attachment_set = new $class_name(); + $this->attachment_set = SwatDBClassMap::new(SiteAttachmentSet::class); $this->attachment_set->setDatabase($this->app->db); $shortname = $this->getAttachmentSetShortname(); if (!$this->attachment_set->loadByShortname($shortname)) { @@ -48,8 +47,7 @@ protected function getAttachmentSet() protected function getNewAttachmentInstance() { - $class_name = SwatDBClassMap::get('SiteAttachment'); - $attachment = new $class_name(); + $attachment = SwatDBClassMap::new(SiteAttachment::class); $attachment->setDatabase($this->app->db); $attachment->attachment_set = $this->getAttachmentSet(); diff --git a/Building/admin/components/Block/Delete.php b/Building/admin/components/Block/Delete.php index 0d3e691..8bebc22 100644 --- a/Building/admin/components/Block/Delete.php +++ b/Building/admin/components/Block/Delete.php @@ -38,7 +38,7 @@ protected function buildInternal() 'select * from Block where id in (%s)', $this->getItemList('integer') ), - SwatDBClassMap::get('BuildingBlockWrapper') + SwatDBClassMap::get(BuildingBlockWrapper::class) ); $view = SiteViewFactory::get($this->app, 'building-block'); diff --git a/Building/admin/components/Block/Edit.php b/Building/admin/components/Block/Edit.php index 5f7b648..2012020 100644 --- a/Building/admin/components/Block/Edit.php +++ b/Building/admin/components/Block/Edit.php @@ -8,7 +8,7 @@ abstract class BuildingBlockEdit extends AdminObjectEdit { protected function getObjectClass() { - return 'BuildingBlock'; + return BuildingBlock::class; } // process phase diff --git a/Building/admin/components/Block/ImageEdit.php b/Building/admin/components/Block/ImageEdit.php index 49b72a5..62873e9 100644 --- a/Building/admin/components/Block/ImageEdit.php +++ b/Building/admin/components/Block/ImageEdit.php @@ -27,8 +27,7 @@ protected function getImageSet() if ($this->getObject()->image instanceof SiteImage) { $this->image_set = $this->getObject()->image->image_set; } else { - $class_name = SwatDBClassMap::get('SiteImageSet'); - $this->image_set = new $class_name(); + $this->image_set = SwatDBClassMap::new(SiteImageSet::class); $this->image_set->setDatabase($this->app->db); $shortname = $this->getImageSetShortname(); if (!$this->image_set->loadByShortname($shortname)) { @@ -47,8 +46,7 @@ protected function getImageSet() protected function getNewImageInstance() { - $class_name = SwatDBClassMap::get('SiteImage'); - $image = new $class_name(); + $image = SwatDBClassMap::new(SiteImage::class); $image->setDatabase($this->app->db); $image->image_set = $this->getImageSet(); diff --git a/Building/admin/components/Block/VideoEdit.php b/Building/admin/components/Block/VideoEdit.php index 7f124bf..817c251 100644 --- a/Building/admin/components/Block/VideoEdit.php +++ b/Building/admin/components/Block/VideoEdit.php @@ -24,12 +24,12 @@ protected function getMedia() } else { $media_id = $this->app->initVar('media'); if ($media_id === null) { + /** @var SwatForm $form */ $form = $this->ui->getWidget('edit_form'); $media_id = $form->getHiddenField('media'); } - $class_name = SwatDBClassMap::get('SiteVideoMedia'); - $this->media = new $class_name(); + $this->media = SwatDBClassMap::new(SiteVideoMedia::class); $this->media->setDatabase($this->app->db); if (!$this->media->load($media_id)) { throw new AdminNotFoundException( @@ -93,12 +93,16 @@ protected function buildInternal() $media = $this->getMedia(); $media->setFileBase('media'); - $this->ui->getWidget('edit_form')->addHiddenField('media', $media->id); + /** @var SwatForm $form */ + $form = $this->ui->getWidget('edit_form'); + $form->addHiddenField('media', $media->id); $player = $media->getMediaPlayer($this->app); ob_start(); $player->display(); - $this->ui->getWidget('player')->content = ob_get_clean(); + /** @var SwatContentBlock $ui_player */ + $ui_player = $this->ui->getWidget('player'); + $ui_player->content = ob_get_clean(); $this->layout->addHtmlHeadEntrySet($player->getHtmlHeadEntrySet()); } diff --git a/Building/dataobjects/BuildingBlock.php b/Building/dataobjects/BuildingBlock.php index bf8101a..b4927bf 100644 --- a/Building/dataobjects/BuildingBlock.php +++ b/Building/dataobjects/BuildingBlock.php @@ -5,6 +5,10 @@ * * @copyright 2014-2016 silverorange * @license http://www.opensource.org/licenses/mit-license.html MIT License + * + * @property ?SiteAttachment $attachment + * @property ?SiteImage $image + * @property ?SiteVideoMedia $media */ class BuildingBlock extends SwatDBDataObject { @@ -14,7 +18,7 @@ class BuildingBlock extends SwatDBDataObject public $id; /** - * @var string + * @var ?string */ public $bodytext; @@ -44,17 +48,17 @@ protected function init() $this->registerInternalProperty( 'attachment', - SwatDBClassMap::get('SiteAttachment') + SwatDBClassMap::get(SiteAttachment::class) ); $this->registerInternalProperty( 'image', - SwatDBClassMap::get('SiteImage') + SwatDBClassMap::get(SiteImage::class) ); $this->registerInternalProperty( 'media', - SwatDBClassMap::get('SiteVideoMedia') + SwatDBClassMap::get(SiteVideoMedia::class) ); $this->id_field = 'integer:id'; diff --git a/Building/dataobjects/BuildingBlockWrapper.php b/Building/dataobjects/BuildingBlockWrapper.php index 460e4e6..852fb34 100644 --- a/Building/dataobjects/BuildingBlockWrapper.php +++ b/Building/dataobjects/BuildingBlockWrapper.php @@ -13,7 +13,7 @@ class BuildingBlockWrapper extends SwatDBRecordsetWrapper protected function init() { parent::init(); - $this->row_wrapper_class = SwatDBClassMap::get('BuildingBlock'); + $this->row_wrapper_class = SwatDBClassMap::get(BuildingBlock::class); $this->index_field = 'id'; } }