Some other points :
- I see a "navbar_list" variable, which is never used anywhere else in the code. Is it normal ? Is is just we will need it in the future ?
- Why did you write a "log in" tab while this code will never be executed when the user is logged out (except if there are more state than logged in and out for a user ?)
- Why do you remove the menu bar when the user is not in the dotlrn anymore ? My understanding is that you want to make the difference between OACS (system admin only) and .LRN (software admin and use), am I right ? if so, I think it is a bad idea. The software is already far too complicated, and say to people "you are using two softwares, and you must be aware of it" is not a way to do. As I already said (even if it may be complicated for developpers (you), an effort should be made IMHO to simplify the administration. Of course, this is just a way to start a debate 😊
Jeremy
