Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when changing file location #1057

Closed
duramato opened this issue Dec 26, 2017 · 36 comments
Closed

Error when changing file location #1057

duramato opened this issue Dec 26, 2017 · 36 comments

Comments

@duramato
Copy link
Contributor

duramato commented Dec 26, 2017

  • Windows 10
  • transgui 5.13

When changing a torrent's file location, I get the following error:

image

But it still changes the file's location.

More info needed just ask

@leonsoft-kras
Copy link
Contributor

leonsoft-kras commented Dec 27, 2017

  1. Can you repeat the problem?
  2. Can you send your ini-file?
    [Hosts] and [Connection ....] can be deleted from the file.

if the problem repeats, please describe in detail your actions
(including the choice of the folder to which you move)

@duramato
Copy link
Contributor Author

  1. Yes, its reproducable
  2. https://gist.github.com/duramato/2f1009af381da750fdf2a00ac5efb466

How to reproduce:

  1. Right click torrent;
  2. Change file location;
  3. Add anything to the end of the existing path, to create a new directory. Like add "123" to "/home/user/downloads" it will become in the end"/home/user/downloads/123";
  4. The error from the OP's scrreenshoot should show up;

@StentMan
Copy link

I'm experiencing the same problem when trying to set data location:
Error: LS-008. Please contact the developer
I just recently reinstalled v5.13 and had to redo all the settings, including the path mappings. I think I got it right, since when I try to open containing folder everything opens just fine.

@panfren
Copy link

panfren commented Jan 2, 2018

Happy new year! Same issue here as well.

@leonsoft-kras
Copy link
Contributor

I could not repeat the situation on your information. I changed the code a bit, maybe this will help. It is necessary to try on your means.

@rd1
Copy link

rd1 commented Jan 8, 2018

same as op, i change location, error pops up but still does the work.

@rd1
Copy link

rd1 commented Jan 8, 2018

also the set data location window doesn't save paths anymore. fresh new install of win10 and fresh transgui 5.13 (only version installed).

only 2 paths in the ini file, original from transmission qt , and other one is from adding a new torrent.

But moving torrents doesn't add new folders to list.

@AOne-T
Copy link

AOne-T commented Jan 14, 2018

Same problem.
Win 10 1709 x64. TransGUI 5.13.
Changing location (every time) pops this dialog. Transmission and locations are on a NAS.

@AOne-T
Copy link

AOne-T commented Jan 24, 2018

I have to add. I found out that it only happend if I want to relocate downloaded torrent to the "new" location that is not in the recent list. If I move to the "known" location - there is no error, but if I move to "new" location - I have this error and this location doesn't save in recent list.

@rd1
Copy link

rd1 commented Jan 27, 2018

i agree, if a path exists in the move box, it's fine, but a new one generates this error and isn't added to the list.

leonsoft-kras added a commit that referenced this issue Jan 28, 2018
@elisimpson
Copy link

I have the same issue. Error LS-008, files are moved anyway, folder location isn't added to the list. Synology NAS.

@leonsoft-kras
Copy link
Contributor

waiting for the next release.
In the meantime, everyone can write "I have the same"
Maybe someone from this information will get better: D

@rd1
Copy link

rd1 commented Feb 5, 2018

works fine in 5.14 once again, thanks

@elisimpson
Copy link

Tested with latest version. The error when moving files is gone, but the new location is still not saved into the list of previously used locations. Here's the relevant part of my .ini file if that helps:

[AddTorrent.DiskStation (LAN)]
PeerLimit=60
FolderCount=1
Folder0=/volume1/download
FolHit0=25
FolExt0=
LastDt0=04.02.2018
Folder1=
FolHit1=-1
FolExt1=
LastDt1=
LastDownloadDir=/volume1/download
LastMoveDir=/volume1/video/Music/Dempagumi.inc/Concert

@cociweb
Copy link

cociweb commented Feb 5, 2018

The error is appeared on 5.14. The move is successful, but the old folder is kept.
Is it a problem only for me?

@leonsoft-kras
Copy link
Contributor

I can't repeat your situation. This function works for me and there is no malfunction.
Minor changes in the function of the result did not bring.

Put a breakpoint in the function and go under the debugger. Tell us where the exception occurs.

@leonsoft-kras leonsoft-kras reopened this Feb 5, 2018
@AOne-T
Copy link

AOne-T commented Feb 9, 2018

Tested with latest version. The error when moving files is gone, but the new location is still not saved into the list of previously used locations.

Me too. Tested on another machine, but the result is the same as @elisimpson wrote.

@leonsoft-kras
Copy link
Contributor

@AOne-T there is a solution. See above.

@AOne-T
Copy link

AOne-T commented Feb 10, 2018

@leonsoft-kras: you mean about the debugger?

@leonsoft-kras
Copy link
Contributor

Yes.
Why write "Me too" ? Tell me what's going on your computer. ))

@rd1
Copy link

rd1 commented Mar 22, 2018

still can't add new paths when moving files in 5.15 , it completes the move but not adding new paths.

@leonsoft-kras
Copy link
Contributor

Of course. Did someone check the code under the debugger?

@AOne-T
Copy link

AOne-T commented Mar 22, 2018

What IDE do you use?

@leonsoft-kras
Copy link
Contributor

Lazarus

@AOne-T
Copy link

AOne-T commented Mar 23, 2018

Project transgui.exe cast an exception 'EStringListError' with message:
Operation not allowed on sorted list

 At address 100082E1B

TMainForm.acMoveTorrentExecute -> SaveDownloadDirs

    if CurFolderParam = 'LastMoveDir' then begin
      if i < 0 then begin
          DeleteDirs (CB, 1);
          CB.Items.Insert   (0, selfolder);  <-- exception here

leonsoft-kras added a commit that referenced this issue Mar 24, 2018
It is necessary to test.
@leonsoft-kras
Copy link
Contributor

Thank you.
It's strange that there are no exceptions to Windows.

@AOne-T
Copy link

AOne-T commented Mar 24, 2018

Yes, it's somewhat suppressed by empty expect block, I think, where original LS-008 exception is commented out.
Will pull changes tonight and test.

@AOne-T
Copy link

AOne-T commented Mar 24, 2018

Checked. Works now!

@leonsoft-kras
Copy link
Contributor

I repeat. At me the code worked completely, the data were added, exceptions did not occur. It is not the existence of an exception handling.
Therefore, it is strange that in different systems the function works differently.

@AOne-T
Copy link

AOne-T commented Mar 30, 2018

When will this issue be closed and released?

@rd1
Copy link

rd1 commented Mar 30, 2018

i was wondering that too.

@leonsoft-kras
Copy link
Contributor

The next release, when there will be some more corrections. Not every day.
Can be said @PeterDaveHello

@PeterDaveHello
Copy link
Member

Will be soon :)

@PeterDaveHello
Copy link
Member

@rd1
Copy link

rd1 commented Mar 31, 2018

yes in deed it seems fixed, tested on win10 x64 , thank you PeterDaveHello !

@PeterDaveHello
Copy link
Member

@rd1 the credits belong to @leonsoft-kras :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

9 participants