Changeset 16634

Timestamp:

11/30/2010 08:50:59 PM (14 years ago)

Author:

nbachiyski

Message:

Run the iis7_supports_permalinks() check first

• It's cheapest, most of the others involve disk operations
• Doesn't go through win_is_writable(), which can't avoid relying on the @ operator

File:

• trunk/wp-admin/includes/misc.php (modified) (1 diff)

trunk/wp-admin/includes/misc.php

@@ -156 +156 @@
 
     // Using win_is_writable() instead of is_writable() because of a bug in Windows PHP
-    if ( ( ! file_exists($web_config_file) && win_is_writable($home_path) && $wp_rewrite->using_mod_rewrite_permalinks() ) || win_is_writable($web_config_file) ) {
-        if ( iis7_supports_permalinks() ) {
-            $rule = $wp_rewrite->iis7_url_rewrite_rules(false, '', '');
-            if ( ! empty($rule) ) {
-                return iis7_add_rewrite_rule($web_config_file, $rule);
-            } else {
-                return iis7_delete_rewrite_rule($web_config_file);
-            }
+    if ( iis7_supports_permalinks() && ( ! file_exists($web_config_file) && win_is_writable($home_path) && $wp_rewrite->using_mod_rewrite_permalinks() ) || win_is_writable($web_config_file) ) {
+        $rule = $wp_rewrite->iis7_url_rewrite_rules(false, '', '');
+        if ( ! empty($rule) ) {
+            return iis7_add_rewrite_rule($web_config_file, $rule);
+        } else {
+            return iis7_delete_rewrite_rule($web_config_file);
         }
     }