Notifiarr / toolbarr

Starr Toolbarr
https://notifiarr.com/discord
MIT License
95 stars 4 forks source link

Migrate from Win to Linux - Path hasn't changed correctly #177

Closed gr1ffo closed 4 months ago

gr1ffo commented 4 months ago

I'm guessing I messed up here. I've tried to amend the path on my Sonarr db file and it doesn't seem to have worked correctly. I followed the walkthrough on the Wiki page.

Original Path: D:\Media\TV Series New Path: /mnt/nas/TV Series

Path as displayed in Sonarr after restoring the db: /mnt/nas/TV Series/D:\Media\TV Series\

I've tried amending the path again but it doesn't remove the D:\ path.

gr1ffo commented 4 months ago

I tried again, in a different order, with better success. I had multiple original paths that I have combined into a single new path. In the first instance, I deleted the paths and then re-added the files to the new path, causing the above issue.

On my second attempt, first, I changed the paths to all records to new linux paths (/mnt/nas/TV Series/, ../TV Series1/ etc), and then deleted the paths to the placeholders (../TV Series1,2,3 etc) and added them to the proper path (../TV Series). This seems to have worked.

I'll try again with Radarr and see how I go.

Thanks very much for this tool.

davidnewhall commented 4 months ago

Hello and thank you for opening an issue! The bug with Windows paths migration has been fixed. I cut a new release with the fix, so please download the new version and try that again.

gr1ffo commented 4 months ago

Thanks David!

I've downloaded the new .zip file. I can now delete instances, but this requires a restart after each time to make it responsive again. I can load a new db in the Radarr option however, it won't let me select either the 'Actions' or 'DB Tools' tab after its saved.

I haven't entered an URL or any other information. I've just uploaded the db.

davidnewhall commented 4 months ago

This sounds very strange. I'm on a Mac and I'm not able to reproduce this behavior you've described using version 0.1.2.

Try putting a Url and API key? it should work without them but may work better with them. Not real sure.

There should also be a log file in the .toolbarr directory in your User folder. C:\Users\you\.toolbarr

gr1ffo commented 4 months ago

I've tried using the URL and API key and still have the same result. To confirm, I'm not using the installer, just the amd64 zip.

Where is the config file stored? Maybe this is corrupt on mine and I need to erase it. I've been jumping between 0.1.1 and 0.1.2 to keep working on it. 0.1.1 has some bugs like not being able to delete a Sonarr or Radarr instance and some other minor things but I can make it work. On 0.1.2 I can delete a Sonarr or Radarr instance (with a restart) and load a db but I can't select another tab or add another instance.

gr1ffo commented 4 months ago

There should also be a log file in the .toolbarr directory in your User folder. C:\Users\you.toolbarr

Sorry just read this. I'll have a look and get back to you.


Here's a section of the log. There's some errors prior to this but they relate to getting the indexers and clients from the URL which I didn't use. Otherwise it's just 'Opened log file'.

2024/05/27 20:26:56 [INFO] Opened log file: C:\Users\me\.toolbarr\toolbarr.log
panic: runtime error: index out of range [3] with length 3

goroutine 38 [running]:
github.com/Notifiarr/toolbarr/pkg/app.(*App).RemoveInstance(0xc0001ec000, 0x3, {0xc00000acd0, 0x6})
    github.com/Notifiarr/toolbarr/pkg/app/instances.go:59 +0x7dd
reflect.Value.call({0x1355f40?, 0xc0001ec000?, 0x30?}, {0x137bfbd, 0x4}, {0xc0001c9680, 0x2, 0xc0001c9680?})
    reflect/value.go:596 +0xca6
reflect.Value.Call({0x1355f40?, 0xc0001ec000?, 0x8?}, {0xc0001c9680?, 0xc000320520?, 0x18eccd0?})
    reflect/value.go:380 +0xb9
github.com/wailsapp/wails/v2/internal/binding.(*BoundMethod).Call(0xc0000a6620, {0xc000274220, 0x2, 0x129ff00?})
    github.com/wailsapp/wails/v2@v2.8.2/internal/binding/boundMethod.go:72 +0x205
github.com/wailsapp/wails/v2/internal/frontend/dispatcher.(*Dispatcher).processCallMessage(0xc00008cfc0, {0xc0003362a0, 0x67}, {0x18ed438, 0xc0000d6800})
    github.com/wailsapp/wails/v2@v2.8.2/internal/frontend/dispatcher/calls.go:45 +0x188
github.com/wailsapp/wails/v2/internal/frontend/dispatcher.(*Dispatcher).ProcessMessage(0x0?, {0xc0003362a0?, 0x0?}, {0x18ed438?, 0xc0000d6800?})
    github.com/wailsapp/wails/v2@v2.8.2/internal/frontend/dispatcher/dispatcher.go:43 +0x56
github.com/wailsapp/wails/v2/internal/frontend/desktop/windows.(*Frontend).processMessage.func1()
    github.com/wailsapp/wails/v2@v2.8.2/internal/frontend/desktop/windows/frontend.go:698 +0x38
created by github.com/wailsapp/wails/v2/internal/frontend/desktop/windows.(*Frontend).processMessage in goroutine 1
    github.com/wailsapp/wails/v2@v2.8.2/internal/frontend/desktop/windows/frontend.go:697 +0x265
davidnewhall commented 4 months ago

Thanks for this!

davidnewhall commented 4 months ago

I made a few small changes. Would you mind grabbing another copy from here and checking to see if it works any better?

gr1ffo commented 4 months ago

Seems to have the same issue. I tested under the Radarr tab.

I deleted the existing instance. I had to restart to get the app to respond again. Then I added back a db file. It appears to save successfully but the only 2 buttons are the 'Save' and 'Test' buttons and the 'Actions' and 'DB Tools' tabs don't open when selected. After restarting, the 'Reset' and 'Delete' buttons have appeared and a new window below the existing instance has appeared, 'Add New Radar Instance' however, selecting it doesn't do anything. The 'Actions' and 'DB Tools' tabs still don't open.

Log file checked and nothing more to add apart from more 'Opened log file' comments.

nalllen commented 4 months ago

Tired re installing it but still the same experience as gr1ffo have in the comment above, tried the one with the changes too but it behaves the same.

davidnewhall commented 4 months ago

@gr1ffo @nalllen - Thank you for your patience while I work on this, and thank you for testing for me. I believe I've located the problem. Please download another copy from here and let me know how it works out.