e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
322 stars 214 forks source link

Login menu fix produces new bug #1059

Closed gary231178 closed 9 years ago

gary231178 commented 9 years ago

Hiya, sorry for lots of posts recently, but love e107 as a user for past 7 years, and just actively started switching to v2.

I just updated using master zip as i noticed recent changes (5 days old) 1 of which references to login menu cleanup. However i have a new bug since updating files showing LOGIN_MENU_L25: code in menu, which wasn't present prior to updating. screenshot_11

CaMer0n commented 9 years ago

hey @gary231178 - we appreciate the posts - keep 'em coming!. If you have used a modified version of bootstrap3 for your theme, you just need to do a search/replace on your files. (most likely theme_config.php)

LOGIN_MENU_L1 should be replaced with LAN_LOGINMENU_1 LOGIN_MENU_L25 should be replaced with LAN_LOGINMENU_25 etc. etc.

,

tgtje commented 9 years ago

Cameron was quicker, your files are probably just over 5 days old (around start of june) .

There L25 still exists in those. As described above> the newer files (5 days) are correct..(you always may update and have all changes...;-) (Note .. info.. : if an upGRade and running langauge packs, you will likely get in trouble....but is should display fine # old lang packs carry _Lnum.. new carries _num

gary231178 commented 9 years ago

@tgtje Nope i saw the change log and as of today the modified files were showing 13 hours old (latest commit 8a22d10325) so i re downloaded the master.zip and overwrote all files so if the master was updated 5 days ago, or even today i am using the latest. No language packs installed or used. Purely an English only site. @CaMer0n i'm using a premium theme by @mcpeace (AKA manatwork) so not sure if its a modded bootstrap3 or not, but ill search out those files and cross-examine

Edit:-can't find the files which need to be modified in the theme, so dropped Mcpeace a message to see if he can help.

Cheers all,

Gary.

gary231178 commented 9 years ago

Found it and fixed!! was in the following:- login_menu_template.php Replaced LOGIN_MENU_L25 should be replaced with LAN_LOGINMENU_25

Cheers all