Moving a Group (shape container) now moves all contained shapes.

OffsetX/Y, ExtentX/Y, Width and Height are always calculated to prevent desync between Group and contained shapes.
Fixed writing Group size into PowerPoint2007 format.

Author: DennisBirkholz

src/PhpPresentation/Shape/Group.php

@@ -83,22 +83,33 @@ public function addShape(AbstractShape $shape): AbstractShape
      */
     public function getOffsetX(): int
     {
-        if (empty($this->offsetX)) {
-            $offsets = GeometryCalculator::calculateOffsets($this);
-            $this->offsetX = $offsets[GeometryCalculator::X];
-            $this->offsetY = $offsets[GeometryCalculator::Y];
+        $offsetX = null;
+
+        foreach ($this->getShapeCollection() as $shape) {
+            if ($offsetX === null) {
+                $offsetX = $shape->getOffsetX();
+            } else {
+                $offsetX = \min($offsetX, $shape->getOffsetX());
+            }
         }
 
-        return $this->offsetX;
+        return $offsetX ?? 0;
     }
 
     /**
-     * Ignores setting the X Offset, preserving the default behavior.
+     * Change the X offset by moving all contained shapes
      *
      * @return $this
      */
-    public function setOffsetX(int $pValue = 0)
+    public function setOffsetX(int $pValue = 0): self
     {
+        $offsetX = $this->getOffsetX();
+        $diff = $pValue - $offsetX;
+
+        foreach ($this->getShapeCollection() as $shape) {
+            $shape->setOffsetX($shape->getOffsetX() + $diff);
+        }
+
         return $this;
     }
 
@@ -107,22 +118,33 @@ public function setOffsetX(int $pValue = 0)
      */
     public function getOffsetY(): int
     {
-        if (empty($this->offsetY)) {
-            $offsets = GeometryCalculator::calculateOffsets($this);
-            $this->offsetX = $offsets[GeometryCalculator::X];
-            $this->offsetY = $offsets[GeometryCalculator::Y];
+        $offsetY = null;
+
+        foreach ($this->getShapeCollection() as $shape) {
+            if ($offsetY === null) {
+                $offsetY = $shape->getOffsetY();
+            } else {
+                $offsetY = \min($offsetY, $shape->getOffsetY());
+            }
         }
 
-        return $this->offsetY;
+        return $offsetY ?? 0;
     }
 
     /**
-     * Ignores setting the Y Offset, preserving the default behavior.
+     * Change the Y offset by moving all contained shapes
      *
      * @return $this
      */
-    public function setOffsetY(int $pValue = 0)
+    public function setOffsetY(int $pValue = 0): self
     {
+        $offsetY = $this->getOffsetY();
+        $diff = $pValue - $offsetY;
+
+        foreach ($this->getShapeCollection() as $shape) {
+            $shape->setOffsetY($shape->getOffsetY() + $diff);
+        }
+
         return $this;
     }
 
@@ -131,35 +153,55 @@ public function setOffsetY(int $pValue = 0)
      */
     public function getExtentX(): int
     {
-        if (null === $this->extentX) {
-            $extents = GeometryCalculator::calculateExtents($this);
-            $this->extentX = $extents[GeometryCalculator::X] - $this->getOffsetX();
-            $this->extentY = $extents[GeometryCalculator::Y] - $this->getOffsetY();
+        $extentX = 0;
+
+        foreach ($this->getShapeCollection() as $shape) {
+            $extentX = \max($extentX, $shape->getOffsetX() + $shape->getWidth());
         }
 
-        return $this->extentX;
+        return $extentX;
     }
 
     /**
      * Get Y Extent.
      */
     public function getExtentY(): int
     {
-        if (null === $this->extentY) {
-            $extents = GeometryCalculator::calculateExtents($this);
-            $this->extentX = $extents[GeometryCalculator::X] - $this->getOffsetX();
-            $this->extentY = $extents[GeometryCalculator::Y] - $this->getOffsetY();
+        $extentY = 0;
+
+        foreach ($this->getShapeCollection() as $shape) {
+            $extentY = \max($extentY, $shape->getOffsetY() + $shape->getHeight());
         }
 
-        return $this->extentY;
+        return $extentY;
+    }
+
+    /**
+     * Calculate the width based on the size/position of the contained shapes
+     * 
+     * @return int
+     */
+    public function getWidth(): int
+    {
+        return $this->getExtentX() - $this->getOffsetX();
     }
 
+    /**
+     * Calculate the height based on the size/position of the contained shapes
+     * 
+     * @return int
+     */
+    public function getHeight(): int
+    {
+        return $this->getExtentY() - $this->getOffsetY();
+    }
+    
     /**
      * Ignores setting the width, preserving the default behavior.
      *
-     * @return self
+     * @return $this
      */
-    public function setWidth(int $pValue = 0)
+    public function setWidth(int $pValue = 0): self
     {
         return $this;
     }
@@ -169,7 +211,7 @@ public function setWidth(int $pValue = 0)
      *
      * @return $this
      */
-    public function setHeight(int $pValue = 0)
+    public function setHeight(int $pValue = 0): self
     {
         return $this;
     }

src/PhpPresentation/Writer/PowerPoint2007/AbstractSlide.php

@@ -1446,8 +1446,8 @@ protected function writeShapeGroup(XMLWriter $objWriter, Group $group, int &$sha
         $objWriter->endElement(); // a:off
         // a:ext
         $objWriter->startElement('a:ext');
-        $objWriter->writeAttribute('cx', CommonDrawing::pixelsToEmu($group->getExtentX()));
-        $objWriter->writeAttribute('cy', CommonDrawing::pixelsToEmu($group->getExtentY()));
+        $objWriter->writeAttribute('cx', CommonDrawing::pixelsToEmu($group->getWidth()));
+        $objWriter->writeAttribute('cy', CommonDrawing::pixelsToEmu($group->getHeight()));
         $objWriter->endElement(); // a:ext
         // a:chOff
         $objWriter->startElement('a:chOff');
@@ -1456,8 +1456,8 @@ protected function writeShapeGroup(XMLWriter $objWriter, Group $group, int &$sha
         $objWriter->endElement(); // a:chOff
         // a:chExt
         $objWriter->startElement('a:chExt');
-        $objWriter->writeAttribute('cx', CommonDrawing::pixelsToEmu($group->getExtentX()));
-        $objWriter->writeAttribute('cy', CommonDrawing::pixelsToEmu($group->getExtentY()));
+        $objWriter->writeAttribute('cx', CommonDrawing::pixelsToEmu($group->getWidth()));
+        $objWriter->writeAttribute('cy', CommonDrawing::pixelsToEmu($group->getHeight()));
         $objWriter->endElement(); // a:chExt
         $objWriter->endElement(); // a:xfrm
         $objWriter->endElement(); // p:grpSpPr