Hi Son,

The issue with this is that you can't actually configure the
"allowcomment" flag of any access rights, in the web interface it
appears as just a overall view setting.

I think either the for loop needs to cover the "objectionable" access
type which doesn't show up anywhere in the access interface, and treat
that differently.

Cheers,

Hugh

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/1095890

Title:
  allowcomments not respected properly

Status in Mahara ePortfolio:
  Invalid

Bug description:
  Not sure why, but the function user_comments_allowed (lib/view.php
  line 3872), seems to check through all the access rights for the
  "allowcomment" flag, which doesn't seem to be configurable from the
  web interface. If the page has allowcomments off, and one of the
  access rights allows comments, then comments are allowed.

  The particular access right i found it on was an "objectionable" type.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1095890/+subscriptions

_______________________________________________
Mailing list: https://launchpad.net/~mahara-contributors
Post to     : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp

Reply via email to