Q7

Folder Context - Silent and Dangerous.

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.3.4
  • Fix Version/s: 1.2.22, 1.3.5
  • Component/s: Core
  • Labels:
    None
  • Test Mode:
    Manual

Description

All my home directory was just deleted by a Folder Context. "Clear folder content..." confused me and I thought that only folder which I added to a context will be cleared. But in fact all root path is cleared when a folder context is executed. Thus if you set something like C://, home://, etc. as a root path and execute a context - you will have a big trouble.

That is potentially dangerous, because "Clear folder content..." is checked by default and because you may not associate "folder content" with root content.

Here are some thoughts about it:

  • Disable "clear content.." option by default;
  • When you enable this option - show a warning dialog with "all data from your root will be deleted.."
  • Replace "root path" with "folder path"

May be something else.

Activity

Hide
komaz added a comment - 11/Jul/13 8:58 PM - edited

fixed. just a note – changes are not yet merged into licensing branch, so dev-1.2 should be used for testing

Show
komaz added a comment - 11/Jul/13 8:58 PM - edited fixed. just a note – changes are not yet merged into licensing branch, so dev-1.2 should be used for testing
Hide
Ulyana Skorokhodova added a comment - 12/Jul/13 10:47 AM

Unchecking "Clear root folder..." checkbox doesn't make ca context editor dirty. So when you reopen it - the option will be selected.

Show
Ulyana Skorokhodova added a comment - 12/Jul/13 10:47 AM Unchecking "Clear root folder..." checkbox doesn't make ca context editor dirty. So when you reopen it - the option will be selected.
Hide
komaz added a comment - 12/Jul/13 10:53 AM

fixed

Show
komaz added a comment - 12/Jul/13 10:53 AM fixed

People

Vote (0)
Watch (0)

Dates

  • Created:
    09/Jul/13 4:17 PM
    Updated:
    12/Jul/13 10:53 AM
    Resolved:
    12/Jul/13 10:53 AM