In-Portal Issue Tracker - In-Portal CMS
Viewing Issue Advanced Details
1389 [In-Portal CMS] Front End bug report always 2012-09-04 09:41 2012-09-04 09:43
resolved 5.0.0  
none 5.2.1-B1
Fixes bug in c_ContentPageLink tag
0001389: Tag c_ContentPageLink builds broken link
When you're printing a sitemap with links to all sections (pages) from a website on it you need to have solid tag to build a link to each page.
Right now there is tag <inp2:CategoryLink template='__default__'/> that does that.

Tag CategoryLink threats a section as a category and populates category_id in a link, but uses keeps template empty (if template parameter is set to "__default__") at the end.
Resulting link is valid because category url itself is a virtual template.

However if developer wants to rewrite some urls on the website and matches template name to an url he sees then nothing would happen, because as I've mentioned template name is empty.
Fortunately we have ContentPageLink that does the job. But over time (since nobody uses it) it broke. Both tags build same url at the end, however with ContentPageLink tag 't' parameter in rewrite listeners isn't empty.
patch contentpagelink_tag_fix.patch (698) 2012-09-04 09:41
Issue History
2012-09-04 09:43 alex Changeset attached 5.2.x r15531
2012-09-04 09:43 alex Note Added: 0005167
2012-09-04 09:43 alex Status reviewed and tested => resolved
2012-09-04 09:43 alex Fixed in Version => 5.2.1-B1
2012-09-04 09:43 alex Resolution open => fixed
2012-09-04 09:43 alex Assigned To !COMMUNITY => alex
2012-09-04 09:43 alex Note Added: 0005166
2012-09-04 09:43 alex Status needs testing => reviewed and tested
2012-09-04 09:43 alex Assigned To => !COMMUNITY
2012-09-04 09:43 alex Developer => alex
2012-09-04 09:43 alex Status active => needs testing
2012-09-04 09:42 alex Reference =>
2012-09-04 09:41 alex New Issue
2012-09-04 09:41 alex File Added: contentpagelink_tag_fix.patch
2012-09-04 09:41 alex Change Log Message => Fixes bug in c_ContentPageLink tag
2012-09-04 09:41 alex Estimate Points => 1

2012-09-04 09:43   
Will test all together later.
2012-09-04 09:43   
Fix committed to 5.2.x branch. Commit Message:

Fixes 0001389: Tag c_ContentPageLink builds broken link