<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 18, 2013 at 10:21 PM, Kasun Chathuranga <span dir="ltr"><<a href="mailto:chathuranga.jayaneththi@gmail.com" target="_blank">chathuranga.jayaneththi@gmail.com</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"><div dir="ltr"><br><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Thu, Apr 18, 2013 at 10:09 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">Kasun Chathuranga a écrit :<br>
<div>> Hi all,<br>
><br>
> As indicated in the tracker I am working on fixing the bug #3879 [1] and<br>
> have submitted a pull request [2]<br>
><br>
> I noticed that in the latest version the interface does not allow changing<br>
> format specific options such as 'Columns separated with', 'Columns enclosed<br>
> with' etc in the import tab. However in the earlier versions these values<br>
> could be specified using the interface.<br>
><br>
> Is this change intentional or is it a regression?<br>
<br>
</div>A regression: see [3].<br>
<br>
[3] <a href="https://sourceforge.net/p/phpmyadmin/bugs/3883/" target="_blank">https://sourceforge.net/p/phpmyadmin/bugs/3883/</a><br>
<br>
<br clear="all"></blockquote></div><div>Thanks for the clarification. I will work on a fix. <br><span class=""></span><br clear="all"></div></div></div></div></blockquote></div>Hi,<br><br></div><div class="gmail_extra">I have fixed the regression and updated my pull request [2].<br>
<br></div><div class="gmail_extra">Most of the format specific options are common to both CSV and CSV with LOAD DATA. Hence, I have pulled the common code for generating these common options to an abstract super class to avoid code duplication.<br>
</div><div class="gmail_extra"><br></div><div class="gmail_extra">It would be great if you could review it.<br><br></div><div class="gmail_extra">-- <br><div dir="ltr"><div style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255)">
Regards<br></div><div style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255)">
Kasun Chathuranga<br><br>[2] <span class=""><font color="#888888"><a href="https://github.com/phpmyadmin/phpmyadmin/pull/267" style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:start;font-style:normal;font-weight:normal;line-height:normal;color:rgb(17,85,204);text-transform:none;font-size:13px;white-space:normal;font-family:arial,sans-serif;word-spacing:0px" target="_blank">https://github.com/phpmyadmin/phpmyadmin/pull/267</a></font></span><br>
</div></div>
</div></div>