Reproduce:

With the linkit create link to the taxonomy term which is one of forum containers or forums taxonomy terms.

For example [path:internal:taxonomy/term/122]

Result:

On the saved page user see unaliased link to the http://domain.com/taxonomy/term/122

Where are next errors appear on the http://domain.com/taxonomy/term/122 page:

* warning: Illegal offset type in isset or empty in /home/data/www/includes/path.inc on line 65.
* warning: Illegal offset type in /home/data/www/includes/path.inc on line 70.
* warning: htmlspecialchars() expects parameter 1 to be string, array given in /home/data/www/includes/bootstrap.inc on line 840.
* warning: Illegal offset type in isset or empty in /home/data/www/includes/path.inc on line 65.
* warning: Illegal offset type in /home/data/www/includes/path.inc on line 70.
* warning: htmlspecialchars() expects parameter 1 to be string, array given in /home/data/www/includes/bootstrap.inc on line 840.
* warning: Illegal offset type in isset or empty in /home/data/www/includes/path.inc on line 65.
* warning: Illegal offset type in /home/data/www/includes/path.inc on line 70.
* warning: htmlspecialchars() expects parameter 1 to be string, array given in /home/data/www/includes/bootstrap.inc on line 840.

Expected result:

Module must create aliased link to the forum container or forum inside container.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anon’s picture

Status: Active » Postponed (maintainer needs more info)

I have tried this now, but I dont get any errors at all.

Link to forum containers and forums taxonomy terms works fine for me.

Please try again and post the result here, Thanks.

anon’s picture

Status: Postponed (maintainer needs more info) » Active
anon’s picture

Status: Active » Closed (won't fix)

I think this is a pathfilter or pathologic bug.

The links shouldnt be going to http://domain.com/taxonomy/term/122 at all.
It should be something like http://domain.com/forum/122

Feel free to open again if you think Im wrong.

yched’s picture

Status: Closed (won't fix) » Active

The issue is that linkit hardcodes taxo links as 'taxonomy/term/[tid]'.
That is not correct. The path of a term page is given by taxonomy_term_path(). By default, this is 'taxonomy/term/[tid]', but not always - and, specifically, *not* for forum terms.

anon’s picture

Version: 6.x-1.5 » 6.x-1.x-dev
Assigned: Unassigned » anon

good point!

anon’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Patch (to be ported)
FileSize
2.33 KB

Ok, I have commited this patch now.

This still needs to be fixed for the 7.x-1.x-dev also.

anon’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
yched’s picture

Wow, that was quick :-) Thanks !
(didn't have time to actually play with the fixed code yet)

Note that taxonomy_term_path() is gone in D7.
URI paths for taxonomy terms are now retrieved by entity_uri('taxonomy_term', $term)

anon’s picture

Status: Patch (to be ported) » Fixed
FileSize
822 bytes

Commited this patch to 7.X.

anon’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.