diff U3 /simplecdn/simplecdn.module /drupal/sites/all/modules/simplecdn/simplecdn.module
--- /simplecdn/simplecdn.module	Sat Feb 21 14:18:30 2009
+++ /drupal/sites/all/modules/simplecdn/simplecdn.module	Wed Jul 21 17:24:56 2010
@@ -26,17 +26,20 @@
  */
 function simplecdn_menu() {
   $items = array();
+
   $items['admin/settings/simplecdn'] = array(
     'title' => 'Simple CDN',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('simplecdn_admin'),
     'access arguments' => array('administer simplecdn')
   );
+
   $items['admin/settings/simplecdn/elements'] = array(
     'title' => 'Elements',
     'type' => MENU_DEFAULT_LOCAL_TASK,
     'weight' => -10,
   );
+
   $items['admin/settings/simplecdn/mirrors'] = array(
     'title' => 'Mirrors',
     'page callback' => 'simplecdn_mirror_list',
@@ -44,6 +47,7 @@
     'access arguments' => array('administer simplecdn'),
     'type' => MENU_LOCAL_TASK,
   );
+
   $items['admin/settings/simplecdn/add-new-mirror'] = array(
     'title' => 'Add new mirror',
     'page callback' => 'drupal_get_form',
@@ -52,6 +56,7 @@
     'type' => MENU_LOCAL_TASK,
     'weight' => 10
   );
+
   $items['admin/settings/simplecdn/mirror/%'] = array(
     'title callback' => 'simplecdn_title_callback',
     'title arguments' => array('Edit mirror: !name', 4),
@@ -60,6 +65,7 @@
     'access arguments' => array('administer simplecdn'),
     'type' => MENU_CALLBACK,
   );
+
   $items['admin/settings/simplecdn/mirror/%/delete'] = array(
     'title callback' => 'simplecdn_title_callback',
     'title arguments' => array('Delete mirror: !name', 4),
@@ -68,6 +74,7 @@
     'access arguments' => array('administer simplecdn'),
     'type' => MENU_CALLBACK,
   );
+
   return $items;
 }
 
@@ -85,14 +92,14 @@
  * Return array of CDN mirrors.
  */
 function simplecdn_mirrors($mid = NULL) {
-  //If cache isn't set, set it.
-  if(cache_get('simplecdn_mirrors') == '') {
+  // If cache isn't set, set it.
+  if (cache_get('simplecdn_mirrors') == '') {
     simplecdn_rebuild_cache();
   }
   
   $mirrors = cache_get('simplecdn_mirrors');
   
-  if($mid != NULL && isset($mirrors->data[$mid])) {
+  if ($mid != NULL && isset($mirrors->data[$mid])) {
     return $mirrors->data[$mid];
   }
   else {
@@ -105,14 +112,16 @@
  */
 function simplecdn_rebuild_cache() {
   $mirrors = array();
-    $result = db_query('SELECT * FROM {simplecdn_mirrors} ORDER BY name');
-    while ($mirror = db_fetch_array($result)) {
-      $mirrors[$mirror['mid']] = $mirror;
-    }
+  $result = db_query('SELECT * FROM {simplecdn_mirrors} ORDER BY name');
+  while ($mirror = db_fetch_array($result)) {
+    $mirrors[$mirror['mid']] = $mirror;
+  }
 
   cache_set('simplecdn_mirrors', $mirrors);
 
-  drupal_set_message(t('CDN Mirror cache updated.'));
+  if (user_access('administer simplecdn')) {
+    drupal_set_message(t('CDN Mirror cache updated.'));
+  }
 }
 
 /**
@@ -121,18 +130,23 @@
 function simplecdn_mirror_list() {
   $header = array(t('Mirror'), t('URL'), t('Actions'));
   $rows = array();
+
   foreach (simplecdn_mirrors() as $mirror) {
     $row = array();
+
     $row[] = l($mirror['name'], 'admin/settings/simplecdn/mirror/'. $mirror['mid']);
     $row[] = $mirror['url'];
-      $links = array();
-      $links[] = l(t('Edit'), 'admin/settings/simplecdn/mirror/'. $mirror['mid']);
-      $links[] = l(t('Delete'), 'admin/settings/simplecdn/mirror/'. $mirror['mid'] .'/delete');
+
+    $links = array();
+    $links[] = l(t('Edit'), 'admin/settings/simplecdn/mirror/'. $mirror['mid']);
+    $links[] = l(t('Delete'), 'admin/settings/simplecdn/mirror/'. $mirror['mid'] .'/delete');
+
     $row[] = implode('    ', $links);
+
     $rows[] = $row;
   }
-  $output = theme('table', $header, $rows);
-  return $output;
+
+  return theme('table', $header, $rows);
 }
 
 /**
@@ -183,9 +197,11 @@
  */
 function simplecdn_mirror_validate($form, &$form_state) {
   $mirrors = simplecdn_mirrors();
-  if(isset($form_state['values']['mid'])) {
+
+  if (isset($form_state['values']['mid'])) {
     unset($mirrors[$form_state['values']['mid']]);
   }
+
   foreach($mirrors as $mid => $mirror) {
     if($mirror['name'] == $form_state['values']['name']) {
       form_set_error('name', t('Mirror name has to be unique.'));
@@ -195,9 +211,11 @@
   if (preg_match('/[^0-9a-zA-Z_-]/', $form_state['values']['name'])) {
     form_set_error('name', t('Please only use alphanumeric characters, underscores (_), and hyphens (-) for player names.'));
   }
+
   if(strlen($form_state['values']['name']) > 20) {
     form_set_error('name', t('Mirror name must be shorter than 20 characters.'));
   }
+
   if(strlen($form_state['values']['url']) > 256) {
     form_set_error('name', t('Mirror URL must be shorter than 256 characters.'));
   }
@@ -278,6 +296,7 @@
  */
 function simplecdn_mirror_delete($form_state, $mid = NULL) {
   $mirror = simplecdn_mirrors($mid);
+
   if (empty($mirror)) {
     drupal_set_message(t('The specified mirror was not found.'), 'error');
     drupal_goto('admin/settings/simplecdn/mirrors');
@@ -285,6 +304,7 @@
 
   $form = array();
   $form['mid'] = array('#type' => 'value', '#value' => $mid);
+
   return confirm_form(
     $form,
     t('Are you sure you want to delete the mirror %name?',
@@ -342,6 +362,7 @@
 
   $mirrors = simplecdn_mirrors();
   $new_mirrors = array('none' => t("Don't use a CDN Mirror"));
+
   foreach($mirrors as $mirror) {
     $new_mirrors[$mirror['mid']] = t($mirror['name'].' ('.$mirror['url'].')');
   }
@@ -378,7 +399,7 @@
   $url = str_ireplace(variable_get('simplecdn_site_url', NULL), $mirror['url'], $url, $count);
 
   //Adds the domain if needed
-  if($count == 0) {
+  if ($count == 0) {
     $url = $mirror['url'].$url;
   }
 
@@ -403,11 +424,13 @@
  */
 function simplecdn_invoke_simplecdnapi($op) {
   $results = array();
+
   foreach (module_implements('simplecdnapi') as $name) {
     $function = $name .'_simplecdnapi';
     $result = $function($op);
     $results[] = $result;
   }
+
   return $results;
 }