mahara-contributors team mailing list archive

[Robert Lyon <1892428@xxxxxxxxxxxxxxxxxx>]

** Changed in: mahara
       Status: Fix Committed => Fix Released

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

Title:
  Turn peer role into configurable role so it can see content

Status in Mahara:
  Fix Released

Bug description:
  It shall be possible for a peer to see the portfolio content.
  Currently, Mahara does not allow someone with the 'Peer' role to see
  the content of the portfolio pages to which the peer has access to.
  This is restrictive for some organisations. Therefore, we are going to
  turn it into an institution setting so that an institution
  administrator can determine how that role shall behave.

  Since this is an assessment feature that may be more often then not be
  determined by an organisation rather than individuals, it is best to
  make it an institution setting and not require the decision to be made
  on each individual peer block.

  We propose to create a new institution setting 'Allow peers to see
  portfolio content' for the peer role with the two possible states:

  1. Can't see any content but can create peer assessment (the current setting)
  2. Can see the entire portfolio and can create peer assessment

  This setting decides what the peer assessor can do when they are
  commenting on a portfolio in that particular institution.

  More details as determined with Cecilia:

  1. A new field will be added to institution settings.

  If set to 'Yes': if the portfolio owner belongs to this institution,
  all account holders that have peer role access to that portfolio will
  be able to see the block contents.

  For multiple institutions: If the owner of the portfolio belongs to
  more than one institution, then the most restrictive rule will apply.
  In other words, if an account holder has peer access to a portfolio,
  they will be able to see the block content only if all the
  institutions the owner belongs to have this setting set to 'Yes'.

  Default value will be 'No'. This will be saved on the
  'institution_config' table on the DB.

  This new field could be placed just below 'Allow public institution
  portfolios'.

  Field name: Allow peers to see portfolio content

  Description: Allow account holders who receive the 'Peer' role on a
  portfolio to see the entire content in the portfolio.

  Help file text: The 'Peer' role is only available for portfolio
  access. If it is set to 'No', the peer can only see the block headings
  on pages on which peer assessments are required. On pages where there
  is no peer assessment, they will not be able to see anything. If an
  account holder has the role of 'Peer and manager', they can always see
  the entire portfolio, no matter this setting here.

  2.Accessing a portfolio by a peer

  The check to see whether a peer can see the block content or not will
  be done when rendering a block instance (on view display).

  This change can be done in the PluginBlocktype class, by adding a code
  to the display_for_roles function. This is the function that defines
  if the block should display content or not.

  If an account holder has more than one portfolio access role, then the more permissive one counts, e.g. account holder has
  a) Peer role with access to content in one institution and peer role without access to content in another institution: Can't see content (see 1 above)
  b) Peer role without access to content and manager role, i.e. 'Peer and manager', then they can see the content on the portfolio because the manager role is more permissive and thus counts.
  b) Peer role with access to content and manager role: Can see content

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