ENH Add generic types

silverstripe · Jan 12, 2024 · 8ebfc7a · 8ebfc7a
1 parent d7dcffe
commit 8ebfc7a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/Extension/Engine/SiteTreePublishingEngine.php b/src/Extension/Engine/SiteTreePublishingEngine.php
@@ -25,6 +25,8 @@
  * * action - name of the executed action: publish or unpublish
  *
  * @see PublishableSiteTree
+ *
+ * @extends DataExtension<SiteTree>
  */
 class SiteTreePublishingEngine extends SiteTreeExtension implements Resettable
 {
@@ -300,7 +302,6 @@ public function flushChanges()
         $urlsToDelete = $this->getUrlsToDelete();
 
         if ($urlsToUpdate) {
-            /** @var UrlBundleInterface $urlService */
             $urlService = Injector::inst()->create(UrlBundleInterface::class);
             $urlService->addUrls($urlsToUpdate);
 
@@ -314,7 +315,6 @@ public function flushChanges()
         }
 
         if ($urlsToDelete) {
-            /** @var UrlBundleInterface $urlService */
             $urlService = Injector::inst()->create(UrlBundleInterface::class);
             $urlService->addUrls($urlsToDelete);
 

diff --git a/src/Extension/Publishable/PublishableSiteTree.php b/src/Extension/Publishable/PublishableSiteTree.php
@@ -19,6 +19,8 @@
  * an extension via the config system ordering (inject your extension "before" the PublishableSiteTree).
  *
  * @see SiteTreePublishingEngine
+ *
+ * @extends DataExtension<SiteTree>
  */
 class PublishableSiteTree extends DataExtension implements StaticallyPublishable, StaticPublishingTrigger
 {
@@ -36,7 +38,7 @@ public function getMyVirtualPages()
     }
 
     /**
-     * @return array|SS_List
+     * @return iterable<SiteTree>
      */
     public function objectsToUpdate($context)
     {
@@ -92,7 +94,7 @@ public function objectsToUpdate($context)
     /**
      * This method controls which caches will be purged
      *
-     * @return array|SS_List
+     * @return iterable<SiteTree>
      */
     public function objectsToDelete($context)
     {