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


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for ownCloud.</div>
<div>By Kamil DomaƄski.</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;">This patch separates custom app configuration records (oc_appconfig) from core app settings (in newly created oc_apps table). The new table contains app id, version string and activation status (boolean) in a single record for each application.

Removal function now removes both core and custom records.
New getApps() now returns apps from core config.
Old getApps() renamed to getAppsUsingConfig().
Installation status now checked with OC_APPCONFIG::isInstalled($app).
Installer now uses new OC_APPCONFIG::installAPP(...) function.
Installer now checks for both existing folder and config record before installing.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">No problems detected.</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>db_structure.xml <span style="color: grey">(599352c)</span></li>

 <li>lib/app.php <span style="color: grey">(e373331)</span></li>

 <li>lib/appconfig.php <span style="color: grey">(4fef754)</span></li>

 <li>lib/installer.php <span style="color: grey">(7ab07bf)</span></li>

</ul>

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




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




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