Disable direction functionality temporarily

Author: clown

Libraries/Itext/Sources/Internal/Writer.cs

@@ -194,11 +194,13 @@ private void SetMetadata(Metadata src, PdfDocument dest)
                     catalog.SetViewerPreferences(vp);
                 }
 
-                // Add Direction to ViewerPreferences
-                var direction = layout == ViewerOption.TwoColumnRight || layout == ViewerOption.TwoPageRight ?
-                                PdfViewerPreferences.PdfViewerPreferencesConstants.RIGHT_TO_LEFT :
-                                PdfViewerPreferences.PdfViewerPreferencesConstants.LEFT_TO_RIGHT ;
-                vp.SetDirection(direction);
+                // TODO: Disable direction functionality (L2R/R2L) temporarily,
+                // need more complicated modification such as adding a new name on the ViewerOption side.
+                //
+                // var direction = layout == ViewerOption.TwoColumnRight || layout == ViewerOption.TwoPageRight ?
+                //                 PdfViewerPreferences.PdfViewerPreferencesConstants.RIGHT_TO_LEFT :
+                //                 PdfViewerPreferences.PdfViewerPreferencesConstants.LEFT_TO_RIGHT ;
+                // vp.SetDirection(direction);
             }
 
             var mode = src.Options.ToPageMode();

Tests/Core/Sources/Itext/ItextWriterTest.cs

@@ -259,7 +259,7 @@ public void SetMetadata(string value)
                 Creator  = value,
                 Producer = value,
                 Version  = new PdfVersion(1, 5),
-                Options  = ViewerOption.TwoColumnLeft,
+                Options  = ViewerOption.SinglePage,
             };
 
             using (var w = new DocumentWriter(new()))
@@ -282,6 +282,44 @@ public void SetMetadata(string value)
             Assert.That(m.Options,       Is.EqualTo(cmp.Options));
         }
 
+        /* ----------------------------------------------------------------- */
+        ///
+        /// SetMetadata
+        ///
+        /// <summary>
+        /// Test to set PDF page layout.
+        /// </summary>
+        ///
+        /* ----------------------------------------------------------------- */
+        [TestCase(ViewerOption.SinglePage)]
+        [TestCase(ViewerOption.OneColumn)]
+        [TestCase(ViewerOption.TwoColumnLeft)]
+        [TestCase(ViewerOption.TwoColumnRight)]
+        [TestCase(ViewerOption.TwoPageLeft)]
+        [TestCase(ViewerOption.TwoPageRight)]
+        public void SetLayout(ViewerOption value)
+        {
+            var src  = GetSource("SampleViewerOption.pdf");
+            var dest = Path(Args(value));
+            var op   = new OpenOption { SaveMemory = true };
+            var cmp  = new Metadata
+            {
+                Version  = new PdfVersion(1, 7),
+                Options  = value,
+            };
+
+            using (var w = new DocumentWriter(new()))
+            {
+                w.Set(cmp);
+                w.Add(new DocumentReader(src, "", op));
+                w.Save(dest);
+            }
+
+            using var r = new DocumentReader(dest, "", op);
+            var m = r.Metadata;
+            Assert.That(m.Options, Is.EqualTo(cmp.Options));
+        }
+
         /* ----------------------------------------------------------------- */
         ///
         /// SetEncryption