ENH Add generic types

code/Controllers/ContentController.php

@@ -45,11 +45,12 @@
  * Subclasses of ContentController are generally instantiated by ModelAsController; this will create
  * a controller based on the URLSegment action variable, by looking in the SiteTree table.
  *
+ * @template T of SiteTree
  */
 class ContentController extends Controller
 {
     /**
-     * @var SiteTree
+     * @var T
      */
     protected $dataRecord;
 
@@ -71,7 +72,7 @@ class ContentController extends Controller
      * The ContentController will take the URLSegment parameter from the URL and use that to look
      * up a SiteTree record.
      *
-     * @param SiteTree $dataRecord
+     * @param T|null $dataRecord
      */
     public function __construct($dataRecord = null)
     {
@@ -110,7 +111,7 @@ public function Link($action = null)
      * Return the children of a given page. The parent reference can either be a page link or an ID.
      *
      * @param string|int $parentRef
-     * @return SS_List
+     * @return SS_List<SiteTree>
      */
     public function ChildrenOf($parentRef)
     {
@@ -188,7 +189,7 @@ protected function init()
      */
     public function handleRequest(HTTPRequest $request): HTTPResponse
     {
-        /** @var SiteTree $child */
+        /** @var SiteTree|null $child */
         $child = null;
         $action = $request->param('Action');
 
@@ -243,6 +244,7 @@ public function project()
 
     /**
      * Returns the associated database record
+     * @return T
      */
     public function data()
     {
@@ -254,7 +256,7 @@ public function data()
     /**
      * Returns a fixed navigation menu of the given level.
      * @param int $level Menu level to return.
-     * @return ArrayList
+     * @return ArrayList<SiteTree>
      */
     public function getMenu($level = 1)
     {
@@ -294,6 +296,9 @@ public function getMenu($level = 1)
         return new ArrayList($visible);
     }
 
+    /**
+     * @return ArrayList<SiteTree>
+     */
     public function Menu($level)
     {
         return $this->getMenu($level);

code/Controllers/LeftAndMainBatchActionsExtension.php

@@ -2,9 +2,13 @@
 
 namespace SilverStripe\CMS\Controllers;
 
+use SilverStripe\Admin\LeftAndMain;
 use SilverStripe\CMS\Controllers\CMSMain;
 use SilverStripe\Core\Extension;
 
+/**
+ * @extends Extension<LeftAndMain>
+ */
 class LeftAndMainBatchActionsExtension extends Extension
 {
     public function updateBatchActionsForm(&$form)

code/Controllers/LeftAndMainPageIconsExtension.php

@@ -5,6 +5,7 @@
 use Psr\SimpleCache\CacheInterface;
 use Psr\SimpleCache\InvalidArgumentException;
 use ReflectionException;
+use SilverStripe\Admin\LeftAndMain;
 use SilverStripe\CMS\Model\SiteTree;
 use SilverStripe\Core\ClassInfo;
 use SilverStripe\Core\Config\Config;
@@ -16,10 +17,11 @@
 
 /**
  * Extension to include custom page icons
+ *
+ * @extends Extension<LeftAndMain>
  */
 class LeftAndMainPageIconsExtension extends Extension implements Flushable
 {
-
     /**
      * @throws InvalidArgumentException
      * @throws ReflectionException

code/Controllers/OldPageRedirector.php

@@ -10,9 +10,11 @@
 use SilverStripe\Control\HTTPResponse_Exception;
 use SilverStripe\Core\Extension;
 
+/**
+ * @extends Extension<Controller>
+ */
 class OldPageRedirector extends Extension
 {
-
     /**
      * On every URL that generates a 404, we'll capture it here and see if we can
      * find an old URL that it should be redirecting to.

code/Forms/InternalLinkModalExtension.php

@@ -9,7 +9,8 @@
 
 /**
  * Decorates ModalController with insert internal link
- * @see ModalController
+ *
+ * @extends Extension<ModalController>
  */
 class InternalLinkModalExtension extends Extension
 {
@@ -22,17 +23,6 @@ class InternalLinkModalExtension extends Extension
         'editorAnchorLink',
     ];
 
-    /**
-     * @return ModalController
-     */
-    public function getOwner()
-    {
-        /** @var ModalController $owner */
-        $owner = $this->owner;
-        return $owner;
-    }
-
-
     /**
      * Form for inserting internal link pages
      *

code/Model/RedirectorPageController.php

@@ -7,6 +7,8 @@
 
 /**
  * Controller for the {@link RedirectorPage}.
+ *
+ * @extends PageController<RedirectorPage>
  */
 class RedirectorPageController extends PageController
 {
@@ -26,7 +28,6 @@ class RedirectorPageController extends PageController
      */
     public function index(HTTPRequest $request)
     {
-        /** @var RedirectorPage $page */
         $page = $this->data();
 
         // Redirect if we can

code/Model/SiteTree.php

@@ -1966,7 +1966,7 @@ public function getLiveURLSegment()
     /**
      * Get the back-link tracking objects that link to this page
      *
-     * @return ArrayList|DataObject[]
+     * @return ArrayList<DataObject>
      */
     public function BackLinkTracking()
     {
@@ -2007,7 +2007,7 @@ public function BackLinkTracking()
      * Returns the pages that depend on this page. This includes virtual pages, pages that link to it, etc.
      *
      * @param bool $includeVirtuals Set to false to exlcude virtual pages.
-     * @return ArrayList|SiteTree[]
+     * @return ArrayList<SiteTree>
      */
     public function DependentPages($includeVirtuals = true)
     {
@@ -2057,7 +2057,7 @@ public function DependentPages($includeVirtuals = true)
     /**
      * Return all virtual pages that link to this page.
      *
-     * @return DataList
+     * @return DataList<SiteTree>
      */
     public function VirtualPages()
     {

code/Model/SiteTreeExtension.php

@@ -7,6 +7,9 @@
 
 /**
  * Plug-ins for additional functionality in your SiteTree classes.
+ *
+ * @template T of SiteTree
+ * @extends DataExtension<T>
  */
 abstract class SiteTreeExtension extends DataExtension
 {

code/Model/SiteTreeLinkTracking.php

@@ -28,6 +28,8 @@
  *
  * @property DataObject|SiteTreeLinkTracking $owner
  * @method ManyManyThroughList<SiteTree> LinkTracking()
+ *
+ * @extends DataExtension<DataObject>
  */
 class SiteTreeLinkTracking extends DataExtension
 {

code/Search/ContentControllerSearchExtension.php

@@ -2,6 +2,7 @@
 
 namespace SilverStripe\CMS\Search;
 
+use SilverStripe\CMS\Controllers\ContentController;
 use SilverStripe\Control\HTTPRequest;
 use SilverStripe\Core\Extension;
 use SilverStripe\Forms\TextField;
@@ -12,6 +13,8 @@
 
 /**
  * Extension to provide a search interface when applied to ContentController
+ *
+ * @extends Extension<ContentController>
  */
 class ContentControllerSearchExtension extends Extension
 {