surya pratap → made their first commit to this issue’s fork.
Patch applied please review
surya pratap → created an issue.
surya pratap → created an issue.
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/entity_visitors.page.inc
-----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------
24 | WARNING | Unused variable $entity_visitors.
-----------------------------------------------------------------------------------------------------
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/entity_visitors.info.yml
-----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------
1 | WARNING | "core_version_requirement" property is missing in the info.yml file
-----------------------------------------------------------------------------------------------------
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/src/Form/EntityVisitorsSettingsForm.php
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------
12 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
--------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/src/Form/EntityVisitiorsConfig.php
------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------
11 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
57 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
114 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/src/EventSubscriber/EntityRouteMatchSubscriber.php
-------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------
64 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/src/EventSubscriber/SendMailOnEntityVisitedSubscriber.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
52 | WARNING | User::load calls should be avoided in classes, use dependency injection instead
--------------------------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/contribution/web/modules/contrib/entity_vistiors-3105864/src/Service/EntityVisitorsManager.php
------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------
16 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
133 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
144 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
surya pratap → made their first commit to this issue’s fork.
Hi @Jay Jangid , Reviewed your merge request. It was successfully updated README.md file.
surya pratap → made their first commit to this issue’s fork.