<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=iso-8859-1"><meta name=Generator content="Microsoft Word 12 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body lang=EN-GB link=blue vlink=purple><div class=WordSection1><p class=MsoNormal>Hi all<o:p></o:p></p><p class=MsoNormal>I know we've discussed this sort of thing before but doesn't seem easy to locate and track back through specific steps but anyway case in point…<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>You may have noticed I responded to the question and thread on the community site re moodle integration and helped Jayne get things working including I think the last issue about logout not working. She hasn't replied and confirmed yet but it seems very likely that the problem with logout is the unecessary onbeforeunload="javascript:logout()" in index.php which I'm pretty sure we've worked through and discussed here before as well as Julian replying to a previous mention of this on the community forums.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>I don't personally find it easy to navigate the commits to spot changes in each branch etc and so not sure whether the rogue code has ever been removed from 2.1 or removed and somehow crept back. Also I seem to recall that Julian orginally added that code to try to resolve a firefox issue but not sure what the final result of that was except that leaving it in causes more problems than it fixes. I did an install of 2.1 earlier this week and with hindsight noticed the issue then too but because of an issue with the latest 2.1 and ldap (another story see below) reverted to an earlier 2.1 revision which must have had that code removed. So a few questions:<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>1. How best to remove the rougue onbeforeunload="javascript:logout()" from 2.1? I could commit the change direct to 2.1 but not sure that's the best way e.g. it's already removed from develop but obviously develop isn't ready for production use.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>2. Are there other differences like this e.g. later changes that aren't related to the new editor but only in later branches and not in 2.1?<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>3. I did an install of the latest 2.1 revision earlier this week and had problems when it came to ldap. Login was working but project folders weren't being created with the username included so breaking. I ended up replace the install with an older revision which worked immediately. There wasn't time to unpick the differences but just wondering if anyone recognises that issue with LDAP and which commit is relevant?<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Cheers<o:p></o:p></p><p class=MsoNormal>Ron<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>BTW apologies if two copies of this message reach the list - the first attempt bounced :-(<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><o:p> </o:p></p></div></body></html>