Content such as Basic Page and Article are not visible to anonymous users. I have set the "Entity: View content entities" for the above pages and the correct view is shown to the super user (user 1). They also seemed correct when I first tested (just after setting permissions).

If an article is promoted to the front page then is only shows for someone who is at least a member.

CommentFileSizeAuthor
#11 group-2857321-11.patch4.48 KBkristiaanvandeneynde
#11 group-2857321-11-TEST-ONLY.patch2.52 KBkristiaanvandeneynde
#9 group-2857321-9.patch3.97 KBkristiaanvandeneynde
#9 group-2857321-9-TEST-ONLY.patch1.99 KBkristiaanvandeneynde
#4 group-permissions_to_view_nodes-2857321-4.patch1.69 KBgaydamaka
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

magpie5212 created an issue. See original summary.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented

Hmm, this should work. Could you provide steps to reproduce?

magpie5212’s picture

magpie5212 CreditAttribution: magpie5212 commented

I have been trying to get this down to some simple steps. I created two group types, Club and Service Team. I set up a view to show a list of the Basic Pages which I placed in a block on the page for both of the group types. As far as I can tell I set the permission the same for both the group types. I find that for Service Team the pages are visible in the list and then accessible to anyone. However, the same view placed on a Club page doesn't show the pages.

The site is on a test domain at the moment at eight.1060.org.uk if that's any help.

gaydamaka’s picture

gaydamaka CreditAttribution: gaydamaka as a volunteer and at FFW commented
FileSize
group-permissions_to_view_nodes-2857321-4.patch1.69 KB

Hi

Maybe I can help for this issue.

gaydamaka’s picture

gaydamaka CreditAttribution: gaydamaka as a volunteer and at FFW commented
Status: Active » Needs review
gaydamaka’s picture

gaydamaka CreditAttribution: gaydamaka as a volunteer and at FFW commented

When we have more one group type in loop every iteration we replace $grants_ao["gnode:$node_type_id"] on new and weight has only last group type.

magpie5212’s picture

magpie5212 CreditAttribution: magpie5212 commented

I applied the patch and it fixes the problem.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented

Thanks for the feedback, this shows that our tests are not covering every scenario. Will try to produce a failing test and see if the patch fixes it.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented
FileSize
group-2857321-9-TEST-ONLY.patch1.99 KB
group-2857321-9.patch3.97 KB

This oughta do it

Status: Needs review » Needs work

The last submitted patch, 9: group-2857321-9.patch, failed testing.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented
Status: Needs work » Needs review
FileSize
group-2857321-11-TEST-ONLY.patch2.52 KB
group-2857321-11.patch4.48 KB

Whoops, didn't adjust test properly.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented
Status: Needs review » Fixed

Other build failed but seems to have stalled. Patch remains the same, so committing. We need better grants tests though, the current ones are rather cryptic :)

Status: Fixed » Closed (fixed)

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