Sam Bauers (@sambauers)

Forum Replies Created

Viewing 25 replies - 401 through 425 (of 1,069 total)
  • @sambauers

    Participant

    I think that query is generated by the forum-last-poster plugin.

    If it is the one that is showing up over and over again, then get rid of it.

    @sambauers

    Participant

    Well, this isn’t happening for me. Is the problem on this site? http://chocolatebydeath.com/forum/

    @sambauers

    Participant

    The only real problem you have is that the Warning is being printed to screen. On a production site you shouldn’t print PHP errors on the screen.

    The warning in that case should be dealt with though and I will do so in trunk for 1.0. If you want to modify the bb_safe_redirect() function then use the below code for a plugin to rid you of the warning.

    <?php
    /*
    Plugin Name: Suppress bb_safe_redirect Warnings
    Plugin URI:
    Description:
    Author:
    Author URI:
    Version: 0.1
    */

    function bb_safe_redirect($location, $status = 302) {

    // Need to look at the URL the way it will end up in wp_redirect()
    $location = wp_sanitize_redirect($location);

    // browsers will assume 'http' is your protocol, and will obey a redirect to a URL starting with '//'
    if ( substr($location, 0, 2) == '//' )
    $location = 'http:' . $location;

    $home = bb_get_uri(null, null, BB_URI_CONTEXT_HEADER);

    if ( !$lp = @parse_url($location) )
    return wp_redirect($home, $status);

    $wpp = parse_url(bb_get_uri());

    $allowed_hosts = (array) apply_filters('allowed_redirect_hosts', array($wpp['host']), isset($lp['host']) ? $lp['host'] : '');

    if ( isset($lp['host']) && !in_array($lp['host'], $allowed_hosts) )
    return wp_redirect($home, $status);

    return wp_redirect($location, $status);
    }
    ?>

    @sambauers

    Participant

    The filter will only be passed one argument by default, so you need to change your add_filter() call to this:

    add_filter('get_user_profile_link', 'ss_get_user_profile_link', 10, 2);

    .

    The third parameter is the priority, the second is the number of arguments to pass.

    @sambauers

    Participant

    I’m not exactly sure what you mean. There comes a point where we can’t reduce query count any further without resorting to complex joins, and to allow ourselves flexibility we need to avoid those. Caching solutions prefer simple queries and 1.0 has much better caching capabilities thanks to the use of the object cache functions in BackPress.

    Also take a look at the WordPress memcached plugin, which is a drop-in replacement for the BackPress object cache functions and class, which hooks into a memcache server.

    MySQL query caching should alleviate most speed problems though.

    @sambauers

    Participant

    If the password fields are left blank they should be ignored.

    @sambauers

    Participant

    You have too much code. You have duplicated the function by the looks of things. Filters don’t work that way.

    Try this documentation on the plugin API for more info:

    https://codex.wordpress.org/Plugin_API

    In reply to: Sitemap generator

    @sambauers

    Participant

    You have a blank line at the start of the plugin file. Remove it.

    In reply to: Accents in username

    @sambauers

    Participant

    Great!

    In reply to: Post and topic status

    @sambauers

    Participant

    For topic_status:

    0 = normal

    1 = deleted

    For post_status:

    0 = normal

    1 = deleted

    2 = spam

    It’s safe to delete these as long as you are careful if you ever backup and restore your database not to have the deleted IDs be allowed to be repopulated.

    There very well could be rows in topicmeta for topics with topic_status > 0

    @sambauers

    Participant

    WordPress and bbPress use the PHPass hashing mechanism to store passwords in the database.

    You can also “odwngrade” the password hashing to MD5 using this plugin…

    https://bbpress.org/plugins/topic/md5-insecurity-for-bbpress/

    @sambauers

    Participant

    Setting $bb->uri in your config file still works (and always should).

    @sambauers

    Participant

    Heh, it almost takes video proof for people to believe that it works sometimes.

    There doesn’t appear to be anything wrong with that bb-config, but you should just let the installer create it for you.

    BackPress is included with bbPress when you grab it via SVN or in the 1.0-Alpha packages. SO there is no need to get it separately.

    In reply to: replacing page title

    @sambauers

    Participant

    The problem with your code was that you were adding a filter to bb_title via an action run when the title was already written to the browser. You should have added the filter at bb_init or just left it on it’s own in the plugin.

    Also, I don’t think there is particularly a need to remove the existing filter on bb_title in your case.

    @sambauers

    Participant

    This works fine for me using trunk, but it does depend on your email client as well (I was using mail.app on Mac OS X).

    There are lots of places that character encoding could fail along the way, PHP could be the culprit, so could the email server that the message is first sent through. Perhaps the mail client you are looking at it in doesn’t support or isn’t setup to view utf-8.

    We have replaced the email sending code in Trunk, and that might be the difference too.

    It would be good to see exactly what’s happening, can you post a link to your forum so we can try signing up to see what happens?

    @sambauers

    Participant

    They are doing it somehow over at edublogs.org but I think that is a highly customised bit of code where they are sharing one bbPress install amongst multiple blogs in WPMU.

    bbPress doesn’t offer this out of the box, no.

    In reply to: Get Current Forum ID?

    @sambauers

    Participant

    Untested and YMMV:

    $current_forum_id = get_forum_id();

    while ( bb_forum() ) {
    $class = '';
    if ($current_forum_id == get_forum_id()) {
    $class = 'on';
    }
    }

    In reply to: Accents in username

    @sambauers

    Participant

    @ meitershaker

    There was a bug in the display name implementation. First, update to the very latest trunk. There is a chance that you might have a user meta item for some users called ‘display_name’. This will be stomping on the real setting in the user table. Use this SQL query to clear out the bad data.

    DELETE FROM bb_usermeta WHERE meta_key = 'display_name';

    Once you’ve updated to the latest trunk and done that, you should be able to use any UTF-8 character in display names.

    @sambauers

    Participant

    WordPress actually keeps a reference to the old URL and sets up a redirect automatically I believe. So there it stays permanent. We don’t do that yet.

    @sambauers

    Participant

    Not really, bbPress and WordPress themes aren’t compatible without some work. I am trying to steer people away from loading WordPress “inside” bbPress.

    Much better approach is to port your WordPress theme across to bbPress.

    @sambauers

    Participant

    No, just install as normal in step 1 add the KEYS as they exist in your WPMU install and by the time you get to step 2 (integration) you should have a bb-config.php file. Add the SALTS in at the start of step 2 before you submit your integration details.

    I made a change today to trunk… https://trac.bbpress.org/changeset/1747

    This should allow you to successfully integrate and select an admin from the blog with ID of 1 (usually the primary blog) in your WPMU install as a keymaster.

    Hopefully this will sort you out.

    @sambauers

    Participant

    I believe that change was your request _ck_

    Happy to help/annoy!

    : )

    @sambauers

    Participant

    Is this a standalone bbPress or integrated with WordPress?

    @sambauers

    Participant

    We aren’t including Snoopy, only WP_Http (which is only around 24k of code if you discount the comments)

    The cron job is triggered by pageviews and then only if there are jobs to run.

    So on a visit to the site, the cron job list is checked (just an option in the meta table) then if there is something there to do, bbPress sends an HTTP request to itself, which the user doesn’t need to wait for.

    In reply to: replacing page title

    @sambauers

    Participant

    +1 johnbillion

Viewing 25 replies - 401 through 425 (of 1,069 total)