Problem with [termpath] [#473368]

Hello,

First of all i want to thank you for this beautiful module. You done a great job !

I found a bug in the filefield paths.

[termpath] - As [term], but including its supercategories separated by /.

Problem is that when i added it to File path and i have a subcategory i get this :

[user-name]/[termpath]/[title] - user/categorysubcategory/nodetitle

Problem is that is not adding the / between category and subcategory.

P.s : the module is in sites/all/modules

I tested it on 2 sites and i get the same problem. I think is a quick fix bug.

Comments

Comment #1

deciphered

English

Ballarat, Victoria, Australia

commented 26 May 2009 at 22:54

Hi Blue,

Firstly, thanks for the kind words :).

I just ran a few tests and was able to reproduce the issue and it appears that the issue is expected behavior by pathauto.
To reproduce I had to turn on the pathauto cleanup option which means that pathauto is cleaning the punctuation ('/') out of the [term-path] token.

Without that setting on I did get the slash, although it was a double slash as reported at (#420856: [termpath] and [termpath-raw] contain extra slash).

I will look into this further and see what can be done, but I'm unsure if anything can be done on FileField Paths side.

Cheers,
Deciphered.

Comment #2

Blue commented 27 May 2009 at 12:41

Yes with all "File path cleanup settings" OFF i get double slash.

Maybe instruct Pathauto to dont remove the / in the [termpath]?

Many thanks.

Comment #3

deciphered

English

Ballarat, Victoria, Australia

commented 10 June 2009 at 00:50

Title:	Problem with [termpath]	» Don't remove
Project:	File (Field) Paths	» Pathauto
Version:	6.x-1.x-dev	» 7.x-1.x-dev

Hi Blue,

Sorry it took so long to get back to this, I looked into this further and it was definitely a problem on my side.
A fix has been created and will be committed shortly.

However, by fixing this issue, the issue of the 2 slashes was re-introduced. I will likely also commit a temporary workaround for this issue until it is fixed in pathauto.

Cheers,
Deciphered.