<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/122197/">https://git.reviewboard.kde.org/r/122197/</a>
     </td>
    </tr>
   </table>
   <br />



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks!</p></pre>
 <br />









<p>- JarosÅ‚aw Staniek</p>


<br />
<p>On February 13th, 2015, 9:13 a.m. CET, Roman Shtemberko wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for Calligra and JarosÅ‚aw Staniek.</div>
<div>By Roman Shtemberko.</div>


<p style="color: grey;"><i>Updated Feb. 13, 2015, 9:13 a.m.</i></p>







<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=336815">336815</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix 336815 (path issue)
Also have fixed with this patch:
Check for an empty name.
Names are no more converted to the lower case (there is some conflicts with this behavior, imported tables can not be opened after this (only after restarting an app (!?)))
Check if name is not starting with a digit (conflicts with '_' at the begining as well).
Default name is added based on sheet selected.</p></pre>
  </td>
 </tr>
</table>



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

 <li>kexi/main/KexiMainWindow.cpp <span style="color: grey">(e82b49e)</span></li>

 <li>kexi/migration/AlterSchemaTableModel.h <span style="color: grey">(2090b35)</span></li>

 <li>kexi/migration/AlterSchemaTableModel.cpp <span style="color: grey">(86ef62e)</span></li>

 <li>kexi/migration/AlterSchemaWidget.h <span style="color: grey">(b29e7f9)</span></li>

 <li>kexi/migration/AlterSchemaWidget.cpp <span style="color: grey">(ea7fedd)</span></li>

 <li>kexi/migration/importtablewizard.h <span style="color: grey">(a0b4dca)</span></li>

 <li>kexi/migration/importtablewizard.cpp <span style="color: grey">(f3d02b9)</span></li>

 <li>kexi/plugins/migration/keximigrationpart.cpp <span style="color: grey">(03c674e)</span></li>

 <li>kexi/widget/KexiConnectionSelectorWidget.cpp <span style="color: grey">(48d3f7e)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/122197/diff/" style="margin-left: 3em;">View Diff</a></p>






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







  </div>
 </body>
</html>