On 09/03/2009 12:14 PM, Michal Zugec wrote:
> ref: refs/heads/network
> commit b72baab724ea256472be56efe577c9231df97881
> Author: Michal Zugec <[email protected]>
> Date:   Thu Sep 3 12:14:49 2009 +0200
> 
>     dns model shows xml list
> ---
>  plugins/network/app/models/dns.rb |   14 +++++++++++---
>  1 files changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git a/plugins/network/app/models/dns.rb 
> b/plugins/network/app/models/dns.rb
> index 6a03736..f153a75 100644
> --- a/plugins/network/app/models/dns.rb
> +++ b/plugins/network/app/models/dns.rb
> @@ -42,9 +42,17 @@ class DNS
>      xml = options[:builder] ||= Builder::XmlMarkup.new(options)
>      xml.instruct! unless options[:skip_instruct]
>  
> -    xml.hostname do
> -      xml.domains @domains
> -      xml.servers @servers
> +    xml.dns do
> +      xml.domains({:type => "array"}) do
> +         domains.each do |@domains, val|
> +               xml.tag!(:domain, domains)
> +         end
> +      end
> +      xml.servers({:type => "array"}) do
> +         servers.each do |@servers, val|
> +               xml.tag!(:nameserver, servers)
> +         end
> +      end
>      end
>    end
>  

Hi, I see it quite often in code and Karel on session say, that instead
  xml.tag!(:domain, domains)
we should use simple
  xml.domain domains

-- 
Josef Reidinger
YaST team
maintainer of perl-Bootloader, YaST2-Repair, webyast modules language
and time
-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to