Just adding the variable should be no problem, but since there is no patch file, it is difficult to review - however, if you avoid syntax errors, I have trouble seeing where you could go wrong :-)
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=1095348
Originator: NO
Agreed, we need to also assign a value to 'xoops_sitename'.
Logged In: YES
user_id=841117
Originator: NO
Feedback as requested at http://xoops.wiki.sourceforge.net/Bug+Fix
Just adding the variable should be no problem, but since there is no patch file, it is difficult to review - however, if you avoid syntax errors, I have trouble seeing where you could go wrong :-)
patch for bug 1599929
Logged In: YES
user_id=1095348
Originator: NO
Added the patch.
Thanks again mith.
File Added: bug-1599929.zip
Logged In: YES
user_id=1384873
Originator: NO
I will try to implement in 2.0.17 core...