Merge branch 'master' into checker
authorAndreas Unterkircher <unki@netshadow.at>
Wed, 28 May 2008 17:23:24 +0000 (19:23 +0200)
committerAndreas Unterkircher <unki@netshadow.at>
Wed, 28 May 2008 17:23:24 +0000 (19:23 +0200)
1  2 
phpfspot.class.php

diff --combined phpfspot.class.php
index df984cb9bc076ecdbe686a48d19ad87eebae32c5,8dec991ebd0b64c2cbc8d70e118bbd21ce3c9ee7..797c6047abbe299d2d15ad432550b5967c6eb775
@@@ -142,9 -142,7 +142,9 @@@ class PHPFSPOT 
  
        /* Check if database file is writeable */
        if(!is_writeable($this->cfg->fspot_db)) {
 -         print $this->cfg->fspot_db ." is not writeable for user ". $this->getuid() ."\n";
 +         print "Error: ". $this->cfg->fspot_db ." is not writeable for user ". $this->getuid() .".\n";
 +         print "Please fix permissions so phpfspot can create indices within the F-Spot database to"
 +            ." speed up some database operations.\n";
           exit(1);
        }
  
  
        /* Check if directory where the database file is stored is writeable  */
        if(!is_writeable(dirname($this->cfg->phpfspot_db))) {
 -         print dirname($this->cfg->phpfspot_db) .": directory is not writeable for user ". $this->getuid() ."\n";
 +         print "Error: ". dirname($this->cfg->phpfspot_db) .": directory is not writeable for user ". $this->getuid() .".\n";
 +         print "Please fix permissions so phpfspot can create its own sqlite database to store some settings.\n";
           exit(1);
        }
  
        /* Check if database file is writeable */
        if(!is_writeable($this->cfg->phpfspot_db)) {
 -         print $this->cfg->phpfspot_db ." is not writeable for user ". $this->getuid() ."\n";
 +         print "Error: ". $this->cfg->phpfspot_db ." is not writeable for user ". $this->getuid() .".\n";
 +         print "Please fix permissions so phpfspot can create its own sqlite database to store some settings.\n";
           exit(1);
        }
  
        }
  
        $this->tmpl->register_function("sort_select_list", array(&$this, "smarty_sort_select_list"), false);
-       $this->tmpl->assign('search_from_date', $this->get_calendar('from'));
-       $this->tmpl->assign('search_to_date', $this->get_calendar('to'));
+       $this->tmpl->assign('search_from_date', $this->get_date_text_field('from'));
+       $this->tmpl->assign('search_to_date', $this->get_date_text_field('to'));
  
        $this->tmpl->assign('preset_selected_tags', $this->getSelectedTags());
        $this->tmpl->assign('preset_available_tags', $this->getAvailableTags());
     public function startSearch()
     {
        /* date search */
-       if(isset($_POST['from']) && $this->isValidDate($_POST['from'])) {
-          $from = $_POST['from'];
+       if(isset($_POST['date_from']) && $this->isValidDate($_POST['date_from'])) {
+          $date_from = $_POST['date_from'];
        }
-       if(isset($_POST['to']) && $this->isValidDate($_POST['to'])) {
-          $to = $_POST['to'];
+       if(isset($_POST['date_to']) && $this->isValidDate($_POST['date_to'])) {
+          $date_to = $_POST['date_to'];
        }
  
        /* tag-name search */
  
        $this->get_tags();
  
-       if(isset($from) && !empty($from))
-          $_SESSION['from_date'] = strtotime($from ." 00:00:00");
+       if(isset($date_from) && !empty($date_from))
+          $_SESSION['from_date'] = strtotime($date_from ." 00:00:00");
        else
           unset($_SESSION['from_date']);
  
-       if(isset($to) && !empty($to))
-          $_SESSION['to_date'] = strtotime($to ." 23:59:59");
+       if(isset($date_to) && !empty($date_to))
+          $_SESSION['to_date'] = strtotime($date_to ." 23:59:59");
        else
           unset($_SESSION['to_date']);
  
     } // _error()
  
     /**
-     * output calendard input fields
+     * get calendar input-text fields
+     *
+     * this function returns a text-field used for the data selection.
+     * Either it will be filled with the current date or, if available,
+     * filled with the date user entered previously.
+     *
      * @param string $mode
      * @return string
      */
-    private function get_calendar($mode)
+    private function get_date_text_field($mode)
     {
-       $year = isset($_SESSION[$mode .'_date']) ? date("Y", $_SESSION[$mode .'_date']) : date("Y");
-       $month = isset($_SESSION[$mode .'_date']) ? date("m", $_SESSION[$mode .'_date']) : date("m");
-       $day = isset($_SESSION[$mode .'_date']) ? date("d", $_SESSION[$mode .'_date']) : date("d");
+       $date = isset($_SESSION[$mode .'_date']) ? date("Y", $_SESSION[$mode .'_date']) : date("Y");
+       $date.= "-";
+       $date.= isset($_SESSION[$mode .'_date']) ? date("m", $_SESSION[$mode .'_date']) : date("m");
+       $date.= "-";
+       $date.= isset($_SESSION[$mode .'_date']) ? date("d", $_SESSION[$mode .'_date']) : date("d");
  
-       $output = "<input type=\"text\" size=\"3\" id=\"". $mode ."year\" value=\"". $year ."\"";
-       if(!isset($_SESSION[$mode .'_date']))
-          $output.= " disabled=\"disabled\"";
-       $output.= " />\n";
-       $output.= "<input type=\"text\" size=\"1\" id=\"". $mode ."month\" value=\"". $month ."\"";
-       if(!isset($_SESSION[$mode .'_date']))
-          $output.= " disabled=\"disabled\"";
-       $output.= " />\n";
-       $output.= "<input type=\"text\" size=\"1\" id=\"". $mode ."day\" value=\"". $day ."\"";
+       $output = "<input type=\"text\" size=\"15\" id=\"date_". $mode ."\" value=\"". $date ."\"";
        if(!isset($_SESSION[$mode .'_date']))
           $output.= " disabled=\"disabled\"";
        $output.= " />\n";
  
        return $output;
  
-    } // get_calendar()
+    } // get_date_text_field()
  
     /**
      * output calendar matrix