WordPress.org

Ready to get started?Download WordPress

Forums

WP Stripe
[resolved] shortcode processing (18 posts)

  1. hmoore71@cox.net
    Member
    Posted 2 years ago #

    The short code processing does not replace the short code with the form html. Instead the html is injected using php code and echos causing the form to always be display at the top of the page.

    http://wordpress.org/extend/plugins/wp-stripe/

  2. Noel Tock
    Human Made
    Plugin Author

    Posted 2 years ago #

    Didn't experience that, but can see it happening. Won't be an issue anymore as it'll be in a modal box down the road. Cheers

  3. hmoore71@cox.net
    Member
    Posted 2 years ago #

    Thanks for the reply.
    BTW when is "down the road?"

    Meantime it is an issue and I have modified the code for it to work properly. A trivial fix.

  4. helgatheviking
    Member
    Posted 2 years ago #

    can confirm this is happening for me too. the shortcode echos when it needs to return a value (a long, concatenated string)

    @hmoore i presume you modified stripe-functions.php and stripe-display.php ? any chance you could pastebin those me?

  5. hmoore71@cox.net
    Member
    Posted 2 years ago #

    @helgatheviking I did modify those files. If you would send me an email and I'll send them to you. Use at you discretion.

  6. hmoore71@cox.net
    Member
    Posted 2 years ago #

    @noel Tock would you like the code I modified to maybe issue an update? I am getting a few requests for the fix.

  7. helgatheviking
    Member
    Posted 2 years ago #

    @noel tock, please don't take away the shortcode form. not everybody (at least not me) would want a modal box.

  8. Noel Tock
    Human Made
    Plugin Author

    Posted 2 years ago #

    Thanks for the feedback, you can surely provide your edits (best is via pastebin or gist.github.com) and I'll take a look. Will save me some work :)

    Cheers

    Noel

  9. hmoore71@cox.net
    Member
    Posted 2 years ago #

    I've placed stripe-display.php and stripe-functions.php at http://pastebin.com/qMxhtnG0

    These changes only address the shortcode fix.

  10. Noel Tock
    Human Made
    Plugin Author

    Posted 2 years ago #

    Thanks for that. I fixed it using output buffering which turned out fine (makes the code a bit cleaner in the process). Thanks again..

  11. hmoore71
    Member
    Posted 2 years ago #

    Yes output buffering is cleaner. I have modified WP-Stripe primarily for a Home Owners Association to use ajax. If interested I have it as a demo at
    http://hmoore71.com/wp/card-stripe.

    I have no plans to release the result but happy to share anything you may be able to use for WP-Stripe.

  12. Noel Tock
    Human Made
    Plugin Author

    Posted 2 years ago #

    Cool fork of the existing plug-in, well done! Need to find some time to tackle the projects component of the original plug-in, and then I may have some time to delve into some of your ajax.

    Cheers!

    Noel

  13. helgatheviking
    Member
    Posted 2 years ago #

    nice work indeed. @noel have you given any thought to hosting your plugin at github so @hmoore and any future others could fork you and send you pull requests?

  14. Noel Tock
    Human Made
    Plugin Author

    Posted 2 years ago #

    Hi Helga, I have, just haven't got around to it :) 90-100 hour weeks don't help, maybe I'll find time over this easter break.. Cheers!

  15. helgatheviking
    Member
    Posted 2 years ago #

    wow, that is keeping pretty busy! sidenote, do you think we should be using an SSL cert to accept donations with your plugin?

  16. hmoore71
    Member
    Posted 2 years ago #

    Stripe certainly recommends it see this.
    But it seems something that is outside of the plugin.

  17. Noel Tock
    Human Made
    Plugin Author

    Posted 2 years ago #

    It's certainly recommended. Stripe.js is already served over SSL, but you'll want to load your page over it too as well as the POST data you push out.

  18. helgatheviking
    Member
    Posted 2 years ago #

    thanks gents. good to know. guess i will have to look into SSL. yay, more stuff to learn. ;)

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags