Skip to content

Dropping a folder onto ifself should be impossible #18850

Closed
#39998
Closed
Dropping a folder onto ifself should be impossible#18850
#39998
Labels
2. developingWork in progress25-feedback26-feedbackbugfeature: filesjavascript

Description

@raimund-schluessler
raimund-schluessler
opened on Jan 12, 2020

Steps to reproduce

  1. Create two folders a and b
  2. Select both folders
  3. Start dragging the two folders by clicking on folder a or b.

Expected behaviour

It should not be possible to drop the two folders onto themself.

Actual behaviour

It is possible to drop the two folders onto the folder which was not clicked for starting the drag, see the image:

dragging

Of course, moving folder b into itself fails and produces an error message:

dragging2

But since I wanted to move both folders, moving folder a into folder b is not the intended result.

Server configuration

Nextcloud version: stable18 as of 8b4b3e1

Activity

  1. raimund-schluessler
    added
    bug
    0. Needs triagePending check for reproducibility or if it fits our roadmap
    feature: files
    javascript
    on Jan 12, 2020

  2. raimund-schluessler commented on Jan 13, 2020

    @raimund-schluessler
    raimund-schluessler
    on Jan 13, 2020
    MemberAuthor

    If someone from @nextcloud/javascript could give me a hint were to start I would look into this.

  3. szaimen commented on May 28, 2021

    @szaimen
    szaimen
    on May 28, 2021
    Contributor

    I could reproduce this on NC21.0.1

    cc @nextcloud/javascript any update here regarding raimund-schuessler's comment above?

  4. szaimen
    added
    1. to developAccepted and waiting to be taken care of
    and removed
    0. Needs triagePending check for reproducibility or if it fits our roadmap
    on May 28, 2021

  5. ChristophWurst commented on May 28, 2021

    @ChristophWurst
    ChristophWurst
    on May 28, 2021
    Member

    Somewhere ™️ in the files app code. But please beware that the files app is our biggest legacy component at the moment and there are strong believes that we'll soon have to rewrite at least parts of it.

  6. szaimen commented on Jan 9, 2023

    @szaimen
    szaimen
    on Jan 9, 2023 · Hidden as resolved
  7. szaimen
    added
    needs info
    0. Needs triagePending check for reproducibility or if it fits our roadmap
    and removed
    1. to developAccepted and waiting to be taken care of
    on Jan 9, 2023
  8. szaimen
    closed this as completedon Mar 6, 2023

  9. raimund-schluessler commented on Mar 6, 2023

    @raimund-schluessler
    raimund-schluessler
    on Mar 6, 2023 · Hidden as resolved
    Author
  10. raimund-schluessler
    reopened this on Mar 6, 2023
  11. szaimen
    added
    25-feedback
    26-feedback
    and removed
    needs info
    on Mar 6, 2023
  12. skjnldsv
    added
    2. developingWork in progress
    and removed
    0. Needs triagePending check for reproducibility or if it fits our roadmap
    on Sep 1, 2023
  13. skjnldsv
    closed this as completedin #39998on Sep 26, 2023
  14. skjnldsv
    added this to Files to vueon Aug 7, 2024
  15. skjnldsv
    moved this to Done in Files to vueon Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    2. developingWork in progress25-feedback26-feedbackbugfeature: filesjavascript

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions