Add some comments
authorJonas Häggqvist <rasher@rasher.dk>
Tue, 17 Mar 2009 21:18:51 +0000 (17 22:18 +0100)
committerJonas Häggqvist <rasher@rasher.dk>
Tue, 17 Mar 2009 21:18:51 +0000 (17 22:18 +0100)
public/admin.php
public/index.php
public/upload.php

index 8a58b2e..9f26acb 100644 (file)
@@ -65,6 +65,7 @@ else {
         $t->assign('themes', $themes);
         $t->assign('approved', $approved);
     }
+    /* Adding a target */
     elseif (isset($_REQUEST['addtarget'])) {
         $site->addtarget(
             $_REQUEST['shortname'],
@@ -76,11 +77,13 @@ else {
         );
         $t->assign('adminmsg', 'Target added');
     }
+    /* Run checkwps on all themes */
     elseif (isset($_REQUEST['runcheckwps'])) {
         $results = $site->checkallthemes();
         $template = 'checkthemes.tpl';
         $t->assign('checkwpsresults', $results);
     }
+    /* Or just show the front page */
     if (!isset($template)) {
         $t->assign('title', 'Admin');
         $t->assign('targets', $site->listtargets());
index b52bab2..33ef49d 100644 (file)
 require_once('preconfig.inc.php');
 
 /* Decide what to do */
+/* This currently doesn't exist - but might show more details about a theme */
 if (isset($_REQUEST['target']) && isset($_REQUEST['themename'])) {
     $site->showtheme($_REQUEST['target'], $_REQUEST['themename']);
 }
+/* Show all themes for a specific target */
 elseif (isset($_REQUEST['target'])) {
     $values['themes'] = $site->listthemes($_REQUEST['target']);
     $lcd = $site->target2lcd($_REQUEST['target']);
     $t->assign('mainlcd', $lcd['mainlcd']);
     $template = 'themelist.tpl';
 }
-elseif (isset($_REQUEST['theme'])) {
-    $site->showtheme($_REQUEST['theme']);
-}
+/* Just show the frontpage */
 else {
     $values['targets'] = $site->listtargets();
     $template = 'frontpage.tpl';
 }
+
 $t->render($template, $values);
 ?>
index a8dd1eb..0c191e7 100644 (file)
@@ -113,11 +113,13 @@ function checkuploadfiles(&$site, &$err) {
 
 if (isset($_REQUEST['author'])) {
     $err = array();
+    /* First we do some checking of the uploaded data */
     checkuploadfields($site, $err);
     checkuploadfiles($site, $err);
     if (count($err) > 0) {
         $t->assign('errors', $err);
     }
+    /* If that went wrong, go on and include the theme */
     else {
         /* 
          * At this stage, the theme has been validated, any possible errors