<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 8, 2014 at 6:01 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-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Le 2014-07-08 07:52, Ashutosh Dhundhara a écrit :<br>
<div class="">> Hi,<br>
><br>
> I have got some ideas regarding RFE #908<br>
</div>> <<a href="http://sourceforge.net/p/phpmyadmin/feature-requests/908/" target="_blank">http://sourceforge.net/p/phpmyadmin/feature-requests/908/</a>> (Improvements for<br>
<div class="">> the table editor (index creation)).<br>
><br>
</div>> *_1. Making Column section on 'Add Index' dialog sort-able._*<br>
<div class="">> With this user will be able to change the order of columns in case of<br>
> composite keys.<br>
><br>
</div>> *_2. Displaying a dialog box when multiple columns for same type of<br>
> index are selected on 'Create table' page._*<br>
<div class="">> This dialog will have two options: 'Create new' and 'Create composite'.<br>
> - 'Create new' will allow user to create a new single column key. (Table<br>
> can have only one Primary key)<br>
> - 'Create composite' will allow user to add that column to a existing<br>
> key (to be created) and sort the order of columns.<br>
><br>
> Will this implement this feature request?<br>
<br>
</div>Hi Ashutosh,<br>
first, the feature request says "If I add two indexed fields to a<br>
table"; remember that this action can be done on initial table creation,<br>
and when adding columns on an existing table.<br></blockquote><div><br></div><div>Thanks for pointing this out. I missed that 'adding column on an existing table' part.</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>
About your suggestion #1, are you talking about the dialog that appears<br>
after choosing table Structure > Indexes > Create an index on x columns<br>
? If so, the user can already choose the columns in the correct order;<br>
so you want to help the user change his mind when creating the index?<br>
<br>
I am not against, but this would be more useful when _editing_ an index.<br>
Or it can be done in both situations.<br>
<br></blockquote><div> </div><div><br></div><div>Yeah, this can be done with the existing system. I failed to see that.</div><div><br></div><div><br></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">

About your suggestion #2, if the user chose PRIMARY in the selector for<br>
two columns, it makes no sense to show this additional dialog (unless<br>
you want him to confirm that the PRIMARY will be composite) but it makes<br>
sense to allow column sorting.<br></blockquote><div><br></div><div>Yeah Marc, I won't show this dialog for PRIMARY key.</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 he chose UNIQUE or INDEX, instead of "Create new" I would say "Create<br>
single-column indexes". The other message could be "Create a composite<br>
index" but do you suggest that this choice opens the already-existing<br>
Edit index dialog, which you would prefill with the chosen column names?<br>
<br>
As per the feature request, the default choice should be to create<br>
single-column indexes.<br></blockquote><div><br></div><div>I think you are getting it wrong. This part is for 'Create table' page and 'add column to existing table' page. Lets say I am creating a new table. I have added a column `col1` and selected UNIQUE index for it (before clicking 'Go'). I won't see any dialog yet. Now lets I have added another column `col2` and selected UNIQUE index for this as well. Now I will get a dialog asking for "Create single-column indexes" or "Create a composite index". If I will choose "Create a composite index", then I will get a list of all temporary UNIQUE indexes and I can merge that column into any of these indexes.</div>
<div>For PRIMARY index, we will need only the composite-index part.</div><div><br></div><div>I think this will also handle the issue highlighted by you in the next mail?</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>
As I told to Smita for her project, it's better to reuse existing<br>
interfaces, in this case the existing index editor. </blockquote></div><br>I believe that the existing index editor works for existing tables (ALTER TABLE statement) but in this case we need to modify the CREATE TABLE query. So no interaction required with the server yet. Still I will try to reuse it, if possible.</div>
<div class="gmail_extra"><br>-- <br><div dir="ltr"><div>Ashutosh Dhundhara</div></div>
</div></div>