fix possible double connect of the Collider m_debugDisplayDataChangedEvent (#1518)

Signed-off-by: John <jonawals@amazon.com>

Gems/PhysX/Code/Editor/DebugDraw.cpp

@@ -200,10 +200,8 @@ namespace PhysX
 
         void Collider::Disconnect()
         {
-            if (AzToolsFramework::ViewportInteraction::ViewportSettingsNotificationBus::Handler::BusIsConnected())
+            m_debugDisplayDataChangedEvent.Disconnect();
-            {
+            AzToolsFramework::ViewportInteraction::ViewportSettingsNotificationBus::Handler::BusDisconnect();
-                AzToolsFramework::ViewportInteraction::ViewportSettingsNotificationBus::Handler::BusDisconnect();
-            }
             AzToolsFramework::EntitySelectionEvents::Bus::Handler::BusDisconnect();
             AzFramework::EntityDebugDisplayEventBus::Handler::BusDisconnect();
             m_displayCallback = nullptr;