Fix needToAuthenticate() so that its operation matches the doc comments

1 files changed, 2 insertions, 2 deletions

diff --git a/libopie2/opiesecurity/multiauthpassword.cpp b/libopie2/opiesecurity/multiauthpassword.cpp
index 6c8944e..fb468d1 100644
--- a/libopie2/opiesecurity/multiauthpassword.cpp
+++ b/libopie2/opiesecurity/multiauthpassword.cpp
@@ -18,99 +18,99 @@
      +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
     : ..    .:,     . . .    without even the implied warranty of
     =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
   _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
 ..}^=.=       =       ;      Library General Public License for more
 ++=   -.     .`     .:       details.
  :     =  ...= . :.=-
  -.   .:....=;==+<;          You should have received a copy of the GNU
   -_. . .   )=.  =           Library General Public License along with
     --        :-=`           this library; see the file COPYING.LIB.
                              If not, write to the Free Software Foundation,
                              Inc., 59 Temple Place - Suite 330,
                              Boston, MA 02111-1307, USA.
 
 */
 
 
 /* OPIE */
 #include <opie2/multiauthcommon.h>
 #include <opie2/multiauthmainwindow.h>
 #include <qpe/config.h>
 #include <qpe/qlibrary.h>
 #include <qpe/qcom.h>
 
 /* QT */
 #include <qapplication.h>
 #include <qvbox.h>
 #include <qpushbutton.h>
 #include <qlabel.h>
 #include <qdir.h>
 
 #include "multiauthpassword.h"
 
 namespace Opie {
 namespace Security {
 
 
 /**
  * Tells if the users requires authentication (used internally to
  * know whether to repaint the screen on resume)
  *
  * \param at_poweron true if we are booting Opie, false if we are resuming it
  * \return true if authenticate() launched right now would trigger an authentication
  */
 bool MultiauthPassword::needToAuthenticate(bool at_poweron)
 {
     Config cfg("Security");
     cfg.setGroup("Misc");
-    if ( !at_poweron && cfg.readBoolEntry("onStart", false) )
+    if ( at_poweron && cfg.readBoolEntry("onStart", false) )
         return true;
-    else if ( at_poweron && cfg.readBoolEntry("onResume", false) )
+    else if ( !at_poweron && cfg.readBoolEntry("onResume", false) )
         return true;
     else
         return false;
 }
 
 
 
 /**
  * \brief Require (if configured so) user authentication to unlock and continue
  *
  * This method will check if you require authentication
  * and then will lock the screen and ask for a successful
  * authentication (explaining what it does or not, depending
  * on your local configuration).
  * It may go into an event loop, but anyhow it will only end
  * when the user has successfully authenticated to the system.
  */
 void MultiauthPassword::authenticate(int lockMode)
 {
     /**
      * \par Conditions
      *
      * If lockMode is an If, it's conditional:
      * \li IfPowerOn will not trigger an authentication if
      *   onStart is set to false in Security.conf,
      * \li IfResume will not trigger an authentication if
      *   onResume is set to false in Security.conf.
      */
     if ( (lockMode == IfPowerOn) || (lockMode == IfResume) )
     {
         Config cfg("Security");
         cfg.setGroup("Misc");
         if ( (
               (lockMode == IfPowerOn) && !cfg.readBoolEntry("onStart", false)
             ) || (
               (lockMode == IfResume) && !cfg.readBoolEntry("onResume", false)
             ) )
             return;
     }
 
     /**
      * \li TestNow will ensure that the authentication window will let
      * people escape through the last screen (which they can reach skipping
      * all the authentication steps)
      * \li LockNow will always go on with the authentication, and won't let
      * people escape.
      */
     bool allowByPass = false;