Update module to utilize Dependency Injection instead of direct Drupal calls

PabloNicolas → changed the visibility of the branch 3441260-update-module-to to hidden.

PabloNicolas → changed the visibility of the branch 3441260-update-module-to to active.

I have checked all conding standard errors via phpcs and it was showing these :

/modules/contrib $ phpcs --standard=DrupalPractice  --extensions=php,module,inc,install tripadvisor_integration/

FILE: /modules/contrib/tripadvisor_integration/src/Form/TripAdvisorIntegrationAdminForm.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 50 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 54 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 59 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 63 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------

/modules/contrib $ phpcs --standard=Drupal  --extensions=php,module,inc,install tripadvisor_integration/

FILE: /modules/contrib/tripadvisor_integration/tripadvisor_integration.module
----------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------
 51 | ERROR | [x] Namespaced classes/interfaces/traits should be referenced with use statements
----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------


FILE: /modules/contrib/tripadvisor_integration/src/Form/TripAdvisorIntegrationAdminForm.php
------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------
 20 | WARNING | Possible useless method overriding detected
------------------------------------------------------------------------------------------------------------------


FILE: /modules/contrib/tripadvisor_integration/src/Plugin/Field/FieldType/TripAdvisorItem.php
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------
 36 | WARNING | Possible useless method overriding detected
--------------------------------------------------------------------------------------------------------------------


FILE:/modules/contrib/tripadvisor_integration/src/Plugin/Field/FieldFormatter/TripAdvisorFieldFormatter.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------------------------------------
 27 | ERROR | Class property $cache_backend should use lowerCamel naming without underscores
 34 | ERROR | Class property $date_time should use lowerCamel naming without underscores
 41 | ERROR | Class property $module_extension_list should use lowerCamel naming without underscores
-----------------------------------------------------------------------------------------------------------------------------------

Most of the errors from above has been fixed except Possible useless method overriding detected.

Thank you for helping on this one mitthukumawat!

My last commit fixed the remaining coding standards warning.

PR merged into the dev release - thanks for your work on this!

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