Tagregator filters every request with function that can throw "undefined index" notices

[coreymckrill]

Tagregator currently has four different "source" classes that it uses to aggregate hashtag data. Each of these classes adds a filter to the request hook so that the items in their Post Lists can be sorted by a particular post meta value. ​The filter checks for an orderby query var, and then compares the source's CPT to the post_type query var.

The problem is that some requests don't have a post_type query var, and if WP_DEBUG is turned on, this will throw an Undefined index: post_type notice, which can be especially problematic for Ajax and REST API requests. It also seems a bit overkill to filter every request when the sorting is only used on very specific admin screens.

[iandunn]

Looks good to me :)

[coreymckrill]

Fixed with ​https://plugins.trac.wordpress.org/changeset/1615209/tagregator