Advertisement:

Author Topic: Permission dramas  (Read 1195 times)

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Permission dramas
« on: March 13, 2019, 03:13:29 AM »
On my forum I have numerous boards but by way of example under category 'Picture and Video Showcase' I have boards

Videos by Drone
Photos by Drone
Panos by drone
etc


ONLY the photos by drone is visible to regular members. As admin I can naturally see ALL of them.

Permissions appear to be set identical - the 'regular members' checkbox is ticked.

I'm not using deny permissions and not using post count based groups permissions.

I did try activating the latter, but given the attributes are all inherited form 'regular member' anyway it of course made no difference.

I have a few boards that I specifically wanted to hide, which I did by accessing board permissions and  under "allowed groups" where for those boards I specifically WANTED to hide, only global mod is checked (not regular members).

So I rechecked all the boards I'm having problems with and they all DO have regular members ticked.



Help?
Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline SychO

  • Lead Customizer
  • SMF Hero
  • *
  • Posts: 1,526
  • Gender: Male
    • SychO9 on GitHub
    • SychO
Re: Permission dramas
« Reply #1 on: March 13, 2019, 05:24:32 AM »
I just did some testing and it appears this is a bug
logged on github https://github.com/SimpleMachines/SMF2.1/issues/5514
« Last Edit: March 13, 2019, 05:42:02 AM by SychO »

Imazighen

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Re: Permission dramas
« Reply #2 on: March 13, 2019, 06:02:02 AM »
Well that's super frustrating.... I've invited people to my new forum and managed to upset them already LOL

Is there a work around?
Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline SychO

  • Lead Customizer
  • SMF Hero
  • *
  • Posts: 1,526
  • Gender: Male
    • SychO9 on GitHub
    • SychO
Re: Permission dramas
« Reply #3 on: March 13, 2019, 06:47:31 AM »
yea, you can either apply the fix I posted here: https://github.com/SimpleMachines/SMF2.1/pull/5515/files
or you can use post based membergroups to allow access to boards

Imazighen

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Re: Permission dramas
« Reply #4 on: March 13, 2019, 06:59:12 AM »
Might sound silly but don't the PBMG just inherit permissions from the default profile?
That also seems a painful workaround for future issues, I'd rather make a code change :)

Would you be kind enough to put here in code tags, like
(find) ...
(add after)...  or
(replace with) ...

as I'm unsure exactly what to do with that GH info. Unless its as simple as deleting the red line and replacing it with the green line?
Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline SychO

  • Lead Customizer
  • SMF Hero
  • *
  • Posts: 1,526
  • Gender: Male
    • SychO9 on GitHub
    • SychO
Re: Permission dramas
« Reply #5 on: March 13, 2019, 07:05:51 AM »
Might sound silly but don't the PBMG just inherit permissions from the default profile?

It does inherit, but the issue is that when you check the box for Regular Membergroups and save, it doesn't actually save the data for Regular Members, that's why it doesn't take effect.

as I'm unsure exactly what to do with that GH info. Unless its as simple as deleting the red line and replacing it with the green line?

exactly

Sources/Subs-Boards.php

Find
Code: [Select]
if (!empty($boardUpdateParameters['deny_groups']))Replace with
Code: [Select]
if (!empty($boardOptions['deny_groups']))
Find
Code: [Select]
if (!empty($boardUpdateParameters['member_groups']))Replace with
Code: [Select]
if (!empty($boardOptions['access_groups']))

Imazighen

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Re: Permission dramas
« Reply #6 on: March 13, 2019, 07:10:12 AM »
Bad news I did those edits and saved it. Opened it again to double check it had been saved.
Still no good I'm afraid.

Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline Arantor

  • Resident Overthinker
  • SMF Friend
  • SMF Legend
  • *
  • Posts: 71,643
    • StoryBB/StoryBB on GitHub
Re: Permission dramas
« Reply #7 on: March 13, 2019, 07:11:12 AM »
Note that actual permissions are different and completely separate from board access and inheritance of permissions does not apply to board access.
Don’t try to tell me that some power can corrupt a person. You haven’t had enough to know what it’s like.

No good deed goes unpunished / No act of charity goes unresented.

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Re: Permission dramas
« Reply #8 on: March 13, 2019, 07:15:35 AM »
Umm so what does that mean in relation to access - should that pull request info (as above) not have fixed the access problem?
Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Re: Permission dramas
« Reply #9 on: March 13, 2019, 07:20:16 AM »
Update: having found that issue, I now have to go into the affected boards, uncheck and then recheck the boxes for groups allowed access, click modify, and it WORKS !!!

Thanks
Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline Arantor

  • Resident Overthinker
  • SMF Friend
  • SMF Legend
  • *
  • Posts: 71,643
    • StoryBB/StoryBB on GitHub
Re: Permission dramas
« Reply #10 on: March 13, 2019, 07:23:42 AM »
Umm so what does that mean in relation to access - should that pull request info (as above) not have fixed the access problem?

It means that the above pull request is right and that you’re conflating two unrelated things ;)
Don’t try to tell me that some power can corrupt a person. You haven’t had enough to know what it’s like.

No good deed goes unpunished / No act of charity goes unresented.

Offline SychO

  • Lead Customizer
  • SMF Hero
  • *
  • Posts: 1,526
  • Gender: Male
    • SychO9 on GitHub
    • SychO
Re: Permission dramas
« Reply #11 on: March 13, 2019, 07:24:39 AM »
yea you have to go back and re-save the board options after making the file edits

anyway, thanks for the report.

Imazighen

Offline woolly bugger

  • Jr. Member
  • **
  • Posts: 159
    • Blue Ridge Fly Fisherman's Forum
Re: Permission dramas
« Reply #12 on: March 13, 2019, 12:15:44 PM »
yeah, this caused some "uncomfortable" moments on my board when the upgrade revealed a secret board to general members, I had to scramble to change permissions, but also had to admit "hurt" members to the secret board....

Offline Kindred

  • The Mean One
  • Support Specialist
  • SMF Legend
  • *
  • Posts: 58,502
  • Gender: Male
    • Kindred-999 on GitHub
Re: Permission dramas
« Reply #13 on: March 13, 2019, 12:28:40 PM »
and, yet another reason to not use beta and RC releases on production sites. :P
Please do not PM, IM or Email me with support questions.  You will get better and faster responses in the support boards.  Thank you.

Offline TurtleKicker

  • Full Member
  • ***
  • Posts: 416
Re: Permission dramas
« Reply #14 on: March 13, 2019, 02:05:21 PM »
and, yet another reason to not use beta and RC releases on production sites. :P
Well, admittedly, the release announcement for RC1 stated:

Quote
The term "Release Candidate" means that the developers believe this version of the software is reliable enough to be installed and used on production websites.


Offline Kindred

  • The Mean One
  • Support Specialist
  • SMF Legend
  • *
  • Posts: 58,502
  • Gender: Male
    • Kindred-999 on GitHub
Re: Permission dramas
« Reply #15 on: March 13, 2019, 03:44:11 PM »
yeah, me and the leads will have to disagree on that point. :P
Please do not PM, IM or Email me with support questions.  You will get better and faster responses in the support boards.  Thank you.

Offline vbgamer45

  • Customizer
  • SMF Super Hero
  • *
  • Posts: 21,559
    • smfhacks on Facebook
    • VBGAMER45 on GitHub
    • @createaforum on Twitter
    • SMF For Free
Re: Permission dramas
« Reply #16 on: March 13, 2019, 03:56:44 PM »
I would agree mixed messages. I wait for final for your main sites. Since you have to redo your mods/theme changes that you make on your site.
Community Suite for SMF - Take your forum to the next level built for SMF, Gallery,Store,Classifieds,Downloads,more!

SMFHacks.com -  Paid Modifications for SMF

Mods:
EzPortal - Portal System for SMF
SMF Gallery Pro
SMF Store SMF Classifieds Ad Seller Pro

Offline Arantor

  • Resident Overthinker
  • SMF Friend
  • SMF Legend
  • *
  • Posts: 71,643
    • StoryBB/StoryBB on GitHub
Re: Permission dramas
« Reply #17 on: March 13, 2019, 03:57:46 PM »
Or, more simply, beliefs can be wrong...
Don’t try to tell me that some power can corrupt a person. You haven’t had enough to know what it’s like.

No good deed goes unpunished / No act of charity goes unresented.

Offline landyvlad

  • Sr. Member
  • ****
  • Posts: 862
  • Gender: Male
    • Michael Reed on Facebook
    • GSX1400 Owners ORG
Re: Permission dramas
« Reply #18 on: March 13, 2019, 11:58:18 PM »
Well I started a brand new site and as RC1 was released I elected to use it.

The existing site (2.0.15) will remain so until 2.1 final is released AND all the mods I use are compatible (or a suitable alternative is).

So I have a foot on both camps !

yea you have to go back and re-save the board options after making the file edits
anyway, thanks for the report.

Cool, just letting you know in case you didn't realise that. But moreso to make it clear to others who may view this thread in future as to what's required to 'fix' the issue.

Thank YOU for the prompt fix :)


Umm so what does that mean in relation to access - should that pull request info (as above) not have fixed the access problem?
It means that the above pull request is right and that you’re conflating two unrelated things ;)

Yeah my fault, a terminology issue.  But all sorted now. Cheers.
Please do not PM, IM or Email me with questions on astrophysics or theology.  You will get better and faster responses by asking homeless people in the street. Thank you.

To paraphrase Kindred: "There are no technical solutions to social problems."

No hack nor blackhats, just persistent asshats.

Offline Sesquipedalian

  • The Mad Doctor
  • Lead Developer
  • Sr. Member
  • *
  • Posts: 969
  • Gender: Male
  • It works! ... in theory.
    • Sesquipedalian on GitHub
Re: Permission dramas
« Reply #19 on: March 14, 2019, 01:28:23 AM »
and, yet another reason to not use beta and RC releases on production sites. :P
Well, admittedly, the release announcement for RC1 stated:

Quote
The term "Release Candidate" means that the developers believe this version of the software is reliable enough to be installed and used on production websites.

Actually, the release announcement stated:
Quote
The term "Release Candidate" means that the developers believe this version of the software is reliable enough to be installed and used on production websites. Nevertheless, you should still expect to encounter minor bugs in SMF 2.1 RC1. We encourage you to report any bugs you find.
You can and should expect bugs.


Or, more simply, beliefs can be wrong...
...which is exactly why that particular word was chosen, yes. :)
« Last Edit: March 14, 2019, 01:52:15 AM by Sesquipedalian »
I promise you nothing.

Sesqu... Sesqui... what?
Sesquipedalian, the best word in the English language.