Search

bbPress Showcase » bbPress Chat

Patches for 2 tickets

RSS
  • Started 1 year ago by Gautam » latest reply by _ck_ (2 voices)
  • 2 posts, 785 views.
  • This topic is not a support question
  • (no related topics found)

  1. Gautam
    Gautam

    junior member
    Joined: Jul '09
    Posts: 6

    offline

    Can you post a patch for these 2 tickets or should we move them forward?:

    1. http://trac.bbpress.org/ticket/1182
    2. http://trac.bbpress.org/ticket/1183

    - Gautam ( http://www.gaut.am )
    Posted 1 year ago #
  2. _ck_
    _ck_

    senior admin
    Joined: Jul '06
    Posts: 908

    offline

    Unfortunately they don't really matter anymore as it would require going back to change plugins to optimize them, which is highly unlikely to ever happen now.

    The first one is not too hard to fix though.

    My best guess would be in functions.bb-core.php

    line 992
    return new BB_Query( 'topic', $query_args, "bb_view_$view" );

    should become

    return empty($query_args) ? false : (new BB_Query( 'topic', $query_args, "bb_view_$view" );

    In theory that should prevent a wasted query when the query is empty.

    Completely untested though and nothing right now would take advantage of it.

    Posted 1 year ago #

RSS feed for this topic

Post a reply to “Patches for 2 tickets”

You must log in to post.

keep _ck_ coding >> donate $5 <<     Theme Switcher:
34 users online from in the past 30 minutes. 15 bots 19 guests
7,421 views today 8,426 yesterday 16,043 peak. Most at once 50 today 62 yesterday 131 peak. Visited today: Amylawrence, guava, JeffreyCasivant, wigbuy123, gdfddfs, nandao, _ck_, summerfortun
2,288 posts in 394 topics over 66 months by 430 of 2,167 members. Latest: watchessaleus, wigbuy123, JeffreyCasivant