publishiza icon indicating copy to clipboard operation
publishiza copied to clipboard

When posts are scheduled, it doesn't storm.

Open aaronjorbin opened this issue 8 years ago • 2 comments

Scheduled storms don't cause any tweets.

I haven't tested, but I believe this is due to the nonce verification in publishiza_publish_post. Since

aaronjorbin avatar Dec 30 '16 05:12 aaronjorbin

Looks like it is also the if ( empty( $_POST['publishiza'] ) ) { check. I wonder if there should be a hook on transition_post_status that:

  1. Looks to see if it's being set to future, and if so, set some meta to check against later.
  2. On future -> Publish, do the storm if the meta set earlier is there.

aaronjorbin avatar Dec 30 '16 05:12 aaronjorbin

Confirmed. I never got around to integrating Publishiza into future dated posts.

And transition_post_status is the place this should happen, if we decide this is a feature worth pursuing. I can see why this would be attractive.

I think the code responsible for doing the actual push to Twitter should be abstracted enough to allow for parsing the text and sending the tweets, but I don't remember. If not, it should be.

JJJ avatar Feb 20 '17 17:02 JJJ