AdbFileManager icon indicating copy to clipboard operation
AdbFileManager copied to clipboard

[BUG]

Open mvargassv opened this issue 1 year ago • 7 comments

Describe the bug The software can't copy files at PC on Windows 11

Unhandled exception log

To Reproduce Steps to reproduce the behavior:

  1. Open any folder from mobile device
  2. Select file to copy
  3. Click right arrow in center screen to copy at PC
  4. nothing is copied

Expected behavior

Screenshots

Desktop (please complete the following information):

  • OS: [e.g. iOS] Windows 11 Pro

Additional context I use version without .NET runtime, I install it separately

mvargassv avatar Sep 20 '24 13:09 mvargassv

Do you have English windows or some other localization? I have noticed there is a bug when copying to the "Pictures" library in non english.

Eg in czech windows, the pictures library is called "Obrázky", but the ADB doesnt copy it to the obrázky folder, but it creates new folder Something similar might be happening to you

T0biasCZe avatar Sep 25 '24 13:09 T0biasCZe

Ahoj, ten program by s tím mohl počítat y vyrobit složku bez mezer, diakritiky atd, na FS mít příznak skrytá a po zkopírování obsah "přesunout" tam, kam to uživatel chtěl. Většinou to budou maximálně stovky souborů, tzn. přesun bude rychlý. Nebo to hlídat per soubor a po každém jenom zkopírování udělat na pozadí přesun, aby se to postupně pěkně objevovalo. Není to dokonalé řešení, ale lepší, aby to fungovalo alespoň nějak?

lordrak007 avatar Sep 27 '24 06:09 lordrak007

To není mezerama/diakritikou. Ono prostě ta složka se na disku jmenuje "Pictures" anglicky, ikdyž ma človek česky windows. Jako "obrázky" se to ukazuje jen naoko, a windows s tím umí pracovat, on si to umí přeložit na Pictures automaticky na pozadí když načítáš soubory atd.
Akorát adb to neumí, takže se to rozbije když nema človek windows v angličtině, a vytvoří to actual složku s tím lokalizovaným jménem...
Tudíž jako fix by mělo stačit vzít ten lokalizovaný path, a přeložit ho zpátky do te anglické formy předtím než se provede ta ADB operace

T0biasCZe avatar Sep 28 '24 18:09 T0biasCZe

Aha, už chápu. A proč se používá překlad, když to možná není třeba? Je to pravděpodobně jen mojí neznalostí použití Shell objektu, ale nestačí změnit v metodě "private void android2pc_Click(object sender, EventArgs e)" string destinationFolder = ShellObject.FromParsingName(explorerBrowser1.NavigationLog.CurrentLocation.ParsingName).Properties.System.ItemPathDisplay.Value; na string destinationFolder = explorerBrowser1.NavigationLog.CurrentLocation.ParsingName; ?

lordrak007 avatar Sep 29 '24 18:09 lordrak007

Spaghetti code moment
mělo by to byt opravene teď...

it should be fixed now for all languages

T0biasCZe avatar Sep 29 '24 21:09 T0biasCZe

https://github.com/T0biasCZe/AdbFileManager/releases/tag/v1.9.1

T0biasCZe avatar Sep 29 '24 21:09 T0biasCZe

Thanks, seems to be fixed. Just had the same problem when clicking on the "Desktop" icon (in a german Windows version) on the right panel as target folder. The folder was set to "C:\Benutzer\XXXXX\Desktop" - which is how Windows displays the folder also in Explorer (using the german word "Benutzer") - but internally on the filesystem the folder is supposd to be "C:\Users\XXXXX\Desktop".

I was really wondering where my pictuers are :D And under C:, I than had two folders named "Benutzer", which was really weird. But yeah, I get it, one is "really" named Benutzer, the other one was named "Users".

However - now it seems fixed. Thanks!

bjspi avatar Oct 12 '24 07:10 bjspi