|
In-Portal Issue Tracker - In-Portal CMS
|
|
Viewing Issue Advanced Details |
|
|
ID:
|
Category:
|
Type:
|
Reproducibility:
|
Date Submitted:
|
Last Update:
|
|
1389 |
[In-Portal CMS] Front End |
bug report |
always |
2012-09-04 09:41 |
2012-09-04 09:43 |
|
|
Reporter:
|
alex |
Platform:
|
|
|
|
Assigned To:
|
alex |
OS:
|
|
|
|
Priority:
|
normal |
OS Version:
|
|
|
|
Status:
|
resolved |
Product Version:
|
5.0.0 |
|
|
Product Build:
|
|
Resolution:
|
fixed |
|
| |
|
ETA:
|
none |
Fixed in Version:
|
5.2.1-B1 |
|
|
Reference:
|
https://groups.google.com/d/topic/in-portal-dev/C-ohuxNHFfc/discussion |
|
Change Log Message:
|
Fixes bug in c_ContentPageLink tag |
|
Estimate Points:
|
1 |
|
|
Summary:
|
0001389: Tag c_ContentPageLink builds broken link |
|
Description:
|
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. |
|
Steps To Reproduce:
|
|
|
Additional Information:
|
|
| Relationships | |
|
Attached Files:
|
 contentpagelink_tag_fix.patch (698) 2012-09-04 09:41
http://tracker.in-portal.org/file_download.php?file_id=1793&type=bug |
|
|
Issue History |
|
Date Modified |
Username |
Field |
Change |
|
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 |
=> https://groups.google.com/d/topic/in-portal-dev/C-ohuxNHFfc/discussion |
|
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 |