Reply to thread

Hmm, you use VideoAspect.Metadata instead of VideoAspect.ASPECT_ID now. Any reason for that?

Another change is that you use GetAttributeValue to get the values. Is there a reason for that? I mean it was basically a Dictionary before and it was less writing to just access the dictionary.

I mean:

item.Aspects[MediaAspect.ASPECT_ID].GetAttributeValue(MediaAspect.ATTR_RECORDINGTIME)).Year

vs:

item[MediaAspect.Metadata].GetAttributeValue(MediaAspect.ATTR_RECORDINGTIME)).Year

I don't really understand the reason of these changes. Sorry for that. I just would like to understand that

Also this:

var ImdbId = movieAspects[MovieAspect.ATTR_IMDB_ID];

vs:

var ImdbId = MediaItemAspect.GetExternalAttribute(item.Aspects, ExternalIdentifierAspect.SOURCE_IMDB, ExternalIdentifierAspect.TYPE_MOVIE);

Any reason for the change? I gues probably the MIA rework, but there must be some fundamental differences?

Otherwise you run a resharper code cleanup to remove all the unused references^^

But the changes to MP2Ext are pretty straight forward, right? Because I moved most Media Library related things to a base class which get accessed over and over again.

I hope you don't have too much compile errors left