From e84df814cdd30508aa87d644f7793328b7127320 Mon Sep 17 00:00:00 2001
From: Guy Sartorelli <guy.sartorelli@silverstripe.com>
Date: Thu, 11 Jan 2024 12:41:31 +1300
Subject: [PATCH] ENH Add generic types

---
 code/Controller/AssetAdminFieldsExtension.php |  4 ++++
 code/Controller/AssetAdminFile.php            |  4 ++--
 code/Extensions/CampaignAdminExtension.php    |  4 +++-
 code/Extensions/RemoteFileModalExtension.php  | 13 ++-----------
 code/Extensions/UsedOnTableExtension.php      |  3 +++
 5 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/code/Controller/AssetAdminFieldsExtension.php b/code/Controller/AssetAdminFieldsExtension.php
index e1bd8cc76..9b4ebe261 100644
--- a/code/Controller/AssetAdminFieldsExtension.php
+++ b/code/Controller/AssetAdminFieldsExtension.php
@@ -2,9 +2,13 @@
 
 namespace SilverStripe\AssetAdmin\Controller;
 
+use SilverStripe\Admin\LeftAndMain;
 use SilverStripe\Core\Extension;
 use SilverStripe\View\Requirements;
 
+/**
+ * @extends Extension<LeftAndMain>
+ */
 class AssetAdminFieldsExtension extends Extension
 {
     public function init()
diff --git a/code/Controller/AssetAdminFile.php b/code/Controller/AssetAdminFile.php
index e20d176fa..7009e1073 100644
--- a/code/Controller/AssetAdminFile.php
+++ b/code/Controller/AssetAdminFile.php
@@ -15,7 +15,7 @@
 /**
  * Update File dataobjects to be editable in this asset admin
  *
- * @property File $owner
+ * @extends DataExtension<File>
  */
 class AssetAdminFile extends DataExtension
 {
@@ -176,7 +176,7 @@ public function humanizedChanges($from, $to)
     /**
      * Get the list of all nested files in use
      *
-     * @return SS_List
+     * @return SS_List<File>
      */
     public function getFilesInUse()
     {
diff --git a/code/Extensions/CampaignAdminExtension.php b/code/Extensions/CampaignAdminExtension.php
index 798e28c3c..20545d102 100644
--- a/code/Extensions/CampaignAdminExtension.php
+++ b/code/Extensions/CampaignAdminExtension.php
@@ -2,6 +2,7 @@
 
 namespace SilverStripe\AssetAdmin\Extensions;
 
+use SilverStripe\AssetAdmin\Forms\FileFormFactory;
 use SilverStripe\Assets\File;
 use SilverStripe\Core\Extension;
 use SilverStripe\Forms\FormAction;
@@ -10,7 +11,8 @@
 /**
  * Extension that updates the Popover menu of `FileFormFactory`.
  * This extension will only be applied if the `campaign-admin` module is installed.
- * @package SilverStripe\AssetAdmin\Extensions
+ *
+ * @extends Extension<FileFormFactory>
  */
 class CampaignAdminExtension extends Extension
 {
diff --git a/code/Extensions/RemoteFileModalExtension.php b/code/Extensions/RemoteFileModalExtension.php
index d2bd74826..a6d6deced 100644
--- a/code/Extensions/RemoteFileModalExtension.php
+++ b/code/Extensions/RemoteFileModalExtension.php
@@ -20,7 +20,8 @@
 
 /**
  * Decorates ModalController with an insert-oembed modal
- * @see ModalController
+ *
+ * @extends Extension<ModalController>
  */
 class RemoteFileModalExtension extends Extension
 {
@@ -30,16 +31,6 @@ class RemoteFileModalExtension extends Extension
         'remoteEditFormSchema',
     );
 
-    /**
-     * @return ModalController
-     */
-    public function getOwner()
-    {
-        /** @var ModalController $owner */
-        $owner = $this->owner;
-        return $owner;
-    }
-
     /**
      * @return HTTPRequest
      */
diff --git a/code/Extensions/UsedOnTableExtension.php b/code/Extensions/UsedOnTableExtension.php
index 0240254dd..ddbfa309c 100644
--- a/code/Extensions/UsedOnTableExtension.php
+++ b/code/Extensions/UsedOnTableExtension.php
@@ -2,6 +2,7 @@
 
 namespace SilverStripe\AssetAdmin\Extensions;
 
+use SilverStripe\Admin\Forms\UsedOnTable;
 use SilverStripe\Assets\Folder;
 use SilverStripe\Assets\Shortcodes\FileLink;
 use SilverStripe\Core\Extension;
@@ -10,6 +11,8 @@
 
 /**
  * Hides several types of DataObjects on the "Used On" tab when viewing files
+ *
+ * @extends Extension<UsedOnTable>
  */
 class UsedOnTableExtension extends Extension
 {