r4110 Some minor cleanups, including:
authorDenis Ovsienko <infrastation@yandex.ru>
Wed, 12 Jan 2011 12:38:23 +0000 (12:38 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Wed, 12 Jan 2011 12:38:23 +0000 (12:38 +0000)
authenticated_via_ldap(): employ exception
render_image.php: using renderError() is more appropriate for browsers expecting IMG

inc/auth.php
inc/init.php
render_image.php

index b1927959e502f3ebbdbb6b049533c4c91df8eafe..10a09a0305a0733d98648a268d863abee5257f37 100644 (file)
@@ -226,14 +226,12 @@ function processAdjustmentSentence ($modlist, &$chain)
 function authenticated_via_ldap ($username, $password, &$ldap_displayname)
 {
        global $LDAP_options;
-       if (
+       if
+       (
                $LDAP_options['cache_retry'] > $LDAP_options['cache_refresh'] or
                $LDAP_options['cache_refresh'] > $LDAP_options['cache_expiry']
        )
-       {
-               showError ('Fatal LDAP configuration error, check secret.php options.', 'inline');
-               die;
-       }
+               throw new RackTablesError ('LDAP misconfiguration: refresh/retry/expiry mismatch', RackTablesError::MISCONFIGURED);
        if ($LDAP_options['cache_expiry'] == 0) // immediate expiry set means disabled cache
                return authenticated_via_ldap_nocache ($username, $password, $ldap_displayname);
        // authenticated_via_ldap_cache()'s way of locking can sometimes result in
index 942d79cd99933481b4356d6fc32a42e2b6b0d63b..416bb58da997e36e183ff5b98077b0edd6ff677d 100644 (file)
@@ -9,11 +9,8 @@
 *
 */
 
-// This is the first thing we need to do.
 require_once 'inc/exceptions.php';
 require_once 'inc/config.php';
-
-// What we need first is database and interface functions.
 require_once 'inc/functions.php';
 require_once 'inc/database.php';
 // Always have default values for these options, so if a user didn't
index 37147abffdd50633f73196b457935eb28edff542..7955b01ae3367a3c65a9d8b49e35118b8499aeb9 100644 (file)
@@ -144,7 +144,7 @@ function colorFromHex ($image, $hex)
 function renderRackThumb ($rack_id = 0)
 {
        // Don't call DB extra times, hence we are most probably not the
-       // only script wishing to acces the same data now.
+       // only script wishing to access the same data now.
        if (NULL !== ($thumbcache = loadThumbCache ($rack_id)))
        {
                header("Content-type: image/png");
@@ -264,19 +264,19 @@ function renderFilePreview ($file_id = 0, $mode = 'view')
                $file = getFile ($file_id);
                if (!in_array ($file['type'], array ('image/jpeg', 'image/png', 'image/gif')))
                {
-                       showError ('Invalid MIME type on file', 'inline');
+                       renderError();
                        break;
                }
                header("Content-type: ${file['type']}");
                echo $file['contents'];
                break;
        case 'preview':
-               if($image = getFileCache($file_id)){ //Cache Hit
+               if ($image = getFileCache ($file_id)) //Cache Hit
+               {
                        header("Content-type: image/jpeg"); 
                        echo $image;
                        break;
                }
-
                //Cache Miss
                $file = getFile ($file_id);
                $image = imagecreatefromstring ($file['contents']);
@@ -304,7 +304,7 @@ function renderFilePreview ($file_id = 0, $mode = 'view')
                }
                break;
        default:
-               showError ('Invalid argument', 'inline');
+               renderError();
                break;
        }
 }