<div dir="ltr">On Mon, Sep 22, 2014 at 5:45 PM, Smita <span dir="ltr"><<a href="mailto:kumarismita62@gmail.com" target="_blank">kumarismita62@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">On Mon, Sep 22, 2014 at 5:37 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><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>
<span>> Hi<br>
><br>
> This is in reference of bug[0] in current master. While I was debugging the<br>
> code to fix the issue, I found that we are trying to force the column to be<br>
> primary key if auto_increment has been checked.<br>
> there is this comment in Table.class.php:<br>
> // Force an auto_increment field to be part of the primary key<br>
>             // even if user did not tick the PK box;<br>
><br>
> anyone have an idea why are we doing so ?<br>
<br>
</span>Hi Smita,<br>
<br>
This is a misinterpretation of the doc. In [1] we see that the column<br>
must be indexed, not necessarily part of the primary key.<br>
<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I have the impression that traditionally, people use the auto_increment<br>
mostly on the primary key but it's not mandatory.<br>
<br></blockquote></span><div>yeah, So I think we should not force the auto_increment to be primary key, right? This will also fix the bug[0].<br></div><span class=""><div><br></div></span></div></div></div></blockquote><div>My mistake, I also misunderstood. We are supposed to force some index.<br>So thing is We can't create a table unless auto_increment column is part of any index. So what do we do to meet this constraint on Create Table interface ?<br></div><div>I suggest, what if when user checks the auto_increment we show a dialog saying it must be indexed, click to add an Index ?<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
><br>
> [0] <a href="https://sourceforge.net/p/phpmyadmin/bugs/4540/" target="_blank">https://sourceforge.net/p/phpmyadmin/bugs/4540/</a><br>
><br>
<br>
[1] <a href="http://dev.mysql.com/doc/refman/5.5/en/create-table.html" target="_blank">http://dev.mysql.com/doc/refman/5.5/en/create-table.html</a><br>
<br>
<br>
--<br>
Marc Delisle (phpMyAdmin)<br>
<br>
------------------------------------------------------------------------------<br>
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer<br>
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports<br>
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper<br>
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer<br>
<a href="http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk" target="_blank">http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk</a><br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net" target="_blank">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></span></div><br></div></div>
</blockquote></div><br></div></div>