mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #06519
[Bug 872619] A change has been merged
Reviewed: https://reviews.mahara.org/785
Committed: http://gitorious.org/mahara/mahara/commit/0043cc8322c59ebd37bf12e0b07372bcdc755f21
Submitter: Richard Mansfield (richardm@xxxxxxxxxx)
Branch: master
commit 0043cc8322c59ebd37bf12e0b07372bcdc755f21
Author: Hugh Davenport <hugh@xxxxxxxxxxxxxxx>
Date: Wed Oct 12 14:38:22 2011 +1300
Fix issue when param_... functions have default of null
Bug #872619
Regression from 79810c3d321b012df0b86a63459cc5de1cfd6e6f
Basically, a default parameter of null was given, then trim
was called, which made it into the string '' which then failed
the test is_null in the files it was used.
This reorders the functions so that the trim is called after
the test to see whether the default should be used. This means
that the default value is never trimmed, and can be null.
One exception to this, is the param_boolean which doesn't test
to see whether it is the default that is returned, so a check
is made first to see whether it is null.
Change-Id: I66d7253a8414dec7eb3ea67ebd363ea068a32012
Signed-off-by: Hugh Davenport <hugh@xxxxxxxxxxxxxxx>
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/872619
Title:
Users cannot access own journal unless multiplejournals enabled
Status in Mahara ePortfolio:
Triaged
Bug description:
a users journal is currently inaccessible to them until they choose
multiple journals. the relevant object appears to not be getting the
owner value populated.
master
linux
postgres
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/872619/+subscriptions
References