Dear Oleg, reviewed the whole patch now.<br><br><br><div class="gmail_quote">On 15 November 2012 00:16, Oleg Kukharchuk <span dir="ltr"><<a href="mailto:oleg.kuh@gmail.com" target="_blank">oleg.kuh@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



 <div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif">
   <table style="border:1px #c9c399 solid" bgcolor="#f9f3c9" cellpadding="8" width="100%">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/107330/" target="_blank">http://git.reviewboard.kde.org/r/107330/</a>
     </td>
    </tr>
   </tbody></table>
   <br>


<table style="background-image:url('');background-repeat:repeat-x;border:1px black solid" bgcolor="#fefadf" cellpadding="8" cellspacing="0" width="100%">
 <tbody><tr>
  <td>

<div>Review request for Calligra, Jarosław Staniek, Adam Pigg, and Dimitrios Tanis.</div>
<div>By Oleg Kukharchuk.</div>







<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1>
 <table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
 <tbody><tr>
  <td>
   <pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">* moved to a KAssistantDialog
* added ability to import into an existant table</pre>
  </td>
 </tr>
</tbody></table>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
 <tbody><tr>
  <td>
   <pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">tested on local branch
and additional testing needed

(code was not cleaned up yet)</pre>
  </td>
 </tr>
</tbody></table>




<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">

 <li>kexi/doc/kexidb/kexidb.doxygen <span style="color:grey">(ebfa7741ac624c818a6ab88470be672095b188ee)</span></li>

 <li>kexi/main/kexinamewidget.h <span style="color:grey">(870c3668128798b25ab7d646393c5c6eb9086b9a)</span></li>

 <li>kexi/plugins/importexport/csv/CMakeLists.txt <span style="color:grey">(f8da4e24dfc19055d009ee845514e567ef843eff)</span></li>

 <li>kexi/plugins/importexport/csv/kexicsvimportdialog.h <span style="color:grey">(15be3504dfc54877bc45d81eb7e59a72f70c9c5a)</span></li>

 <li>kexi/plugins/importexport/csv/kexicsvimportdialog.cpp <span style="color:grey">(18fafd486c13cfae943f2af0078fca93b83c4315)</span></li>

 <li>kexi/widget/fields/KexiFieldListModel.cpp <span style="color:grey">(91ed791065a34d8a137dc84997506bdd0643b231)</span></li>

 <li>libs/db/connection.cpp <span style="color:grey">(d36def0c1b66440c79fa617bc185b47cf2d67d06)</span></li>

 <li>libs/db/drivers/sqlite/sqlitepreparedstatement.cpp <span style="color:grey">(ffb751ebb1c87c32db95f48797e4c358c7b17991)</span></li>

 <li>libs/db/object.cpp <span style="color:grey">(c6585a72626f93d18085c42e8ea19ae92619cfff)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/107330/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>




  </td>
 </tr>
</tbody></table>




  </div>
 </div>


</blockquote></div><br><br clear="all"><br>-- <br>regards / pozdrawiam, Jaroslaw Staniek<br>  Kexi & Calligra & KDE | <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a> | <a href="http://kde.org" target="_blank">http://kde.org</a><br>


  Qt Certified Specialist | <a href="http://qt-project.org" target="_blank">http://qt-project.org</a><br>  <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a><br>