? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
1 May 2020

Pull Request for Issue #28894 .

Summary of Changes

Fix the conditional by accounting also the adapters. Eg if the adapters are different (local and dropbox) the name could be the same (case of copying or moving the file). In sort the previous code will prevent copying/moving files, my bad!

@wilsonge

Testing Instructions

Expected result

Actual result

Documentation Changes Required

avatar dgrammatiko dgrammatiko - open - 1 May 2020
avatar dgrammatiko dgrammatiko - change - 1 May 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 May 2020
Category Administration com_media
avatar wilsonge wilsonge - change - 1 May 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-05-01 13:10:40
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 1 May 2020
avatar wilsonge wilsonge - merge - 1 May 2020
avatar wilsonge
wilsonge - comment - 1 May 2020

Thanks!

avatar wilsonge
wilsonge - comment - 2 May 2020

OK This broke things pretty bad. I think a simpler and working fix is in #28910

avatar dgrammatiko
dgrammatiko - comment - 2 May 2020

My bad it should be $adapter === $destinationAdapter (translating to same adapter same filename, now is different adapter same name). Pfff

Add a Comment

Login with GitHub to post a comment