Skip to content

Commit

Permalink
Merge pull request #104 from djlambert/multipoly
Browse files Browse the repository at this point in the history
Add support for multipolygon geometry
  • Loading branch information
sadortun committed Nov 4, 2015
2 parents 73280d3 + 749641c commit 80d7599
Show file tree
Hide file tree
Showing 6 changed files with 294 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Expand Up @@ -26,7 +26,7 @@ script:
- ./vendor/bin/phpunit -v -c ./tests/travis/travis.$DB.xml --coverage-clover ./build/logs/clover-$TRAVIS_PHP_VERSION-$ORM-$DB.xml

after_script:
- ./vendor/bin/coveralls -v
- ./vendor/bin/coveralls -v --exclude-no-stmt

notifications:
webhooks: https://coveralls.io/webhook?repo_token=$COVERALLS_WEBHOOK
Expand Down
44 changes: 44 additions & 0 deletions lib/CrEOF/Spatial/DBAL/Types/Geometry/MultiPolygonType.php
@@ -0,0 +1,44 @@
<?php
/**
* Copyright (C) 2015 Derek J. Lambert
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/

namespace CrEOF\Spatial\DBAL\Types\Geometry;

use CrEOF\Spatial\DBAL\Types\GeometryType;
use CrEOF\Spatial\PHP\Types\Geometry\GeometryInterface;

/**
* Doctrine MULTIPOLYGON type
*
* @author Derek J. Lambert <dlambert@dereklambert.com>
* @license http://dlambert.mit-license.org MIT
*/
class MultiPolygonType extends GeometryType
{
/**
* {@inheritdoc}
*/
public function getSQLType()
{
return GeometryInterface::MULTIPOLYGON;
}
}
2 changes: 1 addition & 1 deletion lib/CrEOF/Spatial/PHP/Types/Geometry/MultiPolygon.php
Expand Up @@ -26,7 +26,7 @@
use CrEOF\Spatial\PHP\Types\AbstractMultiPolygon;

/**
* Polygon object for POLYGON geometry type
* MultiPolygon object for MULTIPOLYGON geometry type
*
* @author Derek J. Lambert <dlambert@dereklambert.com>
* @license http://dlambert.mit-license.org MIT
Expand Down
153 changes: 153 additions & 0 deletions tests/CrEOF/Spatial/Tests/DBAL/Types/Geometry/MultiPolygonTypeTest.php
@@ -0,0 +1,153 @@
<?php
/**
* Copyright (C) 2015 Derek J. Lambert
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/

namespace CrEOF\Spatial\Tests\DBAL\Types\Geometry;

use Doctrine\ORM\Query;
use CrEOF\Spatial\PHP\Types\Geometry\LineString;
use CrEOF\Spatial\PHP\Types\Geometry\Point;
use CrEOF\Spatial\PHP\Types\Geometry\Polygon;
use CrEOF\Spatial\PHP\Types\Geometry\MultiPolygon;
use CrEOF\Spatial\Tests\Fixtures\MultiPolygonEntity;
use CrEOF\Spatial\Tests\OrmTest;

/**
* MultiPolygonType tests
*
* @author Derek J. Lambert <dlambert@dereklambert.com>
* @license http://dlambert.mit-license.org MIT
*
* @group geometry
*/
class MultiPolygonTypeTest extends OrmTest
{
protected function setUp()
{
$this->usesEntity('multipolygon');
parent::setUp();
}

public function testNullMultiPolygon()
{
$entity = new MultiPolygonEntity();

$this->getEntityManager()->persist($entity);
$this->getEntityManager()->flush();

$id = $entity->getId();

$this->getEntityManager()->clear();

$queryEntity = $this->getEntityManager()->getRepository(self::MULTIPOLYGON_ENTITY)->find($id);

$this->assertEquals($entity, $queryEntity);
}

public function testMultiPolygon()
{
$polygons = array(
new Polygon(
array(
new LineString(
array(
new Point(0, 0),
new Point(10, 0),
new Point(10, 10),
new Point(0, 10),
new Point(0, 0)
)
)
)
),
new Polygon(
array(
new LineString(
array(
new Point(5, 5),
new Point(7, 5),
new Point(7, 7),
new Point(5, 7),
new Point(5, 5)
)
)
)
)
);
$entity = new MultiPolygonEntity();

$entity->setMultiPolygon(new MultiPolygon($polygons));
$this->getEntityManager()->persist($entity);
$this->getEntityManager()->flush();

$id = $entity->getId();

$this->getEntityManager()->clear();

$queryEntity = $this->getEntityManager()->getRepository(self::MULTIPOLYGON_ENTITY)->find($id);

$this->assertEquals($entity, $queryEntity);
}


public function testFindByMultiPolygon()
{
$polygons = array(
new Polygon(
array(
new LineString(
array(
new Point(0, 0),
new Point(10, 0),
new Point(10, 10),
new Point(0, 10),
new Point(0, 0)
)
)
)
),
new Polygon(
array(
new LineString(
array(
new Point(5, 5),
new Point(7, 5),
new Point(7, 7),
new Point(5, 7),
new Point(5, 5)
)
)
)
)
);
$entity = new MultiPolygonEntity();

$entity->setMultiPolygon(new MultiPolygon($polygons));
$this->getEntityManager()->persist($entity);
$this->getEntityManager()->flush();
$this->getEntityManager()->clear();

$result = $this->getEntityManager()->getRepository(self::MULTIPOLYGON_ENTITY)->findByMultiPolygon(new MultiPolygon($polygons));

$this->assertEquals($entity, $result[0]);
}
}
88 changes: 88 additions & 0 deletions tests/CrEOF/Spatial/Tests/Fixtures/MultiPolygonEntity.php
@@ -0,0 +1,88 @@
<?php
/**
* Copyright (C) 2015 Derek J. Lambert
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/

namespace CrEOF\Spatial\Tests\Fixtures;

use CrEOF\Spatial\PHP\Types\Geometry\MultiPolygon;

/**
* Polygon entity
*
* @author Derek J. Lambert <dlambert@dereklambert.com>
* @license http://dlambert.mit-license.org MIT
*
* @Entity
* @Table()
*/
class MultiPolygonEntity
{
/**
* @var int $id
*
* @Id
* @GeneratedValue(strategy="AUTO")
* @Column(type="integer")
*/
protected $id;

/**
* @var MultiPolygon $multiPolygon
*
* @Column(type="multipolygon", nullable=true)
*/
protected $multiPolygon;

/**
* Get id
*
* @return int
*/
public function getId()
{
return $this->id;
}

/**
* Set multipolygon
*
* @param MultiPolygon $multiPolygon
*
* @return self
*/
public function setMultiPolygon(MultiPolygon $multiPolygon)
{
$this->multiPolygon = $multiPolygon;

return $this;
}

/**
* Get multipolygon
*
* @return MultiPolygon
*/
public function getMultiPolygon()
{
return $this->multiPolygon;
}
}
7 changes: 7 additions & 0 deletions tests/CrEOF/Spatial/Tests/OrmTest.php
Expand Up @@ -48,6 +48,7 @@ abstract class OrmTest extends \PHPUnit_Framework_TestCase
const POINT_ENTITY = 'CrEOF\Spatial\Tests\Fixtures\PointEntity';
const LINESTRING_ENTITY = 'CrEOF\Spatial\Tests\Fixtures\LineStringEntity';
const POLYGON_ENTITY = 'CrEOF\Spatial\Tests\Fixtures\PolygonEntity';
const MULTIPOLYGON_ENTITY = 'CrEOF\Spatial\Tests\Fixtures\MultiPolygonEntity';
const GEOGRAPHY_ENTITY = 'CrEOF\Spatial\Tests\Fixtures\GeographyEntity';

/**
Expand Down Expand Up @@ -114,6 +115,11 @@ abstract class OrmTest extends \PHPUnit_Framework_TestCase
'types' => array('polygon'),
'table' => 'PolygonEntity'
),
'multipolygon' => array(
'class' => self::MULTIPOLYGON_ENTITY,
'types' => array('multipolygon'),
'table' => 'MultiPolygonEntity'
),
'geography' => array(
'class' => self::GEOGRAPHY_ENTITY,
'types' => array('geography'),
Expand All @@ -129,6 +135,7 @@ abstract class OrmTest extends \PHPUnit_Framework_TestCase
'point' => 'CrEOF\Spatial\DBAL\Types\Geometry\PointType',
'linestring' => 'CrEOF\Spatial\DBAL\Types\Geometry\LineStringType',
'polygon' => 'CrEOF\Spatial\DBAL\Types\Geometry\PolygonType',
'multipolygon' => 'CrEOF\Spatial\DBAL\Types\Geometry\MultiPolygonType',
'geography' => 'CrEOF\Spatial\DBAL\Types\GeographyType',
'geopoint' => 'CrEOF\Spatial\DBAL\Types\Geography\PointType',
'geolinestring' => 'CrEOF\Spatial\DBAL\Types\Geography\LineStringType',
Expand Down

0 comments on commit 80d7599

Please sign in to comment.