From d6b8ff12e5df8d2ecb4b3d0b2b82b26b6b86fdd2 Mon Sep 17 00:00:00 2001
From: Nick Burch <nick@apache.org>
Date: Mon, 4 Aug 2014 22:00:47 +0000
Subject: [PATCH] Fix bug #56812 - In XSLF provide a way to get the URI of
 externally linked pictures

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1615812 13f79535-47bb-0310-9956-ffa450edef68
---
 .../poi/xslf/usermodel/XSLFPictureShape.java  | 57 ++++++++++++++++++-
 .../org/apache/poi/xslf/TestXSLFBugs.java     | 24 +++++---
 2 files changed, 72 insertions(+), 9 deletions(-)

diff --git a/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPictureShape.java b/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPictureShape.java
index 8663b897fc..6d6e631b52 100644
--- a/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPictureShape.java
+++ b/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPictureShape.java
@@ -23,6 +23,7 @@ import java.awt.Graphics2D;
 import java.awt.geom.Rectangle2D;
 import java.awt.image.BufferedImage;
 import java.io.ByteArrayInputStream;
+import java.net.URI;
 
 import javax.imageio.ImageIO;
 import javax.xml.namespace.QName;
@@ -99,10 +100,27 @@ public class XSLFPictureShape extends XSLFSimpleShape {
             setAnchor(new java.awt.Rectangle(50, 50, 200, 200));
         }
     }
+    
+    /**
+     * Is this an internal picture (image data included within
+     *  the PowerPoint file), or an external linked picture
+     *  (image lives outside)?
+     */
+    public boolean isExternalLinkedPicture() {
+        if (getBlipId() == null && getBlipLink() != null) {
+            return true;
+        }
+        return false;
+    }
 
+    /**
+     * Return the data on the (internal) picture.
+     * For an external linked picture, will return null
+     */
     public XSLFPictureData getPictureData() {
         if(_data == null){
             String blipId = getBlipId();
+            if (blipId == null) return null;
 
             PackagePart p = getSheet().getPackagePart();
             PackageRelationship rel = p.getRelationship(blipId);
@@ -118,10 +136,45 @@ public class XSLFPictureShape extends XSLFSimpleShape {
         }
         return _data;
     }
+    
+    /**
+     * For an external linked picture, return the last-seen
+     *  path to the picture.
+     * For an internal picture, returns null.
+     */
+    public URI getPictureLink() {
+        if (getBlipId() != null) {
+            // Internal picture, nothing to return
+            return null;
+        }
+        
+        String rId = getBlipLink();
+        if (rId == null) {
+            // No link recorded, nothing we can do
+            return null;
+        }
+        
+        PackagePart p = getSheet().getPackagePart();
+        PackageRelationship rel = p.getRelationship(rId);
+        if (rel != null) {
+            return rel.getTargetURI();
+        }
+        return null;
+    }
 
-    private String getBlipId(){
+    private CTBlip getBlip(){
         CTPicture ct = (CTPicture)getXmlObject();
-        return ct.getBlipFill().getBlip().getEmbed();
+        return ct.getBlipFill().getBlip();
+    }
+    private String getBlipLink(){
+        String link = getBlip().getLink();
+        if (link.isEmpty()) return null;
+        return link;
+    }
+    private String getBlipId(){
+        String id = getBlip().getEmbed();
+        if (id.isEmpty()) return null;
+        return id;
     }
 
     @Override
diff --git a/src/ooxml/testcases/org/apache/poi/xslf/TestXSLFBugs.java b/src/ooxml/testcases/org/apache/poi/xslf/TestXSLFBugs.java
index fc422e778d..0e787ba569 100644
--- a/src/ooxml/testcases/org/apache/poi/xslf/TestXSLFBugs.java
+++ b/src/ooxml/testcases/org/apache/poi/xslf/TestXSLFBugs.java
@@ -144,24 +144,34 @@ public class TestXSLFBugs extends POITestCase {
      * there is no data available and XSLFPictureShape.getPictureData()
      * gives a NPE, see bug #56812
      */
-    public void DISABLEDtest56812() throws Exception {
+    public void test56812() throws Exception {
         XMLSlideShow ppt = XSLFTestDataSamples.openSampleDocument("56812.pptx");
         
-        int pictures = 0;
+        int internalPictures = 0;
+        int externalPictures = 0;
         for (XSLFSlide slide : ppt.getSlides()){
             for (XSLFShape shape : slide.getShapes()){
                 assertNotNull(shape);
                 
                 if (shape instanceof XSLFPictureShape) {
-                    XSLFPictureData data = ((XSLFPictureShape) shape).getPictureData();
-                    assertNotNull(data);
-                    assertNotNull(data.getFileName());
-                    pictures++;
+                    XSLFPictureShape picture = (XSLFPictureShape)shape;
+                    if (picture.isExternalLinkedPicture()) {
+                        externalPictures++;
+                        
+                        assertNotNull(picture.getPictureLink());
+                    } else {
+                        internalPictures++;
+                        
+                        XSLFPictureData data = picture.getPictureData();
+                        assertNotNull(data);
+                        assertNotNull(data.getFileName());
+                    }
                 }
             }
         }
         
-        assertEquals(3, pictures);
+        assertEquals(2, internalPictures);
+        assertEquals(1, externalPictures);
     }
     
     protected String getSlideText(XSLFSlide slide) {
-- 
2.39.5