Opened 18 years ago
Closed 18 years ago
#3916 closed defect (bug) (fixed)
Fix for 'link_rating' filter
Reported by: | chriskbchan | Owned by: | rob1n |
---|---|---|---|
Milestone: | 2.2 | Priority: | low |
Severity: | minor | Version: | 2.1.2 |
Component: | General | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
The 'link_rating' filter is not called even 'show_rating' is set in any get links functions. This fix should enable this by calling 'get_linkrating' appropriately.
Attachments (3)
Change History (17)
#2
@
18 years ago
- Keywords has-patch added
- Milestone changed from 2.1.3 to 2.2
- Resolution fixed deleted
- Status changed from closed to reopened
#5
@
18 years ago
Originally I did put the space before the ratings, however, I think it's what to be done by the plugin, as you could check out my blog http://blog.chris-chan.com/, I have the plugin showing an image rather text, without the space it actually looks nicer! Please comment.
#6
@
18 years ago
Without the space, the default behavior (putting the number in) looks very bad. Though I agree not many people are likely to use the un-filtered default behavior.
I don't see any link ratings on your site...
#7
@
18 years ago
You should find the link ratings under the 'Recently Watched Movies' on the sidebar. I have written a very simple plugin to represent the rating with 'stars' according to the rating I have given to a particular movie (that's how I find this patch necessary since without this modification my plugin will not work). As the 'stars' images I have been using have some white spaces before them, with an extra space the 'stars' will be further apart from the link. As we can't assume what will be displayed by the plugin, either Text or Image, I guess the best place to do the spacing will be from the plugin. Just my 20cents.
#8
@
18 years ago
One more thing, if you check the original source, those codes for appending contents for 'show_updated' & 'show_description' didn't assume space before them also. So I think it's better to discard the space.
#9
@
18 years ago
Not quite true. show_description has an extra flag "between" to say what to put between the image and description.
But point taken.
#10
@
18 years ago
- Keywords tested added; reporter-feedback removed
Recommend taking first patch above (without spaces).
I tested this patch, and although it adds the rating text, I think maybe it needs a space in there. The default behavior in today's trunk anyway (2.1.2 version, approximately) is that get_linkrating has no filters, so it is just putting out a number, and it gets appended with no spaces onto whatever else is being printed out (such as the description). Add a space to the patch, right before the rating print-out, and things will be nicer...
Just a thought...