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

Resolve problems pushing to and pulling from Story API #60

Merged
merged 10 commits into from
Sep 27, 2018
Merged

Resolve problems pushing to and pulling from Story API #60

merged 10 commits into from
Sep 27, 2018

Commits on Sep 26, 2018

  1. make sure wp_read_image_metadata() is defined when running media_hand… 

    …le_sideload() during wp_cron

For #59
    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    8a4040f View commit details
    Browse the repository at this point in the history

  2. Fix for Undefined Offset when attempting to delete a story in the API 

    > PHP Notice:  Undefined offset: 0 in /Users/blk/sites/nprapi/wp-content/plugins/nprapi-wordpress/push_story.php on line 99

Seen during GET /wp-admin/post.php?post=6&action=trash&_wpnonce=5b73650f23
    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    c6f505d View commit details
    Browse the repository at this point in the history

  3. Remove duplicate <hr/> in settings for fetching multiple posts; there… 

    …'s a line after this hr for the tags.
    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    c30dde1 View commit details
    Browse the repository at this point in the history

  4. Corrected fix for #59

    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    66e4e9d View commit details
    Browse the repository at this point in the history

  5. Update when the save_post hook is run, because that was not done in #47 

    This fixes the issue described in #57, where the URL parameter ds_npr_update_push was removed along with the 'Push to NPR' submit button, but the URL parameter triggering the nprstory_api_push function was not changed.
    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    8dc1adc View commit details
    Browse the repository at this point in the history

  6. Fix Undefined variable message on "Get NPR Stories" page 

    > PHP Notice:  Undefined variable: story_id in /Users/blk/sites/nprapi/wp-content/plugins/nprapi-wordpress/get_stories.php on line 132

Caused by running is_numeric on a variable that isn't set.
    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    c474542 View commit details
    Browse the repository at this point in the history

  7. Fix for Non-static method DS_NPR_API::load_page_hook() should not be … 

    …called statically

Just had to change how the hook was registered, that's all.
    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    1a55ebf View commit details
    Browse the repository at this point in the history

  8. Modify test to reflect changed action load_page_hook

    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    bd11818 View commit details
    Browse the repository at this point in the history

  9. Update changelog.

    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    5271e25 View commit details
    Browse the repository at this point in the history

  10. Initialize a variable, that will later be used as an array, as an array.

    @benlk
    benlk committed Sep 26, 2018
    Configuration menu
    Copy the full SHA
    63c25f0 View commit details
    Browse the repository at this point in the history