Add support for 'group' and 'having' keys in smart filters

[Dr-Blank]

SmartFilterMixin

• fixes Filter parsing returns unknown keys group and having #1285

Description

excludes keys not belonging to smart filter

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

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

[JonnyWong16]

I was just looking into how to allow group and having in the search filters as well. We can just add them as manual fields here:

python-plexapi/plexapi/library.py

Lines 2725 to 2731 in 47f11f0

	additionalFields = [
	('guid', 'guid', 'Guid'),
	('id', 'integer', 'Rating Key'),
	('index', 'integer', f'{self.type.capitalize()} Number'),
	('lastRatedAt', 'date', f'{self.type.capitalize()} Last Rated'),
	('updatedAt', 'date', 'Date Updated')
	]

        additionalFields = [
            ('guid', 'guid', 'Guid'),
            ('id', 'integer', 'Rating Key'),
            ('index', 'integer', f'{self.type.capitalize()} Number'),
            ('lastRatedAt', 'date', f'{self.type.capitalize()} Last Rated'),
            ('updatedAt', 'date', 'Date Updated'),
            ('group', 'string', 'SQL Group By Statement'),
            ('having', 'string', 'SQL Having Clause')
        ]

group and having should not have the prefix prepended.

python-plexapi/plexapi/library.py

Lines 2780 to 2785 in 47f11f0

	for field, fieldType, fieldTitle in additionalFields:
	fieldXML = (
	f'<Field key="{prefix}{field}" '
	f'title="{fieldTitle}" '
	f'type="{fieldType}"/>'
	)

        for field, fieldType, fieldTitle in additionalFields:
            if field not in {'group', 'having'}:
                field = f"{prefix}{field}"
            fieldXML = (
                f'<Field key="{field}" '
                f'title="{fieldTitle}" '
                f'type="{fieldType}"/>'
            )

This should allow passing the parsed filters back into search().