Fix: Raise an exception when trying to create a smart collection/playlist with items

[glensc]

Description

Fixes #1375

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated the docstring for new or existing methods
• I have added tests when applicable

[glensc]

actually, i'm not sure what was happening. as the "items" doesn't seem to be passed to the _createSmart method

[glensc]

so, what was happening was that it called this URL:

• /playlists?smart=1&title=The Melchior Collection&type=video&uri=server://[redacted]/com.plexapp.plugins.library/library/sections/1/all?includeGuids=1&type=1

which seems like smart playlist with no filters (so it contains whole library)

[glensc]

Maybe what could be done here instead is to raise an error if no filters are provided

[JonnyWong16]

I would expect a smart playlist to contain the entire library if no filters are provided. I don't think we should restrict that.

[glensc]

So the change here should be fine. It would disallow using smart and items, possibly not confusing the user.

user can still create smart playlist without filters as long as they don't also pass items.

[JonnyWong16]

Could you also add the same exception for smart collections?

python-plexapi/plexapi/collection.py

Lines 505 to 506 in abcab4f

	if smart:
	return cls._createSmart(server, title, section, limit, libtype, sort, filters, **kwargs)