[PATCH aeolus-cli] BZ821341: Error messages needs to be in synced in UI and CLI

Jason Guiditta jguiditt at redhat.com
Wed Sep 26 13:24:12 UTC 2012


https://bugzilla.redhat.com/show_bug.cgi?id=821341

Nokogiri behave differently if it is initialized with strict parsing
config, as we were already doing in the TemplateXML object
initializer.
---
 lib/aeolus_cli/command/base_command.rb  |    6 +++++-
 lib/aeolus_cli/command/build_command.rb |    1 -
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/lib/aeolus_cli/command/base_command.rb b/lib/aeolus_cli/command/base_command.rb
index 8ae8429..6ac250d 100644
--- a/lib/aeolus_cli/command/base_command.rb
+++ b/lib/aeolus_cli/command/base_command.rb
@@ -79,7 +79,11 @@ module Aeolus
 
       def validate_xml_document(schema_path, xml_string)
         schema = Nokogiri::XML::RelaxNG(File.read(schema_path))
-        doc = Nokogiri::XML xml_string
+	begin
+          doc = Nokogiri::XML(xml_string) { |config| config.strict }
+	rescue Nokogiri::XML::SyntaxError => e
+	  return [e]
+	end
         schema.validate(doc)
       end
 
diff --git a/lib/aeolus_cli/command/build_command.rb b/lib/aeolus_cli/command/build_command.rb
index c2978e9..c219c18 100644
--- a/lib/aeolus_cli/command/build_command.rb
+++ b/lib/aeolus_cli/command/build_command.rb
@@ -66,7 +66,6 @@ module Aeolus
       def validate_xml_schema(xml)
         errors = validate_xml_document(File.dirname(__FILE__) + "/../../../examples/tdl.rng", xml)
         if errors.length > 0
-          puts "ERROR: The given Template does not conform to the TDL Schema, see below for specific details:"
           errors.each do |error|
             puts "- Line: " + error.line.to_s + " => " + error.message
           end
-- 
1.7.7.6




More information about the aeolus-devel mailing list