This topic was marked solved by its starter, on November 14th, 2012, 09:05 AM
Delete member does not delete PM rules

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670
A confident man keeps quiet.whereas a frightened man keeps talking, hiding his fear.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
Re: Delete member does not delete PM rules
« Reply #1, on November 5th, 2012, 12:30 AM »
Aren't PM rules stored in the members table?

EDIT: I'm thinking of labels, not rules, ignore me.
Posted: November 5th, 2012, 12:27 AM

Fixed locally, will commit in next round of commits, thanks.
When we unite against a common enemy that attacks our ethos, it nurtures group solidarity. Trolls are sensational, yes, but we keep everyone honest. | Game Memorial

Nao

  • Dadman with a boy
  • Posts: 16,082
Re: Delete member does not delete PM rules
« Reply #2, on November 5th, 2012, 12:38 AM »
In the same vein, maybe it should be pointed out that the bug that prevented testers from downloading Wedge for a couple of days was due to the fact that the media profile permissions weren't set for them.

Which leads me to two remarks about permission copying:

- we should make it clearer in the permissions page what the difference between 'inherit' and 'base' is (I actually had to go through the source code to be sure I was going to 'copy' them from the Regular Members table!)
- when copying permissions from another membergroup, also copy their media profile permissions!!
i.e. go through all media permission profiles, get the original membergroup's permissions, and copy them for the new membergroup.

I know it's a PITA, but it's safer that way.
Do you think you can do it, Pete? I'm way too uncomfortable with permission code. But OTOH, here it's mostly media permissions, which you may not be as well versed into. My knowledge about them is not perfect either -- that feature was broken by Dragooon, and I had to learn how to use it first. And it was years ago...

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
Re: Delete member does not delete PM rules
« Reply #4, on November 5th, 2012, 01:28 AM »
Fixed original issue in r1768. Not marking solved since I think the above two posts should be in another topic but will leave for now.