FUDforum
Fast Uncompromising Discussions. FUDforum will get your users talking.

Home » FUDforum Development » Bug Reports » Some small issues (XHTML header, undefined variable, check of confirmation password)
Show: Today's Messages :: Polls :: Message Navigator
Return to the default flat view Create a new topic Submit Reply
Re: Some small issues [message #162156 is a reply to message #162151] Sat, 27 March 2010 08:33 Go to previous messageGo to previous message
naudefj is currently offline  naudefj   South Africa
Messages: 3771
Registered: December 2004
Karma:
Senior Member
Administrator
Core Developer
JanRei wrote:
The language referenced in the XHTML header is not correct in my installation. I have chosen to install the forum in German, but directly after install I get this: <html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" lang="en" xml:lang="en">
And after rebuilding of the theme it's empty: <html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" lang="" xml:lang="">

I do not understand the issue completely yet, but it seems related to use of pspell_lang for this. My PHP installation does not have pspell support enabled. Maybe we could use the language code instead?


In v3.0.0 pspell_lang was the only language code we had. However, as you've indicated, we can use lang in v3.0.1. Please change it in compiler.inc and commit (look for case 'LANG':).

JanRei wrote:
I get a notice that the variable $login in include\theme\default\adm_acc.inc is not defined. The login name would be available in $GLOBALS['r']->login.


Well spotted, you can fix it in adm_acc.inc.t by adding a "$login = ...".

JanRei wrote:
The check of the confirmation password does not work correctly for me. As far as I know $('#foobar') looks for the element with id "foobar", but the input element has no id at the moment. So it should work if we add a corresponding id. Or alternatively the JavaScript can be modified like this:
-	if ($(password2).attr("value") != $('#'+ password1).attr('value')) {
+	if ($(password2).attr("value") != $('[name='+password1+']').attr('value')) {



You are welcome to add ID's (if I missed any). The JQuery code is fine as-is.

Best regards.

Frank
[Message index]
 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: Messages are garbled after mporting from backup
Next Topic: Minimize message icon appears twice?
Goto Forum:
  

-=] Back to Top [=-
[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Sun Nov 24 21:23:30 GMT 2024

Total time taken to generate the page: 0.03855 seconds