From: Andrey Kutejko <andy128k@gmail.com>
Date: Sun, 7 Jul 2013 06:32:01 +0000 (+0300)
Subject: simplify default settings
X-Git-Tag: 0.5~185
X-Git-Url: https://git.andy128k.dev/?a=commitdiff_plain;h=27deff33e7d4741905c3e48609f44846053cc7b3;p=ipf.git

simplify default settings
---

diff --git a/ipf.php b/ipf.php
index c70afa2..6123db7 100644
--- a/ipf.php
+++ b/ipf.php
@@ -2,11 +2,20 @@
 
 final class IPF
 {
-    private static $settings = array();
+    private static $settings = array(
+        'app_base'          => '',
+        'debug'             => false,
+        'media_url'         => '/media/',
+        'static_url'        => '/static/',
+        'session_cookie_id' => 'sessionid',
+        'dir_permission'    => 0777,
+        'file_permission'   => 0666,
+        'time_zone'         => 'America/Toronto',
+    );
 
     private static function applySettings($settings)
     {
-        foreach($settings as $key=>$val)
+        foreach ($settings as $key => $val)
             IPF::$settings[strtolower($key)] = $val;
     }
 
@@ -51,51 +60,17 @@ final class IPF
                 IPF::$settings['template_dirs'][] = IPF::$settings['ipf_path'].DIRECTORY_SEPARATOR.'ipf'.DIRECTORY_SEPARATOR.'admin'.DIRECTORY_SEPARATOR.'templates';
         }
 
-        if (!isset(IPF::$settings['debug'])){
-            IPF::$settings['debug'] = false;
-        }
-
         if (!isset(IPF::$settings['admin_title'])){
             IPF::$settings['admin_title'] = 'IPF Administration';
         }
 
-        if (!isset(IPF::$settings['app_base'])){
-            IPF::$settings['app_base'] = '/index.php';
-        }
-
-        if (!isset(IPF::$settings['append_slash'])){
-            IPF::$settings['append_slash'] = true;
-        }
-
-        if (!isset(IPF::$settings['media_url'])){
-            IPF::$settings['media_url'] = '/media/';
-        }
-
-        if (!isset(IPF::$settings['static_url'])){
-            IPF::$settings['static_url'] = '/static/';
-        }
-
         if (!isset(IPF::$settings['tiny_mce_url'])){
-            IPF::$settings['tiny_mce_url'] = '/static/admin/tiny_mce/';
+            IPF::$settings['tiny_mce_url'] =  IPF::$settings['static_url'] . 'admin/tiny_mce/';
         }
 
         if (!isset(IPF::$settings['urls'])){
             throw new IPF_Exception_Settings('Specify site url routes');
         }
-
-        if (!isset(IPF::$settings['session_cookie_id'])){
-            IPF::$settings['session_cookie_id'] = 'sessionid';
-        }
-
-        if (!isset(IPF::$settings['dir_permission']))
-            IPF::$settings['dir_permission'] = 0777;
-
-        if (!isset(IPF::$settings['file_permission']))
-            IPF::$settings['file_permission'] = 0666;
-
-        if (!isset(IPF::$settings['time_zone'])){
-            IPF::$settings['time_zone'] = 'America/Toronto';
-        }
     }
 
     private static function requestedFileExists()
diff --git a/ipf/middleware/common.php b/ipf/middleware/common.php
index 8826459..389d3cd 100644
--- a/ipf/middleware/common.php
+++ b/ipf/middleware/common.php
@@ -4,9 +4,9 @@ class IPF_Middleware_Common
 {
     function processRequest(&$request)
     {
-        if (IPF::get('append_slash')){
-            $url = $request->http_host.IPF_HTTP_URL::getAction();
-            if (substr($url,-1)!='/'){
+        if (IPF::get('append_slash', true)) {
+            $url = $request->http_host . IPF_HTTP_URL::getAction();
+            if (substr($url,-1)!='/') {
                 $url = $request->addUrlrotocol($url).'/';
                 return new IPF_HTTP_Response_Redirect($url);
             }
diff --git a/t/project/project/settings.php b/t/project/project/settings.php
index da2bad6..a499421 100644
--- a/t/project/project/settings.php
+++ b/t/project/project/settings.php
@@ -9,8 +9,6 @@ $set['tmp'] = $project_root . '/tmp';
 
 $set['upload_url'] = '/media/upload/';
 $set['upload_path'] = $project_root . '/htdocs/media/upload/';
-$set['media_path'] = $project_root . '/htdocs/media/';
-$set['media_url'] = '/media/';
 
 $set['secret_key'] = '123456';