YAMJ / yamj-v2

Yet Another Movie Jukebox (YAMJ) v2
GNU General Public License v3.0
28 stars 11 forks source link

Nest multiple seasons of a series under a master entry #113

Closed Omertron closed 9 years ago

Omertron commented 9 years ago

Original issue 113 created by Omertron on 2008-07-17T05:28:33.000Z:

Note, this is somewhat related to item # 58.

What steps will reproduce the problem?

  1. Scan a movie library with several seasons of a show

What is the expected output? What do you see instead?

Desired enhancement automatically (or with a property flag?) nest all seasons of a particular show as sub-index pages under one master entry for the show.

Example, right now I see 6 or 7 Sopranos icons in my 'S' index. Ideally I would like to see one Sopranos icon, then a link for each season, and on each season page the links for the individual episodes. There would be bonus points awarded for making the system smart enough to do this only if there was more than 1 season of episodes for any given show.

Omertron commented 9 years ago

Comment #1 originally posted by Omertron on 2008-07-17T06:14:21.000Z:

Need to agree on some sort of naming convention so that people know how to organise seasons of a show.

Omertron commented 9 years ago

Comment #2 originally posted by Omertron on 2008-07-17T08:38:31.000Z:

something like this http://img129.imageshack.us/img129/3092/11112222ck4.jpg

Omertron commented 9 years ago

Comment #3 originally posted by Omertron on 2008-07-22T15:02:23.000Z:

I also think this is very important. It just looks silly to have 11 seasons of Fraiser for example taking up the screen. The picture above is exactly what I've imagined it should be like.

Omertron commented 9 years ago

Comment #4 originally posted by Omertron on 2008-08-13T14:32:54.000Z:

+1

very good point

Omertron commented 9 years ago

Comment #5 originally posted by Omertron on 2008-08-17T21:40:58.000Z:

If the picture above could be implemented, it would greatly advance the usability in the application by lessening the clutter with all the icons for multi-season TV programs.

Omertron commented 9 years ago

Comment #6 originally posted by Omertron on 2008-08-25T05:36:55.000Z:

If this cannot be done to the thumbnails, perhaps another option is...

I have added issue 151 to request that Season number show up in the text shown at the bottom when you go over the thumbnails.

Omertron commented 9 years ago

Comment #7 originally posted by Omertron on 2008-09-09T11:02:36.000Z:

+1... Great!

Omertron commented 9 years ago

Comment #8 originally posted by Omertron on 2008-09-12T14:45:06.000Z:

Love the idea!

Omertron commented 9 years ago

Comment #9 originally posted by Omertron on 2008-10-21T19:33:39.000Z:

Love this idea. Hope it gets implemented soon.

Omertron commented 9 years ago

Comment #10 originally posted by Omertron on 2008-10-22T15:41:05.000Z:

We've heard no comments from the developers on this enhancement?

Omertron commented 9 years ago

Comment #11 originally posted by Omertron on 2008-10-23T03:00:28.000Z:

Sure you have...the first two comments, including the issue reporter, were from developers. ;)

Omertron commented 9 years ago

Comment #12 originally posted by Omertron on 2008-10-24T15:54:09.000Z:

Hmm, I see... Well, just more reason for the developers to implement this feature.

I have many seasons of a lot of shows, that I would like to index and have on my PCH, but so far I only have one or two season on my PCH and in my Jukebox, because they would take too much "space" on the front of my Jukebox, if I had my 10 seasons of Friends, 9 Seasons of X-files, 6 seasons of MythBusters etc.

Omertron commented 9 years ago

Comment #13 originally posted by Omertron on 2008-10-27T16:07:47.000Z:

I'd love to see this happen! 11 seasons of Frasier and 11 of MASH....ugh, so much of the main menu eaten up!

Omertron commented 9 years ago

Comment #14 originally posted by Omertron on 2008-11-10T15:31:44.000Z:

Yes I am hoping for this too.

Omertron commented 9 years ago

Comment #15 originally posted by Omertron on 2008-11-25T20:55:10.000Z:

Yes Please

Omertron commented 9 years ago

Comment #16 originally posted by Omertron on 2008-11-25T20:56:31.000Z:

+1 this is a great idea. The photo example is exactly how I pictured it happening.

Omertron commented 9 years ago

Comment #17 originally posted by Omertron on 2008-11-25T23:00:15.000Z:

please please please

Omertron commented 9 years ago

Comment #18 originally posted by Omertron on 2008-11-26T03:27:34.000Z:

+1

Omertron commented 9 years ago

Comment #19 originally posted by Omertron on 2008-11-26T09:16:52.000Z:

Yep, this would be good, no need for a page full of scrubs :)

Omertron commented 9 years ago

Comment #20 originally posted by Omertron on 2008-11-26T10:02:24.000Z:

I think this would be a great enhancment if it could be implemented.

Omertron commented 9 years ago

Comment #21 originally posted by Omertron on 2008-11-26T15:32:14.000Z:

Either we just got a huge influx of new users, or some very industrious person who really wants this feature just registered a bunch of Google accounts to try and bump this request up. :) funny

Omertron commented 9 years ago

Comment #22 originally posted by Omertron on 2008-11-26T18:29:09.000Z:

The issue link was posted on the forum thread about tv series, so I guess that's everyone from there.

Kinda think this would be good to implement when the TV nfo enhancement is done

Omertron commented 9 years ago

Comment #23 originally posted by Omertron on 2008-11-26T18:47:42.000Z:

Right. There's sort of a dependency to this TV stuff. I would like to get the current TheTVDB api stuff that's currently implemented out there in a release for everyone to use. Then, when those enhancement requests start to come in, we can deal with those changes along with the implementation of the tvshow NFO. etc etc

Omertron commented 9 years ago

Comment #24 originally posted by Omertron on 2008-11-26T19:01:14.000Z:

"There would be bonus points awarded for making the system smart enough to do this only if there was more than 1 season of episodes for any given show."

This is really important so series like "Band of Brothers" and others wouldn't have a "Band of Brothers" page and then only one icon for "season 1"

Omertron commented 9 years ago

Comment #25 originally posted by Omertron on 2008-11-28T02:43:01.000Z:

+1 comment # 2 has the idea spot on.

 24 raises a good point. You don't what an extra layer of menus unless there are

multiple season.

Omertron commented 9 years ago

Comment #26 originally posted by Omertron on 2008-12-17T20:24:55.000Z:

Would really like this one. The Simpsons is what, 20 seasons now?

 2's format looks great.

Omertron commented 9 years ago

Comment #27 originally posted by Omertron on 2008-12-22T09:22:45.000Z:

Would love this as well, for sure.

Omertron commented 9 years ago

Comment #28 originally posted by Omertron on 2008-12-22T09:30:21.000Z:

As much as I love the potential in YAMJ and test it a lot, this issue is the one reason why I'm, in fact, not using YAMJ yet for everyday use.

Omertron commented 9 years ago

Comment #29 originally posted by Omertron on 2009-01-10T02:59:05.000Z:

+1 Would be nice to not have to Scroll down through a series like Take home chef with 90 Episodes in one season too.....The Folder Structure should dictate the structure of the corresponding menus,

Omertron commented 9 years ago

Comment #30 originally posted by Omertron on 2009-01-24T14:26:25.000Z:

 2 format is great

Omertron commented 9 years ago

Comment #31 originally posted by Omertron on 2009-01-30T22:29:00.000Z:

How do things stand on this issue from the perspective of the coder(s)?

Omertron commented 9 years ago

Comment #32 originally posted by Omertron on 2009-02-07T22:29:20.000Z:

Nobody here but us chickens?

Omertron commented 9 years ago

Comment #33 originally posted by Omertron on 2009-02-08T17:09:14.000Z:

I noticed someone built something called YASP that works with YAMJ to create this function, however I'm trying to find a way to use it with the myihome plugin.

Omertron commented 9 years ago

Comment #34 originally posted by Omertron on 2009-02-08T18:33:31.000Z:

YASP can't be used with alternate skins though. The functionality really needs to be built into YAMJ so that everyone can use it. This is probably the number 1 on my 'hope its implemented soon' list :)

Omertron commented 9 years ago

Comment #35 originally posted by Omertron on 2009-02-09T16:02:12.000Z:

We may have a new developer joining us soon. Perhaps we can assign this to him as his first task. "Throw him to the wolves" as it were. :)

Omertron commented 9 years ago

Comment #36 originally posted by Omertron on 2009-02-09T17:21:59.000Z:

Sounds like a plan. :)

Omertron commented 9 years ago

Comment #37 originally posted by Omertron on 2009-02-09T17:55:33.000Z:

Oh, Mr Altman, you are evil!

Omertron commented 9 years ago

Comment #38 originally posted by Omertron on 2009-02-09T18:50:08.000Z:

God i hope you're not talking about me :)

Omertron commented 9 years ago

Comment #39 originally posted by Omertron on 2009-02-09T19:41:01.000Z:

ha. no not talking about you, James. I was talking about Artem. But he has some decent experience and is well versed in XSLT. I mentioned it to him, if he wants to pick it up, cool. If not, then that's ok too.

Omertron commented 9 years ago

Comment #40 originally posted by Omertron on 2009-02-09T20:49:04.000Z:

whew... :)

Thats good, because I think my coding is gonna fairly limited until after june or so. I got 5 tests left to take (maybe 6) and 4.5 months to take them. So i'm kinda under the gun.

Omertron commented 9 years ago

Comment #41 originally posted by Omertron on 2009-02-09T23:20:45.000Z:

This feature is a must. It will make a huge difference, as mentioned above, 10 folders of friends/fraisure is quite annoying.

Omertron commented 9 years ago

Comment #42 originally posted by Omertron on 2009-02-10T08:11:28.000Z:

Seems like there is a fix on the forum. I've asked if we can include it here: http://www.networkedmediatank.com/showthread.php?tid=16758

I've tested it and it works well with the default skins, but probably needs a moviejukebox.property to turn on/off

Omertron commented 9 years ago

Comment #43 originally posted by Omertron on 2009-02-11T08:12:52.000Z:

<empty>

Omertron commented 9 years ago

Comment #44 originally posted by Omertron on 2009-02-11T08:18:54.000Z:

YES :D

Omertron commented 9 years ago

Comment #45 originally posted by Omertron on 2009-02-14T06:44:44.000Z:

I've submitted my code for this in r735. It is controlled by the configuration property mjb.singleSeriesPage. It defaults to false, so you will have to enable it if you want to use it.

Warning 1: It is broken when you have "special" (season 0) episodes. If you have them, don't enable this yet.

Warning 2: This exposes what I would call a bug in the RobinsJExcal skin. If you want to use that skin with single series pages, please implement this fix. It applies to the version of the skin released on 4 Feb 09.

Omertron commented 9 years ago

Comment #46 originally posted by Omertron on 2009-02-14T11:11:15.000Z:

http://www.networkedmediatank.com/showthread.php?tid=16758&pid=144170#pid144170

Omertron commented 9 years ago

Comment #47 originally posted by Omertron on 2009-02-14T19:07:58.000Z:

I have merged and rechecked our code changes.

There are still no changes in XSL.

The poster and name of the series is taken from the very first available season. There must be possibility to assign different poster to series master (not implemented yet).

There must not be problems with "special" season 0 anymore. Unfortunately I do not have a test scenario for this case.

@all: Please test the r737 with your collections.

Omertron commented 9 years ago

Comment #48 originally posted by Omertron on 2009-02-15T20:45:41.000Z:

Just a note, r738 is back to using characters that windows will not accept for filenames:

Indexing libraries... Error creating XML file: .\temp\Jukebox\TV Series_M_A_S*H_1.xml Exception in thread "main" java.lang.NullPointerException at com.moviejukebox.tools.XMLWriter.writeStartDocument(XMLWriter.java:40 ) at com.moviejukebox.writer.MovieJukeboxXMLWriter.writeMovieXML(MovieJuke boxXMLWriter.java:742) at com.moviejukebox.MovieJukebox.generateLibrary(MovieJukebox.java:364) at com.moviejukebox.MovieJukebox.main(MovieJukebox.java:179)

Omertron commented 9 years ago

Comment #49 originally posted by Omertron on 2009-02-16T19:49:21.000Z:

I guess I can set this to fixed. (see r742) See also Issue 524. It is very similar to this.