From 51e18d25ecc43440a269c83d5c52960b59a88356 Mon Sep 17 00:00:00 2001 From: Chris Kankiewicz Date: Thu, 6 Aug 2015 14:32:52 -0700 Subject: [PATCH] Cleaned up PRs --- resources/DirectoryLister.php | 6 +++--- resources/default.config.php | 10 +++++----- resources/themes/bootstrap/index.php | 2 +- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/resources/DirectoryLister.php b/resources/DirectoryLister.php index 5b0cd1a..f42e4c2 100644 --- a/resources/DirectoryLister.php +++ b/resources/DirectoryLister.php @@ -208,8 +208,8 @@ class DirectoryLister { * @return boolean Returns true if in config is enabled open links in another window, false if not * @access public */ - public function determineOpenLinksAnotherWindow() { - return $this->_config['open_links_another_window']; + public function externalLinksNewWindow() { + return $this->_config['external_links_new_window']; } @@ -695,7 +695,7 @@ class DirectoryLister { protected function _getAppUrl() { // Get the server protocol - if (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS']!=="off") { + if (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') { $protocol = 'https://'; } else { $protocol = 'http://'; diff --git a/resources/default.config.php b/resources/default.config.php index 09d9405..7ceb32e 100644 --- a/resources/default.config.php +++ b/resources/default.config.php @@ -3,11 +3,11 @@ return array( // Basic settings - 'hide_dot_files' => true, - 'list_folders_first' => true, - 'list_sort_order' => 'natcasesort', - 'theme_name' => 'bootstrap', - 'open_links_another_window' => true, + 'hide_dot_files' => true, + 'list_folders_first' => true, + 'list_sort_order' => 'natcasesort', + 'theme_name' => 'bootstrap', + 'external_links_new_window' => true, // Hidden files 'hidden_files' => array( diff --git a/resources/themes/bootstrap/index.php b/resources/themes/bootstrap/index.php index 418f250..5285569 100644 --- a/resources/themes/bootstrap/index.php +++ b/resources/themes/bootstrap/index.php @@ -110,7 +110,7 @@ containsIndex($fileInfo['file_path'])): ?> - determineOpenLinksAnotherWindow()) echo 'target="_blank"'?> > + externalLinksNewWindow()): ?>target="_blank">