From 556cb3663879845d3ae454f74bfdf32399b847e5 Mon Sep 17 00:00:00 2001
From: Mojmir Fendek <mfendek@silverstripe.com>
Date: Mon, 14 Oct 2024 13:37:13 +1300
Subject: [PATCH] PR fixes.

---
 code/ReportAdmin.php | 20 +++++++-------------
 composer.json        |  2 +-
 2 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/code/ReportAdmin.php b/code/ReportAdmin.php
index 46f2a05d..8bd38099 100644
--- a/code/ReportAdmin.php
+++ b/code/ReportAdmin.php
@@ -119,9 +119,7 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider
         }
 
         return $output
-            // Provide a good default for the reports order, otherwise this will be using sort by namespace
             ->sort('Title', 'ASC')
-            // Provide a good default for the data class, otherwise this will use the class of the first item in the list
             ->setDataClass(Report::class);
     }
 
@@ -250,14 +248,16 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider
                 GridFieldPaginator::create()
             );
 
+            $titleLabel = _t('SilverStripe\\Reports\\ReportAdmin.ReportTitle', 'Title');
+            $descriptionLabel = _t('SilverStripe\\Reports\\ReportAdmin.ReportDescription', 'Description');
+
             // Configure the filter header filter search form
             $generalField = BasicSearchContext::config()->get('general_search_field_name');
             $searchFieldList = FieldList::create([
                 HiddenField::create($generalField),
-                TextField::create('Title'),
-                TextField::create('Description'),
+                TextField::create('Title', $titleLabel),
+                TextField::create('Description', $descriptionLabel),
             ]);
-
             $searchContext = BasicSearchContext::create(Report::class);
             $searchContext->setFields($searchFieldList);
 
@@ -266,22 +266,16 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider
                 'Title',
                 'Description',
             ];
-
             foreach ($filters as $fieldName) {
                 $fieldFilter = PartialMatchFilter::create($fieldName);
-                $fieldFilter->setModifiers([
-                    // We want case-insensitive match to be consistent with other areas of the CMS
-                    'nocase',
-                ]);
                 $searchContext->addFilter($fieldFilter);
             }
-
             $filterHeader->setSearchContext($searchContext);
 
             $gridField = GridField::create('Reports', false, $this->Reports(), $gridFieldConfig);
             $columns->setDisplayFields(array(
-                'title' => _t('SilverStripe\\Reports\\ReportAdmin.ReportTitle', 'Title'),
-                'description' => _t('SilverStripe\\Reports\\ReportAdmin.ReportDescription', 'Description'),
+                'title' => $titleLabel,
+                'description' => $descriptionLabel,
             ));
 
             $columns->setFieldFormatting([
diff --git a/composer.json b/composer.json
index 4f009e31..c6927d20 100644
--- a/composer.json
+++ b/composer.json
@@ -21,7 +21,7 @@
     ],
     "require": {
         "php": "^8.1",
-        "silverstripe/framework": "^5",
+        "silverstripe/framework": "^5.2",
         "silverstripe/admin": "^2",
         "silverstripe/versioned": "^2",
         "silverstripe/config": "^2",