Skip to content
This repository has been archived by the owner on Mar 26, 2021. It is now read-only.

Fixed several bugs, added critical filters, added composer.json #38

Merged
merged 4 commits into from
Dec 20, 2016

Conversation

mikeschinkel
Copy link
Contributor

@mikeschinkel mikeschinkel commented Aug 21, 2016

So you have version 1.6 on WPackagist.org but there are fixes on GitHub that have not made it into an update on WordPress.org.

By adding this composer.json we can use Composer to pull "dev-master" even if you have not officially released the code yet to WordPress.org.

Merge pull request npr#32 from newclarity/master
…st_meta(), wp_set_post_categories() and resolving category names upstream of wp_set_post_categories.

Fixed require_once() statements to include full path and avoid weird path-related errors.
Fixed misspelled variable named $params
Replaced call to missing t() function with call to __().
@mikeschinkel
Copy link
Contributor Author

mikeschinkel commented Dec 13, 2016

Hi there.

I have just added to this pull request including fixing a few bugs. Here is what I added today:

  • Fixed misspelled variable named $params
  • Replaced call to missing t() function with call to __().
  • Fixed require_once() statements to include full path and avoid weird path-related errors.
  • Added hooks to filter post $args prior to wp_insert_post(), update_post_meta(), wp_set_post_categories() and resolving category names upstream of wp_set_post_categories.

This last one is a set of hooks critical for a station that needs to customize the import of content for their needs.

Any chance you guys can review this any time soon?

@mikeschinkel mikeschinkel changed the title Added a composer.json so "dev-master" can be accessed from Composer Fixed several bugs, added critical filters, and added a composer.json Dec 13, 2016
@mikeschinkel mikeschinkel changed the title Fixed several bugs, added critical filters, and added a composer.json Fixed several bugs, added critical filters, added composer.json Dec 13, 2016
@mikeschinkel mikeschinkel reopened this Dec 13, 2016
@mike-douglas
Copy link
Contributor

Thank you for submitting this, and apologies for not getting to it sooner!

@mike-douglas mike-douglas merged commit 1e9c7d4 into npr:master Dec 20, 2016
@mikeschinkel
Copy link
Contributor Author

@mike-douglas Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants