I actually have one to add to the list. Comments below ...
Elias Torres wrote:
Hi Guys, I'm out for the day (classes, meetings, etc) so I won't be able to develop but I want to give you a heads up before you deploy to BSC that I found a couple of missing things in the tagging implementation. Sorry for the bad timing. Basically, I found that in one of the VM models (I believe URLModel) where we compute URLs $url.entries.rss(), ... we are taking in $model.categoryPath, but we are not taking tags, so any URLs like <link rel="self"/> in feeds will not contain the tags from the request. Plus I noticed that I did not check the deprecated VM files that could need access to tags.
Yes, we can add a new method for getting a url to rss/atom feeds by tag in the URLModel, that's easy to do. However, we are not going to make any changes to the deprecated models and macros to add new functionality. Those should only get changed to ensure that they continue working, but not for any other reason.
One more thing, I also need to add getPopularTags() to the PageModel like I did for SiteModel().
Nope, you already have it in WebsiteData, so it's available via $model.weblog.getPopularTags()
However, one item we haven't figured out which should probably be addressed is what to do about tag urls which specify tags which don't exist. i.e. /weblog/tags/lkweehlkwer
The way things are setup right now that is a valid page which will simply not show any entries, and I don't think that's what we want to do for a couple reasons. 1) It's a bit inefficient and silly to have *all* tag urls be assumed correct and 2) It's horrible for the cache since that means a rogue spider or malicious user could completely empty the cache of valid content by filling it with pages/feeds from bogus tag urls.
I think what we should probably do is add some tag checking to the validation sections of both the page and feed servlets to actually check that if tags were specified that they actually exist. if none of the tags specified exist then we can offer a 404.
I could get to these changes tomorrow, but Allen, maybe you could help me if you guys are releasing today.
I will take a look at what I can today and see if I can get these done. -- Allen
BTW, are we branching 3.1 today? -Elias
