From 1e303faceb58a2d39ba971534d7f1ec2bb3e068d Mon Sep 17 00:00:00 2001
From: Benjamin Bentmann <bentmann@apache.org>
Date: Sat, 9 May 2009 22:58:36 +0000
Subject: [PATCH] o Changed boolean field to String to enable both filtering
 and three-valued logic for proper merging

git-svn-id: https://svn.apache.org/repos/asf/maven/components/branches/MNG-2766@773283 13f79535-47bb-0310-9956-ffa450edef68
---
 maven-model/src/main/mdo/maven.mdo | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/maven-model/src/main/mdo/maven.mdo b/maven-model/src/main/mdo/maven.mdo
index 48e3454d21..a53e2be2a3 100644
--- a/maven-model/src/main/mdo/maven.mdo
+++ b/maven-model/src/main/mdo/maven.mdo
@@ -1356,11 +1356,16 @@
         <field>
           <name>optional</name>
           <version>4.0.0</version>
-          <description>Indicates the dependency is optional for use of this library. While the
+          <description>
+            <![CDATA[
+            Indicates the dependency is optional for use of this library. While the
             version of the dependency will be taken into account for dependency calculation if the
-            library is used elsewhere, it will not be passed on transitively.</description>
-          <type>boolean</type>
-          <defaultValue>false</defaultValue>
+            library is used elsewhere, it will not be passed on transitively. Note: While the type
+            of this field is <code>String</code> for technical reasons, the semantic type is actually
+            <code>Boolean</code>. Default value is <code>false</code>.
+            ]]>
+          </description>
+          <type>String</type>
         </field>
       </fields>
       <codeSegments>
@@ -1382,6 +1387,16 @@
           <version>4.0.0</version>
           <code>
             <![CDATA[
+    public boolean isOptional()
+    {
+        return ( optional != null ) ? Boolean.parseBoolean( optional ) : false;
+    }
+
+    public void setOptional( boolean optional )
+    {
+        this.optional = String.valueOf( optional );
+    }
+
     /**
      * @see java.lang.Object#toString()
      */