writer configuration

ACadSharp.Tests/IO/DWG/DwgWriterSingleObjectTests.cs

@@ -45,7 +45,7 @@ protected virtual void writeDwgFile(SingleCaseGenerator data, ACadVersion versio
 			string path = this.getPath(data.Name, "dwg", version);
 
 			data.Document.Header.Version = version;
-			DwgWriter.Write(path, data.Document, this.onNotification);
+			DwgWriter.Write(path, data.Document, notification: this.onNotification);
 		}
 	}
 }

ACadSharp/IO/CadWriterBase.cs

@@ -6,7 +6,36 @@
 
 namespace ACadSharp.IO
 {
-	public abstract class CadWriterBase : ICadWriter
+	/// <summary>
+	/// Configuration for the <see cref="CadWriterBase{T}"/> class.
+	/// </summary>
+	public class CadWriterConfiguration
+	{
+		/// <summary>
+		/// The writer will close the stream once the operation is completed.
+		/// </summary>
+		/// <value>
+		/// default: true
+		/// </value>
+		public bool CloseStream { get; set; } = true;
+
+		/// <summary>
+		/// Will not ignore the <see cref="ACadSharp.Objects.XRecord"/> objects in the document.
+		/// </summary>
+		/// <remarks>
+		/// Due the complexity of XRecords, if this flag is set to true, it may cause a corruption of the file if the records have been modified manually.
+		/// </remarks>
+		/// <value>
+		/// default: false
+		/// </value>
+		public bool WriteXRecords { get; set; } = false;
+	}
+
+	/// <summary>
+	/// Base class for the CAD writers.
+	/// </summary>
+	public abstract class CadWriterBase<T> : ICadWriter
+		where T : CadWriterConfiguration, new()
 	{
 		/// <summary>
 		/// Notification event to get information about the writing process.
@@ -17,12 +46,9 @@ public abstract class CadWriterBase : ICadWriter
 		public event NotificationEventHandler OnNotification;
 
 		/// <summary>
-		/// Notifies the writer to close the stream once the operation is completed.
+		/// Configuration for the writer.
 		/// </summary>
-		/// <value>
-		/// default: true
-		/// </value>
-		public bool CloseStream { get; set; } = true;
+		public T Configuration { get; set; } = new T();
 
 		protected Stream _stream;
 

ACadSharp/IO/DWG/DwgStreamWriters/DwgObjectWriter.Objects.cs

@@ -33,7 +33,7 @@ private void writeObject(CadObject obj)
 					return;
 			}
 
-			if (obj is XRecord && this.IgnoreXRecords)
+			if (obj is XRecord && !this.WriteXRecords)
 			{
 				return;
 			}
@@ -127,7 +127,7 @@ private void writeDictionary(CadDictionary dictionary)
 			//Common:
 			foreach (var item in dictionary)
 			{
-				if (item is XRecord && this.IgnoreXRecords)
+				if (item is XRecord && !this.WriteXRecords)
 				{
 					return;
 				}

ACadSharp/IO/DWG/DwgStreamWriters/DwgObjectWriter.cs

@@ -21,7 +21,7 @@ internal partial class DwgObjectWriter : DwgSectionIO
 		/// </summary>
 		public Dictionary<ulong, long> Map { get; } = new Dictionary<ulong, long>();
 
-		public bool IgnoreXRecords { get; }
+		public bool WriteXRecords { get; }
 
 		private Dictionary<ulong, CadDictionary> _dictionaries = new();
 
@@ -39,14 +39,14 @@ internal partial class DwgObjectWriter : DwgSectionIO
 
 		private Entity _next;
 
-		public DwgObjectWriter(Stream stream, CadDocument document, bool ignoreXRecords = true) : base(document.Header.Version)
+		public DwgObjectWriter(Stream stream, CadDocument document, bool writeXRecords = true) : base(document.Header.Version)
 		{
 			this._stream = stream;
 			this._document = document;
 
 			this._msmain = new MemoryStream();
 			this._writer = DwgStreamWriterBase.GetMergedWriter(document.Header.Version, this._msmain, TextEncoding.Windows1252());
-			this.IgnoreXRecords = ignoreXRecords;
+			this.WriteXRecords = writeXRecords;
 		}
 
 		public void Write()

ACadSharp/IO/DWG/DwgWriter.cs

@@ -12,7 +12,7 @@ namespace ACadSharp.IO
 	/// <summary>
 	/// Class for writing a DWG from a <see cref="CadDocument"/>.
 	/// </summary>
-	public class DwgWriter : CadWriterBase
+	public class DwgWriter : CadWriterBase<CadWriterConfiguration>
 	{
 		private ACadVersion _version { get { return this._document.Header.Version; } }
 
@@ -72,7 +72,7 @@ public override void Write()
 
 			this._stream.Flush();
 
-			if (this.CloseStream)
+			if (this.Configuration.CloseStream)
 			{
 				this._stream.Close();
 			}
@@ -89,11 +89,17 @@ public override void Dispose()
 		/// </summary>
 		/// <param name="filename"></param>
 		/// <param name="document"></param>
+		/// <param name="configuration"></param>
 		/// <param name="notification"></param>
-		public static void Write(string filename, CadDocument document, NotificationEventHandler notification = null)
+		public static void Write(string filename, CadDocument document, CadWriterConfiguration configuration = null, NotificationEventHandler notification = null)
 		{
 			using (DwgWriter writer = new DwgWriter(filename, document))
 			{
+				if(configuration != null)
+				{
+					writer.Configuration = configuration;
+				}
+
 				writer.OnNotification += notification;
 				writer.Write();
 			}
@@ -287,7 +293,7 @@ private void writeRevHistory()
 		private void writeObjects()
 		{
 			MemoryStream stream = new MemoryStream();
-			DwgObjectWriter writer = new DwgObjectWriter(stream, this._document);
+			DwgObjectWriter writer = new DwgObjectWriter(stream, this._document, this.Configuration.WriteXRecords);
 			writer.OnNotification += this.triggerNotification;
 			writer.Write();
 

ACadSharp/IO/DXF/DxfWriter.cs

@@ -7,18 +7,13 @@ namespace ACadSharp.IO
 	/// <summary>
 	/// Class for writing a DXF from a <see cref="CadDocument"/>.
 	/// </summary>
-	public class DxfWriter : CadWriterBase
+	public class DxfWriter : CadWriterBase<DxfWriterConfiguration>
 	{
 		/// <summary>
 		/// Flag indicating if the dxf will be writen as a binary file
 		/// </summary>
 		public bool IsBinary { get; }
 
-		/// <summary>
-		/// DXF writer configuration.
-		/// </summary>
-		public DxfWriterConfiguration Configuration { get; set; } = new DxfWriterConfiguration();
-
 		private IDxfStreamWriter _writer;
 		private CadObjectHolder _objectHolder = new CadObjectHolder();
 
@@ -71,7 +66,7 @@ public override void Write()
 
 			this._writer.Flush();
 
-			if (this.CloseStream)
+			if (this.Configuration.CloseStream)
 			{
 				this._writer.Close();
 			}

ACadSharp/IO/DXF/DxfWriterConfiguration.cs

@@ -8,7 +8,7 @@ namespace ACadSharp.IO
 	/// <summary>
 	/// Configuration for writing DWG files.
 	/// </summary>
-	public class DxfWriterConfiguration
+	public class DxfWriterConfiguration : CadWriterConfiguration
 	{
 		/// <summary>
 		/// Variables that must be writen in a dxf file