Fix the issues reported by phpcs

Above warnings has been fixed.

Reviewed patch #2 LGTM, all reported issue fixed. +1RTBC

Attaching SS for reference.

-    $page = $this->getSession()->getPage();
+    // $page = $this->getSession()->getPage();

If a line is initializing a variable that is never used, that line needs to be removed, not commented out.
Also, the variable could be not necessary, but $this->getSession()->getPage() could still be necessary. Has that been checked?

Adding patch and interdiff file, please review

Hi @everyone,

The patch @chaitanyadessai provided confirmed removed the unused variable as raised by comment #5, but one error were still reported that needs to be fixed. Please see below:

readmore_extrafield git:(3.1.x) curl https://www.drupal.org/files/issues/2023-05-13/3348047-6.patch | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1440  100  1440    0     0   5198      0 --:--:-- --:--:-- --:--:--  5373
patching file src/Plugin/ExtraField/Display/ReadmoreExtrafield.php
patching file tests/src/FunctionalJavascript/ReadMoreExtraFieldFunctionalJavascriptTest.php
➜  readmore_extrafield git:(3.1.x) ✗ cd ..
➜  contrib git:(master) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig readmore_extrafield

FILE: /Users/PrometInterns/Demo-site/drupal-orgissue-v9/web/modules/contrib/readmore_extrafield/src/Plugin/ExtraField/Display/ReadmoreExtrafield.php
----------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Url.
----------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------------

Time: 221ms; Memory: 10MB

Kindly check

Thanks,
Jake

@a.aaronjake please review.

Hi @chaitanyadessai,

The patch you provided did fix the errors.

readmore_extrafield git:(3.1.x) ✗ curl https://www.drupal.org/files/issues/2024-07-23/3348047-8.patch | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1603  100  1603    0     0   6515      0 --:--:-- --:--:-- --:--:--  6735
patching file src/Plugin/ExtraField/Display/ReadmoreExtrafield.php
patching file tests/src/FunctionalJavascript/ReadMoreExtraFieldFunctionalJavascriptTest.php
➜  readmore_extrafield git:(3.1.x) ✗ cd ..
➜  contrib git:(master) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig readmore_extrafield
➜  contrib git:(master) ✗

Will now move this to RTBC

Thanks,
Jake

Let's create a merge request, now that patches are no longer tested.