First of all, having this module is really great, well done!!

When trying the RC1 just now, I noticed that anonymous users didn't have access to any content anymore, until I though of rebuilding permissions, after that it worked again. So it seems that this is a necessary step. That's why I would suggest to bring up a message after installation, reminding admins to do this. Many other modules do this, so I guess there must be some mechanism for this.

CommentFileSizeAuthor
#4 domain_adv-354392.patch597 bytesJuliaKM

Comments

bookish’s picture

bookish commented
Assigned: Unassigned » bookish
burningdog’s picture

burningdog commented

You could use drupal_set_message() to do this :)

Skirr’s picture

Skirr commented

As minimum, its very import to write about it in readme.file

JuliaKM’s picture

JuliaKM commented
Status: Active » Needs review
StatusFileSize
new domain_adv-354392.patch597 bytes

This patch adds a .install file which will set the flag to rebuild permissions when the module is enabled or disabled.

aaron1234nz’s picture

aaron1234nz commented

I found that too, however, rebuilding the node access permissions with domain advanced turned on removes the node access permissions for sites (in the node access table, all 'domain_id' grants are removed and 'all' grants are inserted (ie default drupal behavior))

aaron1234nz’s picture

aaron1234nz commented

Ignore previous comment. This bug is fixed with Commit #279638 and the latest -dev version is working for me.