mahara-contributors team mailing list archive

[Mahara Bot <1911822@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/11525
Committed: https://git.mahara.org/mahara/mahara/commit/846052adf2cddef2aa93e86203c2c5011d5c3478
Submitter: Lisa Seeto (lisaseeto@xxxxxxxxxxxxxxx)
Branch:    master

commit 846052adf2cddef2aa93e86203c2c5011d5c3478
Author: Doris Tam <doristam@xxxxxxxxxxxxxxx>
Date:   Tue Jan 26 12:40:35 2021 +1300

Bug 1911822: Added a check for if a collection will be copied as a
personal one

Before it checks for evidence views to copy over to a new personal
collection

Change-Id: Icb57ef4c51dde1e107bcaea6201fd96ad941f06d

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1911822

Title:
  Copying a portfolio with SmartEvidence into the group/institution/site
  level throws errors

Status in Mahara:
  Fix Committed

Bug description:
  When you created a portfolio with SmartEvidence and want to add that
  to the group, institution, or site level, you get the following error
  message:

  [WAR] 80 (lib/collection.php:406) Undefined property: stdClass::$owner
  Call stack (most recent first):

      log_message("Undefined property: stdClass::$owner", 8, true, true, "/home/kristina/code/mahara/htdocs/lib/collection.p...", 406) at /home/kristina/code/mahara/htdocs/lib/errors.php:515
      error(8, "Undefined property: stdClass::$owner", "/home/kristina/code/mahara/htdocs/lib/collection.p...", 406, array(size 21)) at /home/kristina/code/mahara/htdocs/lib/collection.php:406
      Collection::create_from_template(array(size 5), 1) at /home/kristina/code/mahara/htdocs/lib/view.php:7625
      createview_submit(object(Pieform), array(size 5)) at /home/kristina/code/mahara/htdocs/lib/pieforms/pieform.php:568
      Pieform->__construct(array(size 8)) at /home/kristina/code/mahara/htdocs/lib/pieforms/pieform.php:168
      Pieform::process(array(size 8)) at /home/kristina/code/mahara/htdocs/lib/mahara.php:5507
      pieform(array(size 8)) at /home/kristina/code/mahara/htdocs/view/choosetemplate.php:30

  
  Note: Groups cannot yet handle SmartEvidence. Therefore, that page would be ignored during the copying process.

  To replicate:

  1. Create a collection in a non-site admin account. Make sure it includes SmartEvidence. Note: I saw the problem without adding annotations etc.
  2. Make the collection copyable to all registered people.
  3. Copy the collection into an institution, the site, or the group.
  Expected result: It works.
  Actual result: You get the above error message.

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