• A simple to use plugin which is a very useful addition to any site that wants subscribers. It would be even more useful if it provided “text to show before form” and “text to show after form” filtering hooks though.

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    I knew I thought this sounded familiar when I responded to the support thread. Didn’t catch it was a double-posting.

    Could you let me know your thoughts on a custom filter name vs the generic one? Will be looking to push this out soonish once hearing back from you (and knocking up the stars would be nice too 😉 )

    Cheers.

    Thread Starter Pizdin Dim

    (@pizdin_dim)

    I was specifically referring to allowing for hooks into the “text to show after form” and “text to show after form” textareas.

    I hacked the class-bjgk-genesis-enews-extended.php file like this:

    // Allow filtering on widget text.
    $instance[‘text’] = apply_filters(‘widget_text’, $instance[‘text’], $instance, $this->id_base);

    // Allow filtering on widget text.
    $instance[‘after_text’] = apply_filters(‘widget_text’, $instance[‘after_text’], $instance, $this->id_base);

    In my case I’m trying to allow for Markdown to be used in the two textareas.

    Thread Starter Pizdin Dim

    (@pizdin_dim)

    Sorry, I meant “text to show before form” and “text to show after form” textareas.

    Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    Right, that’s the areas the commit changed, though looking at a specific filter, rather than using widget_text since I’m not sure that plugin devs/site admins using that filter (meant for the default text widget in WP) would expect it to fire on those areas. Thoughts?

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Very useful’ is closed to new replies.