Changeset:
        553b69e54c49
        
https://sourceforge.net/p/mrbs/hg-code/ci/553b69e54c495a28f5309ba20a671bb315d7777a
Author:
        Campbell Morrison <cimorri...@hg.code.sf.net>
Date:
        Thu Mar 16 11:11:16 2017 +0000
Log message:

Restructured code

diffstat:

 web/session/functions_session.inc |  9 ++++++++-
 web/session/session_cookie.inc    |  8 ++++----
 web/session/session_joomla.inc    |  8 ++++----
 web/session/session_php.inc       |  8 ++++----
 web/session/session_wordpress.inc |  6 +++---
 5 files changed, 23 insertions(+), 16 deletions(-)

diffs (149 lines):

diff -r 60eca9fd7e31 -r 553b69e54c49 web/session/functions_session.inc
--- a/web/session/functions_session.inc Thu Mar 16 10:49:36 2017 +0000
+++ b/web/session/functions_session.inc Thu Mar 16 11:11:16 2017 +0000
@@ -1,11 +1,18 @@
 <?php
 namespace MRBS;
 
+
+function print_login_failed()
+{
+  echo "<p>" . get_vocab('unknown_user') . "</p>\n";
+}
+
+
 /*
   Display the login form. 
   Will eventually return to $target_url with query string returl=$returl
 */
-function printLoginForm($action, $target_url, $returl)
+function print_login_form($action, $target_url, $returl)
 {
   $html = '';
   
diff -r 60eca9fd7e31 -r 553b69e54c49 web/session/session_cookie.inc
--- a/web/session/session_cookie.inc    Thu Mar 16 10:49:36 2017 +0000
+++ b/web/session/session_cookie.inc    Thu Mar 16 11:11:16 2017 +0000
@@ -75,8 +75,8 @@
     if (($valid_username = authValidateUser($NewUserName, $NewUserPassword)) 
=== false)
     {
       print_header();
-      echo "<p>".get_vocab('unknown_user')."</p>\n";
-      printLoginForm(this_page(), $target_url, $returl);
+      print_login_failed();
+      print_login_form(this_page(), $target_url, $returl);
       exit();
     }
     else
@@ -151,7 +151,7 @@
 if (isset($Action) && ($Action == "QueryName"))
 {
   print_header();
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
   exit();
 }
 
@@ -175,7 +175,7 @@
   {
     $target_url = $target_url . "?" . $QUERY_STRING;
   }
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
 
   exit();
 }
diff -r 60eca9fd7e31 -r 553b69e54c49 web/session/session_joomla.inc
--- a/web/session/session_joomla.inc    Thu Mar 16 10:49:36 2017 +0000
+++ b/web/session/session_joomla.inc    Thu Mar 16 11:11:16 2017 +0000
@@ -44,8 +44,8 @@
     if (!$mainframe->login($credentials))
     {
       print_header();
-      echo "<p>".get_vocab('unknown_user')."</p>\n";
-      printLoginForm(this_page(), $target_url, $returl);
+      print_login_failed();
+      print_login_form(this_page(), $target_url, $returl);
       exit();
     }
   }
@@ -83,7 +83,7 @@
 if (isset($Action) && ($Action == "QueryName"))
 {
   print_header();
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
   exit();
 }
 
@@ -107,7 +107,7 @@
   {
     $target_url = $target_url . "?" . $QUERY_STRING;
   }
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
 
   exit();
 }
diff -r 60eca9fd7e31 -r 553b69e54c49 web/session/session_php.inc
--- a/web/session/session_php.inc       Thu Mar 16 10:49:36 2017 +0000
+++ b/web/session/session_php.inc       Thu Mar 16 11:11:16 2017 +0000
@@ -103,8 +103,8 @@
     if (($valid_username = authValidateUser($NewUserName, $NewUserPassword)) 
=== false)
     {
       print_header();
-      echo "<p>".get_vocab('unknown_user')."</p>\n";
-      printLoginForm(this_page(), $target_url, $returl);
+      print_login_failed();
+      print_login_form(this_page(), $target_url, $returl);
       exit();
     }
     $_SESSION["UserName"] = $valid_username;
@@ -145,7 +145,7 @@
 if (isset($Action) && ($Action == "QueryName"))
 {
   print_header();
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
   exit();
 }
 
@@ -170,7 +170,7 @@
   {
     $target_url = $target_url . "?" . $QUERY_STRING;
   }
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
 
   exit();
 }
diff -r 60eca9fd7e31 -r 553b69e54c49 web/session/session_wordpress.inc
--- a/web/session/session_wordpress.inc Thu Mar 16 10:49:36 2017 +0000
+++ b/web/session/session_wordpress.inc Thu Mar 16 11:11:16 2017 +0000
@@ -26,7 +26,7 @@
 if (isset($Action) && ($Action == "QueryName"))
 {
   print_header();
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
   exit();
 }
 
@@ -53,7 +53,7 @@
       $errors = $wp_user;  // $errors is a WordPress global
       print_header();
       echo "<p>" . apply_filters ('login_errors', 
$wp_user->get_error_message()) . "</p>\n";
-      printLoginForm(this_page(), $target_url, $returl);
+      print_login_form(this_page(), $target_url, $returl);
       exit();
     }
   }
@@ -105,7 +105,7 @@
   {
     $target_url = $target_url . "?" . $QUERY_STRING;
   }
-  printLoginForm(this_page(), $target_url, $returl);
+  print_login_form(this_page(), $target_url, $returl);
 
   exit();
 }

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Mrbs-commits mailing list
Mrbs-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mrbs-commits

Reply via email to