Minor code cleanup

src/modules/HttpServerPackages.rb

@@ -34,19 +34,19 @@ def main
     #
     # @param pattern [String] a regex pattern to match, no escaping done
     # @return list of matching package names
-    def by_pattern(pattern)
+    def by_provides_regexp(pattern)
       raise ArgumentError, "Missing search pattern" if pattern.nil? || pattern.empty?
 
       # NOTE: - Resolvable.find takes POSIX regexp, later select uses Ruby regexp
       # - Resolvable.find supports regexps only for dependencies, so we need to
       # filter result according to package name
-      Y2Packager::Resolvable.find({ provides_regexp: "^#{pattern}$" }, [:name])
+      Y2Packager::Resolvable.find({ kind: :package, provides_regexp: "^#{pattern}$" }, [])
         .select { |p| p.name =~ /\A#{pattern}\z/ }
         .map(&:name)
         .uniq
     end
 
-    publish function: :by_pattern, type: "list <string> (string)"
+    publish function: :by_provides_regexp, type: "list <string> (string)"
 
   end
 

src/modules/YaPI/HTTPDModules.pm

@@ -780,7 +780,7 @@ sub ServerModules {
         %modules = ( %modules,
             'php' . $php_version => {
                     summary   => __("Provides support for PHP dynamically generated pages"),
-                    packages  => ["apache2-mod_php" . YaST::HTTPDPhpModule->Version()],
+                    packages  => ["apache2-mod_php" . $php_version],
                     default   => 0,
                     position  => 490
             }

src/modules/YaST/HTTPDPhpModule.pm

@@ -10,7 +10,7 @@ our %TYPEINFO;
 BEGIN { $TYPEINFO{Version} = ["function", "string" ]; }
 sub Version {
     # when function returns an array, we get reference to it
-    $l = HttpServerPackages->by_pattern("php[0-9]{1,2}");
+    $l = HttpServerPackages->by_provides_regexp("php[0-9]{1,2}");
 
     return if(!$l);