Some more improvements:
authorRoland Haeder <roland@mxchange.org>
Wed, 16 Apr 2014 19:27:16 +0000 (21:27 +0200)
committerRoland Haeder <roland@mxchange.org>
Wed, 16 Apr 2014 19:27:16 +0000 (21:27 +0200)
- 'provider' renamed to 'registration_provider' (GET parameter)
- Added developer-only code (do only use if you are developing on the
  registration and don't want to have registrations, e.g. because you are
  implementing a new registration provider.

Signed-off-by: Roland H├Ąder <roland@mxchange.org>
inc/filter/register_filter.php
inc/filter/wernis_filter.php
inc/libs/register_functions.php
inc/libs/wernis_functions.php
inc/modules/guest/what-register.php
templates/de/html/guest/guest_registration_provider_row.tpl
templates/de/html/guest/guest_wernis_registration_form.tpl
templates/de/html/guest/guest_wernis_registration_rpc_form.tpl

index 1eac558..6ae8442 100644 (file)
@@ -123,7 +123,7 @@ function FILTER_GENERIC_USER_REGISTRATION ($filterData) {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if ((isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'register')) {
+       if ((isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'register')) {
                // Run it
                $filterData['status'] = doGenericUserRegistration();
 
@@ -143,7 +143,7 @@ function FILTER_GENERIC_USER_REGISTRATION_CHECK () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is the registration provider set?
-       if ((isFormSent()) && (isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'register')) {
+       if ((isFormSent()) && (isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'register')) {
                // Check form
                $isFormSent = isRegistrationDataComplete();
 
@@ -161,7 +161,7 @@ function FILTER_GENERIC_USER_REGISTRATION_DONE () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if ((isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'register')) {
+       if ((isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'register')) {
                // Run it
                displayMessage('{--REGISTRATION_DONE--}');
 
@@ -178,7 +178,7 @@ function FILTER_GENERIC_USER_REGISTRATION_FAILED () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if ((isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'register')) {
+       if ((isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'register')) {
                // This should not be reached
                reportBug(__FUNCTION__, __LINE__, 'This filter should not handle it.');
        } // END - if
@@ -192,7 +192,7 @@ function FILTER_GENERIC_USER_REGISTRATION_FORM () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if (((isGetRequestElementSet('provider')) && (getRequestElement('provider') == 'register')) || ((!isGetRequestElementSet('provider')) && (getDefaultRegistrationProvider() == 'register'))) {
+       if (((isGetRequestElementSet('registration_provider')) && (getRequestElement('registration_provider') == 'register')) || ((!isGetRequestElementSet('registration_provider')) && (getDefaultRegistrationProvider() == 'register'))) {
                // Display generic form
                doDisplayGenericUserRegistrationForm();
 
index 0de661b..902473a 100644 (file)
@@ -45,7 +45,7 @@ function FILTER_WERNIS_USER_REGISTRATION ($filterData) {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if ((isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'wernis')) {
+       if ((isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'wernis')) {
                // Run it
                $filterData['status'] = doWernisUserRegistration();
 
@@ -64,7 +64,7 @@ function FILTER_WERNIS_USER_REGISTRATION_DONE () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if ((isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'wernis')) {
+       if ((isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'wernis')) {
                // Run it
 
                // Interrupt filter chain
@@ -82,7 +82,7 @@ function FILTER_WERNIS_USER_REGISTRATION_FAILED () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is generic user registration selected?
-       if ((isPostRequestElementSet('provider')) && (postRequestElement('provider') == 'wernis')) {
+       if ((isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'wernis')) {
                // Run it
 
                // Interrupt filter chain
@@ -100,7 +100,7 @@ function FILTER_WERNIS_USER_REGISTRATION_FORM () {
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Is WDS66 user registration selected?
-       if (((isGetRequestElementSet('provider')) && (getRequestElement('provider') == 'wernis')) || ((!isGetRequestElementSet('provider')) && (getDefaultRegistrationProvider() == 'wernis'))) {
+       if (((isGetRequestElementSet('registration_provider')) && (getRequestElement('registration_provider') == 'wernis')) || ((!isGetRequestElementSet('registration_provider')) && (getDefaultRegistrationProvider() == 'wernis'))) {
                // Run it
                doDisplayWernisUserRegistrationForm();
 
@@ -114,31 +114,42 @@ function FILTER_WERNIS_USER_REGISTRATION_FORM () {
 }
 
 // Filter to run WDS66 API user registation check
-function FILTER_WERNIS_USER_REGISTRATION_CHECK ($filterData) {
+function FILTER_WERNIS_USER_REGISTRATION_CHECK () {
+       // Default is form is not sent
+       $isFormSent = FALSE;
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
+       // Is the registration provider set?
+       if ((isFormSent('wernis_register')) && (isPostRequestElementSet('registration_provider')) && (postRequestElement('registration_provider') == 'wernis')) {
+               // Check form
+               $isFormSent = isRegistrationDataComplete();
+
+               // Interrupt filter chain
+               interruptFilterChain();
+       } // END - if
+
        // Return it
        //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Done!');
-       return $filterData;
+       return $isFormSent;
 }
 
 // Filter for WDS66 API user login
 // @TODO 0% done
 function FILTER_WERNIS_USER_LOGIN ($filterData) {
-       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
+       /* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Return filter data
-       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Done!');
+       /* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Done!');
        return $filterData;
 }
 
 // Filter for WDS66 API user login check
 // @TODO 0% done
 function FILTER_WERNIS_USER_LOGIN_CHECK ($filterData) {
-       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
+       /* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called!');
 
        // Return filter data
-       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Done!');
+       /* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Done!');
        return $filterData;
 }
 
index 0880221..c6f458a 100644 (file)
@@ -328,6 +328,14 @@ function doGenericUserRegistration () {
                return FALSE;
        } // END - if
 
+       // These elements must be set
+       assert(isset($GLOBALS['register_country_row']));
+       assert(isset($GLOBALS['register_country_data']));
+       assert(isset($GLOBALS['register_confirm_hash']));
+
+       // Only comment this in if you develop
+       //* DEVELOPER-CODE: */ return TRUE;
+
        // Create user's account...
        sqlQueryEscaped("INSERT INTO
        `{?_MYSQL_PREFIX?}_user_data`
index b17121d..7a3adca 100644 (file)
@@ -529,13 +529,13 @@ function doDisplayWernisUserRegistrationForm () {
 
 // Finish user registration with WDS66 API
 function doWernisFinishUserRegistration ($challenge, $challengeResponse, $status) {
-       // Check status from GET parameters and stored value
+       // Check status from GET parameters and if the form has been sent
        if (($status == '1') && (isFormSent('wernis_register'))) {
-               // Form has been sent
-               die(__FUNCTION__ . ':' . __LINE__ . ': Reached!');
+               // The form has been sent and challenge is fine
+               die(__FUNCTION__ . ':' . __LINE__ . '<pre>'.print_r(postRequestArray(), TRUE).'</pre>');
        } elseif ($status == '1') {
                // Get mapped data based on challenge
-               $return = getWernisMapedDataFromApiByChallenge($challenge, $status);
+               $return = getWernisMappedDataFromApiByChallenge($challenge, $status);
 
                // Is the array filled?
                if ((count($return['mapped_data']) > 0) && (empty($return['message']))) {
@@ -565,7 +565,7 @@ function doWernisFinishUserRegistration ($challenge, $challengeResponse, $status
 }
 
 // "Getter" for mapped data by calling the API and given challenge and status
-function getWernisMapedDataFromApiByChallenge ($challenge, $status) {
+function getWernisMappedDataFromApiByChallenge ($challenge, $status) {
        // Get stored registration data
        $rows = getWernisRegistrationDataByKey('api_redirect_challenge', $challenge);
 
@@ -799,7 +799,7 @@ function doWernisAuthAccepted ($args) {
        // auth_key and wernis_userid must be set
        assert(isset($args['auth_key']));
        assert(isset($args['wernis_userid']));
-       die(__FUNCTION__ . '<pre>'.print_r($args, TRUE).'</pre>');
+       die(__FUNCTION__ . ':' . __LINE__ . '<pre>' . print_r($args, TRUE) . '</pre>');
 }
 
 //-----------------------------------------------------------------------------
index 6d158ca..c174554 100644 (file)
@@ -64,7 +64,7 @@ if (runFilterChain('check_user_registration') === TRUE) {
        }
 } else {
        // Is a registration provider selected?
-       if (!isGetRequestElementSet('provider')) {
+       if (!isGetRequestElementSet('registration_provider')) {
                // Display selector form
                loadTemplate('guest_registration_select');
        } // END - if
index 53ae0cc..560c0df 100644 (file)
@@ -1 +1 @@
-<li><a href="{%url=modules.php?module=index&amp;what=register&amp;provider=$content[provider_extension]%}">{%pipe,translateRegistrationProviderName=$content[provider_name]%}</a></li>
+<li><a href="{%url=modules.php?module=index&amp;what=register&amp;registration_provider=$content[provider_extension]%}">{%pipe,translateRegistrationProviderName=$content[provider_name]%}</a></li>
index 8fed6a7..2494f4e 100644 (file)
@@ -1,6 +1,6 @@
 <div align="center">
-{%form,formMethodPost,formNameWernis_register=modules.php?module=index&amp;what=register&amp;provider=wernis&amp;status=1&amp;challenge=$content[challenge]&amp;__challenge_response=$content[__challenge_response]%}
-<input type="hidden" name="registration_provider" value="register" />
+{%form,formMethodPost,formNameWernis_register=modules.php?module=index&amp;what=register&amp;registration_provider=wernis&amp;status=1&amp;challenge=$content[challenge]&amp;__challenge_response=$content[__challenge_response]%}
+<input type="hidden" name="registration_provider" value="wernis" />
 <table border="0" cellspacing="0" cellpadding="0" width="640" class="table dashed">
        <tr>
                <td colspan="2" class="table_header bottom">
index 74d606a..04ed087 100644 (file)
@@ -1,6 +1,6 @@
 <div class="para">
 <div align="center">
-       {%form,formMethodPost=modules.php?module=index&amp;what=register&amp;provider=wernis%}
+       {%form,formMethodPost=modules.php?module=index&amp;what=register&amp;registration_provider=wernis%}
        <div class="table dashed">
                <div class="table_header bottom">
                        {--GUEST_WERNIS_REGISTRATION_TITLE--}