Remove the method isValid for Collections

Author: jmcarcell

include/podio/CollectionBase.h

@@ -48,8 +48,6 @@ class CollectionBase {
 
   virtual bool hasID() const = 0;
 
-  virtual bool isValid() const = 0;
-
   /// number of elements in the collection
   virtual size_t size() const = 0;
 

include/podio/UserDataCollection.h

@@ -157,12 +157,6 @@ class UserDataCollection : public CollectionBase {
         getID() != static_cast<uint32_t>(podio::ObjectID::invalid);
   }
 
-  [[deprecated("isValid will be removed, use hasID() if you want to check if it has an ID, otherwise assume the "
-               "collection is valid")]]
-  bool isValid() const override {
-    return hasID();
-  }
-
   /// number of elements in the collection
   size_t size() const override {
     return _vec.size();

include/podio/detail/LinkCollectionImpl.h

@@ -261,12 +261,6 @@ class LinkCollection : public podio::CollectionBase {
         getID() != static_cast<uint32_t>(podio::ObjectID::invalid);
   }
 
-  [[deprecated("isValid will be removed, use hasID() if you want to check if it has an ID, otherwise assume the "
-               "collection is valid")]]
-  bool isValid() const override {
-    return hasID();
-  }
-
   podio::CollectionWriteBuffers getBuffers() override {
     return m_storage.getCollectionBuffers(m_isSubsetColl);
   }

python/templates/Collection.h.jinja2

@@ -161,12 +161,6 @@ public:
         getID() != static_cast<uint32_t>(podio::ObjectID::invalid);
   }
 
-  [[deprecated("isValid will be removed, use hasID() if you want to check if it has an ID, otherwise assume the "
-               "collection is valid")]]
-  bool isValid() const final {
-    return hasID();
-  }
-
   size_t getDatamodelRegistryIndex() const final;
 
   // support for the iterator protocol