- Remove some unused variables - Initialize variables which may otherwise not be available / default case for `switch()` - Better default value for `populate_file_path()` parameter - Error prevention: check that that slug is actually in use - Error prevention: make doubly sure the filesystem is available before using it in a location were we *should* have it as we initialized it ourselves earlier - Some spelling
An already existing filter on individual items is called `tgmpa_table_data_item`. It would therefore be more logical for this filter on the array of items to be called `tgmpa_table_data_items` instead of `tgmpa_plugin_table_items`. This shouldn't give any issues with "existing implementations" as the filter was only introduced about a week ago.
This filter has become superfluous now people can set a `parent_slug` through the config array. Also, the logic of which admin_menu call to use didn't account for that properly and now does.
…e logical in use. Previously `false` had to be returned for TGMPA to load which was confusing. This shouldn't cause issues for "existing implementations" as the filter has only been introduced a week ago and I doubt there are any existing implementations yet.