From: Rafael C. <raf...@gm...> - 2013-12-07 17:45:16
|
Hi, After the latest changes in LanguageSetup.php, I think the line 96 is no longer necessary in UserSettings.php. In my system works fine without this line. Comments? Regards, Rafael. |
From: Phil D. <ph...@lo...> - 2013-12-07 19:32:11
|
Line 96 putenv('LANGUAGE=' . $_SESSION['Language']); I think this was to do with different OS windows and linux use different variable names. There was fair bit of trial and error when we first set this up - would be reluctant to change without someone with a selection of OS's trying and proving it was ok to remove it. Phil Phil Daintree Logic Works Ltd - +64 (0)275 567890 http://www.logicworks.co.nz On 08/12/13 06:45, Rafael Chacón wrote: > Hi, > > After the latest changes in LanguageSetup.php, I think the line 96 is > no longer necessary in UserSettings.php. In my system works fine > without this line. > > Comments? > > Regards, Rafael. > > > ------------------------------------------------------------------------------ > Sponsored by Intel(R) XDK > Develop, test and display web and hybrid apps with a single code base. > Download it for free now! > http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk > > > _______________________________________________ > Web-erp-developers mailing list > Web...@li... > https://lists.sourceforge.net/lists/listinfo/web-erp-developers |
From: Rafael C. <raf...@gm...> - 2013-12-07 21:53:42
|
Hi Tim, That was the origin of my confusion. At the begging, I marked this line (UserSettings.php line 96) with "/*" and "*/", and tested. It works. When I was documenting that, I realized that it was "LanguageSetup.php", not "LanguagesArray.php". As I understand, this script is to (a) setup gettext and (b) load LanguagesArray.php... But LanguagesArray.php do not need to be re-loaded, and (I think) gettext must be setup once. Unfortunately, we still do not understand very well the operation of LanguageSetup.php and UserSettings.php, so I prefer someone more experienced to review it. Best regards, Rafael. > > > 2013/12/7 Tim Schofield <tim...@gm...> > >> Hi, >> >> includes/LanguageSetup.php is the script that actually sets up the >> languages for gettext to use in the translations, which is why it >> needs to be included when the language gets changed. >> >> I think you may be mixing it up with includes/LanguagesArray.php? >> >> Thanks >> Tim >> >> On 7 December 2013 20:08, Rafael Chacón <raf...@gm...> >> wrote: >> > Hi, >> > >> > I think that "include ('includes/LanguageSetup.php');" was required when >> > languages names were in different languages, but now they are in their >> local >> > language. Therefore, I think it is not necessary to update along with >> the >> > rest of the session variables to reflect user changes on-the-fly. >> > >> > My tests were successful, but I am not completely sure. I do not want >> that >> > delete this line causes an error in the future. This is the reason why >> I ask >> > this question. >> > >> > Best regards, Rafael. >> > >> > >> > 2013/12/7 Tim Schofield <tim...@gm...> >> >> >> >> Phil, this is the line that Rafael is referring to. >> >> >> >> include ('includes/LanguageSetup.php'); // After last changes in >> >> LanguageSetup.php, is it required to update? >> >> >> >> I think its there to update the language when the user changes rather >> >> than waiting for them to log out and back in. >> >> >> >> Tim >> >> >> >> On 7 December 2013 19:31, Phil Daintree <ph...@lo...> wrote: >> >> > Line 96 >> >> > >> >> > putenv('LANGUAGE=' . $_SESSION['Language']); >> >> > >> >> > I think this was to do with different OS windows and linux use >> different >> >> > variable names. There was fair bit of trial and error when we first >> set >> >> > this up - would be reluctant to change without someone with a >> selection >> >> > of >> >> > OS's trying and proving it was ok to remove it. >> >> > >> >> > Phil >> >> > >> >> > Phil Daintree >> >> > Logic Works Ltd - +64 (0)275 567890 >> >> > http://www.logicworks.co.nz >> >> > >> >> > On 08/12/13 06:45, Rafael Chacón wrote: >> >> > >> >> > Hi, >> >> > >> >> > After the latest changes in LanguageSetup.php, I think the line 96 >> is no >> >> > longer necessary in UserSettings.php. In my system works fine without >> >> > this >> >> > line. >> >> > >> >> > Comments? >> >> > >> >> > Regards, Rafael. >> >> > >> >> > >> >> > >> >> > >> ------------------------------------------------------------------------------ >> >> > Sponsored by Intel(R) XDK >> >> > Develop, test and display web and hybrid apps with a single code >> base. >> >> > Download it for free now! >> >> > >> >> > >> http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk >> >> > >> >> > >> >> > >> >> > _______________________________________________ >> >> > Web-erp-developers mailing list >> >> > Web...@li... >> >> > https://lists.sourceforge.net/lists/listinfo/web-erp-developers >> >> > >> >> > >> >> > >> >> > >> >> > >> ------------------------------------------------------------------------------ >> >> > Sponsored by Intel(R) XDK >> >> > Develop, test and display web and hybrid apps with a single code >> base. >> >> > Download it for free now! >> >> > >> >> > >> http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk >> >> > _______________________________________________ >> >> > Web-erp-developers mailing list >> >> > Web...@li... >> >> > https://lists.sourceforge.net/lists/listinfo/web-erp-developers >> >> > >> >> >> >> >> >> >> >> -- >> >> Course View Towers, >> >> Plot 21 Yusuf Lule Road, >> >> Kampala >> >> T +256 (0) 312 314 418 >> >> M +256 (0) 752 963 325 >> >> www.weberpafrica.com >> >> Twitter: @TimSchofield2 >> >> Blog: http://weberpafrica.blogspot.co.uk/ >> > >> > >> >> >> >> -- >> Course View Towers, >> Plot 21 Yusuf Lule Road, >> Kampala >> T +256 (0) 312 314 418 >> M +256 (0) 752 963 325 >> www.weberpafrica.com >> Twitter: @TimSchofield2 >> Blog: http://weberpafrica.blogspot.co.uk/ >> > > |