Server refactoring: Added sample native service module; fixed typo

Author: dmarteau

CMakeLists.txt

@@ -445,6 +445,8 @@ IF (WIN32)
   SET (DEFAULT_PLUGIN_SUBDIR  plugins)
   SET (DEFAULT_INCLUDE_SUBDIR include)
 
+  SET (DEFAULT_SERVER_MODULE_SUBDIR server)
+
   IF (MSVC)
     SET (DEFAULT_BIN_SUBDIR bin)
     SET (DEFAULT_CGIBIN_SUBDIR bin)
@@ -518,6 +520,9 @@ ELSE (WIN32)
     SET (DEFAULT_PLUGIN_SUBDIR   ../PlugIns/qgis)
     SET (QGIS_PLUGIN_SUBDIR_REV  ../../MacOS)
     SET (DEFAULT_INCLUDE_SUBDIR  include/qgis)
+
+    SET (DEFAULT_SERVER_MODULE_SUBDIR ../PlugIns/server)
+
     # path for framework references when running from build directory
     # changed later to reference in-app resources upon install
     SET (CMAKE_INSTALL_NAME_DIR ${CMAKE_BINARY_DIR}/output/lib)
@@ -541,6 +546,8 @@ ELSE (WIN32)
     SET (DEFAULT_LIBEXEC_SUBDIR lib${LIB_SUFFIX}/qgis)
     SET (DEFAULT_PLUGIN_SUBDIR  lib${LIB_SUFFIX}/qgis/plugins)
     SET (DEFAULT_INCLUDE_SUBDIR include/qgis)
+
+    SET (DEFAULT_SERVER_MODULE_SUBDIR lib${LIB_SUFFIX}/qgis/server)
   ENDIF (APPLE)
 
 ENDIF (WIN32)
@@ -587,6 +594,8 @@ SET (QGIS_DATA_SUBDIR    ${DEFAULT_DATA_SUBDIR}    CACHE STRING "Subdirectory wh
 SET (QGIS_PLUGIN_SUBDIR  ${DEFAULT_PLUGIN_SUBDIR}  CACHE STRING "Subdirectory where plugins will be installed")
 SET (QGIS_INCLUDE_SUBDIR ${DEFAULT_INCLUDE_SUBDIR} CACHE STRING "Subdirectory where header files will be installed")
 
+SET (QGIS_SERVER_MODULE_SUBDIR  ${DEFAULT_SERVER_MODULE_SUBDIR}  CACHE STRING "Subdirectory where server modules will be installed")
+
 # mark *_SUBDIR variables as advanced as this is not something
 # that an average user would use
 MARK_AS_ADVANCED (QGIS_BIN_SUBDIR QGIS_CGIBIN_SUBDIR QGIS_LIB_SUBDIR QGIS_LIBEXEC_SUBDIR QGIS_DATA_SUBDIR QGIS_PLUGIN_SUBDIR QGIS_INCLUDE_SUBDIR)
@@ -600,6 +609,8 @@ SET (QGIS_DATA_DIR    ${QGIS_DATA_SUBDIR})
 SET (QGIS_PLUGIN_DIR  ${QGIS_PLUGIN_SUBDIR})
 SET (QGIS_INCLUDE_DIR ${QGIS_INCLUDE_SUBDIR})
 
+SET (QGIS_SERVER_MODULE_DIR ${QGIS_SERVER_MODULE_SUBDIR})
+
 # set the default locations where the targets (executables, libraries) will land when compiled
 # this is to allow running qgis from the source tree without having to actually do a "make install"
 SET (QGIS_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/output)

python/server/qgsserverrequest.sip

@@ -37,7 +37,7 @@ class QgsServerRequest
        /**
         * Constructor
         *
-        * @param url the lurl string
+        * @param url the url string
         * @param method the request method
         */
        QgsServerRequest( const QString& url, Method method );

python/server/qgsserverresponse.sip

@@ -38,7 +38,7 @@ class QgsServerResponse
 
        /** Set Header entry 
         *  Add Header entry to the response 
-        *  Note that it is usually an error to set Header after writng data
+        *  Note that it is usually an error to set Header after writing data
         */
         virtual void setHeader( const QString& key, const QString& value ) = 0;
 
@@ -49,8 +49,8 @@ class QgsServerResponse
 
         /**
          * Send error
-         * This method delegate error handling at the server level. This is different
-         * from calling setReturnCodei() along with and a specific response body.
+         * This method delegates error handling at the server level. This is different
+         * from calling setReturnCode() along with and a specific response body.
          * @param code HHTP return code value
          * @param message An informative error message
          */
@@ -65,9 +65,9 @@ class QgsServerResponse
 
        /**
         * Write chunk of data
-        * They is a convenient method that will write directly to the 
+        * This is a convenient method that will write directly to the 
         * underlying I/O device
-        * @return the number of bytes that were actually writtene
+        * @return the number of bytes that were actually written
         */ 
         virtual qint64 write(const QByteArray& byteArray );
 

python/server/qgsservicemodule.sip

@@ -22,7 +22,7 @@
  * QgsServiceModule
  * Class defining the service module interface for QGIS server services
  *
- * This class act as a service registrar for services. 
+ * This class acts as a service registrar for services. 
  *
  * For dynamic modules, a QgsServiceModule instance is returned from the QGS_ServiceModule_Init() entry point 
  */

python/server/qgsserviceregistry.sip

@@ -20,7 +20,7 @@
 /**
  * \ingroup server
  * QgsServiceRegistry
- * Class defining the cegistry manager for QGIS server services
+ * Class defining the registry manager for QGIS server services
  *
  * This class provides methods for registering and retrieving 
  * services.

src/server/CMakeLists.txt

@@ -14,6 +14,8 @@ IF (ENABLE_TESTS)
    ADD_DEFINITIONS(-DENABLE_MS_TESTS=1)
 ENDIF (ENABLE_TESTS)
 
+ADD_SUBDIRECTORY(services)
+
 ########################################################
 # Files
 
@@ -49,7 +51,6 @@ SET ( qgis_mapserv_SRCS
   qgssldconfigparser.cpp
   qgsconfigparserutils.cpp
   qgsserver.cpp
-#XXX https://github.com/qgis/QGIS-Enhancement-Proposals/issues/74
   qgsservice.cpp
   qgsservicemodule.cpp
   qgsserviceloader.cpp

src/server/qgsserverrequest.h

@@ -28,7 +28,7 @@
  */
 
 // Note about design: this intreface must be passed along to python and thus signatures methods must be 
-// compatible with pyQGS/pyQT api and rules.
+// compatible with pyQGIS/pyQT api and rules.
 
 class SERVER_EXPORT QgsServerRequest
 {
@@ -41,7 +41,7 @@ class SERVER_EXPORT QgsServerRequest
        /**
         * Constructor
         *
-        * @param url the lurl string
+        * @param url the url string
         * @param method the request method
         */
        QgsServerRequest( const QString& url, Method method );

src/server/qgsserverresponse.h

@@ -44,7 +44,7 @@ class SERVER_EXPORT QgsServerResponse
 
        /** Set Header entry 
         *  Add Header entry to the response 
-        *  Note that it is usually an error to set Header after writng data
+        *  Note that it is usually an error to set Header after writing data
         */
         virtual void setHeader( const QString& key, const QString& value ) = 0;
 
@@ -55,8 +55,8 @@ class SERVER_EXPORT QgsServerResponse
 
         /**
          * Send error
-         * This method delegate error handling at the server level. This is different
-         * from calling setReturnCodei() along with and a specific response body.
+         * This method delegates error handling at the server level. This is different
+         * from calling setReturnCode() along with and a specific response body.
          * @param code HHTP return code value
          * @param message An informative error message
          */
@@ -70,10 +70,10 @@ class SERVER_EXPORT QgsServerResponse
         virtual void write( const QString& data );
 
        /**
-        * Write chunk af data
+        * Write chunk of data
         * This is a convenient method that will write directly
         * to the underlying I/O device
-        * @creturn the number of bytes that were actually written
+        * @return the number of bytes that were actually written
         */
         virtual qint64 write( const QByteArray &byteArray );
 
@@ -84,7 +84,7 @@ class SERVER_EXPORT QgsServerResponse
         * to the underlying I/O device
         * @return the number of bytes written
         *
-        *  @note not available in pything bindings
+        *  @note not available in python bindings
         */
         virtual qint64 write( const char* data, qint64 maxsize);
 
@@ -95,7 +95,7 @@ class SERVER_EXPORT QgsServerResponse
         * to the underlying I/O device
         * @return the number of bytes written
         *
-        * @note not available in pything bindings
+        * @note not available in python bindings
         */
         virtual qint64 write( const char* data );
 

src/server/qgsservicemodule.h

@@ -28,7 +28,7 @@ class QgsServiceRegistry;
  * QgsServiceModule
  * Class defining the service module interface for QGIS server services
  *
- * This class act as a service registrar for services. 
+ * This class acts as a service registrar for services. 
  *
  * For dynamic modules, a QgsServiceModule instance is returned from the QGS_ServiceModule_Init() entry point 
  */

src/server/qgsserviceregistry.h

@@ -33,7 +33,7 @@ class QgsService;
 /**
  * \ingroup server
  * QgsServiceRegistry
- * Class defining the cegistry manager for QGIS server services
+ * Class defining the registry manager for QGIS server services
  *
  * This class provides methods for registering and retrieving 
  * services.
@@ -59,7 +59,7 @@ class SERVER_EXPORT QgsServiceRegistry
      * @param version the version string (optional) 
      * @return QgsService
      *
-     * If the version is not provided the higher version of the service is rerturnod 
+     * If the version is not provided the higher version of the service is returned 
      */
     QgsService* getService( const QString& name, const QString& version = QString() );