fix: SCHWED-4345 fix validity of views too

modules/mol/base/src/atom_view.cc

@@ -167,6 +167,11 @@ unsigned long AtomView::GetHashCode() const
   return reinterpret_cast<unsigned long>(data_.get());
 } 
 
+bool AtomView::IsValid() const
+{
+  return Impl().get()!=0 && Impl()->GetEntity();
+}
+
 }} // ns
 
 

modules/mol/base/src/atom_view.hh

@@ -47,7 +47,7 @@ public:
   operator bool() const { return this->IsValid(); }
   /// \brief check validity of handle
   /// \sa #operator bool()
-  bool IsValid() const { return data_.get()!=0; }
+  bool IsValid() const;
   //@}
   // constructors  
   AtomView();

modules/mol/base/src/chain_view.cc

@@ -483,5 +483,11 @@ unsigned long ChainView::GetHashCode() const
   return reinterpret_cast<unsigned long>(data_.get());
 }
 
+bool ChainView::IsValid() const
+{
+  return Impl().get()!=0 && Impl()->GetEntity();
+}
+
+
 }} // ns
 

modules/mol/base/src/chain_view.hh

@@ -56,7 +56,7 @@ public:
   operator bool() const { return this->IsValid(); }
   /// \brief check validity of handle
   /// \sa #operator bool()
-  bool IsValid() const { return data_.get()!=0; }
+  bool IsValid() const;
   //@}
   
   /// \brief Get parent entity view

modules/mol/base/src/residue_view.cc

@@ -291,4 +291,9 @@ unsigned long ResidueView::GetHashCode() const
   return reinterpret_cast<unsigned long>(data_.get());
 }
 
+bool ResidueView::IsValid() const
+{
+  return Impl().get()!=0 && Impl()->GetEntity();
+}
+
 }} //ns

modules/mol/base/src/residue_view.hh

@@ -65,7 +65,7 @@ public:
   operator bool() const { return this->IsValid(); }
   /// \brief check validity of handle
   /// \sa #operator bool()
-  bool IsValid() const { return data_.get()!=0; }
+  bool IsValid() const;
   //@}
   
   /// \name internal