2007-01-10 Marek Sieradzki <marek.sieradzki@gmail.com>
[mono.git] / mcs / class / Microsoft.Build.Engine / Microsoft.Build.BuildEngine / BuildPropertyGroup.cs
index cea9b78ecea1b4d0814a9fa992f17debe3fa8827..d20027559483678e4edc059062834cfd4f5c8486 100644 (file)
@@ -29,7 +29,7 @@
 
 using System;
 using System.Collections;
-using System.Collections.Specialized;
+using System.Collections.Generic;
 using System.Reflection;
 using System.Text;
 using System.Xml;
@@ -37,34 +37,39 @@ using System.Xml;
 namespace Microsoft.Build.BuildEngine {
        public class BuildPropertyGroup : IEnumerable {
        
+               bool                    read_only;
+               ImportedProject         importedProject;
                XmlElement              propertyGroup;
-               bool                    isImported;
                GroupingCollection      parentCollection;
                Project                 parentProject;
-               IList                   properties;
-               IDictionary             propertiesByName;
+               List <BuildProperty>    properties;
+               Dictionary <string, BuildProperty>      propertiesByName;
 
                public BuildPropertyGroup ()
-                       : this (null, null)
+                       : this (null, null, null, false)
                {
                }
 
-               internal BuildPropertyGroup (XmlElement xmlElement, Project project)
+               internal BuildPropertyGroup (XmlElement xmlElement, Project project, ImportedProject importedProject, bool readOnly)
                {
-                       this.isImported = false;
+                       this.importedProject = importedProject;
                        this.parentCollection = null;
                        this.parentProject = project;
-                       this.isImported = false;
                        this.propertyGroup = xmlElement;
+                       this.read_only = readOnly;
 
                        if (FromXml) {
-                               this.properties = new ArrayList ();
-                               foreach (XmlElement xe in propertyGroup.ChildNodes) {
+                               this.properties = new List <BuildProperty> ();
+                               foreach (XmlNode xn in propertyGroup.ChildNodes) {
+                                       if (!(xn is XmlElement))
+                                               continue;
+                                       
+                                       XmlElement xe = (XmlElement) xn;
                                        BuildProperty bp = new BuildProperty (parentProject, xe);
                                        AddProperty (bp);
                                } 
                        } else
-                               this.propertiesByName = CollectionsUtil.CreateCaseInsensitiveHashtable ();
+                               this.propertiesByName = new Dictionary <string, BuildProperty> (StringComparer.InvariantCultureIgnoreCase);
                }
 
                public BuildProperty AddNewProperty (string propertyName,
@@ -82,19 +87,20 @@ namespace Microsoft.Build.BuildEngine {
                        if (FromXml) {
                                XmlElement xe;
                                
-                               xe = propertyGroup.OwnerDocument.CreateElement (propertyName);
+                               xe = propertyGroup.OwnerDocument.CreateElement (propertyName, Project.XmlNamespace);
                                propertyGroup.AppendChild (xe);
-                               if (treatPropertyValueAsLiteral) {
+                               
+                               if (treatPropertyValueAsLiteral)
                                        xe.InnerText = Utilities.Escape (propertyValue);
-                               } else {
+                               else
                                        xe.InnerText = propertyValue;
-                               }
+                               
                                prop = new BuildProperty (parentProject, xe);
-                       } else {
-                               prop = new BuildProperty (propertyName, propertyValue);
-                       }
-                       AddProperty (prop);
-                       return prop;
+                               AddProperty (prop);
+                               parentProject.EvaluatedProperties.AddProperty (prop);
+                               return prop;
+                       } else
+                               throw new InvalidOperationException ("This method is only valid for persisted <System.Object[]> elements.");
                }
 
                internal void AddProperty (BuildProperty property)
@@ -102,8 +108,8 @@ namespace Microsoft.Build.BuildEngine {
                        if (FromXml) {
                                properties.Add (property);
                        } else {
-                               if (propertiesByName.Contains (property.Name)) {
-                                       BuildProperty existing = (BuildProperty) propertiesByName [property.Name];
+                               if (propertiesByName.ContainsKey (property.Name)) {
+                                       BuildProperty existing = propertiesByName [property.Name];
                                        if (property.PropertyType <= existing.PropertyType) {
                                                propertiesByName.Remove (property.Name);
                                                propertiesByName.Add (property.Name, property);
@@ -116,37 +122,52 @@ namespace Microsoft.Build.BuildEngine {
                
                public void Clear ()
                {
+                       if (FromXml) {
+                               propertyGroup.RemoveAll ();
+                               properties = new List <BuildProperty> ();
+                       } else
+                               propertiesByName = new Dictionary <string, BuildProperty> ();
                }
 
+               [MonoTODO]
                public BuildPropertyGroup Clone (bool deepClone)
                {
-                       return null;
+                       throw new NotImplementedException ();
                }
 
                public IEnumerator GetEnumerator ()
                {
-                       if (properties != null) {
+                       if (FromXml)
                                foreach (BuildProperty bp in properties)
                                        yield return bp;
-                       } else if (propertiesByName != null) {
-                               foreach (DictionaryEntry de in propertiesByName)
-                                       yield return (BuildProperty) de.Value;
-                       } else
-                               throw new Exception ("PropertyGroup is not initialized.");
+                       else 
+                               foreach (KeyValuePair <string, BuildProperty> kvp in propertiesByName)
+                                       yield return kvp.Value;
                }
 
                public void RemoveProperty (BuildProperty propertyToRemove)
                {
-                       if (properties == null)
-                               throw new Exception ("PropertyGroup is not initialized.");
-                       properties.Remove (propertyToRemove);
+                       if (FromXml)
+                               properties.Remove (propertyToRemove);
+                       else {
+                               foreach (KeyValuePair <string, BuildProperty> kvp in propertiesByName)
+                                       if (kvp.Value == propertyToRemove) {
+                                               propertiesByName.Remove (kvp.Key);
+                                               break;
+                                       }
+                       }
                }
 
                public void RemoveProperty (string propertyName)
                {
-                       if (propertiesByName == null)
-                               throw new Exception ("PropertyGroup is not initialized.");
-                       propertiesByName.Remove (propertyName);
+                       if (FromXml) {
+                               foreach (BuildProperty bp in properties)
+                                       if (bp.Name == propertyName) {
+                                               properties.Remove (bp);
+                                               break;
+                                       }
+                       } else
+                               propertiesByName.Remove (propertyName);
                }
 
                public void SetProperty (string propertyName,
@@ -155,16 +176,27 @@ namespace Microsoft.Build.BuildEngine {
                        SetProperty (propertyName, propertyValue, false);
                }
                
-               // FIXME: use treatPropertyValueAsLiteral
-               [MonoTODO]
+               // FIXME: add a test for SetProperty on property from xml
                public void SetProperty (string propertyName,
                                         string propertyValue,
                                         bool treatPropertyValueAsLiteral)
                {
-                       if (propertiesByName.Contains (propertyName) == false) {
-                               AddNewProperty (propertyName, propertyValue);
-                       }
-                       ((BuildProperty) propertiesByName [propertyName]).Value = propertyValue;
+                       if (read_only)
+                               return;
+                       
+                       if (propertiesByName.ContainsKey (propertyName))
+                               propertiesByName.Remove (propertyName);
+
+                       BuildProperty bp;
+                       if (treatPropertyValueAsLiteral)
+                               bp = new BuildProperty (propertyName, Utilities.Escape (propertyValue));
+                       else
+                               bp = new BuildProperty (propertyName, propertyValue);
+
+                       AddProperty (bp);
+
+                       if (IsGlobal)
+                               parentProject.ProcessXml ();
                }
                
                internal void Evaluate ()
@@ -173,7 +205,13 @@ namespace Microsoft.Build.BuildEngine {
                                throw new InvalidOperationException ();
                        }
                        foreach (BuildProperty bp in properties) {
-                               bp.Evaluate ();
+                               if (bp.Condition == String.Empty)
+                                       bp.Evaluate ();
+                               else {
+                                       ConditionExpression ce = ConditionParser.ParseCondition (bp.Condition);
+                                       if (ce.BoolEvaluate (parentProject))
+                                               bp.Evaluate ();
+                               }
                        }
                }
                
@@ -185,25 +223,24 @@ namespace Microsoft.Build.BuildEngine {
                        }
                        set {
                                if (!FromXml)
-                                       throw new InvalidOperationException ("Can only set condition on xml elements.");
+                                       throw new InvalidOperationException (
+                                       "Cannot set a condition on an object not represented by an XML element in the project file.");
                                propertyGroup.SetAttribute ("Condition", value);
                        }
                }
 
                public int Count {
                        get {
-                               if (properties != null)
+                               if (FromXml)
                                        return properties.Count;
-                               else if (propertiesByName != null)
-                                       return propertiesByName.Count;
                                else
-                                       throw new Exception ("PropertyGroup is not initialized.");
+                                       return propertiesByName.Count;
                        }
                }
 
                public bool IsImported {
                        get {
-                               return isImported;
+                               return importedProject != null;
                        }
                }
 
@@ -213,13 +250,21 @@ namespace Microsoft.Build.BuildEngine {
                        }
                }
 
-               public BuildProperty this[string propertyName] {
+               bool IsGlobal {
                        get {
-                               if (propertiesByName.Contains (propertyName)) {
-                                       return (BuildProperty) propertiesByName [propertyName];
-                               } else {
+                               return parentProject != null && propertyGroup == null;
+                       }
+               }
+
+               public BuildProperty this [string propertyName] {
+                       get {
+                               if (FromXml)
+                                       throw new InvalidOperationException ("Properties in persisted property groups cannot be accessed by name.");
+                               
+                               if (propertiesByName.ContainsKey (propertyName))
+                                       return propertiesByName [propertyName];
+                               else
                                        return null;
-                               }
                        }
                        set {
                                propertiesByName [propertyName] = value;
@@ -230,6 +275,10 @@ namespace Microsoft.Build.BuildEngine {
                        get { return parentCollection; }
                        set { parentCollection = value; }
                }
+
+               internal XmlElement XmlElement {
+                       get { return propertyGroup; }
+               }
        }
 }