Update unit test and changelog
This commit is contained in:
parent
abc67edb50
commit
e78489b36e
|
|
@ -31,6 +31,7 @@ This release marked heavy refactorings on internal code structure with the creat
|
||||||
- Endnote: Ability to add endnotes - @ivanlanin
|
- Endnote: Ability to add endnotes - @ivanlanin
|
||||||
- ListItem: Ability to create custom list and reset list number - @ivanlanin GH-10 GH-198
|
- ListItem: Ability to create custom list and reset list number - @ivanlanin GH-10 GH-198
|
||||||
- ODT Writer: Basic table writing support - @ivanlanin
|
- ODT Writer: Basic table writing support - @ivanlanin
|
||||||
|
- Image: Keep image aspect ratio if only 1 dimension styled - @japonicus GH-194
|
||||||
|
|
||||||
### Bugfixes
|
### Bugfixes
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -133,14 +133,15 @@ class Image extends AbstractElement
|
||||||
$this->style = $this->setStyle(new ImageStyle(), $style, true);
|
$this->style = $this->setStyle(new ImageStyle(), $style, true);
|
||||||
$styleWidth = $this->style->getWidth();
|
$styleWidth = $this->style->getWidth();
|
||||||
$styleHeight = $this->style->getHeight();
|
$styleHeight = $this->style->getHeight();
|
||||||
|
list($actualWidth, $actualHeight) = $imgData;
|
||||||
if (!($styleWidth && $styleHeight)) {
|
if (!($styleWidth && $styleHeight)) {
|
||||||
if ($styleWidth == null && $styleHeight == null) {
|
if ($styleWidth == null && $styleHeight == null) {
|
||||||
$this->style->setWidth($imgData[0]);
|
$this->style->setWidth($actualWidth);
|
||||||
$this->style->setHeight($imgData[1]);
|
$this->style->setHeight($actualHeight);
|
||||||
} else if ($styleWidth) {
|
} elseif ($styleWidth) {
|
||||||
$this->style->setHeight($imgData[1] * ($styleWidth / $imgData[0]));
|
$this->style->setHeight($actualHeight * ($styleWidth / $actualWidth));
|
||||||
} else {
|
} else {
|
||||||
$this->style->setWidth($imgData[0] * ($styleHeight / $imgData[1]));
|
$this->style->setWidth($actualWidth * ($styleHeight / $actualHeight));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
$this->setImageFunctions();
|
$this->setImageFunctions();
|
||||||
|
|
|
||||||
|
|
@ -129,7 +129,7 @@ class ImageTest extends \PHPUnit_Framework_TestCase
|
||||||
public function testPNG()
|
public function testPNG()
|
||||||
{
|
{
|
||||||
$src = __DIR__ . "/../_files/images/firefox.png";
|
$src = __DIR__ . "/../_files/images/firefox.png";
|
||||||
$oImage = new Image($src);
|
$oImage = new Image($src, array('width' => 100));
|
||||||
|
|
||||||
$this->assertInstanceOf('PhpOffice\\PhpWord\\Element\\Image', $oImage);
|
$this->assertInstanceOf('PhpOffice\\PhpWord\\Element\\Image', $oImage);
|
||||||
$this->assertEquals($oImage->getSource(), $src);
|
$this->assertEquals($oImage->getSource(), $src);
|
||||||
|
|
@ -146,7 +146,7 @@ class ImageTest extends \PHPUnit_Framework_TestCase
|
||||||
public function testGIF()
|
public function testGIF()
|
||||||
{
|
{
|
||||||
$src = __DIR__ . "/../_files/images/mario.gif";
|
$src = __DIR__ . "/../_files/images/mario.gif";
|
||||||
$oImage = new Image($src);
|
$oImage = new Image($src, array('height' => 100));
|
||||||
|
|
||||||
$this->assertInstanceOf('PhpOffice\\PhpWord\\Element\\Image', $oImage);
|
$this->assertInstanceOf('PhpOffice\\PhpWord\\Element\\Image', $oImage);
|
||||||
$this->assertEquals($oImage->getSource(), $src);
|
$this->assertEquals($oImage->getSource(), $src);
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue