Search

bbPress Showcase » bbPress Chat

Patches for 2 tickets

RSS
  • Started 1 year ago by Gautam » latest reply by _ck_ (2 voices)
  • 2 posts, 759 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: 884

    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:
42 users online from in the past 30 minutes. 18 bots 24 guests
1,634 views today 8,665 yesterday 14,010 peak. Most at once 42 today 48 yesterday 131 peak. Visited today: Dickinsonjohn9, shairah27, xiaotian
2,245 posts in 385 topics over 63 months by 426 of 1,989 members. Latest: dxdiag02, Pablo01, Dickinsonjohn9