Jump to content

Auto Organize Smart Match, picking up wrong shows


GiGo

Recommended Posts

GiGo

Sounds like you are on it. I do hate the Error Null dialog, would be great to know what the heck went wrong! So glad someone is giving the Auto Organize plugin so attention. Please keep me updated.

Link to comment
Share on other sites

chef

I have been able to circumvent the  "OverwriteExistsingFile" option, when using  the "Identify" modal in the configuration, and allow the item to copy over to the library, while having the "OverwriteExistsingFile" option set to false.

I believe this is a more intuitive way to handle.

Next I'll find the "Are you sure" modal and allow "OverwriteExistsingFile" as well.

  • Like 1
Link to comment
Share on other sites

chef

@GiGo

Time to test this thing out. This thing is about 90% "fixed".

It's safe to test.

 

I would LOVE to know all the weird buggy stuff you might find here, and anything that isn't working intuitively.

 

Download moved down to the next release.

@cayars do you have a moment to load this thing and let me know if I have made things worse or better?

@rbjtech are you around to try this thing out too? 

 

 

 

 

Edited by chef
Link to comment
Share on other sites

GiGo
1 hour ago, chef said:

@GiGo

Time to test this thing out. This thing is about 90% "fixed".

It's safe to test.

 

I would LOVE to know all the weird buggy stuff you might find here, and anything that isn't working intuitively.

Emby.AutoOrganize.zip 123.49 kB · 0 downloads

 

@cayars do you have a moment to load this thing and let me know if I have made things worse or better?

@rbjtech are you around to try this thing out too? 

 

 

 

 

Cheers will give this a go, will report back later :)

Link to comment
Share on other sites

GiGo

@chef um... I'm not seeing any changes?

I've sinced moved Origin manually and now it matches, but I assumed that with a match already in place I would get an option on the right to change it? I have overwritten the old Auto Organize dll and rebooted the server, version of AutoOrganize is 1.6.0.0, I'm running the latest beta version of emby.

Auto Organize.png

Link to comment
Share on other sites

chef
5 hours ago, GiGo said:

@chef um... I'm not seeing any changes?

I've sinced moved Origin manually and now it matches, but I assumed that with a match already in place I would get an option on the right to change it? I have overwritten the old Auto Organize dll and rebooted the server, version of AutoOrganize is 1.6.0.0, I'm running the latest beta version of emby.

Auto Organize.png

You replaced the dll in "programdata/plugins"?

 

Might have to clear browser data. 

That is still the old UI. 

Link to comment
Share on other sites

GiGo
1 minute ago, chef said:

You replaced the dll in "programdata/plugins"?

 

Might have to clear browser data. 

That is still the old UI. 

Yep that solved it!

WOW That looks SO SO MUCH BETTER and it WORKS!

GREAT WORK! You need to roll this out to everyone it's so much better!

  • Like 1
Link to comment
Share on other sites

GiGo

FYI;

Just tried a file that I already had in my collection as I wanted to updated it, it would of had the same file name when I click the Organize button it came up with a blank error 'Internal Server Erro' message, however if I click the new Identify button and select the correct show it moves it perfectly without any error message.

Link to comment
Share on other sites

chef
1 minute ago, GiGo said:

FYI;

Just tried a file that I already had in my collection as I wanted to updated it, it would of had the same file name when I click the Organize button it came up with a blank error 'Internal Server Erro' message, however if I click the new Identify button and select the correct show it moves it perfectly without any error message.

Right! I have to track down that error. I saw it too.

When the plugin has the option to "overwrite" unchecked, we send a new param (on button click) to "requestOverwrite" to the server.

The second button doesn't send that request yet. I'd better track that down today before I send it to Luke.

Link to comment
Share on other sites

rbjtech
8 hours ago, chef said:

 

@rbjtech are you around to try this thing out too? 

 

yep - gimme an hour or so and I'll happily test - thanks :)

  • Thanks 1
Link to comment
Share on other sites

chef

Here is a version which should fix the button click.

Clear browser data, because we changed the Javascript. :)

 

Edited by chef
Link to comment
Share on other sites

rbjtech

This was on the previous version - but is looking much more polished than the previous plugin and working well.

Is there anything in particular you want me to test, or test everything ? ;)

auto.thumb.PNG.91c0b771e50393963cfe81172b82b9b3.PNG

 

auto2.thumb.PNG.217110c9a993cca43bf65d0eccbcd6dc.PNG

auto3.thumb.PNG.9dac5c536aca08ed1a18d07498a88dde.PNG

Link to comment
Share on other sites

chef
58 minutes ago, rbjtech said:

This was on the previous version - but is looking much more polished than the previous plugin and working well.

Is there anything in particular you want me to test, or test everything ? ;)

auto.thumb.PNG.91c0b771e50393963cfe81172b82b9b3.PNG

 

auto2.thumb.PNG.217110c9a993cca43bf65d0eccbcd6dc.PNG

auto3.thumb.PNG.9dac5c536aca08ed1a18d07498a88dde.PNG

Wondering about those failures.

I notice the identify button is not available.

Does the Correction Modal appear when you click the "Move" button beside items like "Top Gear-s01e16-USA Road Trip.mp4"?

 

I also winder is there is a better word to use in the UI besides "Failure". Which sounds worse then an "un-identified"  item.

Edited by chef
Link to comment
Share on other sites

GiGo
8 minutes ago, chef said:

I also winder is there is a better word to use in the UI besides "Failure". Which sounds worse then an "un-identified"  item.

'Attention' maybe? But honestly don't think it matters.

  • Like 2
Link to comment
Share on other sites

rbjtech

They are previous failures not yet run though the new version - I'm just going to re-identify them and we should be all green again.

Gonna need an hour or so - pesky work is getting in the way .. 😆

Link to comment
Share on other sites

chef
1 minute ago, rbjtech said:

They are previous failures not yet run though the new version - I'm just going to re-identify them and we should be all green again.

Gonna need an hour or so - pesky work is getting in the way .. 😆

Thanks for your time. 

Link to comment
Share on other sites

chef
4 minutes ago, GiGo said:

'Attention' maybe? But honestly don't think it matters.

Yep! That is much better.

Link to comment
Share on other sites

chef

I submitted the work on GitHub. But I am absolutely the worst at properly submitting a pull request, so I submitted the work to Luke directly, as well.

There were some other feature requests from users, so maybe if there is time, I'll look at those as well.

Edited by chef
  • Like 3
Link to comment
Share on other sites

rbjtech
4 hours ago, chef said:

Thanks for your time. 

These are all cleared now @chef - bbc iplayer and thetvdb episode numbering does not match, so I just need to rename to thetvdb naming and all is good.

Thanks for the much needed refresh of this essential plugin. 👍

  • Thanks 1
Link to comment
Share on other sites

GiGo

Just noticed an issue when organising, can't see the 'remember this correction' tick box

Auto Organize.png

  • Thanks 1
Link to comment
Share on other sites

chef
2 hours ago, GiGo said:

Just noticed an issue when organising, can't see the 'remember this correction' tick box

Auto Organize.png

I'll fix it right away. Thank you. 

Link to comment
Share on other sites

chef

Here we go, let me know if this fixes the margin issue in the modal :)

 

 

This also fixes Movie organization.

Edited by chef
Link to comment
Share on other sites

chef

@rbjtech 

do you know what is the difference between the Auto-detect movies, and auto organize movies options in the auto organize plugin?

 

What is create sub directory per movie? Does that mean in the watched folder or in the actual library?

 

Edited by chef
Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...