hansfn wrote:First of all, making a modified version of makeCategoryLink seems like the obvious thing to do. However, there were some reasons I didn't suggest that - the same reasons we haven't created the function yet:
1) A category link will give you all entries in that category (independent of weblog). However, a user link will give you only the entries written by that user in a specific weblog.
2) A user can have written both entries and page, but the user link only returns the entries. (Pages don't have categories so there is no such problem for categories.)
So to make it clear what you get from a user link a decided to include the weblog parameter. When we have fixed the user link, at least issue 1 above, I'll create a function makeUserLink (or makeAuthorLink).
Okay, I don't understand what difference the weblog parameter makes or doesn't make. Perhaps its because my site isn't as extensive as some others. I'm simply trying to create a link to a page that shows the entries a user has authored from the blogstats snippet.
The snippet doesn't count pages, so that point is moot for the extension's purposes.
However, what I might do, since the extension counts an author's entries based on included categories, is create a link that looks like ?u={author}&c={included cats}. However, I don't know if that would be any more or less desirable than passing a weblog parameter. It would, however, keep the output consistent with the count of the extension. For instance, we want to count an author's entries in three categories of a 6-category weblog. The link with his username, therefore, would only show his entries in those three categories (i.e., ?u=author&c=cat1,cat2,cat3).
hansfn wrote:The mod_rewrite parameter/prefix for a user link is "author" - see example.htaccess.
Yeah, I thought of this before coming back to the keyboard this evening and made the appropriate changes
hansfn wrote:NB! Never give a function a name that is likely to be used in the core. Multiple definitions of the same function is a fatal error.
I wasn't planning on keeping that in the code, at least not in its present form. I started with that as a base since I was just copying and pasting (and modifying). Thanks for the tip, though.