Add remaining deprecated onXXX handlers to GattServer::EventHandler

Author: AGlass0fMilk

connectivity/FEATURE_BLE/include/ble/GattServer.h

@@ -132,6 +132,41 @@ class GattServer {
             (void)params;
         }
 
+        /**
+         * Function invoked when a client writes an attribute
+         *
+         * @note params has a temporary scope and should be copied by the
+         * application if needed later
+         */
+        virtual void onDataWritten(const GattWriteCallbackParams *params) {
+            (void)params;
+        }
+
+        /**
+         * Function invoked when a client reads an attribute
+         *
+         * @note  This functionality may not be available on all underlying stacks.
+         * Application code may work around that limitation by monitoring read
+         * requests instead of read events.
+         *
+         * @note params has a temporary scope and should be copied by the
+         * application if needed later
+         *
+         * @see GattCharacteristic::setReadAuthorizationCallback()
+         * @see isOnDataReadAvailable().
+         */
+        virtual void onDataRead(const GattReadCallbackParams *params) {
+            (void)params;
+        }
+
+        /**
+         * Function invoked when the GattServer instance is about
+         * to be shut down. This can result in a call to reset() or BLE::reset().
+         */
+        virtual void onShutdown(const GattServer *server) {
+            (void)server;
+        }
+
         /**
          * Function invoked when the client has subscribed to characteristic updates
          *

connectivity/FEATURE_BLE/source/cordio/source/GattServerImpl.cpp

@@ -878,6 +878,11 @@ GapAdvertisingData::Appearance GattServer::getAppearance()
 ble_error_t GattServer::reset(ble::GattServer* server)
 {
     /* Notify that the instance is about to shutdown */
+    if(eventHandler) {
+        eventHandler->onShutdown(server);
+    }
+
+    // Execute callbacks added with deprecated API
     shutdownCallChain.call(server);
     shutdownCallChain.clear();
 
@@ -1480,11 +1485,21 @@ GattServer::EventHandler *GattServer::getEventHandler()
 
 void GattServer::handleDataWrittenEvent(const GattWriteCallbackParams *params)
 {
+    if(eventHandler) {
+        eventHandler->onDataWritten(params);
+    }
+
+    // Execute callbacks added with deprecated API
     dataWrittenCallChain.call(params);
 }
 
 void GattServer::handleDataReadEvent(const GattReadCallbackParams *params)
 {
+    if(eventHandler) {
+        eventHandler->onDataRead(params);
+    }
+
+    // Execute callbacks added with deprecated API
     dataReadCallChain.call(params);
 }