Project

General

Profile

Actions

Bug #274

closed

Recent Comments on Academic News Blog

Added by Sarah Morgano almost 14 years ago. Updated over 13 years ago.

Status:
Resolved
Priority name:
Low
Assignee:
Ron Rennick
Category name:
WordPress (misc)
Target version:
Start date:
2010-07-20
Due date:
% Done:

100%

Estimated time:
1.50 h
Deployment actions:

Description

I noticed that all of the links from the comments on the Academic News Blog go to my '6 degrees of facebook' post instead of each respective post:

Recent Comments
Scott Voth on Apture Plug-in for WordPress
Scott Voth on Apture Plug-in for WordPress
cunycommons on Apture Plug-in for WordPress
Brian Foote on The Round Up I Wrote After Not Seeing Any Fireworks 6/28 – 7/4
zlobley on The Round Up I Wrote After Not Seeing Any Fireworks 6/28 – 7/4

Best,

Sarah

Actions #1

Updated by Boone Gorges almost 14 years ago

  • Assignee set to Ron Rennick
  • Priority name changed from Normal to Low

Ron, can I ask you or Andrea to take a look at this? (No huge rush.) The blog in question is on cdev at news.cdev.gc.cuny.edu. It's populated by FeedWordPress, pulling in posts from a couple other blogs on the network. Check out the hrefs on the post titles over on the Recent Comments widget. They're all point toward the URL of the very last post that is appearing on the front page post loop.

Haven't looked at the code, but I imagine that the query isn't being reset, or that the widget is using the_permalink() outside of the proper loop, or something along those lines. Might also be connected with the fact that FWP sets the guid to something other than the address of the post on the current blog. Those links ought to point to the original post.

Actions #2

Updated by Ron Rennick almost 14 years ago

That's running a filter on the_permalink. If you hover over the titles in the posts, they link to the original posts. Probably just needs a "in the loop" check.

Actions #3

Updated by Boone Gorges over 13 years ago

Ron, would you mind making the appropriate updates to the theme?

Actions #4

Updated by Boone Gorges over 13 years ago

  • Target version set to 1.0.1
Actions #5

Updated by Ron Rennick over 13 years ago

I'm away until Tuesday. I'll have a look when I return.

Actions #6

Updated by Boone Gorges over 13 years ago

  • Target version changed from 1.0.1 to 1.0.2

Bumping to 1.0.2 so we can get 1.0.1 out the door.

Actions #7

Updated by Boone Gorges over 13 years ago

(Thanks, btw!)

Actions #8

Updated by Ron Rennick over 13 years ago

I have this fixed on my local copy. Am I still updating master?

Actions #9

Updated by Boone Gorges over 13 years ago

Sure, the master is fine.

Actions #10

Updated by Ron Rennick over 13 years ago

  • % Done changed from 0 to 100
  • Estimated time set to 1.50 h

I re-read your notes on the 1.0 branch & pushed the changes to that branch instead of the master.

Actions #11

Updated by Boone Gorges over 13 years ago

  • Status changed from New to Resolved

Looks fantastic, Ron. Your push to the 1.0-branch made my life a little easier :) and the fix looks perfect. Thanks.

Actions

Also available in: Atom PDF