brent0r (@brent0r)

Forum Replies Created

Viewing 22 replies - 1 through 22 (of 22 total)

brent0r
Participant

@brent0r

I’m not sure if this is displayed correctly or not. Appears the li code is dropping outside of the post. It also breaks the template.


brent0r
Participant

@brent0r

So you’re wanting the forum to display within the buddy press template, correct?


brent0r
Participant

@brent0r

I too would be interested in a drop-down solution. I’ll trial a few options this coming weekend. I like the idea of having a button that replaces the form and opens a new page too.


brent0r
Participant

@brent0r

@kdelsimone
If you want the link changed from the topic to the reply, I suggest copying loop-search-reply.php from /wp-content/plugins/bbpress/templates/default/bbpress to your child themes bbpress directory @ /wp-content/themes/theme-child/bbpress
From there you can edit it and change line 25 from this:
<a class="bbp-topic-permalink" href="<?php bbp_topic_permalink( bbp_get_reply_topic_id() ); ?>"><?php bbp_topic_title( bbp_get_reply_topic_id() ); ?></a></h3>

to this:
<a class="bbp-topic-permalink" href="<?php bbp_reply_url(); ?>"><?php bbp_topic_title( bbp_get_reply_topic_id() ); ?></a></h3>


brent0r
Participant

@brent0r

You can also go directly to the reply by clicking the post #number on the top right of each result.
Are you wanting to change your search results so that “In reply to: Topic Title” redirects to the #post within that topic?


brent0r
Participant

@brent0r

functions.php

add_filter( 'bbp_get_topic_freshness_link', 'rew_freshness_no_date_link' , 100, 5) ; 
	
function rew_freshness_no_date_link($anchor, $topic_id, $time_since, $link_url, $title) {
	if (!empty ($topic_id)) {
	?>
	<a href="<?php echo $link_url; ?>"><?php echo $title ;?></a> (<?php echo $time_since ; ?>)
	<?php
	}
	else echo 'No topics' ;
}

It works perfectly when viewing the category page that lists all topics. When I click the topic however and view the post it displays the following in the breadcrumbs:
› Forums › General › Mobile General › hi Reply To: test (3 weeks, 2 days ago)

See how it adds the topic title with (the date) that has its link removed, to the end of the breadcrumb.


brent0r
Participant

@brent0r

Yes, that also looks like you did change that previously.
It’s like the breadcrumbs catching this change and adding it.


brent0r
Participant

@brent0r

So the last revision helps a little, but in my case there is a last post date in the breadcrumbs. (https://goo.gl/6iA5Sn)
Do you get that when viewing a post?


brent0r
Participant

@brent0r

Thanks again Robin, yes you’ve understood exactly what I was trying to achieve. I tested the above example, it throws Notice: Use of undefined constant active_id – assumed ‘active_id’ in /var/www/html/wp-content/themes/name/functions.php on line 94

Once I remove bbp_topic_permalink($active_id) eg. bbp_topic_permalink() it works as it should. (I’m not sure if removing that is the right way to go, but doing so seems to help it work)

There is one little issue though which I’ve noticed since applying the above. The breadcrumb links now have an additional link and date at the end of the breadcrumb. (example: https://goo.gl/9q3NjW) Posts and the forum index are not effected though.

Thanks once again 🙂


brent0r
Participant

@brent0r

When disabling that theme does the layout change? To me there looks like something else changing the layout, if you haven’t played around with it in anyway.
Try disabling any plugins too, if you have any.
Failing that, is this your first time installing bbpress? How did you go about installing/setting it up?


brent0r
Participant

@brent0r

When I use your code above, I get the 3 last posts without a link on the date/time.

When I am not using your 3 last posts code (different site) from above, bbpress naturally links the last post.
How would I best go about removing just the link from a single last post in that example?

I’ve figured that I could use your code as above but change to ($count == 1). I’m obviously not experienced with writing code but to me it seems an excessive way to go about it.


brent0r
Participant

@brent0r

Link from OP.

It all looks pretty good. There’s a couple of problem areas I can see though. Are you using any custom CSS or themes?


brent0r
Participant

@brent0r

Update (ran out of editing time) – in my post earlier I used bbp_forum_last_active_time which displays no link. Unfortunately forums with no posts don’t display “No topics” but instead show a time (which is likely when that forum was created?).


brent0r
Participant

@brent0r

Thanks Robin W, working really well. And thanks for taking your time to help.

One last question which is kind of related. Is it possible to remove the href link from the date/time of the last post (clean bbpress install)?

Thanks.


brent0r
Participant

@brent0r

I’ve managed to find a solution, I will add styling later to both topic title and date.
<a href="<?php bbp_forum_last_reply_url(); ?>"><?php bbp_forum_last_topic_title() && bbp_forum_last_topic_title(); ?> (<?php bbp_forum_last_active_time(); ?>)</a>

I’d still love to find a way to display 3 last posts rather than just one.

In reply to: Freshness Link

brent0r
Participant

@brent0r

Is it possible to post your final, working code used?


brent0r
Participant

@brent0r

Hi Robin W, I am using bbp style pack (your plugin, right?) to change freshness to Last Post. BTW, love your plugin!

Also, as well as wanting to have the topic title and date on one line, is it possible to have the 3 most last posts displayed rather than one?

Thanks again.


brent0r
Participant

@brent0r

I really like what you’ve done. I’m struggling to make changes to several areas and it looks like you’ve managed to get it perfect!

Specifically the “latest post / freshness” changes on the forum index. I love how its formatted with picture on left, followed by forum and dates. Also the Replies: Views: changed on each section is really nice too 🙂

Any chance you would share the template changes, it would really help me achieve similar results and learn how you got there.

Cheers 🙂

Viewing 22 replies - 1 through 22 (of 22 total)