Merge pull request #63 from openxml/explicit-false

Turn off the bold!

Author: boblail

lib/openxml/docx/properties.rb

@@ -14,6 +14,7 @@ module Properties
 require "openxml/docx/properties/string_property"
 require "openxml/docx/properties/on_off_property"
 require "openxml/docx/properties/toggle_property"
+require "openxml/docx/properties/explicit_toggle_property"
 require "openxml/docx/properties/string_property"
 require "openxml/docx/properties/container_property"
 require "openxml/docx/properties/transparent_container_property"

lib/openxml/docx/properties/bold.rb

@@ -1,7 +1,7 @@
 module OpenXml
   module Docx
     module Properties
-      class Bold < ToggleProperty
+      class Bold < ExplicitToggleProperty
         tag :b
 
       end

lib/openxml/docx/properties/explicit_toggle_property.rb

@@ -0,0 +1,15 @@
+module OpenXml
+  module Docx
+    module Properties
+      class ExplicitToggleProperty < ToggleProperty
+
+        def to_xml(xml)
+          return if value.nil?
+          return xml["w"].public_send(tag, "w:val" => value) if value == false
+          xml["w"].public_send(tag)
+        end
+
+      end
+    end
+  end
+end

lib/openxml/docx/properties/italics.rb

@@ -1,7 +1,7 @@
 module OpenXml
   module Docx
     module Properties
-      class Italics < ToggleProperty
+      class Italics < ExplicitToggleProperty
         tag :i
 
       end

spec/properties/bold_spec.rb

@@ -12,7 +12,7 @@
 
   with_value(false) do
     it_should_work
-    it_should_output ""
+    it_should_output "<w:b w:val=\"false\"/>"
   end
 
   with_value(nil) do

spec/properties/italics_spec.rb

@@ -12,7 +12,7 @@
 
   with_value(false) do
     it_should_work
-    it_should_output ""
+    it_should_output "<w:i w:val=\"false\"/>"
   end
 
   with_value(nil) do