[GeoMechanicsApplication] Fix constructor destructor code smells in c…

…ustom elements (#12249)
KratosMultiphysics · Apr 5, 2024 · 7bf35ed · 7bf35ed
1 parent 6370d5d
commit 7bf35ed
Show file tree

Hide file tree

Showing 27 changed files with 46 additions and 73 deletions.
diff --git a/applications/GeoMechanicsApplication/custom_elements/U_Pw_small_strain_FIC_element.hpp b/applications/GeoMechanicsApplication/custom_elements/U_Pw_small_strain_FIC_element.hpp
@@ -54,8 +54,10 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) UPwSmallStrainFICElement
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    UPwSmallStrainFICElement(IndexType NewId = 0) : UPwSmallStrainElement<TDim, TNumNodes>(NewId) {}
+    explicit UPwSmallStrainFICElement(IndexType NewId = 0)
+        : UPwSmallStrainElement<TDim, TNumNodes>(NewId)
+    {
+    }
 
     /// Constructor using an array of nodes
     UPwSmallStrainFICElement(IndexType NewId, const NodesArrayType& ThisNodes, std::unique_ptr<StressStatePolicy> pStressStatePolicy)
@@ -78,8 +80,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) UPwSmallStrainFICElement
     {
     }
 
-    /// Destructor
-    ~UPwSmallStrainFICElement() override {}
+    ~UPwSmallStrainFICElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/U_Pw_small_strain_interface_element.hpp b/applications/GeoMechanicsApplication/custom_elements/U_Pw_small_strain_interface_element.hpp
@@ -45,8 +45,10 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) UPwSmallStrainInterfaceElement
     using UPwBaseElement<TDim, TNumNodes>::CalculateDerivativesOnInitialConfiguration;
     using UPwBaseElement<TDim, TNumNodes>::mThisIntegrationMethod;
 
-    /// Default Constructor
-    UPwSmallStrainInterfaceElement(IndexType NewId = 0) : UPwBaseElement<TDim, TNumNodes>(NewId) {}
+    explicit UPwSmallStrainInterfaceElement(IndexType NewId = 0)
+        : UPwBaseElement<TDim, TNumNodes>(NewId)
+    {
+    }
 
     /// Constructor using an array of nodes
     UPwSmallStrainInterfaceElement(IndexType                          NewId,

diff --git a/...ions/GeoMechanicsApplication/custom_elements/U_Pw_small_strain_link_interface_element.hpp b/...ions/GeoMechanicsApplication/custom_elements/U_Pw_small_strain_link_interface_element.hpp
@@ -72,8 +72,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) UPwSmallStrainLinkInterfaceElement
     {
     }
 
-    // Destructor
-    ~UPwSmallStrainLinkInterfaceElement() override {}
+    ~UPwSmallStrainLinkInterfaceElement() = default;
 
     //----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/U_Pw_updated_lagrangian_FIC_element.hpp b/applications/GeoMechanicsApplication/custom_elements/U_Pw_updated_lagrangian_FIC_element.hpp
@@ -118,8 +118,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) UPwUpdatedLagrangianFICElement
     {
     }
 
-    /// Destructor
-    ~UPwUpdatedLagrangianFICElement() override {}
+    ~UPwUpdatedLagrangianFICElement() = default;
 
     /**
      * @brief Creates a new element

diff --git a/applications/GeoMechanicsApplication/custom_elements/U_Pw_updated_lagrangian_element.hpp b/applications/GeoMechanicsApplication/custom_elements/U_Pw_updated_lagrangian_element.hpp
@@ -117,8 +117,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) UPwUpdatedLagrangianElement
     {
     }
 
-    /// Destructor
-    ~UPwUpdatedLagrangianElement() override {}
+    ~UPwUpdatedLagrangianElement() = default;
 
     int Check(const ProcessInfo& rCurrentProcessInfo) const override;
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/drained_U_Pw_small_strain_element.hpp b/applications/GeoMechanicsApplication/custom_elements/drained_U_Pw_small_strain_element.hpp
@@ -50,8 +50,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) DrainedUPwSmallStrainElement
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    DrainedUPwSmallStrainElement(IndexType NewId = 0)
+    explicit DrainedUPwSmallStrainElement(IndexType NewId = 0)
         : UPwSmallStrainElement<TDim, TNumNodes>(NewId)
     {
     }
@@ -81,8 +80,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) DrainedUPwSmallStrainElement
     {
     }
 
-    /// Destructor
-    ~DrainedUPwSmallStrainElement() override {}
+    ~DrainedUPwSmallStrainElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
     Element::Pointer Create(IndexType               NewId,

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cable_element.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_cable_element.hpp
@@ -54,7 +54,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCableElement : public GeoTrussEle
     using GeoTrussElement<TDim, TNumNodes>::mInternalStressesFinalizedPrevious;
     using GeoTrussElement<TDim, TNumNodes>::mInternalStresses;
 
-    GeoCableElement(){};
+    GeoCableElement() = default;
     GeoCableElement(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoCableElement(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_2D2N.cpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_2D2N.cpp
@@ -49,9 +49,6 @@ Element::Pointer GeoCrBeamElement2D2N::Create(IndexType               NewId,
     return Kratos::make_intrusive<GeoCrBeamElement2D2N>(NewId, pGeom, pProperties);
 }
 
-//----------------------------------------------------------------------------------------------------
-GeoCrBeamElement2D2N::~GeoCrBeamElement2D2N() {}
-
 //----------------------------------------------------------------------------------------------------
 void GeoCrBeamElement2D2N::InitializeSolutionStep(const ProcessInfo& rCurrentProcessInfo)
 {

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_2D2N.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_2D2N.hpp
@@ -51,11 +51,11 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCrBeamElement2D2N : public CrBeam
     using EquationIdVectorType = BaseType::EquationIdVectorType;
     using DofsVectorType       = BaseType::DofsVectorType;
 
-    GeoCrBeamElement2D2N(){};
+    GeoCrBeamElement2D2N() = default;
     GeoCrBeamElement2D2N(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoCrBeamElement2D2N(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 
-    ~GeoCrBeamElement2D2N() override;
+    ~GeoCrBeamElement2D2N() = default;
 
     /**
      * @brief Creates a new element

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_3D2N.cpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_3D2N.cpp
@@ -49,9 +49,6 @@ Element::Pointer GeoCrBeamElement3D2N::Create(IndexType               NewId,
     return Kratos::make_intrusive<GeoCrBeamElement3D2N>(NewId, pGeom, pProperties);
 }
 
-//----------------------------------------------------------------------------------------------------
-GeoCrBeamElement3D2N::~GeoCrBeamElement3D2N() {}
-
 //-------------------------------------------------------------------------------------------------
 void GeoCrBeamElement3D2N::Initialize(const ProcessInfo& rCurrentProcessInfo)
 {

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_3D2N.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_3D2N.hpp
@@ -51,11 +51,11 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCrBeamElement3D2N : public CrBeam
     using EquationIdVectorType = BaseType::EquationIdVectorType;
     using DofsVectorType       = BaseType::DofsVectorType;
 
-    GeoCrBeamElement3D2N(){};
+    GeoCrBeamElement3D2N() = default;
     GeoCrBeamElement3D2N(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoCrBeamElement3D2N(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 
-    ~GeoCrBeamElement3D2N() override;
+    ~GeoCrBeamElement3D2N() = default;
 
     /**
      * @brief Creates a new element

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_2D2N.cpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_2D2N.cpp
@@ -52,9 +52,6 @@ Element::Pointer GeoCrBeamElementLinear2D2N::Create(IndexType               NewI
     return Kratos::make_intrusive<GeoCrBeamElementLinear2D2N>(NewId, pGeom, pProperties);
 }
 
-//-------------------------------------------------------------------------------------------------
-GeoCrBeamElementLinear2D2N::~GeoCrBeamElementLinear2D2N() {}
-
 //----------------------------------------------------------------------------------------
 void GeoCrBeamElementLinear2D2N::ResetConstitutiveLaw()
 {

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_2D2N.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_2D2N.hpp
@@ -72,7 +72,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCrBeamElementLinear2D2N : public
                             NodesArrayType const&   ThisNodes,
                             PropertiesType::Pointer pProperties) const override;
 
-    ~GeoCrBeamElementLinear2D2N() override;
+    ~GeoCrBeamElementLinear2D2N() = default;
 
     void Initialize(const ProcessInfo& rCurrentProcessInfo) override;
 
@@ -97,7 +97,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCrBeamElementLinear2D2N : public
     void ResetConstitutiveLaw() override;
 
 protected:
-    GeoCrBeamElementLinear2D2N(){};
+    GeoCrBeamElementLinear2D2N() = default;
 
     Vector mInternalGlobalForcesFinalized         = ZeroVector(msElementSize);
     Vector mInternalGlobalForcesFinalizedPrevious = ZeroVector(msElementSize);

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_3D2N.cpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_3D2N.cpp
@@ -18,7 +18,6 @@
 #include "custom_elements/geo_cr_beam_element_linear_3D2N.hpp"
 #include "custom_utilities/static_condensation_utility.h"
 #include "geo_mechanics_application_variables.h"
-#include "geo_mechanics_application_variables.h"
 #include "includes/define.h"
 
 namespace Kratos
@@ -53,9 +52,6 @@ Element::Pointer GeoCrBeamElementLinear3D2N::Create(IndexType               NewI
     return Kratos::make_intrusive<GeoCrBeamElementLinear3D2N>(NewId, pGeom, pProperties);
 }
 
-//-------------------------------------------------------------------------------------------------
-GeoCrBeamElementLinear3D2N::~GeoCrBeamElementLinear3D2N() = default;
-
 //----------------------------------------------------------------------------------------
 void GeoCrBeamElementLinear3D2N::ResetConstitutiveLaw()
 {

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_3D2N.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_cr_beam_element_linear_3D2N.hpp
@@ -49,11 +49,11 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCrBeamElementLinear3D2N : public
     using EquationIdVectorType = BaseType::EquationIdVectorType;
     using DofsVectorType       = BaseType::DofsVectorType;
 
-    GeoCrBeamElementLinear3D2N(){};
+    GeoCrBeamElementLinear3D2N() = default;
     GeoCrBeamElementLinear3D2N(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoCrBeamElementLinear3D2N(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 
-    ~GeoCrBeamElementLinear3D2N() override;
+    ~GeoCrBeamElementLinear3D2N() = default;
 
     /**
      * @brief Creates a new element

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_curved_beam_element.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_curved_beam_element.hpp
@@ -63,8 +63,10 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCurvedBeamElement
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    GeoCurvedBeamElement(IndexType NewId = 0) : GeoStructuralBaseElement<TDim, TNumNodes>(NewId) {}
+    explicit GeoCurvedBeamElement(IndexType NewId = 0)
+        : GeoStructuralBaseElement<TDim, TNumNodes>(NewId)
+    {
+    }
 
     /// Constructor using an array of nodes
     GeoCurvedBeamElement(IndexType NewId, const NodesArrayType& ThisNodes)
@@ -85,8 +87,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoCurvedBeamElement
         mThisIntegrationMethod = this->GetIntegrationMethod();
     }
 
-    /// Destructor
-    virtual ~GeoCurvedBeamElement() {}
+    ~GeoCurvedBeamElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_linear_truss_element.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_linear_truss_element.hpp
@@ -57,7 +57,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoLinearTrussElement
 
     using GeoTrussElementBase<TDim, TNumNodes>::mpConstitutiveLaw;
 
-    GeoLinearTrussElement(){};
+    GeoLinearTrussElement() = default;
     GeoLinearTrussElement(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoLinearTrussElement(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_linear_truss_element_base.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_linear_truss_element_base.hpp
@@ -53,7 +53,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoTrussElementLinearBase
 
     using GeoTrussElementBase<TDim, TNumNodes>::mpConstitutiveLaw;
 
-    GeoTrussElementLinearBase(){};
+    GeoTrussElementLinearBase() = default;
     GeoTrussElementLinearBase(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoTrussElementLinearBase(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_structural_base_element.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_structural_base_element.hpp
@@ -34,8 +34,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoStructuralBaseElement : public El
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    GeoStructuralBaseElement(IndexType NewId = 0) : Element(NewId) {}
+    explicit GeoStructuralBaseElement(IndexType NewId = 0) : Element(NewId) {}
 
     /// Constructor using an array of nodes
     GeoStructuralBaseElement(IndexType NewId, const NodesArrayType& ThisNodes)
@@ -56,8 +55,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoStructuralBaseElement : public El
         mThisIntegrationMethod = this->GetIntegrationMethod();
     }
 
-    /// Destructor
-    virtual ~GeoStructuralBaseElement() {}
+    ~GeoStructuralBaseElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_truss_element.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_truss_element.hpp
@@ -56,7 +56,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoTrussElement : public GeoTrussEle
 
     KRATOS_CLASS_INTRUSIVE_POINTER_DEFINITION(GeoTrussElement);
 
-    GeoTrussElement(){};
+    GeoTrussElement() = default;
     GeoTrussElement(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoTrussElement(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/geo_truss_element_base.hpp b/applications/GeoMechanicsApplication/custom_elements/geo_truss_element_base.hpp
@@ -57,7 +57,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) GeoTrussElementBase : public Element
     using FullDofMatrixType    = BoundedMatrix<double, TDim * TNumNodes, TDim * TNumNodes>;
     using FullDofVectorType    = BoundedVector<double, TDim * TNumNodes>;
 
-    GeoTrussElementBase(){};
+    GeoTrussElementBase() = default;
     GeoTrussElementBase(IndexType NewId, GeometryType::Pointer pGeometry);
     GeoTrussElementBase(IndexType NewId, GeometryType::Pointer pGeometry, PropertiesType::Pointer pProperties);
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/steady_state_Pw_element.hpp b/applications/GeoMechanicsApplication/custom_elements/steady_state_Pw_element.hpp
@@ -54,8 +54,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) SteadyStatePwElement : public Transi
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    SteadyStatePwElement(IndexType NewId = 0) : BaseType(NewId) {}
+    explicit SteadyStatePwElement(IndexType NewId = 0) : BaseType(NewId) {}
 
     /// Constructor using an array of nodes
     SteadyStatePwElement(IndexType NewId, const NodesArrayType& ThisNodes, std::unique_ptr<StressStatePolicy> pStressStatePolicy)
@@ -78,8 +77,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) SteadyStatePwElement : public Transi
     {
     }
 
-    /// Destructor
-    ~SteadyStatePwElement() override {}
+    ~SteadyStatePwElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/steady_state_Pw_interface_element.hpp b/applications/GeoMechanicsApplication/custom_elements/steady_state_Pw_interface_element.hpp
@@ -56,8 +56,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) SteadyStatePwInterfaceElement
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    SteadyStatePwInterfaceElement(IndexType NewId = 0)
+    explicit SteadyStatePwInterfaceElement(IndexType NewId = 0)
         : TransientPwInterfaceElement<TDim, TNumNodes>(NewId)
     {
     }
@@ -87,8 +86,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) SteadyStatePwInterfaceElement
     {
     }
 
-    /// Destructor
-    ~SteadyStatePwInterfaceElement() override {}
+    ~SteadyStatePwInterfaceElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/steady_state_Pw_piping_element.hpp b/applications/GeoMechanicsApplication/custom_elements/steady_state_Pw_piping_element.hpp
@@ -56,8 +56,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) SteadyStatePwPipingElement
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    SteadyStatePwPipingElement(IndexType NewId = 0)
+    explicit SteadyStatePwPipingElement(IndexType NewId = 0)
         : SteadyStatePwInterfaceElement<TDim, TNumNodes>(NewId)
     {
     }
@@ -87,8 +86,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) SteadyStatePwPipingElement
     {
     }
 
-    /// Destructor
-    ~SteadyStatePwPipingElement() override {}
+    ~SteadyStatePwPipingElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/transient_Pw_element.hpp b/applications/GeoMechanicsApplication/custom_elements/transient_Pw_element.hpp
@@ -54,8 +54,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) TransientPwElement : public UPwSmall
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
-    /// Default Constructor
-    TransientPwElement(IndexType NewId = 0) : BaseType(NewId) {}
+    explicit TransientPwElement(IndexType NewId = 0) : BaseType(NewId) {}
 
     /// Constructor using an array of nodes
     TransientPwElement(IndexType NewId, const NodesArrayType& ThisNodes, std::unique_ptr<StressStatePolicy> pStressStatePolicy)
@@ -78,8 +77,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) TransientPwElement : public UPwSmall
     {
     }
 
-    /// Destructor
-    ~TransientPwElement() override {}
+    ~TransientPwElement() = default;
 
     ///----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 

diff --git a/applications/GeoMechanicsApplication/custom_elements/transient_Pw_interface_element.hpp b/applications/GeoMechanicsApplication/custom_elements/transient_Pw_interface_element.hpp
@@ -51,8 +51,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) TransientPwInterfaceElement
     using InterfaceElementVariables = typename BaseType::InterfaceElementVariables;
     using SFGradAuxVariables        = typename BaseType::SFGradAuxVariables;
 
-    /// Default Constructor
-    TransientPwInterfaceElement(IndexType NewId = 0)
+    explicit TransientPwInterfaceElement(IndexType NewId = 0)
         : UPwSmallStrainInterfaceElement<TDim, TNumNodes>(NewId)
     {
     }
@@ -82,7 +81,7 @@ class KRATOS_API(GEO_MECHANICS_APPLICATION) TransientPwInterfaceElement
     {
     }
 
-    ~TransientPwInterfaceElement() override                                    = default;
+    ~TransientPwInterfaceElement()                                             = default;
     TransientPwInterfaceElement(const TransientPwInterfaceElement&)            = delete;
     TransientPwInterfaceElement& operator=(const TransientPwInterfaceElement&) = delete;
     TransientPwInterfaceElement(TransientPwInterfaceElement&&)                 = delete;