Bug when merging applications
Tony Lambregts
tony.lambregts at gmail.com
Fri Nov 11 11:23:40 CST 2005
Jonathan Ernst wrote:
>Hi,
>
>Try to merge the newly submitted Fritz with the old one and you'll get
>an "application not found" error.
>
>
>
Fix for this
Change Log: Allow the merging of versions into exisiting applications again.
Files changed admin/adminAppQueue.php
-------------- next part --------------
Index: admin/adminAppQueue.php
===================================================================
RCS file: /home/wine/appdb/admin/adminAppQueue.php,v
retrieving revision 1.48
diff -u -r1.48 adminAppQueue.php
--- admin/adminAppQueue.php 30 Oct 2005 22:27:14 -0000 1.48
+++ admin/adminAppQueue.php 11 Nov 2005 17:22:58 -0000
@@ -46,7 +46,7 @@
//display row
echo "<tr class=$bgcolor>\n";
/* map the merging of the current app to the app we are displaying in the table */
- echo " <td>".html_ahref($ob->appName,"adminAppQueue.php?sub=duplicate&appId=".$currentAppId."&appIdMergeTo=".$ob->appId)."</td>\n";
+ echo " <td>".html_ahref($ob->appName,"adminAppQueue.php?sub=duplicate&apptype=application&appId=".$currentAppId."&appIdMergeTo=".$ob->appId)."</td>\n";
echo " <td>$y->versions versions </td>\n";
echo "</tr>\n\n";
@@ -172,9 +172,10 @@
/* move this version submission under the existing app */
$oVersion->iAppId = $_REQUEST['appIdMergeTo'];
$oVersion->update();
-
+
/* delete the appId that is the duplicate */
- $oApp->delete();
+ $oAppDelete = new Application($oApp->iAppId);
+ $oAppDelete->delete();
}
/* redirect back to the main page */
More information about the wine-patches
mailing list