#79 Pretty Aplaws URLs

[Steve Pugh]

This patch replaces ugly aplaws URLs of the format:

/redirect?oid=[com.arsdigita.cms.contentassets.RelatedLink:{id=125011}]

with shorter and neater URLs that look like this:

/redirect?oid=RelatedLink-id-125011

This has the additional benefit that percent-encoding
issues associated with the unusual characters [ { } ]
etc are avoided.

After applying the patch, aplaws will use the neater
format by default, however the old-style URLs are still
parsed correctly.

There is more discussion of this issue here:
http://sourceforge.net/forum/forum.php?thread_id=1491506&forum_id=368401

Steve

[Steve Pugh]

URL format patch

[Steve Pugh]

Logged In: YES
user_id=1271522

ADDITIONAL:

I have just discovered a problem with this patch.
If you go to "/ccm/admin/terms/" and select for example
"Local Custom Navigation", it fails because the name-value
pair is
key=APLAWS-CUSTOM
i.e. the value contains a hyphen but this character is used
as the field separator.

Solutions:
1. Use a different field separator character.
2. Change patch to interpret VALUE differently
oid=TYPE-NAME-VALUE
where the last token (VALUE) may contain hyphens
3. change the keys of the navigation lists so that they
don't contain hyphens.

Locally, I have changed the code using solution (2). If you
want to know the details let me know.

[chris gilbert]

Logged In: YES
user_id=1069385
Originator: NO

Hi Steve,

Could you send me details of your local solution to the problem - a straight copy of your OID class would be easier than a diff file - another good reason for implementing this patch is that stable links can be used in emails (current ugly urls are not recognised as urls by some mail clients and so are not rendered as links)

Thanks - please send to chris.gilbert{at}westsussex.gov.uk or to my sourceforge address

Chris......

[Steve Pugh]

Logged In: YES
user_id=1271522
Originator: YES

Committed revision 1591