Semantic Error : Attribute 'si' should be present when the value of attribute 't' is 'shared'.

Is this a:

• Issue with the OpenXml library
• Question on library usage

Description

Semantic Error : Attribute ‘si’ should be present when the value of attribute ‘t’ is ‘shared’. On 2 separate valid Office 365 Excel documents.

Information

• .NET Target: NET 4.6.2
• DocumentFormat.OpenXml Version: 2.11.0

Repro

class Program
    {
        static void Main(string[] args)
        {
            OpenXmlPackage spreadsheetDocument1 = SpreadsheetDocument.Open("DiversenBerekeningen.xlsm", false);
            List<ValidationErrorInfo> errorInfos1 = ValidateDocument(spreadsheetDocument1);

            foreach (ValidationErrorInfo info in errorInfos1)
            {
                Console.WriteLine(string.Format("DiversenBerekeningen.xlsm Error Occurred {0}", info.Description)); 
            }

            OpenXmlPackage spreadsheetDocument2 = SpreadsheetDocument.Open("DiversenGrafiekenPeltier.xlsm", false);
            List<ValidationErrorInfo> errorInfos2 = ValidateDocument(spreadsheetDocument2);

            foreach (ValidationErrorInfo info in errorInfos2)
            {
                Console.WriteLine(string.Format("DiversenGrafiekenPeltier.xlsm Validaton Error Occurred {0}", info.Description));
            }

            Console.ReadKey();
        }

        static List<ValidationErrorInfo> ValidateDocument(OpenXmlPackage package)
        {
            OpenXmlValidator validator = new OpenXmlValidator(FileFormatVersions.Office2019);
            List<ValidationErrorInfo> validationErrors = validator.Validate(package).ToList();

            return validationErrors;
        }
    }

Observed

The following validation error was reported on both files.

Semantic Error : Attribute ‘si’ should be present when the value of attribute ‘t’ is ‘shared’

Expected

Both files should have been validated successfully without errors.

Invalid Files.zip

[twsouthwick]

Cool. It’s merged in now and will be available via CI. We’ll probably cut a 2.11.1 release soon as we’ve got a few other fixes to get out.

[06needhamt]

@twsouthwick Thanks for the fix. I should have been more clear that I was referring to that one specific validation issue the others that you reported were expected.