Feeling Lucky
?
[#1834] - Notice in JFactory::getUser because it could return null and try to access id of null for no avail
- Closed
- 13 Oct 2013
- Medium
- Build: master
- # 1834
(for version 2.5 at least)
There is a notice (NullPointer!) because of not testing return value of JSession.get('user').
The faulted lines are :
$current = self::getSession()->get('user');
if ($current->id != $id) {…
as by construction JSession.get($name,$default = nul…)
could return null either because the non constructed error (still a shameful TODO here !) or if no such entry in the current session, then the following access to
null->id raise a Notice.
This should be much more defensive.
zero-24
- | Labels |
Added:
?
Removed: ? ? |
||
| Build | ⇒ | staging | |
Thanks for reporting this. At this time we are only using github as the place to submit code fixes so I am closing the report here. The actual reporting of issues and testing fixes is still taking place on Joomlacode.
As it has been some time since you opened this issue can you please confirm that it is still valid with the current Master or Joomla 3.2 beta.
If it is still valid please can you open an item on the Joomlacode tracker in the appropriate area.
CMS Bug Reports: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8103
CMS Feature Requests: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8549