menu_per_role.info is missing the configure = admin/config/system/menu_per_role entry

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markusd1984 created an issue. See original summary.

jigish.addweb’s picture

Status: Active » Needs review
FileSize
349 bytes

@markusd1984, Please find the attached patch for the same.

Thanks!..

krina.addweb’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
28.7 KB
29.72 KB
183.02 KB

@jigish.addweb, Thanks for the patch. Your patch is working fine as I tested the mentioned changes using simplytest.me. PFA

Grimreaper’s picture

Hello,

Thanks for reporting the issue, making a patch and testing it.

I also test the patch and it is ok. I will merge it. Here is my review anyway.

+++ b/menu_per_role.info
@@ -3,3 +3,4 @@ description = Allows restricting access to menu items per role.
\ No newline at end of file

Files must end with an empty new line.

I will reorder the entries in the .info file.

Grimreaper’s picture

Status: Reviewed & tested by the community » Fixed

And this is merged now.

Status: Fixed » Closed (fixed)

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