** Also affects: mahara/18.04
   Importance: Medium
     Assignee: Robert Lyon (robertl-9)
       Status: Fix Committed

** Also affects: mahara/18.10
   Importance: Undecided
       Status: New

** Changed in: mahara/18.04
    Milestone: 18.10.0 => 18.04.0

** Changed in: mahara/18.10
    Milestone: None => 18.10.0

** Changed in: mahara/18.10
   Importance: Undecided => Medium

** Changed in: mahara/18.10
       Status: New => Fix Committed

-- 
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/1758757

Title:
  Leap2A export warnings on comment/annotation

Status in Mahara:
  Fix Committed
Status in Mahara 18.04 series:
  Fix Committed
Status in Mahara 18.10 series:
  Fix Committed

Bug description:
  On doing an export in PHP 7.0/7.1 I see these warnings

  [WAR] 4a (export/leap/lib.php:406) Parameter 1 to
  LeapExportElementComment::setup_links() expected to be a reference,
  value given, referer: http://testing/export/index.php

  [WAR] 4a (export/leap/lib.php:406) Parameter 1 to
  LeapExportElementAnnotation::setup_links() expected to be a reference,
  value given, referer: http://testing/export/index.php

  In the Mahara code it says:
  "You must explicitly pass variables by reference when calling call_user_func, 
..." 

  But in PHP manual itself it says "Note that the parameters for
  call_user_func() are not passed by reference."

  So there is something not quite right here

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1758757/+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