[PartDesign] Remove useless function overriding

Double-click is already handled by PartDesign::ViewProvider::doubleClicked()
 Overridings in Loft & Pipe brings no specific behavior
 By removing overridings, behavioral consistency is improved both in ...
 ... undo commands & body activation management when PartDesign items ...
 ... are double-clicked

src/Mod/PartDesign/Gui/ViewProviderLoft.cpp

@@ -81,11 +81,6 @@ void ViewProviderLoft::setupContextMenu(QMenu* menu, QObject* receiver, const ch
     PartDesignGui::ViewProvider::setupContextMenu(menu, receiver, member);
 }
 
-bool ViewProviderLoft::doubleClicked(void)
-{
-    return PartDesignGui::setEdit(pcObject);
-}
-
 bool ViewProviderLoft::setEdit(int ModNum)
 {
     if (ModNum == ViewProvider::Default)

src/Mod/PartDesign/Gui/ViewProviderLoft.h

@@ -41,7 +41,6 @@ public:
     /// grouping handling 
     std::vector<App::DocumentObject*> claimChildren(void)const;
     void setupContextMenu(QMenu*, QObject*, const char*);
-    bool doubleClicked();
 
     virtual bool onDelete(const std::vector<std::string> &);
     void highlightReferences(const bool on, bool auxiliary);

src/Mod/PartDesign/Gui/ViewProviderPipe.cpp

@@ -89,11 +89,6 @@ void ViewProviderPipe::setupContextMenu(QMenu* menu, QObject* receiver, const ch
     PartDesignGui::ViewProvider::setupContextMenu(menu, receiver, member);
 }
 
-bool ViewProviderPipe::doubleClicked(void)
-{
-    return PartDesignGui::setEdit(pcObject);
-}
-
 bool ViewProviderPipe::setEdit(int ModNum) {
     if (ModNum == ViewProvider::Default )
         setPreviewDisplayMode(true);

src/Mod/PartDesign/Gui/ViewProviderPipe.h

@@ -49,7 +49,6 @@ public:
     /// grouping handling 
     std::vector<App::DocumentObject*> claimChildren(void)const;
     void setupContextMenu(QMenu*, QObject*, const char*);
-    bool doubleClicked();
 
     virtual bool onDelete(const std::vector<std::string> &);
     void highlightReferences(Reference mode, bool on);