From f7ba5d54c25ac0a340591695cf5f4704f0b39298 Mon Sep 17 00:00:00 2001 From: Guy Sartorelli Date: Thu, 11 Jan 2024 16:03:38 +1300 Subject: [PATCH] ENH Add generic types --- code/Control/UserDefinedFormAdmin.php | 4 ---- code/Control/UserDefinedFormController.php | 3 +-- code/Extension/UpgradePolymorphicExtension.php | 2 ++ code/Extension/UserFormFieldEditorExtension.php | 7 ++++--- code/Extension/UserFormFileExtension.php | 4 ++-- code/Extensions/UsedOnTableExtension.php | 5 +++-- code/FormField/UserFormsCheckboxSetField.php | 2 -- code/FormField/UserFormsCompositeField.php | 1 - code/Model/EditableCustomRule.php | 1 - code/Model/EditableFormField.php | 2 -- .../EditableFormField/EditableMultipleOptionField.php | 1 - code/Model/Recipient/EmailRecipient.php | 9 ++++----- code/Model/Recipient/UserFormRecipientItemRequest.php | 2 +- code/Task/RecoverUploadLocationsHelper.php | 2 +- code/Task/UserFormsColumnCleanTask.php | 1 - code/UserForm.php | 2 +- 16 files changed, 19 insertions(+), 29 deletions(-) diff --git a/code/Control/UserDefinedFormAdmin.php b/code/Control/UserDefinedFormAdmin.php index 91aee8f..7cc75c2 100644 --- a/code/Control/UserDefinedFormAdmin.php +++ b/code/Control/UserDefinedFormAdmin.php @@ -60,7 +60,6 @@ class UserDefinedFormAdmin extends LeftAndMain */ private static function getRestrictedAccessField(string $folder, string $title) { - /** @var TextField $textField */ $textField = TextField::create('CreateFolder', ''); /** @var Folder $formSubmissionsFolder */ @@ -121,7 +120,6 @@ class UserDefinedFormAdmin extends LeftAndMain if ($editableFormField instanceof EditableFileField) { $folderId = $editableFormField->FolderID; } - /** @var Folder $folder */ $folder = Folder::get()->byID($folderId); if (!$folder) { $folder = $this->getFormSubmissionFolder(); @@ -222,7 +220,6 @@ class UserDefinedFormAdmin extends LeftAndMain if (!$id) { throw new HTTPResponse_Exception(_t(__CLASS__.'.INVALID_REQUEST', 'This request was invalid.'), 400); } - /** @var EditableFileField $editableFileField */ $editableFormField = EditableFormField::get()->byID($id); if (!$editableFormField) { $editableFormField = Versioned::get_by_stage(EditableFormField::class, Versioned::DRAFT)->byID($id); @@ -276,7 +273,6 @@ class UserDefinedFormAdmin extends LeftAndMain { $folderID = $this->getRequest()->requestVar('FolderID'); if ($folderID) { - /** @var Folder $folder */ $folder = Folder::get()->byID($folderID); if (!$folder) { throw new HTTPResponse_Exception(_t(__CLASS__.'.INVALID_REQUEST', 'This request was invalid.'), 400); diff --git a/code/Control/UserDefinedFormController.php b/code/Control/UserDefinedFormController.php index 201cc18..3837690 100644 --- a/code/Control/UserDefinedFormController.php +++ b/code/Control/UserDefinedFormController.php @@ -38,7 +38,7 @@ use Swift_RfcComplianceException; /** * Controller for the {@link UserDefinedForm} page type. * - * @package userforms + * @extends PageController */ class UserDefinedFormController extends PageController { @@ -200,7 +200,6 @@ class UserDefinedFormController extends PageController $watch = []; if ($formFields) { - /** @var EditableFormField $field */ foreach ($formFields as $field) { if ($result = $field->formatDisplayRules()) { $watch[] = $result; diff --git a/code/Extension/UpgradePolymorphicExtension.php b/code/Extension/UpgradePolymorphicExtension.php index e40bd9a..2221234 100644 --- a/code/Extension/UpgradePolymorphicExtension.php +++ b/code/Extension/UpgradePolymorphicExtension.php @@ -23,6 +23,8 @@ use SilverStripe\UserForms\UserForm; * certain amount of manual checking is required to ensure that upgrades are performed smoothly. * * @internal This API is likely to be removed in later major versions of silverstripe/userforms + * + * @extends DataExtension */ class UpgradePolymorphicExtension extends DataExtension { diff --git a/code/Extension/UserFormFieldEditorExtension.php b/code/Extension/UserFormFieldEditorExtension.php index a1ede83..62b37e1 100644 --- a/code/Extension/UserFormFieldEditorExtension.php +++ b/code/Extension/UserFormFieldEditorExtension.php @@ -15,6 +15,7 @@ use SilverStripe\Forms\GridField\GridFieldDetailForm; use SilverStripe\Forms\GridField\GridFieldToolbarHeader; use SilverStripe\ORM\DataExtension; use SilverStripe\ORM\DataList; +use SilverStripe\ORM\DataObject; use SilverStripe\UserForms\Form\GridFieldAddClassesButton; use SilverStripe\UserForms\Model\EditableFormField; use SilverStripe\UserForms\Model\EditableFormField\EditableFieldGroup; @@ -228,11 +229,11 @@ class UserFormFieldEditorExtension extends DataExtension /** * When duplicating a UserDefinedForm, duplicate all of its fields and display rules * - * @see \SilverStripe\ORM\DataObject::duplicate - * @param \SilverStripe\ORM\DataObject $oldPage + * @see DataObject::duplicate + * @param DataObject $oldPage * @param bool $doWrite * @param string $manyMany - * @return \SilverStripe\ORM\DataObject + * @return DataObject */ public function onAfterDuplicate($oldPage, $doWrite, $manyMany) { diff --git a/code/Extension/UserFormFileExtension.php b/code/Extension/UserFormFileExtension.php index 3e00d65..82f5a60 100644 --- a/code/Extension/UserFormFileExtension.php +++ b/code/Extension/UserFormFileExtension.php @@ -10,11 +10,12 @@ use SilverStripe\ORM\DataObject; use SilverStripe\ORM\Queries\SQLUpdate; use SilverStripe\UserForms\Control\UserDefinedFormController; use SilverStripe\UserForms\Model\Submission\SubmittedFileField; -use SilverStripe\Versioned\Versioned; /** * @property string $UserFormUpload * @method SubmittedFileField SubmittedFileField() + * + * @extends DataExtension */ class UserFormFileExtension extends DataExtension { @@ -45,7 +46,6 @@ class UserFormFileExtension extends DataExtension */ public function updateTrackedFormUpload(&$value): void { - /** @var File|Versioned|UserFormFileExtension $file */ $file = $this->owner; if ($file->UserFormUpload != self::USER_FORM_UPLOAD_UNKNOWN) { $value = $file->UserFormUpload == self::USER_FORM_UPLOAD_TRUE; diff --git a/code/Extensions/UsedOnTableExtension.php b/code/Extensions/UsedOnTableExtension.php index 734fe74..e9c084f 100644 --- a/code/Extensions/UsedOnTableExtension.php +++ b/code/Extensions/UsedOnTableExtension.php @@ -2,6 +2,7 @@ namespace SilverStripe\UserForms\Extensions; +use SilverStripe\Admin\Forms\UsedOnTable; use SilverStripe\Core\Extension; use SilverStripe\ORM\DataObject; use SilverStripe\UserForms\Model\EditableFormField; @@ -11,10 +12,11 @@ use SilverStripe\UserForms\Model\UserDefinedForm; /** * Update DataObjects on the file "Used On" table + * + * @extends Extension */ class UsedOnTableExtension extends Extension { - /** * Link submitted file fields to their parent page * @@ -26,7 +28,6 @@ class UsedOnTableExtension extends Extension if (!($dataObject instanceof SubmittedFileField)) { return; } - /** @var SubmittedForm $submittedForm */ $submittedForm = $dataObject->Parent(); if (!$submittedForm) { $dataObject = null; diff --git a/code/FormField/UserFormsCheckboxSetField.php b/code/FormField/UserFormsCheckboxSetField.php index 7d9092f..8242d4c 100644 --- a/code/FormField/UserFormsCheckboxSetField.php +++ b/code/FormField/UserFormsCheckboxSetField.php @@ -33,8 +33,6 @@ class UserFormsCheckboxSetField extends CheckboxSetField /** * jQuery validate requires that the value of the option does not contain * the actual value of the input. - * - * @return ArrayList */ public function getOptions() { diff --git a/code/FormField/UserFormsCompositeField.php b/code/FormField/UserFormsCompositeField.php index c1bf8ac..7a8cd6c 100644 --- a/code/FormField/UserFormsCompositeField.php +++ b/code/FormField/UserFormsCompositeField.php @@ -40,7 +40,6 @@ abstract class UserFormsCompositeField extends CompositeField implements UserFor if (get_class($field) === EditableFormField::class || !$field->getFormField()) { return $this; } - /** @var EditableFormField $formField */ $formField = $field->getFormField(); // Save this field diff --git a/code/Model/EditableCustomRule.php b/code/Model/EditableCustomRule.php index 66afe78..08fa1b4 100644 --- a/code/Model/EditableCustomRule.php +++ b/code/Model/EditableCustomRule.php @@ -152,7 +152,6 @@ class EditableCustomRule extends DataObject */ public function buildExpression() { - /** @var EditableFormField $formFieldWatch */ $formFieldWatch = $this->ConditionField(); //Encapsulated the action to the object $action = $formFieldWatch->getJsEventHandler(); diff --git a/code/Model/EditableFormField.php b/code/Model/EditableFormField.php index 9f0799a..08d2dd2 100755 --- a/code/Model/EditableFormField.php +++ b/code/Model/EditableFormField.php @@ -950,10 +950,8 @@ class EditableFormField extends DataObject ]; // Check for field dependencies / default - /** @var EditableCustomRule $rule */ foreach ($this->DisplayRules() as $rule) { // Get the field which is effected - /** @var EditableFormField $formFieldWatch */ $formFieldWatch = DataObject::get_by_id(EditableFormField::class, $rule->ConditionFieldID); // Skip deleted fields if (!$formFieldWatch) { diff --git a/code/Model/EditableFormField/EditableMultipleOptionField.php b/code/Model/EditableFormField/EditableMultipleOptionField.php index ab28482..feb2a98 100644 --- a/code/Model/EditableFormField/EditableMultipleOptionField.php +++ b/code/Model/EditableFormField/EditableMultipleOptionField.php @@ -122,7 +122,6 @@ class EditableMultipleOptionField extends EditableFormField $clonedNode = parent::duplicate(true); foreach ($this->Options() as $field) { - /** @var EditableOption $newField */ $newField = $field->duplicate(false); $newField->ParentID = $clonedNode->ID; $newField->Version = 0; diff --git a/code/Model/Recipient/EmailRecipient.php b/code/Model/Recipient/EmailRecipient.php index dbf39f5..0923a07 100644 --- a/code/Model/Recipient/EmailRecipient.php +++ b/code/Model/Recipient/EmailRecipient.php @@ -464,7 +464,6 @@ class EmailRecipient extends DataObject // Check all rules $isAnd = $this->CustomRulesCondition === 'And'; foreach ($customRules as $customRule) { - /** @var EmailRecipientCondition $customRule */ $matches = $customRule->matches($data); if ($isAnd && !$matches) { return false; @@ -715,7 +714,7 @@ class EmailRecipient extends DataObject } /** - * @return DataList|null + * @return DataList|null */ protected function getMultiOptionFields() { @@ -726,7 +725,7 @@ class EmailRecipient extends DataObject } /** - * @return ArrayList|null + * @return ArrayList|null */ protected function getValidSubjectFields() { @@ -745,7 +744,7 @@ class EmailRecipient extends DataObject } /** - * @return DataList|null + * @return DataList|null */ protected function getValidEmailFromFields() { @@ -758,7 +757,7 @@ class EmailRecipient extends DataObject } /** - * @return ArrayList|DataList|null + * @return ArrayList|DataList|null */ protected function getValidEmailToFields() { diff --git a/code/Model/Recipient/UserFormRecipientItemRequest.php b/code/Model/Recipient/UserFormRecipientItemRequest.php index bdfe544..38fe2b5 100644 --- a/code/Model/Recipient/UserFormRecipientItemRequest.php +++ b/code/Model/Recipient/UserFormRecipientItemRequest.php @@ -48,7 +48,7 @@ class UserFormRecipientItemRequest extends GridFieldDetailForm_ItemRequest /** * Get some placeholder field values to display in the preview * - * @return ArrayList + * @return ArrayList */ protected function getPreviewFieldData() { diff --git a/code/Task/RecoverUploadLocationsHelper.php b/code/Task/RecoverUploadLocationsHelper.php index 1961c83..75764f8 100644 --- a/code/Task/RecoverUploadLocationsHelper.php +++ b/code/Task/RecoverUploadLocationsHelper.php @@ -560,7 +560,7 @@ limit 100 * Returns DataList object containing every * uploaded file record * - * @return DataList + * @return DataList */ private function getCountQuery() { diff --git a/code/Task/UserFormsColumnCleanTask.php b/code/Task/UserFormsColumnCleanTask.php index b207314..e62a086 100644 --- a/code/Task/UserFormsColumnCleanTask.php +++ b/code/Task/UserFormsColumnCleanTask.php @@ -30,7 +30,6 @@ class UserFormsColumnCleanTask extends MigrationTask */ public function run($request) { - /** @var \SilverStripe\ORM\DataObjectSchema $schema */ $schema = DataObject::getSchema(); foreach ($this->tables as $db) { diff --git a/code/UserForm.php b/code/UserForm.php index e29414d..2fcffc7 100644 --- a/code/UserForm.php +++ b/code/UserForm.php @@ -350,7 +350,7 @@ SQL; * Converts the RelationList to an ArrayList so that manipulation * of the original source data isn't possible. * - * @return ArrayList + * @return ArrayList */ public function FilteredEmailRecipients($data = null, $form = null) {