added try catch

Author: mkaszewiak

src/DocumentFormat.OpenXml.Features/DocumentFormat.OpenXml.Features.csproj

@@ -25,4 +25,16 @@
     </PackageReference>
   </ItemGroup>
 
+  <ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.0'">
+    <PackageReference Include="System.IO.Packaging" />
+  </ItemGroup>
+
+  <ItemGroup Condition="'$(TargetFramework)' == 'net8.0'">
+    <PackageReference Include="System.IO.Packaging" />
+  </ItemGroup>
+
+  <ItemGroup Condition="'$(TargetFramework)' == 'net10.0'">
+    <PackageReference Include="System.IO.Packaging" />
+  </ItemGroup>
+
 </Project>

src/DocumentFormat.OpenXml.Framework/Packaging/OpenXmlPart.cs

@@ -43,9 +43,25 @@ internal void Load(OpenXmlPackage? openXmlPackage, OpenXmlPart? parent, Uri uriT
 
             SetPackage(openXmlPackage, parent);
 
-            // TODO: should we delay load?
-            var part = _openXmlPackage.Features.GetRequired<IPackageFeature>().Package.GetPart(uriTarget);
+            // add the _uri to be reserved
+            IPackagePart? part = null;
+            try
+            {
+                part = _openXmlPackage.Features.GetRequired<IPackageFeature>().Package.GetPart(uriTarget);
+            }
+            catch (InvalidOperationException ex)
+            {
+                var errorMessage = SR.Format(
+                    ExceptionMessages.SpecifiedPartNotFound,
+                    uriTarget.OriginalString, ex.StackTrace);
+                throw new OpenXmlPackageException(errorMessage);
+            }
+            catch (Exception)
+            {
+                throw;
+            }
 
+            // TODO: should we delay load?
             Features.Set<IPackagePartFeature>(new PackagePartFeature(part));
 
             if (IsContentTypeFixed && !IsValidContentType(part))
@@ -59,6 +75,17 @@ internal void Load(OpenXmlPackage? openXmlPackage, OpenXmlPart? parent, Uri uriT
                 throw new OpenXmlPackageException(errorMessage);
             }
 
+            if (part is null)
+            {
+                var errorMessage = SR.Format(
+                    ExceptionMessages.InvalidPartContentType,
+                    part.Uri.OriginalString,
+                    part.ContentType,
+                    ContentType);
+
+                throw new OpenXmlPackageException(errorMessage);
+            }
+
             // add the _uri to be reserved
             Features.GetRequired<IPartUriFeature>().ReserveUri(ContentType, Uri);
         }

src/DocumentFormat.OpenXml.Framework/Resources/ExceptionMessages.Designer.cs

@@ -417,4 +417,8 @@
   <data name="EncryptedPackageNotSupported" xml:space="preserve">
     <value>Encrypted packages are not supported.</value>
   </data>
+  <data name="SpecifiedPartNotFound" xml:space="preserve">
+    <value> Part: {0} doesn't exist in the package.
+ {1}</value>
+  </data>
 </root>

src/DocumentFormat.OpenXml.Framework/Resources/ExceptionMessages.resx

@@ -6,4 +6,7 @@
   <ItemGroup>
     <EmbeddedResource Include="assets\**\*" />
   </ItemGroup>
+  <ItemGroup>
+    <PackageReference Include="System.IO.Packaging" />
+  </ItemGroup>
 </Project>