Some changes to the API documentation.

Added missing makefile definitions.

library/network/src/Makefile.am

@@ -45,7 +45,8 @@ yfwdlib_DATA = \
   lib/y2firewall/firewalld/relations.rb \
   lib/y2firewall/firewalld/service.rb \
   lib/y2firewall/firewalld/zone.rb \
-  lib/y2firewall/firewalld/zone_parser.rb
+  lib/y2firewall/firewalld/zone_reader.rb \
+  lib/y2firewall/firewalld/service_reader.rb
 
 yfwdapilibdir = @ylibdir@/y2firewall/firewalld/api/
 yfwdapilib_DATA = \

library/network/src/lib/y2firewall/firewalld/api.rb

@@ -95,10 +95,12 @@ def running?
         state == "running"
       end
 
+      # Enables the firewalld service
       def enable!
         offline? ? run_command("--enable") : Yast::Service.Enable("firewalld")
       end
 
+      # Disables the firewalld service
       def disable!
         offline? ? run_command("--disable") : Yast::Service.Disable("firewalld")
       end
@@ -129,9 +131,9 @@ def default_zone
       # Set the default zone
       #
       # @param zone [String] The firewall zone
-      # @return [String] default zone
+      # @return [Boolean] true if the default zone was modified correctly
       def default_zone=(zone)
-        run_command("--set-default-zone=#{zone}") == "success"
+        string_command("--set-default-zone=#{zone}") == "success"
       end
 
       # Do a reload of the firewall if running. In offline mode just return
@@ -163,8 +165,7 @@ def log_denied_packets?(kind)
 
       # @param kind [String] Denied packets to log. Possible values are:
       #   all, unicast, broadcast, multicast and off
-      # @return [Boolean] True if desired packet type was set to being logged
-      #   when denied
+      # @return [Boolean] true if the type of packages to log was set correctly
       def log_denied_packets=(kind)
         string_command("--set-log-denied=#{kind}") == "success"
       end

library/network/src/lib/y2firewall/firewalld/api/services.rb

@@ -93,8 +93,8 @@ def service_description(service, permanent: permanent?)
         # @param service [String] The firewall service
         # @param long_description [String] the new service description
         def service_description=(service, long_description)
-          run_command("--service=#{service}", "--set-description=#{long_description}",
-            permanent: !offline?)
+          string_command("--service=#{service}", "--set-description=#{long_description}",
+            permanent: !offline?) == "success"
         end
 
         # Returns whether the service definition for the service name given is