xbmc/xbmc — pull request #654 by jmarshallnz (2012-01-24 01:38:44+00:00)
Title: Set nfo scraping - fixes for Eden

This pull req takes care of several related things:

1. Mixed .nfo tag content now overrides any content returned from the scraper. In particular, additive information (genres, sets, cast etc.) is not just tacked on, but instead is overwritten should it be available in the .nfo.

2. The clear attribute was ignored for music video artists.

3. Cosmetic changes to the naming of variables in the infotag loaders. Primarily I renamed chained to append (as chained is a scraper-specific term, and these functions are also used elsewhere) and prefix to prioritise, as not all information is actually prefixed.

4. Added a NEW setting (and NEW language string, as none was available) to the user interface to allow users to control the grouping/flattening of sets in movie title listings. The rationale for doing so is that themoviedb now has the set information, the scraper returns it by default (more information == a good thing) but with the existing advancedsettings only being defaulted to group sets together, this would give anybody not previously using .nfo files (i.e. most users) a change in behaviour. Downside is only the new string, as anyone using as.xml for this feature previously would have had it set to what is now the default.

Please review for stupids, as this is Eden material.