In-Portal Issue Tracker - In-Portal CMS
Viewing Issue Advanced Details
1429 [In-Portal CMS] Template System bug report always 2012-11-05 10:43 2012-11-07 06:10
alex  
alex  
normal  
resolved 5.1.0  
fixed  
 
none 5.2.1-B1  
https://groups.google.com/d/topic/in-portal-bugs/kf-oMaCAmL8/discussion
Fixes misleading path being used by ModuleInclude tag
1
0001429: Incorrectly included templates via <inp2:m_ModuleInclude tag
In-Portal has 2 tags, that allow to use same template multiple times on other different templates:

* <inp2:m_Include - includes 1 given template
* <inp2:m_ModuleInclude - includes given template from each module

There is a problem with m_ModuleInclude tag in Admin Console, where he uses "core/" prefix to include templates from core module (see attached image). This works of course (otherwise whole "Structure & Data" section would be dead), but when you need to replace template, that is included (e.g. "catalog_tab" from core module), then you need to write "core/" before replaced template name.

What I've described might not be a bug, but because of "Sections" tab in "Structure & Data" page is loaded via ajax and is accessing "catalog_tab" (and not "core/catalog_tab", that was replaced) resulting page uses original non-replaced template.

Solution:
Inside m_ModuleInclude tag when we have "core/" as module path, then strip it.

This way template replacement will work.
We might try to risk and actually remove "core/" from Modules table and see how it will work afterwards.
png inportal_module_include_admin.png (12,100) 2012-11-07 05:54
http://tracker.in-portal.org/file_download.php?file_id=1855&type=bug
png

patch moduleinclude_and_core_fix.patch (3,064) 2012-11-07 06:08
http://tracker.in-portal.org/file_download.php?file_id=1856&type=bug
Issue History
2012-11-07 06:10 alex Changeset attached 5.2.x r15615
2012-11-07 06:10 alex Note Added: 0005264
2012-11-07 06:10 alex Status reviewed and tested => resolved
2012-11-07 06:10 alex Fixed in Version => 5.2.1-B1
2012-11-07 06:10 alex Resolution open => fixed
2012-11-07 06:10 alex Assigned To !COMMUNITY => alex
2012-11-07 06:09 alex Note Added: 0005263
2012-11-07 06:09 alex Status needs testing => reviewed and tested
2012-11-07 06:09 alex Assigned To => !COMMUNITY
2012-11-07 06:09 alex Developer => alex
2012-11-07 06:09 alex Status active => needs testing
2012-11-07 06:08 alex File Added: moduleinclude_and_core_fix.patch
2012-11-07 05:54 alex File Added: inportal_module_include_admin.png
2012-11-05 10:43 alex New Issue
2012-11-05 10:43 alex Reference => https://groups.google.com/d/topic/in-portal-bugs/kf-oMaCAmL8/discussion
2012-11-05 10:43 alex Change Log Message => Fixes misleading path being used by ModuleInclude tag
2012-11-05 10:43 alex Estimate Points => 1

Notes
(0005263)
alex   
2012-11-07 06:09   
Will test all together later.
(0005264)
alex   
2012-11-07 06:10   
Fix committed to 5.2.x branch. Commit Message:

Fixes 0001429: Incorrectly included templates via <inp2:m_ModuleInclude tag