Fix

By default locked property is true and not specified in protection element
nissl-lab · Aug 31, 2023 · c64e40b · c64e40b
1 parent ce0e0cc
commit c64e40b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/ooxml/XSSF/UserModel/XSSFCellStyle.cs b/ooxml/XSSF/UserModel/XSSFCellStyle.cs
@@ -695,7 +695,11 @@ public bool IsHidden
         {
             get
             {
-                return _cellXf.IsSetProtection() && _cellXf.protection.IsSetHidden() && _cellXf.protection.hidden;
+                if (!_cellXf.IsSetProtection() || !_cellXf.protection.IsSetHidden())
+                {
+                    return false;
+                }
+                return _cellXf.protection.hidden;
             }
             set
             {
@@ -804,7 +808,7 @@ public bool IsLocked
             {
                 if (!_cellXf.IsSetProtection())
                 {
-                    return false;
+                    return true;
                 }
                 return _cellXf.protection.locked;
             }

diff --git a/testcases/ooxml/XSSF/UserModel/TestXSSFCellStyle.cs b/testcases/ooxml/XSSF/UserModel/TestXSSFCellStyle.cs
@@ -718,7 +718,7 @@ public void TestGetSetHidden()
         [Test]
         public void TestGetSetLocked()
         {
-            Assert.IsFalse(cellStyle.IsLocked);
+            Assert.IsTrue(cellStyle.IsLocked);
             cellStyle.IsLocked = (true);
             Assert.IsTrue(cellStyle.IsLocked);
             cellStyle.IsLocked = (false);