Little bug: assign groups to roles

  • Resolved metiu92

    (@metiu92)


    8 years ago

    Hello!
    I just saw two little bugs: after getting to “Assign groups to roles” page, if a role is not assigned to any group an “undefined index” is thrown.
    Another similar error is thrown from page “Management information” when a group is created without setting a name. If at a later time I set a name everything is ok (also if I re-set the name with an empty one).

    I got a little question also: sorry if this seems to be silly but I didn’t find anything about it, can you tell me how to remove groups?

    Thank you in advance and beautiful plugin!

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Robin W

    (@robin-w)

    Thanks, I’ll take a look at those errors and get them fixed in the next version

    There is no remove groups – sorry it is occasionally asked for, but the coding to check that nothing is set etc. before allowing deletion is too painful for me to get the energy to write !!

    You can just change the name of an unused group to say ‘Not Used’ or indeed re-use it.

    Plugin Author Robin W

    (@robin-w)

    hmm… I can’t see either of those errors, but if you can give me the full and exact error I suspect I can work out what is wrong

    Thread Starter metiu92

    (@metiu92)

    Ok, consider I have xdebug active so maybe this is why I can see the error. The messages I see are these:

    1. from “Assign groups to roles” page:Notice: Undefined index: in ****\wp-content\plugins\bbp-private-groups\includes\role_assignment.php on line 61
    2. from “From management information” page: Notice: Undefined index: group3 in ****\wp-content\plugins\bbp-private-groups\includes\settings.php on line 315

    The group3 is just a new group I’ve created without setting a name. I just think some isset() checks are missing maybe, but I didn’t look it out very well.

    Thank you for your fast answer!

    Plugin Author Robin W

    (@robin-w)

    Thanks for that – I’ve just released version 3.4.0 which should fix both, but please could you confirm for me.

    Thread Starter metiu92

    (@metiu92)

    Perfect, everything is fixed now! Thank you!!

    Have a nice day ??

    Plugin Author Robin W

    (@robin-w)

    Great – thanks for letting me know and for reporting it – I never like errors in code !

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Little bug: assign groups to roles’ is closed to new replies.