Ferreteria/v0.5/@cls/login/login/trylogin: Difference between revisions

From Woozle Writes Code
< Ferreteria‎ | v0.5‎ | @cls‎ | login‎ | login
Jump to navigation Jump to search
(Created page with "=users\csLogin\TryLogin(string $sUser,string $sPass) : bool= ==About== * '''File''': {{l/ferreteria/code|login/status.php}} * '''Returns''': TRUE iff the submitted credentials...")
 
Line 15: Line 15:
* '''2022-03-17''' SRAL: PassMatches() is now a Card method, so $rc is now converted to a Card
* '''2022-03-17''' SRAL: PassMatches() is now a Card method, so $rc is now converted to a Card
* '''2022-03-18''' adapting from session\cNativeRow->UserLogin()
* '''2022-03-18''' adapting from session\cNativeRow->UserLogin()
* '''2022-03-18''' SRAL: The return value for this (cLoginStatus) is now a static class (csLoginStatus),
* '''2022-03-18''' SRAL: The return value for this (cLoginStatus) is now a static class (csLoginStatus), so there's no point in returning it. Will instead return a bool indicating success/failure.
          so there's no point in returning it. Will instead return a bool indicating success/failure.
* '''2022-03-29''' SRAL: It no longer makes sense to me to have this routine handle bookkeeping; it should do the minimum necessary and get out. It's probably duplicating functionality. ...and most of what we need is in SRAL:FindLoginName() -- so let's just consolidate this back into the only thing that calls it: Feature->TryLogin()
* '''2022-03-29''' SRAL: It no longer makes sense to me to have this routine handle bookkeeping; it should do the minimum necessary and get out. It's probably duplicating functionality. ...and most of what we need is in SRAL:FindLoginName() -- so let's just consolidate this back into the only thing that calls it: Feature->TryLogin()
* '''2022-03-29''' Moving TryLogin() from the Account Feature to csLogin.
* '''2022-03-29''' Moving TryLogin() from the Account Feature to csLogin.

Revision as of 00:23, 30 March 2022

users\csLogin\TryLogin(string $sUser,string $sPass) : bool

About

  • File: login/status.php
  • Returns: TRUE iff the submitted credentials are correct and there is no other reason to deny the login

Action

  • checks credentials against account data for the requested login slug
  • if successful:
  • else (not successful):

History

SRAL = StorageRow()->AuthorizeLogin()

  • 2020-01-10 SRAL: Rewriting began several days ago.
  • 2021-01-16 SRAL: removed $osLogin->SetAccountRecord(TRUE,$rc) after "$rc =" because it's now redundant
  • 2022-02-10 SRAL: moved from Feature to Storage Row
  • 2022-03-17 SRAL: PassMatches() is now a Card method, so $rc is now converted to a Card
  • 2022-03-18 adapting from session\cNativeRow->UserLogin()
  • 2022-03-18 SRAL: The return value for this (cLoginStatus) is now a static class (csLoginStatus), so there's no point in returning it. Will instead return a bool indicating success/failure.
  • 2022-03-29 SRAL: It no longer makes sense to me to have this routine handle bookkeeping; it should do the minimum necessary and get out. It's probably duplicating functionality. ...and most of what we need is in SRAL:FindLoginName() -- so let's just consolidate this back into the only thing that calls it: Feature->TryLogin()
  • 2022-03-29 Moving TryLogin() from the Account Feature to csLogin.