code cleanups in rpc.php

Signed-off-by: Andreas Unterkircher <unki@netshadow.at>

1 files changed, 34 insertions, 24 deletions

diff --git a/rpc.php b/rpc.php
index e4fecf8..bdb6206 100644
--- a/rpc.php
+++ b/rpc.php
@@ -24,10 +24,20 @@
 
 require_once "phpfspot.class.php";
 
+/**
+ * PHPFSPOT_RPC class
+ *
+ * handles AJAX-RPC calls from client browsers
+ * @package phpfspot
+ */
 class PHPFSPOT_RPC {
 
+   /**
+    * PHPFSPOT_RPC constructor
+    */
    public function __construct()
    {
+      /* start PHP session */
       session_start();
 
    } // __construct()
@@ -39,7 +49,7 @@ class PHPFSPOT_RPC {
       $server = new HTML_AJAX_Server();
       $server->handleRequest();
 
-      $fspot = new PHPFSPOT();
+      $phpfspot = new PHPFSPOT();
 
       /* if no action is specified, no need to further process this
        * function here.
@@ -55,45 +65,45 @@ class PHPFSPOT_RPC {
       switch($action) {
          case 'showphoto':
             if(isset($_GET['id']) && is_numeric($_GET['id'])) {
-               $fspot->showPhoto($_GET['id']);
+               $phpfspot->showPhoto($_GET['id']);
             }
             break;
    
          case 'getxmltaglist':
-            print $fspot->get_xml_tag_list();
+            print $phpfspot->get_xml_tag_list();
             break;
 
          case 'show_available_tags':
-            print $fspot->getAvailableTags();
+            print $phpfspot->getAvailableTags();
             break;
 
          case 'show_selected_tags':
-            print $fspot->getSelectedTags();
+            print $phpfspot->getSelectedTags();
             break;
 
          case 'addtag':
             if(isset($_POST['id']) && is_numeric($_POST['id'])) {
-               print $fspot->addTag($_POST['id']);
+               print $phpfspot->addTag($_POST['id']);
             }
             break;
 
          case 'deltag':
             if(isset($_POST['id']) && is_numeric($_POST['id'])) {
-               print $fspot->delTag($_POST['id']);
+               print $phpfspot->delTag($_POST['id']);
             }
             break;
 
          case 'reset':
-            $fspot->resetTagSearch();
-            $fspot->resetNameSearch();
-            $fspot->resetTags();
-            $fspot->resetDateSearch();
-            $fspot->resetPhotoView();
+            $phpfspot->resetTagSearch();
+            $phpfspot->resetNameSearch();
+            $phpfspot->resetTags();
+            $phpfspot->resetDateSearch();
+            $phpfspot->resetPhotoView();
             break;
 
          case 'tagcondition':
             if(isset($_POST['mode']) && in_array($_POST['mode'], Array('or', 'and'))) {
-               print $fspot->setTagCondition($_POST['mode']);
+               print $phpfspot->setTagCondition($_POST['mode']);
             }
             break;
 
@@ -107,54 +117,54 @@ class PHPFSPOT_RPC {
             if(isset($_GET['last_photo']) && is_numeric($_GET['last_photo']))
                $_SESSION['last_photo'] = $_GET['last_photo'];
 
-            $fspot->showPhotoIndex();
+            $phpfspot->showPhotoIndex();
             break;
    
          case 'showcredits':
-            $fspot->showCredits();
+            $phpfspot->showCredits();
             break;
 
          case 'search':
-            print $fspot->startSearch();
+            print $phpfspot->startSearch();
             break;
 
          case 'update_sort_order':
             if(isset($_POST['value']) && is_string($_POST['value'])) {
-               print $fspot->updateSortOrder($_POST['value']);
+               print $phpfspot->updateSortOrder($_POST['value']);
             }
             break;
 
          case 'get_export':
             /* $_GET['mode'] will be validated by getExport() */
-            $fspot->getExport($_GET['mode']);
+            $phpfspot->getExport($_GET['mode']);
             break;
 
          case 'get_photo_to_show':
-            $fspot->getCurrentPhoto();
+            $phpfspot->getCurrentPhoto();
             break;
 
          case 'get_calendar_matrix':
             if((is_numeric($_GET['year']) || !isset($_GET['year'])) &&
                (is_numeric($_GET['month']) || !isset($_GET['month'])) &&
                (is_numeric($_GET['day']) || !isset($_GET['day']))) {
-               $fspot->get_calendar_matrix($_GET['year'], $_GET['month'], $_GET['day']);
+               $phpfspot->get_calendar_matrix($_GET['year'], $_GET['month'], $_GET['day']);
             }
             break;
 
          case 'what_to_do':
-            print $fspot->whatToDo();
+            print $phpfspot->whatToDo();
             break;
 
          case 'reset_slideshow':
-            print $fspot->resetSlideShow();
+            print $phpfspot->resetSlideShow();
             break;
 
          case 'get_next_slideshow_img':
-            print $fspot->getNextSlideShowImage();
+            print $phpfspot->getNextSlideShowImage();
             break;
          
          case 'get_prev_slideshow_img':
-            print $fspot->getPrevSlideShowImage();
+            print $phpfspot->getPrevSlideShowImage();
             break;
 
       }