Sucess message shown while deleting a non empty psuedo folder for the first time

Bug #1347598 reported by Ashish Chandra
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Masco

Bug Description

Steps to reproduce:

1) Create a container and create a psuedo folder into it.
2) Navigate to newly created psuedo folder, and upload a couple of objects.
3) Navigate to object store - > Containers in left pane.
4) Click on name of the container, to get the container view.
5) Now against the newly created psuedo folder, click on Delete Object.

A popup is displayed displaying message "Success: Deleted Object: new-psuedo-folder"

Again try to delete the psuedo folder for the second time, this time correct message is shown in the popup.
"Error: Unable to delete object: new-psuedo-folder"

Please refer to attached Video, to see the process in action.

Revision history for this message
Ashish Chandra (ashish-chandra) wrote :
description: updated
Changed in horizon:
status: New → Confirmed
importance: Undecided → Low
Masco (masco)
Changed in horizon:
assignee: nobody → Masco Kaliyamoorthy (masco)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/120755

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/120755
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=1565819d597b1abbd3934c6b2ec3d4e10a4a0549
Submitter: Jenkins
Branch: master

commit 1565819d597b1abbd3934c6b2ec3d4e10a4a0549
Author: Masco Kaliyamoorthy <email address hidden>
Date: Thu Sep 11 17:21:46 2014 +0530

    wrong message while deleting non empty pseudo folder

    while try to delete a pseudo folder which contains some objects is
    showing the success message but the folder is not actually deleted

    added a check similar to delete container, the new check will
    verify the pseudo folder is empty before call the swift client
    call. if the folder is not empty it will throw a exception
    with a appropriate error message and the message will be shown to
    the user in the pop up message box.

    Change-Id: I0bbea6ef6ac26ef357240610535b9d84053a89c2
    Closes-Bug: #1347598

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.