Chuckie (@ajtruckle)

Forum Replies Created

Viewing 25 replies - 1 through 25 (of 278 total)

  • Chuckie
    Participant

    @ajtruckle

    That is interesting to know. I didn’t realise there was a filter.

    The logic you describe sounds sensible. I have had to cache values in my InnoSetup installers for performance reasons too.

    Maybe one day I will have a look at this and see if I can write a hook. Is it documented? For example, how do we know what the function definition should be in the php file?


    Chuckie
    Participant

    @ajtruckle

    Fixed:

    input[type="text"]#bbp_topic_title,
    input[type="text"]#bbp_topic_tags {
    	width: 100%;
    }

    This way I don’t affect the checkbox type controls. Now all OK.


    Chuckie
    Participant

    @ajtruckle

    I checked the functions.php file, line 353:

    	// Forums
    	$forum_count = ! empty( $r['count_forums'] )
    		? wp_count_posts( bbp_get_forum_post_type() )->publish
    		: 0;

    So this is a bbPress issue, part of the function bbp_get_statistics. Is there anyway for me to correct this logic without modifying the core …?


    Chuckie
    Participant

    @ajtruckle

    The reason for the issue is because I have this custom CSS that has a side effect on the check box:

    /* Topic Title */
    input#bbp_topic_title, input#bbp_topic_tags {
    	width: 100%;
    }

    This is causing that check box to take on 100% with instead of 13px like the other check boxes. I need to find a way of being able to make my changes above without causing the side effect to the third check box.

    You see, when creating a topic or reply we have a list of tags in a input box and in that context I want 100% width. But when merging topics we are using the same ID for some reason for tags check box and as a result I have a conflict.

    Is there a way to resolve it? Thanks.


    Chuckie
    Participant

    @ajtruckle

    I find it interesting that even with my new theme (which behaves way better than my previous) that the check box is still central:

    View post on imgur.com


    Chuckie
    Participant

    @ajtruckle

    This is resolved for me now as I am using a new theme Unos Business premium. 🙂


    Chuckie
    Participant

    @ajtruckle

    The only limitation is that of WordPress itself. You can’t have more than the fixed number of sub-menus. But atleast you can configure many things in a nice way. 🙂 Now my menu in the backend has some kind of order! 🙂


    Chuckie
    Participant

    @ajtruckle

    If you are interested, I ended up using EnlighterJS. It has a fantastic WordPress module that is compatible with bbPress and is fantastic. User support has been great via GitHub:

    Enlighter – Customizable Syntax Highlighter

    If you need to show code snippets in your forum then this should be the one to use – hands down.


    Chuckie
    Participant

    @ajtruckle

    I used bbp style pack to rename Registered Users as Members. 🙂


    Chuckie
    Participant

    @ajtruckle

    Just to mention that Advanced Menu Editor is a very nice plugin for managing menu positions.

    Admin Menu Editor


    Chuckie
    Participant

    @ajtruckle

    Sounds cool. Unfortunately I am not in a position to purchase the pro versions at the time. But it will be a good change for your users. Well done. 🙂


    Chuckie
    Participant

    @ajtruckle

    But is simply does not make sense. WordPress / bbPress should return public count if not logged on and complete count if logged on – surely. It is false and misleading.

    If they are only listing just the public content then the rest of the figures should match that.

    Not to worry.

    Can we change the terms at all so that they are more consistent with the advanced stats?

    Side point: I put a query in your GD Power Search forum 🙂

    Thanks!


    Chuckie
    Participant

    @ajtruckle

    These are my forums:

    View post on imgur.com

    General Information is public.
    Frequently Asked Questions is public

    That amounts to 2.

    The rest are all private. But even when I am logged in it says to.

    Even the At a Glance area says 2:

    View post on imgur.com


    Chuckie
    Participant

    @ajtruckle

    I have delved into the database using php admin on my domain portal to see what is happening.

    All of the “null” entries are where we have revisions for “replies” to “topics”. So if a “topic” has a revision it is listed. But if it is a reply, it is not listed. I verified this by doing the sql query in the database and those fields are empty.

    So whilst it looks “odd” that there is these null items they are valid. Technically the wp sweep should be improved to obtain those names from the primary topic and list it rather than null.

    But at least we know why now.


    Chuckie
    Participant

    @ajtruckle

    Yes, thanks though!


    Chuckie
    Participant

    @ajtruckle

    Here are the results after running the sweep for revisions:

    View post on imgur.com

    So 3 are good. I am not sure about the two “null” ones though?

    The terms are now purged. 🙂

    Thanks for your help.


    Chuckie
    Participant

    @ajtruckle

    Ok! Then I will go ahead and sweep those terms. Thanks.


    Chuckie
    Participant

    @ajtruckle

    Your new code is working fine for “revisions” 🙂

    It turns out that in the backend after I run your code I am left with two revisions. Both of which are forum posts 🙂 Cool .. well done.

    Maybe we can do the same for the terms (tags)?

    Andrew


    Chuckie
    Participant

    @ajtruckle

    Hi @robin-w

    Thanks for the revised code which I have yet to try.

    There are two aspects in general to consider with respects to the plugin:

    1. The actual sweeping (which you have been looking at)
    2. The actual stats provided on the plugin page.

    Both ideally need to be addressed. For example, you mention about the tags, but in his backend they are referred to as terms:

    View post on imgur.com

    I think terms are unused_terms in the function you referred to. But as mentioned, for this sweeping change to make sense the stats also need revising so they are not included in the counts or detail lists.


    Chuckie
    Participant

    @ajtruckle

    Thanks. I will try this on my beta site tomorrow.

    Will this work for both:

    Revision
    Tag

    ?

    Andy


    Chuckie
    Participant

    @ajtruckle

    The author told me to clone his plugin and extend it to support what we require. Oh well.

    Andrew


    Chuckie
    Participant

    @ajtruckle

    This is resolved.


    Chuckie
    Participant

    @ajtruckle

    If you know how to do the code in php you might be able to build a URL that effectively jumps to that reply in the topic (just like you can see on the standard forum pages).


    Chuckie
    Participant

    @ajtruckle

    Not that I know of. I see what you mean though. You can jump to that post from the parent topic list by clicking on the freshness link. Or by clicking on that page link.

    But at the top where the 1 2 3 > are lists I don’t know how.


    Chuckie
    Participant

    @ajtruckle

    I completely understand where you are coming from. This is only a cosmetic issue and I think I will leave it as it is. But I appreciate the resolution provided.

Viewing 25 replies - 1 through 25 (of 278 total)