Replace uses of /Drupal within classes

Hi

I fix the all the issues except one Dependency Injection because i didn't get object for that class.

Please review,

Hi dineshkumarbollu,

I applied your patch #2 into Event Platform version 1.0.x-dev and confirmed that most of the PHPCS errors were fixed except the dependency injection you mentioned.

Please look at the screenshots attached for your reference.

Thank you

+  /**
+   * The configuration factory service.
+   *
+   * @var \Drupal\Core\Config\ConfigFactoryInterface
+   */
+  protected $configFactory;

The config factory. is sufficient.

+  /**
+   * Constructs a new CustomBlock object.
+   *
+   * @param \Drupal\Core\Config\ConfigFactoryInterface $configFactory
+   *   The configuration factory service.
+   */
+  public function __construct(ConfigFactoryInterface $configFactory) {

The class name is missing its namespace.

-      return ['event_platform_header_cta' => [
-        '#cache' => [
-          'tags' => ['config_pages_list'],
+      return [
+        'event_platform_header_cta' => [
+          '#cache' => [
+            'tags' => ['config_pages_list'],
+          ],

The existing code is already correct. The first line does not even exceed 80 characters.

 /**
- * Class SchedulerSettingsForm.
+ * Defines SchedulerSettingsForm class.

That description is still repeating the class name.

 /**
  * @file
- * Install, update and uninstall functions for the Event Platform Sessions module.
+ * Install, update and uninstall functions for Event Platform Sessions module.
  */

/**
  * @file
- * Install, update and uninstall functions for the Event Platform Sponsors module.
+ * Install, update and uninstall functions for Event Platform Sponsors module.
  */

The existing line just needed a comma after update. Removing the definite article is not the correct change, since in English for the module is the correct phrase to use.

I ended up fixing the code standards issues in a separate issue, so updating this thread to focus on replacing the usage of /Drupal static calls within classes. The code in the merge request seemed to break my Drupal 10 site, so I did some refactoring to get it functional. I also added a couple of other fixes for where /Drupal calls were being made within classes. The resulting changes are now merged in.

Automatically closed - issue fixed for 2 weeks with no activity.