Merge pull request #18 from Soullivaneuh/points-collection-deprecation

Deprecate PointsCollection class
Algatux · Jun 3, 2016 · 9aa7987 · 9aa7987
2 parents d58ef5a + 94d4065
commit 9aa7987
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/UPGRADE-1.x.md b/UPGRADE-1.x.md
@@ -21,6 +21,7 @@ Some class are deprecated and should not be used anymore:
 * `Algatux\InfluxDbBundle\Services\Clients\InfluxDbClientFactory`
 * `Algatux\InfluxDbBundle\Services\Clients\WriterClient`
 * `Algatux\InfluxDbBundle\Services\Clients\ReaderClient`
+* `Algatux\InfluxDbBundle\Model\PointsCollection`
 
 ## Deprecated interfaces
 

diff --git a/src/Model/PointsCollection.php b/src/Model/PointsCollection.php
@@ -8,8 +8,15 @@
 use InfluxDB\Database;
 use InfluxDB\Point;
 
+@trigger_error(
+    'The '.__NAMESPACE__.'\PointsCollection class is deprecated since version 1.1 and will be removed in 2.0.',
+    E_USER_DEPRECATED
+);
+
 /**
  * Class PointsCollection.
+ *
+ * @deprecated Since version 1.1, to be removed in 2.0.
  */
 class PointsCollection extends ArrayCollection
 {

diff --git a/tests/unit/Events/InfluxDbEventTest.php b/tests/unit/Events/InfluxDbEventTest.php
@@ -11,6 +11,9 @@
 
 class InfluxDbEventTest extends \PHPUnit_Framework_TestCase
 {
+    /**
+     * @group legacy
+     */
     public function test_event_interface()
     {
         $event = new HttpEvent(new PointsCollection([1, 2, 3]), ClientInterface::HTTP_CLIENT);