[Plugin: WPBook Lite] Minor typo in wpbook-cron

  • Resolved Jeffrey Schutzman

    (@jeffpyebrookcom)


    11 years, 10 months ago

    Hi,

    Using version 1.4.2

    Line 226 missing quotes, is
    $my_timestamp = $my_timestamp_results[meta_value];
    should be
    $my_timestamp = $my_timestamp_results[‘meta_value’];

    BTW, I love the plugin. I have done a little hacking to make a couple of enhancements that could be integrated into the main version if you are interested. Right now they are in a plugin so as not to distrub the code base. The changes I made let the plugin handle custom post types (commented out the one line). More interesting the publish hooks don’t directly send to facebook. Instead a cron wakes and look for posts to publish. If it finds one it sends it, and then sleeps. The cycle handles custom post types so it will send one of our designs, then one of our portfolio entries, then one of our posts. THere is a throttle build in so fb doesn’t get bombarded and people don’t get too annoyed. Let me know if you want the code. It is small and all it needs is a configuration UI.

    thanks again.

    http://wordpress.org/extend/plugins/wpbook-lite/

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter Jeffrey Schutzman

    (@jeffpyebrookcom)

    11 years, 10 months ago

    Also,

    Line 129 in wpbook-lite.php changing the ‘8’ to ‘manage_options’ seems to eliminate a deprecated argument warning?

    It should look something like this:

    $wpbook_lite_plugin_page = add_options_page(‘WPBook Lite’, ‘WPBook Lite’, ‘manage_options’, basename(__FILE__), ‘wpbook_lite_subpanel’);

    Plugin Author John Eckman

    (@johneckman)

    11 years, 10 months ago

    Thanks Jeff – I’d love to see the code you’ve done.

    I’ve been exploring moving the whole plugin over to github and only posting new releases the the plugin repo – that would make it easier for folks to contribute patches.

    I think we need a UI for custom post types though – people should be able to choose which custom post types WPBook should publish and which not.

    The cron approach to publishing could be interesting too as it would lessen the time delay when people hit publish – it’s already triggered on post transition for scheduled posts, but for “live publish” it feels a bit sluggish

    Plugin Author John Eckman

    (@johneckman)

    11 years, 10 months ago

    Fixed both of these in 1.4.3, just committed.

    Also migrated the code to github – https://github.com/jeckman/wpbook-lite – so you can send me a pull request or diffs of whatever.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘[Plugin: WPBook Lite] Minor typo in wpbook-cron’ is closed to new replies.