<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Wed, Feb 26, 2014 at 10:08 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Smita a écrit :<br>
<div><div class="h5">> Hi,<br>
><br>
> I was looking into the bug <a href="http://sourceforge.net/p/phpmyadmin/bugs/4040/" target="_blank">http://sourceforge.net/p/phpmyadmin/bugs/4040/</a> ,<br>
> I am able to reproduce it, I get the fatal error in apache error log saying<br>
> "<br>
>  PHP Fatal error:  Call to a member function getPath() on a non-object in<br>
> /var/www/phpmyadmin/libraries/Util.class.php on line 184"<br>
> Though there is no error shown on UI, at the user interface I get the<br>
> expected message "Configuration has been saved" .<br>
> Also the error comes only when active theme is "pmahomme". Because in the<br>
> code we are unsetting the SESSION['PMA_Theme_Manager'] and<br>
> $_SESSION['PMA_Theme'] if the current theme is not "original".<br>
><br>
> So I have a Question:<br>
> Currently when we reset the settings, active theme doesn't reset to default<br>
> one, which is "pmahomme", it remains same. Is it the correct behaviour or<br>
> should it reset the current theme to default one?<br>
><br>
> So if we don't need to reset the theme to default one, then just removing<br>
> the unset statement which is unsetting themes session variables should fix<br>
> the issue. Otherwise instead of unsetting,  setting the themes session<br>
> variables to default theme will fix the issue.<br>
><br>
> Please let me know if any concerns, so that I can proceed with fixing this<br>
> bug.<br>
><br>
> Thanks and Regards<br>
> Smita Kumari<br>
<br>
</div></div>Smita,<br>
I'm not sure that it will help, but when someone chooses a theme<br>
different than the default one, a persistent cookie named "pma_theme" is<br>
sent to the browser. Therefore, this user choice should be respected.<br>
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a> | <a href="http://phpmyadmin.net" target="_blank">http://phpmyadmin.net</a><br>
<br>
------------------------------------------------------------------------------<br>
Flow-based real-time traffic analytics software. Cisco certified tool.<br>
Monitor traffic, SLAs, QoS, Medianet, WAAS etc. with NetFlow Analyzer<br>
Customize your own dashboards, set traffic alerts and generate reports.<br>
Network behavioral analysis & security monitoring. All-in-one tool.<br>
<a href="http://pubads.g.doubleclick.net/gampad/clk?id=126839071&iu=/4140/ostg.clktrk" target="_blank">http://pubads.g.doubleclick.net/gampad/clk?id=126839071&iu=/4140/ostg.clktrk</a><br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br></blockquote><div> </div></div>So in that case when we reset the settings, we should not unset the SESSION['PMA_Theme_Manager'] and<br>
<div class="gmail_extra">$_SESSION['PMA_Theme'] variables If I get it right ? What do you think ?<br><br></div>Thanks<br>Smita Kumari</div></div>