FILE: /var/www/html/drupal-contrib/modules/contrib/hubspot_integration/hubspot_integration.routing.yml
------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------------------------------------
7 | WARNING | The administration page callback should probably use "administer site configuration" - which implies the user can change something - rather than "access
| | administration pages" which is about viewing but not changing configurations.
---------------------------------------------------------------------------------------------------------

CommentFileSizeAuthor
#2 3125656-2.patch571 bytesprabha1997

Comments

prabha1997 created an issue. See original summary.

prabha1997’s picture

prabha1997 commented
Assigned: prabha1997 » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3125656-2.patch571 bytes

Kindly review patch

shreyal999’s picture

shreyal999 commented
Status: Needs review » Reviewed & tested by the community

Patch Reviewed. Working as expected. No Warnings.

romainj’s picture

romainj commented

I think that we should use the Administer hubspot integration permission instead.

romainj’s picture

romainj commented

In fact I suggest we keep using the access administration pages permission as this route use a core controller.

romainj’s picture

romainj commented
Status: Reviewed & tested by the community » Closed (won't fix)