2002-11-15 Tim Coleman <tim@timcoleman.com>

        * System.Data.Common/DbDataPermission.cs:
        * System.Data.SqlClient/SqlClientPermission.cs:
                Make these agree on the class status page.
        * System.Data.SqlClient/SqlCommand.cs:
                - Fix up handling of GUID and [Var]Binary, and Image types
        * System.Data.SqlClient/SqlParameter.cs:
                - Provide support for conversion between Type,
                  DbType, SqlDbType, and the SQL server type names.
                - Fix up handling of GUID and [Var]Binary types
        * System.Data.SqlClient/SqlParameterCollection.cs:
                Correct all of the Add methods.
        * Test/SqlTest.cs:
                - Add more types to test: unique identifier, binary,
                  image, smalldatetime, money, smallmoney, timestamp

svn path=/trunk/mcs/; revision=9003

1 files changed, 34 insertions, 15 deletions

diff --git a/mcs/class/System.Data/System.Data.Common/DbDataPermission.cs b/mcs/class/System.Data/System.Data.Common/DbDataPermission.cs
index faf4ee88c61..0d157b60e6c 100644
--- a/mcs/class/System.Data/System.Data.Common/DbDataPermission.cs
+++ b/mcs/class/System.Data/System.Data.Common/DbDataPermission.cs
@@ -16,41 +16,60 @@ namespace System.Data.Common {
 	[Serializable]
 	public abstract class DBDataPermission : CodeAccessPermission, IUnrestrictedPermission
 	{
-		private bool allowBlankPassword;
-		private PermissionState permissionState;
+		#region Fields
+
+		bool allowBlankPassword;
+		PermissionState state;
+
+		#endregion // Fields
+
+		#region Constructors
 
 		protected DBDataPermission () 
+			: this (PermissionState.None, false)
 		{
-			allowBlankPassword = false;
-			permissionState = PermissionState.None;
 		}
 
 		protected DBDataPermission (PermissionState state) 
+			: this (state, false)
 		{
-			allowBlankPassword = false;
-			permissionState = state;
 		}
 
-		public DBDataPermission (PermissionState state, bool abp) 
+		public DBDataPermission (PermissionState state, bool allowBlankPassword) 
 		{
-			allowBlankPassword = abp;
-			permissionState = state;
+			this.state = state;
+			this.allowBlankPassword = allowBlankPassword;
 		}
 
+		#endregion // Constructors
+
+		#region Properties
+
 		public bool AllowBlankPassword {
 			get { return allowBlankPassword; }
 			set { allowBlankPassword = value; }
 		}
+		
+		internal PermissionState State {
+			get { return state; }
+			set { state = value; }
+		}
+
+		#endregion // Properties
+
+		#region Methods
 
 		public override IPermission Copy () 
 		{
-			throw new NotImplementedException ();
+			DBDataPermission copy = CreateInstance ();
+			copy.AllowBlankPassword = this.allowBlankPassword;
+			copy.State = this.state;
+			return copy;
 		}
 
-		[MonoTODO]
 		protected virtual DBDataPermission CreateInstance ()
 		{
-			throw new NotImplementedException ();
+			return (DBDataPermission) Activator.CreateInstance (this.GetType ());
 		}
 
 		[MonoTODO]
@@ -73,9 +92,7 @@ namespace System.Data.Common {
 
 		public bool IsUnrestricted () 
 		{
-			if (permissionState == PermissionState.Unrestricted)
-				return true;
-			return false;
+			return (state == PermissionState.Unrestricted);
 		}
 
 		[MonoTODO]
@@ -89,5 +106,7 @@ namespace System.Data.Common {
 		{
 			throw new NotImplementedException ();
 		}
+
+		#endregion // Methods
 	}
 }