Fixed according to comments

VERSION

@@ -1 +1 @@
-3.1.0
+3.1.1

package/yast2-installation.changes

@@ -3,6 +3,7 @@ Tue Oct 15 14:15:31 CEST 2013 - locilka@suse.com
 
 - Added support for ignore[d][_]feature[s] (FATE#314982) allowing
   to skip some unwanted features of the installer
+- 3.1.1
 
 -------------------------------------------------------------------
 Thu Oct 10 14:48:46 CEST 2013 - locilka@suse.com

package/yast2-installation.spec

@@ -17,7 +17,7 @@
 
 
 Name:           yast2-installation
-Version:        3.1.0
+Version:        3.1.1
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

src/modules/InstFunctions.rb

@@ -70,15 +70,13 @@ def reset_ignored_features
     # @param [String] feature_name
     # @return [Boolean] whether it's ignored
     def feature_ignored?(feature_name)
-      if (feature_name == nil)
+      if feature_name == nil
         Builtins.y2warning("Undefined feature to check")
         return false
       end
 
-      ignored_features()
-
       feature = feature_name.downcase.tr("-_", "")
-      @ignored_features.include?(feature)
+      ignored_features.include?(feature)
     end
 
     publish :function => :ignored_features, :type => "list ()"

test/inst_functions.rb

@@ -54,15 +54,15 @@
 
   it "returns one feature set on commandline by one entry" do
     linuxrc_commandline = "i-g-n-o-r-e_feature=fff"
-    ignored_features    = ["fff"].sort
+    ignored_features    = ["fff"]
 
     Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features
   end
 
   it "returns one feature set on commandline by one entry using up/down case" do
     linuxrc_commandline = "Ignore_FeaTUres=ffF"
-    ignored_features    = ["fff"].sort
+    ignored_features    = ["fff"]
 
     Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features