Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All folders will be deleted from Teampass system in case you delete parent and its child at the same time #1608

Closed
Antoniano opened this issue Dec 29, 2016 · 0 comments

Comments

@Antoniano
Copy link

Steps to reproduce

  1. Create some folders (at least 2 parents) and at least one child in any parent folder. Example: Admin passwords (parent), Kyiv (child), London (child); QA passwords (parent).
  2. Choose to delete parent and his child at the same time. Example: choose Admin passwords and Kyiv folders at the same time
  3. Delete folders, pressing the "trash" button. Totally all the folders will be deleted from Teampass instance, but not only 2 that have been chosen.

Expected behaviour

Only chosen folders must be deleted. If parent and its child are chosen to be deleted at the same time - then delete parent and all its children (also would be nice to see notification that parent will be deleted showing all the children that would be lost in case user proceed).

Actual behaviour

All folders will be totally deleted from Teampass system.

Server configuration

Operating system: Ubuntu 14.04

Web server: Apache2

Database: MySQL

PHP version: 5.5.9

Teampass version: 2.1.26

Updated from an older Teampass or fresh install: fresh install

Client configuration

Browser: Mozilla

Operating system: Windows 10

Logs

Web server error log

Insert your webserver log here

Firebug log (How to?)

Insert the Firebug log here
nilsteampassnet pushed a commit that referenced this issue Jan 2, 2017
Several improvements:
-Folders deletion protocol has been securized to prevent unconsistencies in folders tree
- New login availability is checked "live"
- Filtering counters in datatables
- Users Management dialogbox improved

Fix for #1608, #1592
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants