| T O P I C R E V I E W |
| Prinz |
Posted - 12 Sep 2010 : 01:50:42
I know we had already talk about this, but i think the amount of variables for the Movie Card is almost in reverse.
There are 100 (Front-)Cover Variables. But only 20 Fanart Variables. I think in Reality the probability is much higher that for one Movie or especially TV Show there are more Fanart's available than Covers.
I looked in my test database, the highest number for Cover was 48 (a rare example) and the highest number of Fanart was 89 for one TV Show. Then i looked in the thetvdb there a few Big TV Shows (House, Fringe, Smallville, Bones, Doctor Who,...) that have 60-70 and more Fanarts available.
I think i would be more realistically to reduce the number of Cover-variables (if necessary because of the performance) and increase the variables for Fanart.
Maybe some other people could write here what the highest number of Covers, Backcovers and Fanarts for one Movie/TV Show are in there Databases. |
| 20 L A T E S T R E P L I E S (Newest First) |
| JDommi |
Posted - 21 Nov 2010 : 11:20:17
@Prinz:
Yesterday I have uploaded a new CoverManager_Test.rar with ability to batch rename all covers to fit the MovieID.
Requirements:
1. Check IDs if every cover type (cover, backcover, fanart) has the same ID
2. If the check finds any differences: Equalize IDs
3. Rename all covers to fit the MovieID and update their base names in database
To point 3.: I proceed only movies that have a valid entry in field "Cover". The other pics will keep old name.
About 1000 pics per minute were processed on my computer.
A complete renumbering to eliminate the gap of deleted movies is not possible as I don't wanna build the whole table new.
Beside that it's only a work around until Ale has time to correct the Renumbering "bug" in XMM. |
| JDommi |
Posted - 19 Nov 2010 : 22:45:49
Oh yeah, Ale ;)
Seems to be a lot of "fun" for you *ggg*
You have to correct the renumbering bug and add a few more cover categories - but read it (after enjoying to be back again!) |
| Alessio Viti |
Posted - 19 Nov 2010 : 22:36:48
Sorry guys, I haven't read the topic deeply, I have only understand that there is more work for me, right? 
Ale |
| JDommi |
Posted - 19 Nov 2010 : 21:11:26
GREAT, Prinz!
Now let's all call "Ale" to keep him a little further work *lol* |
| Prinz |
Posted - 19 Nov 2010 : 20:56:09
_MOVIE_SEASON_BANNERS_
_MOVIE_SEASON_COVERS_
I got a solution for that. We don't use these Variables.
I would suggest a Naming pattern like this:
MOVIEID-TITLE_BANNER_SEASON01_NUMBER.jpg
example: 301-Chuck_BANNER_SEASON01_1.jpg
MOVIEID-TITLE_COVER_SEASON01_NUMBER.jpg
example: 301-Chuck_COVER_SEASON01_1.jpg
And as the Variables:
_MOVIE_SEASON_BASENAME_
This Variable has always to be set, regardless if there are any Images. Its only used to generate the real filenames with javascript. It's contents is the Path and Basefilename.
Example: C:\extremmoviemanger database\standard_cover\301-Chuck_SEASON.jpg
_MOVIE_SEASON_BANNERSCOUNTS_
_MOVIE_SEASON_COVERSCOUNTS_
These Variables should have the number of Images for each season separated with a commas always beginning with the season 0 up to the last season imported for the TV Show, but at least 2 Entries. For seasons that aren't in the database set the number to -1.
Example what the Variable should give back: -1, 0, 3, 5
It has always to give back at least: -1, -1
These Variables will be used as a javascript array:
var tvshowbannercount = new Array(_MOVIE_SEASON_BANNERSCOUNTS_);
var tvshowcovercount = new Array(_MOVIE_SEASON_COVERSCOUNTS_);
When replaced it's a javascript array:
var tvshowbannercount = new Array(-1, 0, 3, 5);
var tvshowcovercount = new Array(-1, 2, 1, 0);
Because of the use as an array the variables have at least 2 Numbers separated with a comma like i sad.
With this all real Filenames can be generates with javascript. |
| JDommi |
Posted - 19 Nov 2010 : 20:15:52
Okay, that's an argument!
Thx for quick reply ;) |
| Prinz |
Posted - 19 Nov 2010 : 20:14:15
quote:
Originally posted by JDommi
*EDIT*
Btw. why do you need 3 variables? The first additional cover has only an appended _1 to the primary/base name!?!
Speed of the replacement is the simple reason. It's faster to replace the _1 with other numbers. |
| JDommi |
Posted - 19 Nov 2010 : 19:55:53
As soon as Ale (hopefully) implements that (and corrects the renumbering bug) I can activate the new categories in my CM. Five additional types are already possible...
*EDIT*
Btw. why do you need 3 variables? The first additional cover has only an appended _1 to the primary/base name!?! |
| Prinz |
Posted - 19 Nov 2010 : 13:15:33
quote:
_MOVIE_POSTER_
_MOVIE_COVER_ (for DVD, Blue-ray, VHS)
_MOVIE_FRONTCOVER_ (for Jewelcase)
_MOVIE_BACKCOVER_ (for Jewelcase)
_MOVIE_CD_DVD_
_MOVIE_FANART_
_MOVIE_SCREENSHOTS_
_MOVIE_BANNERS_
To add all these new Category's to XMM shouldn't be a big problem anymore. Because the main reason against it doesn't exist anymore. There are only 3 Variables for each new Category necessary anymore. (The primary Image, the first additional Image (_1.jpg) and the COUNT Variable)
quote:
_MOVIE_SEASON_BANNERS_
_MOVIE_SEASON_COVERS_
To integrate these is in XMM will be a bit more difficult, because there have to be grouped as a (Sub-)category for each Season of a TV Show. Especially how to integrate it/display them in the the Edit-Dialog will be a little Challenge.
A special function to import these for the magic script engine should be not that of a problem.
And i would suggest to make all 3 Variables (The primary Image, the first additional Image (_1.jpg) and the COUNT Variable)
of all Category's also available on the Episode Card. Adding the Actors Displaying just like on the movie card to the episode card would be the icing on the cake.
*Edit*
The Variables for these Category's could also be a bit problematic because you would have to have a set for each Season.  That will be a even bigger challenge. I'm thinking about it... |
| katleeh |
Posted - 12 Nov 2010 : 04:19:47
On the concept of having the other variables available, has anyone made a moviecard using those yet? IE: the banner and poster one?
- KatLeeH |
| Alessio Viti |
Posted - 10 Nov 2010 : 13:43:54
Perfect! Thank you again!
Ale |
| Prinz |
Posted - 10 Nov 2010 : 13:36:32
In my tests everything work as it should. |
| Alessio Viti |
Posted - 10 Nov 2010 : 13:31:48
Great Prinz!
So, if everything is OK for you I will upload the new release in few hours...
The main moviecard will be movie_bigcover
Ale |
| Prinz |
Posted - 10 Nov 2010 : 13:05:37
New exe is working.
Here are the new Movie Cards that use the new function.
As an example a added a Card: movie_bigcover-limited.htm that shows how to limit the amount of image that are shown (the old limit 20/50 is used in this example movie card):
quote:
var relTo = document.getElementById ("extracover");
var relTo2 = document.getElementById ("Additional_Fanart");
var fmc = $('#mcov1').text();
var fmf = $('#mfan1').text();
var mc_count = _MOVIE_COVERCOUNT_;
if (mc_count > 20) {mc_count = 20;}
var mf_count = _MOVIE_FANARTCOUNT_;
if (mf_count > 50) {mf_count = 50;}
for (var mic = 1;mic < mc_count;mic++) {
relTo.insertAdjacentHTML('BeforeEnd','<img src="' + fmc.replace('_1.','_' + mic + '.') + '" onerror="ImgError2(this);" class="my_img">#013;');}
for (var mic = 1;mic < mf_count;mic++) {
relTo2.insertAdjacentHTML('BeforeEnd','<img src="' + fmf.replace('_1.','_' + mic + '.') + '" onerror="ImgError2(this);" class="my_img">#013;');}
http://www.mediafire.com/download.php?cyjrrl0yl1kpzfe
PS: On my PC the new movie cards are even loaded quicker with all images as before the old "hard-coded variables" movie cards and less images. |
| Alessio Viti |
Posted - 10 Nov 2010 : 12:08:55
OK Prinz...
I will leave the number to 20 for the REPLACE of _MOVIE_COVER_ (to support the old moviecards without java) and I will remove the limit for count the number of covers.
Thank you! |
| Prinz |
Posted - 10 Nov 2010 : 11:51:08
quote:
Originally posted by Alessio Viti
Because count the number of images (i.e. check if the files exist) can take some time too, right?
You mean you have to check if a file exists inside XMM itself? I don't know how much time your program needs to make this checks and get the image count for the variables...
There is no need to check if the file exists inside the movie card anymore, because if the variable says there are for example 103 only these images are added via javascript to the card. That makes the loading of the card faster if there is only 1 Image only 1 is added to the Card, if there is none there is no image added to the card... If you wont to add less then the maximum to a card a simple additional check in javascript will reduce the amount of added image.
So from the Movie Card HTML javascript programing side there is no need for any restriction to that number, that won't change the loading time at all. That is the good thing about adding the images this way.
PS.: I had to change the moviecard a little bit, so don't use the test card that i upload. That has a few bugs... |
| Alessio Viti |
Posted - 10 Nov 2010 : 11:30:57
Hello Prinz,
Yes, you are right, if you have less images you get the exact number, otherwise you get the maximum.
OK, if we are ready I will raise this number and maybe upload the new release...
How many covers I should put as maximum? Because count the number of images (i.e. check if the files exist) can take some time too, right?
Thank you
Ale |
| Prinz |
Posted - 10 Nov 2010 : 10:35:13
Somehow the variables don't allways give back the correct number of Images. It seams that the numbers use the old maximum restriction. So for normal covers the number is never higher than 20 and for fanart never higher than 50. But the should give back the real number of Images for that movie/tv show. Restricting the number of shown images is easily done on the movie cards itself with the javascript function.
PS.: Even now the loading time is noticeable faster on my pc. |
| Alessio Viti |
Posted - 10 Nov 2010 : 04:27:17
Ops...
OK, I sent new exe with the changed variables.
Thank you! |
| Prinz |
Posted - 09 Nov 2010 : 15:45:46
The exe you sent doesn't work. It replaces the first part with _MOVIE_COVER_COUNT_ with the picture, because that part also matches that variable.
I think we have to rename the variables for the image count for example to:
_MOVIE_COVERCOUNT_
_MOVIE_BACKCOVERCOUNT_
_MOVIE_FANARTCOUNT_
so that the magic script engine doesn't confuse the variables. |
