MNT Remove TODO comments

src/ChangeSet.php

@@ -213,8 +213,6 @@ public function removeObject(DataObject $object)
         ])->first();
 
         if ($item) {
-            // TODO: Handle case of implicit added item being removed.
-
             $item->delete();
         }
 

src/DataDifferencer.php

@@ -188,8 +188,6 @@ protected function getObjectDisplay($object = null)
         }
 
         // Use image tag
-        // TODO Use CMSThumbnail instead to limit max size, blocked by DataDifferencerTest and GC
-        // not playing nice with mocked images
         if ($object instanceof Image) {
             return $object->getTag();
         }

src/RecursivePublishable.php

@@ -201,7 +201,6 @@ public function findOwners($recursive = true, $list = null)
         }
 
         // Build reverse lookup for ownership
-        // @todo - Cache this more intelligently
         $rules = $this->lookupReverseOwners();
 
         // Hand off to recursive method

src/RecursivePublishableHandler.php

@@ -12,7 +12,6 @@ class RecursivePublishableHandler extends Extension
 {
     /**
      * Ensure that non-versioned records are published on save.
-     * @todo Build this action into explicit UX action: https://github.com/silverstripe/silverstripe-versioned/issues/71
      * @param DataObject $record
      */
     public function onAfterSave(DataObject $record)

src/Versioned.php

@@ -2021,7 +2021,6 @@ public function Versions($filter = "", $sort = "", $limit = "", $join = "", $hav
 
         $list = DataObject::get(DataObject::getSchema()->baseDataClass($owner), $filter, $sort, $join, $limit);
         if ($having) {
-            // @todo - This method doesn't exist on DataList
             $list->having($having);
         }
 
@@ -2656,8 +2655,6 @@ public static function get_latest_version($class, $id)
     /**
      * Returns whether the current record is the latest one.
      *
-     * @todo Performance - could do this directly via SQL.
-     *
      * @see get_latest_version()
      * @see latestPublished
      *

tests/php/DataDifferencerTest.php

@@ -70,8 +70,7 @@ public function testArrayValues()
         $obj1v2 = Versioned::get_version(DataDifferencerTest\TestObject::class, $obj1->ID, $afterVersion);
         $differ = new DataDifferencer($obj1v1, $obj1v2);
         $obj1Diff = $differ->diffedData();
-        // TODO Using getter would split up field again, bug only caused by simulating
-        // an array-based value in the first place.
+
         $this->assertContainsIgnoreWhitespace('<del>a,b</del><ins>a</ins>', $obj1Diff->getField('Choices'));
     }