← Back to team overview

mahara-contributors team mailing list archive

[Bug 1308294] A patch has been submitted for review

 

Patch for "master" branch: https://reviews.mahara.org/3266

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

Title:
  Error in profile_icon_url call

Status in Mahara ePortfolio:
  In Progress

Bug description:
  In htdocs/lib/user.php (approx line 2504) we have:

   if (!property_exists($user, 'profileicon') || !property_exists($user, 'email')) {
     ...
          log_debug("profile_icon_url was passed a user without profileicon & email properties");
     ...
   }

  The warn is for AND but the check is for OR.

  The check should be for AND.

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


References