<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Aug 6, 2014 at 2:46 AM, 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-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Le 2014-08-05 14:35, Ashutosh Dhundhara a écrit :<br>
<div class="">> Hi,<br>
><br>
> I was starting to work on RFE #1123<br>
</div>> <<a href="https://sourceforge.net/p/phpmyadmin/feature-requests/1123/" target="_blank">https://sourceforge.net/p/phpmyadmin/feature-requests/1123/</a>> (Zeroconf<br>
<div class="">> PMA tables support).<br>
><br>
> Is it something like enabling/disabling the PMA configuration storage<br>
> feature through UI with default table names?<br>
><br>
> Also, automatically create these tables when user has enabled this<br>
> feature through UI?<br>
<br>
</div>Hi Ashutosh,<br></blockquote><div><br></div><div>Thanks Marc and Isaac.</div><div><br></div><div>The RFE was a bit confusing but the insight provided by you both clarified everything.</div><div>However, I have a few queries and I am posting them in-line.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
I have read again the RFE and the comments in the ticket; then I read<br>
the details of your proposal on <a href="http://google-melange.com" target="_blank">google-melange.com</a>.<br>
<br>
I don't have all the answers but can give you some pointers; others can<br>
give ideas too or contradict mine :)<br>
<br>
* The solution has to work for a single-user installation and also for a<br>
multi-user installation<br></blockquote><div><br></div><div>How single-user and multi-user installation is going to affect this feature?</div><div>Is it something like if the user does not have access to the database defined in the</div>
<div>directive $cfg['Servers'][$i]['pmadb'], then create the pma tables in the database on which he/she</div><div>has access? </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
* Priority should go to use the values from config.inc.php (the control<br>
user, pmadb and all pmadb tables) if present. This is mainly for<br>
backward compatibility and also because I suspect it's the only way to<br>
support a multi-user installation<br>
<br>
* In case there is no such configuration, the logic could check the<br>
presence of the default database name and table names, as suggested in<br>
examples/create_tables.sql<br>
<br>
* I'm not sure on which panel is the best place to do this verification,<br>
but it could be at the same places where the warnings about the<br>
configuration storage are currently done (home page and each database ><br>
Operations page<br>
<br>
* If this default database name "phpmyadmin" does not exist, the logic<br>
could look for the default table names in the current database (this is<br>
in case the user has access to only one database)</blockquote><div><br></div><div>Lets say user sees the warning on homepage. Then how to decide the current db?</div><div>Is it something like creating these tables in the first database on which the user has CREATE access?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
* If the default db and/or default table names are not found, the logic<br>
could pop a dialog, offering to create these in the current db and<br>
asking for confirmation; I hope it's possible to directly read<br>
examples/create_tables.sql for this, in order to avoid discrepancies<br>
between the logic and changes in create_tables.sql<br>
<br>
* Something will have to be done with the logic that gives a warning<br>
about the configuration storage; it would probably lose its purpose<br></blockquote><div><br></div><div>Yes Marc. We will have to do something for this.</div><div>How about keeping this message/warning intact and when the user clicks on 'here' of "<span style="color:rgb(0,0,0);font-family:sans-serif;font-size:13px;background-color:rgb(255,192,203)">To find out why click </span><a href="http://127.0.0.1/pma/chk_rel.php?token=ee04c7e45192bb15a4151248028e905f" style="color:rgb(35,90,129);outline:none;font-family:sans-serif;font-size:13px">here</a><span style="color:rgb(0,0,0);font-family:sans-serif;font-size:13px;background-color:rgb(255,192,203)">.</span>", the user</div>
<div>will be directed to diagnostic page as it is currently done. And on that page we can provide a link with text something like "Fix it" and when user will click this link, the new logic will be executed?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
* Maybe we'll need a new directive in config.inc.php, so that the<br>
installer can decide that the zero conf is active or not (would be<br>
active by default)<br>
<br>
* I am not in favor of designing a UI to set up the configuration<br>
storage, as this already exists in /setup (with the goal of producing a<br>
config.inc.php)<br>
<br>
* I am not in favor of programmatically creating a .php file in the<br>
installation directory or doing any modification in config.inc.php<br>
<br></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div>Ashutosh Dhundhara</div></div>
</div></div>