mahara-contributors team mailing list archive

[Robert Lyon <robertl@xxxxxxxxxxxxxxx>]

Public bug reported:

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.

** Affects: mahara
     Importance: Low
     Assignee: Robert Lyon (robertl-9)
         Status: In Progress

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