@@ -11,9 +11,9 @@ if($_POST['formSubmitted'] == "true" || ((isset($_GET['id']) && is_numeric($_GET
if(isset($_GET['deleteAddon']) && $_GET['deleteAddon']) $deleteAddon = true;
if($deleteAddon){
if(deleteAddon($curseAddonID)){
- $_SESSION['message'] = "Deleted Addon.";
+ $_SESSION['message'] = "The Addon $addonName has been deleted.";
}else{
- $_SESSION['message'] = "Deletting Addon failed.";
+ $_SESSION['message'] = "Deleting Addon failed.";
}
}else{
if(updateAddon($curseAddonID)){
@@ -84,7 +84,7 @@ if($numRows != 0){
}
////
echo "<td align=\"center\"><a href=\"../cachedZips/".$row['addonName'].".zip\"><img src=\"".$baseURL."images/download.gif\" width=\"25\" height=\"25\" border=\"0\"></a></td>" .
- "<td align=\"center\"><a href=\"".$_SERVER['PHP_SELF']."?deleteAddon=true&id=".$row['id']."&sort=".$orderBy."&direction=".$orderDir."\"><img src=\"".$baseURL."images/delete.png\" width=\"25\" height=\"25\" border=\"0\"></a></td>" .
+ "<td align=\"center\"><a href=\"".$_SERVER['PHP_SELF']."?deleteAddon=true&id=".$row['curseAddonID']."&sort=".$orderBy."&direction=".$orderDir."\"><img src=\"".$baseURL."images/delete.png\" width=\"25\" height=\"25\" border=\"0\"></a></td>" .
"</tr>";
$i++;
}
admin/listAddons.php
@@ -8,6 +8,8 @@ TODO:
-Fix it to die gracefully if curse isn't providing the xml correctly
-Updated blah, but blah was missing
-Use the server's tmp directory
+-return stuff in arrays
+-Improve session messaging system
*/
class ParseXML{
function GetChildren($vals, &$i) {
@@ -205,7 +207,8 @@ function md5Addon($addonName){
$md5Location = $baseURL.'cachedZips/'.$addonName.'.dir/versions/'.$addonName.'.md5';
$md5Directory = $baseURL.'cachedZips/'.$addonName.'.dir/versions/';
if(!file_exists($md5Directory)) mkdir($md5Directory);
- //file_put_contents($md5Location, $md5Hash);
+ //die('Dir: '.$md5Directory . ' | Loc: ' . $md5Location . ' | Hash: ' . $md5Hash);
+ file_put_contents($md5Location, $md5Hash);
return;
}
@@ -269,14 +272,15 @@ function deleteAddon($curseAddonID){
if(!result) return false;
while($row = mysql_fetch_array($result, MYSQL_ASSOC)){
$addonName = trim($row['addonName']);
+
}
- if(file_exists("$baseURLcachedZips/$addonName.dir")){
+ if(file_exists($baseURL.'cachedZips/'.$addonName.'.dir')){
shell_exec('rm -rf "'.$baseURL.'cachedZips/'.$addonName.'.dir"');
}
- if(file_exists("$baseURLcachedZips/$addonName.zip")){
+ if(file_exists($baseURL.'cachedZips/'.$addonName.'.zip')){
shell_exec('rm -rf "'.$baseURL.'cachedZips/'.$addonName.'.zip"');
}
- $query = "DELETE FROM amz_addonsList WHERE id=".$curseAddonID;
+ $query = "DELETE FROM amz_addonsList WHERE curseAddonID=".$curseAddonID;
$deleteResult = mysql_query($query);
if(!$deleteResult) return false;
return true;
functions.php
90147e69cdbfee98d3ba67b2642777cc25efc06a
Eric Boehs
ericboehs@gmail.com
http://github.com/ericboehs/wow-php-add-on-manager/commit/f803158fdcc57eebd090f0d36009dec01c6b688e
f803158fdcc57eebd090f0d36009dec01c6b688e
2008-10-28T09:27:54-07:00
2008-10-28T09:27:54-07:00
Made the deleteAddon() syntax look a little better and fixed listAddons.php to send the curseAddonID instead of the id in the sql table when deleting an addon (which resulted in the addon not getting deleted properly) - Also updated listAddon.php to return the name of the Addon deleted
d64edbf086b13587493475cb1add7ea91889daa1
Eric Boehs
ericboehs@gmail.com