user/view.php PHP Fatal error in non-auth access

Here a fix:

https://gitorious.org/~juancs/mahara/juancs-mahara/commit/adbc4d3e64e9c44ca54b7eaa1bc7d89ea597190a

Hello Juan, thanks for your patch can you please push it to gerrit for revision, see https://wiki.mahara.org/index.php/Developer_Area/Contributing_Code for details. Thanks!

Ops... sorry ;) Done ;)

Hello Juan, I can't reproduce it for some reason. $view object always exist, does not matter if you are authenticated or not (lines 58-65). And if you try accessing non-existing user it will throw the exception earlier.

I've realised that the error isn't raised with all users, just those without a profile view. You can try with any of these:

   select id from usr where id not in ( select distinct owner from view where type='profile' );

Take one id and access /user/view.php?id={one of the preovious id}

Maybe is not a general problem... just a problem with the way we are creating our users... mmmm... is it possible to have a user created without a profile view?

Look at get_view_by_type() method. If profile view doesn't exists and you're not logged in, null is returned. That's what I'm getting here :/

That is true, attempt to access user that has been created but does not have profile will cause fatal error.

Regarding your question, yes, it is possible to have the user without profile. The profile view is created on the first user login, see auth/user.php, load_views() call from authenticate().

https://reviews.mahara.org/#change,828

Also fixed on master at https://bugs.launchpad.net/mahara/+bug/837801

Reviewed: https://reviews.mahara.org/828
Committed: http://gitorious.org/mahara/mahara/commit/edffa944101f1e8918f0aa7a4e08aca2c07742e4
Submitter: Richard Mansfield (<email address hidden>)
Branch: 1.4_STABLE

commit edffa944101f1e8918f0aa7a4e08aca2c07742e4
Author: Juan Segarra Montesinos <email address hidden>
Date: Thu Nov 3 09:45:04 2011 +0100

    Fix PHP Fatal Error in user/view.php

    Reported in Bug #885588

    Change-Id: I10e2277e3b0d0e83ae2348ea3212c711e42ca353
    Signed-off-by: Juan Segarra Montesinos <email address hidden>

Thanks Juan again, well-spotted issue ;)