Merge pull request #4295 from owncloud/fix/invalid_copy_into_descenda…

…nt_in_different_spaces

[BUG] Invalid copy into descendant in different spaces

CHANGELOG.md

@@ -35,6 +35,7 @@ ownCloud admins and users.
 * Bugfix - Conflict in copy with files without extension: [#4222](https://github.com/owncloud/android/issues/4222)
 * Bugfix - Add "scope" parameter to /token endpoint HTTP requests: [#4260](https://github.com/owncloud/android/pull/4260)
 * Bugfix - Fix in the handling of the base URL: [#4279](https://github.com/owncloud/android/issues/4279)
+* Bugfix - Copy folder into descendant in different spaces: [#4293](https://github.com/owncloud/android/issues/4293)
 * Change - Android library as a module instead of submodule: [#3962](https://github.com/owncloud/android/issues/3962)
 * Enhancement - Koin DSL: [#3966](https://github.com/owncloud/android/pull/3966)
 * Enhancement - Unit tests for datasources classes - Part 1 & Fixes: [#4063](https://github.com/owncloud/android/issues/4063)
@@ -122,6 +123,14 @@ ownCloud admins and users.
    https://github.com/owncloud/android/issues/4279
    https://github.com/owncloud/android/pull/4287
 
+* Bugfix - Copy folder into descendant in different spaces: [#4293](https://github.com/owncloud/android/issues/4293)
+
+   Copying a folder into another folder with the same name in a different space now
+   works correctly.
+
+   https://github.com/owncloud/android/issues/4293
+   https://github.com/owncloud/android/pull/4295
+
 * Change - Android library as a module instead of submodule: [#3962](https://github.com/owncloud/android/issues/3962)
 
    Android library, containing all networking stuff, is now the 5th module in the

changelog/unreleased/4295

@@ -0,0 +1,6 @@
+Bugfix: Copy folder into descendant in different spaces
+
+Copying a folder into another folder with the same name in a different space now works correctly.
+
+https://github.com/owncloud/android/issues/4293
+https://github.com/owncloud/android/pull/4295

owncloudDomain/src/main/java/com/owncloud/android/domain/files/usecases/CopyFileUseCase.kt

@@ -49,7 +49,7 @@ class CopyFileUseCase(
     fun validateOrThrowException(listOfFilesToCopy: List<OCFile>, targetFolder: OCFile) {
         require(listOfFilesToCopy.isNotEmpty())
 
-        if (listOfFilesToCopy.any { targetFolder.remotePath.startsWith(it.remotePath) }) {
+        if (listOfFilesToCopy.any { targetFolder.remotePath.startsWith(it.remotePath) && targetFolder.spaceId == it.spaceId }) {
             throw CopyIntoDescendantException()
         }
     }