This is the tracker for the Core of the Nucleus CMS-project (Nucleus CMS Website)
FS#91 — Initialize member variables in MEMBER class correctly
Attached to Project— Nucleus Core
Opened by Wouter Demuynck (karma) - Friday, 19 August 2005, 06:49PM
Opened by Wouter Demuynck (karma) - Friday, 19 August 2005, 06:49PM
| Improvement | |
| Functionality | |
| CLOSED | |
| No-one | |
| All |
| Low | |
| Normal | |
| CVS | |
| 3.22 | |
| Undecided | |
 |
Mail from andrew:
"The other bug is that some values in the MEMBER class aren't
initialized, and probably should be.
Those valuese are $loggedin (should be inited to 0), $id (should this
be inited to -1?), $admin (should be inited to 0), and $canlogin
(should be inited to 0). These 4 values can all be initialized when
they are declared (or I think that's the case). The other value that
should probably be initialized is $language, and given the value of
$CONF['Language'], but this has to be done in the constructor, as I
don't think you can initialize using globals in the variable
declaration."
"The other bug is that some values in the MEMBER class aren't
initialized, and probably should be.
Those valuese are $loggedin (should be inited to 0), $id (should this
be inited to -1?), $admin (should be inited to 0), and $canlogin
(should be inited to 0). These 4 values can all be initialized when
they are declared (or I think that's the case). The other value that
should probably be initialized is $language, and given the value of
$CONF['Language'], but this has to be done in the constructor, as I
don't think you can initialize using globals in the variable
declaration."
This task depends upon
This task blocks these from closing
Date: Sunday, 21 August 2005, 08:14PM
Reason for closing: Implemented
Additional comments about closing: Main branch now also has the fix
Nucleus-3-2 branch has this. Still needs to be merged with main branch
MEMBER.php