Merge pull request #1 from EllisLab/bug/20

Bug/20
ExpressionEngine · Dec 3, 2018 · 981499a · 981499a
2 parents 0ec2a93 + cbee096
commit 981499a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/changelogs/patch.rst b/changelogs/patch.rst
@@ -22,7 +22,7 @@ Patch Release
 - Fixed a bug where a supplied class was not added to "select" fields in the shared form view.
 - Fixed a potential malformed query issue in the relationships_query hook.
 - Fixed a potential PHP error (#21) when saving option-type Grid columns.
-
+- Fixed a bug where the installer checks if the user theme directory is writable even when not installing the default theme.
 
 EOF MARKER: This line helps prevent merge conflicts when things are
 added on the bottoms of lists
diff --git a/system/ee/installer/controllers/wizard.php b/system/ee/installer/controllers/wizard.php
@@ -961,7 +961,7 @@ public function template_path_writeable($radio)
 
 	public function themes_user_writable($radio)
 	{
-		if ( ! is_really_writable($this->root_theme_path.'user'))
+		if ( $radio == 'y' && ! is_really_writable($this->root_theme_path.'user'))
 		{
 			ee()->form_validation->set_message(
 				'themes_user_writable',