appdb/admin editBundle.php
WineHQ
wineowner at wine.codeweavers.com
Thu Jun 22 22:42:08 CDT 2006
ChangeSet ID: 25980
CVSROOT: /opt/cvs-commit
Module name: appdb
Changes by: wineowner at winehq.org 2006/06/22 22:42:08
Modified files:
admin : editBundle.php
Log message:
Chris Morgan <cmorgan at alum.wpi.edu>
Fix editing bundles, they've been broken since appdb was imported into cvs and we haven't noticed, ha
Patch: http://cvs.winehq.org/patch.py?id=25980
Old revision New revision Changes Path
1.7 1.8 +51 -48 appdb/admin/editBundle.php
Index: appdb/admin/editBundle.php
diff -u -p appdb/admin/editBundle.php:1.7 appdb/admin/editBundle.php:1.8
--- appdb/admin/editBundle.php:1.7 23 Jun 2006 3:42: 8 -0000
+++ appdb/admin/editBundle.php 23 Jun 2006 3:42: 8 -0000
@@ -21,73 +21,76 @@ function build_app_list()
echo "</select>\n";
}
-
-if($cmd)
+if($_REQUEST['cmd'])
{
- if($cmd == "delete")
+ if($_REQUEST['cmd'] == "delete")
{
- $hResult = query_appdb("DELETE FROM appBundle WHERE appId = $appId AND bundleId = $bundleId");
+ $hResult = query_appdb("DELETE FROM appBundle WHERE appId =".$_REQUEST['appId']." AND bundleId =".$_REQUEST['bundleId']);
if($hResult)
addmsg("App deleted from bundle", "green");
+ else
+ addmsg("Failed to delete app from bundle!", "red");
}
- if($cmd == "add")
+ if($_REQUEST['cmd'] == "add")
{
- $hResult = query_appdb("INSERT INTO appBundle VALUES ($bundleId, $appId)");
+ $hResult = query_appdb("INSERT INTO appBundle VALUES (".$_REQUEST['bundleId'].", ".$_REQUEST['appId'].")");
if($hResult)
- addmsg("App $appId added to Bundle $bundleId", "green");
+ addmsg("App $appId added to Bundle".$_REQUEST['bundleId'], "green");
}
- redirectref();
- exit;
}
-else
-{
- apidb_header("Edit Application Bundle");
- $hResult = query_appdb("SELECT bundleId, appBundle.appId, appName FROM appBundle, appFamily ".
- "WHERE bundleId = $bundleId AND appFamily.appId = appBundle.appId");
- if($hResult && mysql_num_rows($hResult))
- {
- echo html_frame_start("Apps in this Bundle","300",'',0);
- echo "<table width='100%' border=0 cellpadding=3 cellspacing=0>\n\n";
-
- echo "<tr class=color4>\n";
- echo " <td><font color=white> Application Name </font></td>\n";
- echo " <td><font color=white> Delete </font></td>\n";
- echo "</tr>\n\n";
+apidb_header("Edit Application Bundle");
+
+$hResult = query_appdb("SELECT bundleId, appBundle.appId, appName FROM appBundle, appFamily ".
+ "WHERE bundleId = ".$_REQUEST['bundleId']." AND appFamily.appId = appBundle.appId");
+
+echo html_frame_start("Apps in this Bundle","300",'',0);
+echo "<table width='100%' border=0 cellpadding=3 cellspacing=0>\n\n";
- $c = 1;
- while($oRow = mysql_fetch_object($hResult))
- {
- //set row color
- if ($c % 2 == 1) { $bgcolor = 'color0'; } else { $bgcolor = 'color1'; }
+echo "<tr class=color4>\n";
+echo " <td><font color=white> Application Name </font></td>\n";
+echo " <td><font color=white> Delete </font></td>\n";
+echo "</tr>\n\n";
+
+if($hResult && mysql_num_rows($hResult))
+{
+ $c = 1;
+ while($oRow = mysql_fetch_object($hResult))
+ {
+ //set row color
+ if ($c % 2 == 1) { $bgcolor = 'color0'; } else { $bgcolor = 'color1'; }
- $delete_link = "[<a href='editBundle.php?cmd=delete&bundleId=$bundleId&appId=$oRow->appId'>delete</a>]";
+ $delete_link = "[<a href='editBundle.php?cmd=delete&bundleId=".$_REQUEST['bundleId']."&appId=$oRow->appId'>delete</a>]";
- echo "<tr class=$bgcolor>\n";
- echo " <td>$oRow->appName </td>\n";
- echo " <td>$delete_link </td>\n";
- echo "</tr>\n\n";
+ echo "<tr class=$bgcolor>\n";
+ echo " <td>$oRow->appName </td>\n";
+ echo " <td>$delete_link </td>\n";
+ echo "</tr>\n\n";
- $c++;
- }
-
- echo "</table>\n\n";
- echo html_frame_end();
+ $c++;
}
+} else if($hResult && !mysql_num_rows($hResult))
+{
+ /* indicate to the user that there are no apps in this bundle at the moment */
+ echo "<tr>\n";
+ echo " <td colspan=2 align=center><b>No applications in this bundle</b></td>\n";
+ echo "</tr>\n";
+}
- echo "<form method=post action=editBundle.php>\n";
+echo "</table>\n\n";
+echo html_frame_end();
- echo html_frame_start("Application List (double click to add)","",'',2);
- build_app_list();
- echo html_frame_end();
+echo "<form method=post action=editBundle.php>\n";
+
+echo html_frame_start("Application List (double click to add)","",'',2);
+build_app_list();
+echo html_frame_end();
- echo "<input type=hidden name=bundleId value=$bundleId>\n";
- echo "<input type=hidden name=cmd value=add>\n";
- echo "</form>\n";
+echo "<input type=\"hidden\" name=\"bundleId\" value=\"".$_REQUEST['bundleId']."\">\n";
+echo "<input type=\"hidden\" name=\"cmd\" value=\"add\">\n";
+echo "</form>\n";
- apidb_footer();
-}
-
+apidb_footer();
?>
More information about the wine-cvs
mailing list