Discussion Forums  >  Self Hosted Control Panels

Replies: 4    Views: 72

ehwright
Aspiring developer
Profile
Posts: 57
Reg: Feb 21, 2012
Perth
2,370
02/16/14 10:10 PM (11 years ago)

BT_menu.php error

Not sure if it's already been fixed (I'm using a self hosted panel), but there is an error in BT_menu.php. On line 265 it reads $menuItemJsonVars .= ""titleText":"" . $addMenuItemText . """; It should have a comma like this $menuItemJsonVars .= ""titleText":"" . $addMenuItemText . "","; It causes the json data to be invalid by missing the comma between the title text and loadscreenid fields.
 
SmugWimp
Smugger than thou...
Profile
Posts: 6316
Reg: Nov 07, 2012
Tamuning, GU
81,410
like
02/16/14 10:47 PM (11 years ago)
Interesting. Nice catch! Cheers! -- Smug
 
chris1
Code is Art
Profile
Posts: 3862
Reg: Aug 10, 2012
Austin, TX
50,120
like
02/16/14 11:55 PM (11 years ago)
Thanks - I've added that to our bug list for David to take a look at.
 
SmugWimp
Smugger than thou...
Profile
Posts: 6316
Reg: Nov 07, 2012
Tamuning, GU
81,410
like
02/17/14 03:01 AM (11 years ago)
How odd. I just went to edit my bt_menu.php file on that line (265) and mine looks like this: $menuItemJsonVars .= "\"titleText\":\"" . $addMenuItemText . "\""; which I changed to: $menuItemJsonVars .= "\"titleText\":\"" . $addMenuItemText . "\", "; //<-- added comma But I'm curious. Yours did or did not have a backslash? did you type or paste that? Cheers! -- Smug
 
ehwright
Aspiring developer
Profile
Posts: 57
Reg: Feb 21, 2012
Perth
2,370
like
02/17/14 03:13 AM (11 years ago)
I pasted it. Weird, now I'm going to go back and have another look
 

Login + Screen Name Required to Post

pointerLogin to participate so you can start earning points. Once you're logged in (and have a screen name entered in your profile), you can subscribe to topics, follow users, and start learning how to make apps like the pros.