The DvG-domains module implements hook_dvg_default_permissions_alter. Therefore it loops through all provided default permissions and applies them accordingly.

There is a flaw in the code where the variable $domain_part is set, and used to determine if the permission should be set, but not initialized blank at the start of the loop. So the value can stay the same in two loops, while it should not.

CommentFileSizeAuthor
#2 dvg_domains-init-var-3068657-2.patch690 bytesjoshahubbers

Comments

JoshaHubbers created an issue. See original summary.

joshahubbers’s picture

StatusFileSize
new690 bytes

This patch initializes the variable at the start of each loop so the value of the previous loop will not affect the current loop.

Danny.Wouters’s picture

Status: Needs review » Reviewed & tested by the community

Nice catch! The variable should indeed receive a default value during each loop.
The patch from #2 fixes this problem.

paulvandenburg’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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