Edit Library and Show's Advanced Settings

[blacktwin]

Allow for editing a show's advanced settings

show.editAdvanced(showOrdering='aired', autoDeletionItemPolicyWatchedLibrary=0)

Allow for editing a library's advanced settings

library.editAdvanced(enableBIFGeneration=0, collectionMode=2)

[blacktwin]

I can move/add these methods to settings.Preferences if I carry over the show's key into settings.Preferences's _initpath.

    def preferences(self):
        """ Returns a list of :class:`~plexapi.settings.Preferences` objects. """
        items = []
        data = self._server.query(self._details_key)
        for item in data.iter('Preferences'):
            for elem in item:
                setting = settings.Preferences(data=elem, server=self._server)
                setting._initpath = self.key
                items.append(setting)

        return items

This way the settings could be updated from the show object in mass or from the preference object.

[blacktwin]

I should add some tests to this. Switch settings, reload, assert.

[blacktwin]

@Hellowlol this should be ready for review.

[Hellowlol]

Is it still needed to decode this? py2 has been dropped.

[blacktwin]

.default is a value so it also is a byte that is either an int or str.

[Hellowlol]

same again, why do we need to decode?

[blacktwin]

The .value can be classed as a byte that is either an int or str. Majority of the time they're int -1, 1, 2 something like that.

[blacktwin]

    _bool_cast = lambda x: True if x == 'true' or x == '1' else False
    _bool_str = lambda x: str(x).lower()
    _str = lambda x: str(x).encode('utf-8') # py2 dependant?
    TYPES = {
        'bool': {'type': bool, 'cast': _bool_cast, 'tostr': _bool_str},
        'double': {'type': float, 'cast': float, 'tostr': _str},
        'int': {'type': int, 'cast': int, 'tostr': _str},
        'text': {'type': str, 'cast': _str , 'tostr': _str},
    }

Updating the the tostr: _str references to tostr: str should resolve the issue.

[jjlawren]

Calling PlexServer.settings.get("FriendlyName") is returning an empty value for some reason in this test, and only for this PR. Could one of the changes/tests be inadvertently clearing it somehow? Or maybe it just needs a reload() at the start of the failing test?

[coveralls]

Coverage increased (+0.4%) to 72.024% when pulling 0123904 on show_advanced_setting into 737401b on master.

[blacktwin]

This PR should be gtg.