[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] Akonadi Commandline Interface Project
From:       Lindsay Mathieson <lindsay.mathieson () gmail ! com>
Date:       2014-02-28 21:56:33
Message-ID: 53110611.7020509 () gmail ! com
[Download RAW message or body]

On 01/03/14 03:17, Jonathan Marten wrote:
> Hello Bhaskar,
> This is a dangerous command (extremely so!).  So as to avoid any user
> prompting for confirmation (which is not so useful for command line
> scripting), there is a check for such commands which needs a magic
> variable to be set in the environment in order to be able to execute
> such commands.  So, in ClearCommand::start(), the first line should
> be:
>
>    if (!allowDangerousOperation()) emit finished(RuntimeError);


Perhaps output a console msg as well, along the lines of:

  Dangerous operation cancelled. Set "ALLOW_DANGEROUS_OPERATION=1" to
continue.

So users know what has happened and what to do if they really want.


_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic