There should be separate permissions defined for:

  • Administering setting for the module.
  • Accessing the search results page.
CommentFileSizeAuthor
#4 2924580-permissions-4.patch759 bytesajits
#4 interdiff.txt441 bytesajits
#2 2924580-permissions.patch318 bytesnikitaaswani

Comments

AjitS created an issue. See original summary.

nikitaaswani’s picture

nikitaaswani commented
Status: Active » Needs review
StatusFileSize
new 2924580-permissions.patch318 bytes

Uploading a patch adding permissions.

ajits’s picture

ajits
he/him
Primary language Marathi
Location India
commented
Status: Needs review » Needs work

Thank you for the patch! This adds the permissions alright however missed to use it on the search results page. The page still has access content permission.

ajits’s picture

ajits
he/him
Primary language Marathi
Location India
commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new interdiff.txt441 bytes
new 2924580-permissions-4.patch759 bytes

Correcting on commit.

  • AjitS committed 2cf38ae on 8.x-1.x authored by nikitaaswani 
    Issue #2924580 by AjitS, nikitaaswani: Define permission to access...
ajits’s picture

ajits
he/him
Primary language Marathi
Location India
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x

Status: Fixed » Closed (fixed)

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