[geeklog-devel] Links categories (was: Upgrading from 1.4.1 to 1.5)

[Web Site Master]

I had mentioned this before but can we change the name of the links category
autotag to something else?  Something like [link_category:] is a bit more
descriptive. Links isn't the only plugin to have categories and it probably
will get confusing for some people. 

This would probably be a good idea for all plugins to do in the future, have
the name of the plugin at the start of the tag so 2 plugins with the same
autotag name will never be a problem.


Tom


-----Original Message-----
From: geeklog-devel-bounces at lists.geeklog.net
[mailto:geeklog-devel-bounces at lists.geeklog.net] On Behalf Of Dirk Haun
Sent: December-31-07 5:56 AM
To: geeklog-devel
Subject: [geeklog-devel] Links categories (was: Upgrading from 1.4.1 to 1.5)

Dirk Haun wrote:

>Looks like the update for the Links plugin isn't working properly - all
>my links lost their category

While attempting to fix that, I ran across a few oddities and issues
with backward compatibility. I fixed those but would like to hear some
comments to see what others think, please.

Before 1.5, the link categories served as both an ID and a (descriptive)
label. The new Links plugin separates that, so now we have a link ID
(cid), a category name and even a category description.

When upgrading from earlier versions, the code in CVS tried to assign
new IDs to the categories. Also, the URLs for the categories on the
public list of links were changed to use that new cid. That, however,
broke existing links pointing to categories.

For example, on geeklog.net we have a category "Tech. Sites". The link
to that category is .../links/index.php?category=Tech.+Sites

After the upgrade, that would have become .../index.php?
cid=20071231114832121 and the old URL would have stopped working.

So I decided to rewrite that part and keep the old URLs intact - and
therefore allow a cid to include spaces (and other special characters),
just like before. So the cid is not really an ID in the sense we use
that elsewhere (e.g. in stories).

Anyone see a problem with that?

Also, the Links plugin now has a [category:] autotag (new in 1.5).
However, that won't work with these sorts of IDs, e.g.

   [category:Tech. Sites]

The way autotags work, the blank would be seen as the end of the ID, so
it wouldn't work. Instead of trying to come up with fancy workarounds
(e.g. use Tech.+Sites as the ID, i.e. urlencoded) I'd suggest we simply
drop that autotag.

Objections?

bye, Dirk


-- 
http://www.haun-online.de/
http://spam.tinyweb.net/

_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://eight.pairlist.net/mailman/listinfo/geeklog-devel

__________ NOD32 2758 (20071231) Information __________

This message was checked by NOD32 antivirus system.
http://www.eset.com