In-Portal Issue Tracker - In-Portal CMS
Viewing Issue Advanced Details
445 [In-Portal CMS] Other bug report always 2009-12-16 10:40 2011-03-14 02:10
alex  
alex  
normal  
closed 5.0.1  
fixed  
 
none 5.0.2-B1  
Google Groups http://groups.google.com/group/in-portal-bugs/browse_thread/thread/e00f519eaed911f9
0
0000445: Enable Mod-rewrite for Front-end in Admin
Currently mod-rewrite mode (on/off) is ignored, when Front-End is viewed via
right frame in administrative console. This is no big deal, because all url
building methods build urls, that will work in both cases (with and without
mod-rewrite). But, with new rewrite listener approach (added in 5.0.1) we
could came across situation, when developer could create such rewrite
listener, that must be always present for customized urls to work. See
example provided below:

Example #1:
mod-rewrite off:
http://www.site.com/index.php?env=-path/to/template:m555--2--s-
mod-rewrite on:
http://www.site.com/russian/path/to/category/path/to/template.html

In example #1 such transformation to url happens:

   - "-2-" is language id, which is transformed to "/russian/";
   - "m555" is category id, which is transformed to "/path/to/category/";
   - "path/to/template" is just added to url's ending without
   transformations.

In this scenario url parser/builder always could determine what url part is
transformed/parsed to what ID or any other environment variable ("env"
variable in non-mod-rewrite url) part.

Example #2:
mod-rewrite off:
http://www.site.com/index.php?env=-path/to/template:m0--1--s-
mod-rewrite on: http://www.site.com/completely/different/template.html

In example #2 only one transformation happens: real template named
"path/to/template" is replaced with nice (to create SEO urls) name
"completely/different/template", which of course doesn't exist on disk.
Using this approach only rewrite listener (which made transformation) knows
what real templates should be used for each of given virtual urls.
If mod-rewrite is turned off, then such url won't work (in case if someone
placed direct link to it). Based on my experience I could say, that, when
mod-rewrite is enabled on site, that it's never going to be disabled.

So, for large projects, when there is very complex url building/parsing
schemes it's hard to always track url compatibility between mod-rewrite and
non-mod-rewrite modes.

Based on all above I propose to use mod-rewrite urls even, when Front-End is
viewed in right frame in administrative console (see attached patch).
child of 0000098closed  (5.0.1)alex Redo Mod-Rewrite for Better Flexibility 
patch admin_rewrite_urls.patch (652) 2009-12-16 10:40
http://tracker.in-portal.org/file_download.php?file_id=159&type=bug
Issue History
2011-03-14 02:10 alex Relationship added child of 0000098
2010-01-11 22:01 Dmitry Note Added: 0001313
2010-01-11 22:01 Dmitry Status resolved => closed
2009-12-20 06:51 alex Note Added: 0001182
2009-12-20 06:51 alex Status reviewed and tested => resolved
2009-12-20 06:51 alex Fixed in Version => 5.0.2-B1
2009-12-20 06:51 alex Resolution open => fixed
2009-12-20 06:51 alex Assigned To !COMMUNITY => alex
2009-12-20 06:51 alex Changeset attached 5.0.x r12953
2009-12-20 02:04 Dmitry Note Added: 0001178
2009-12-20 02:04 Dmitry Status needs testing => reviewed and tested
2009-12-16 10:41 Dmitry Reporter Dmitry => alex
2009-12-16 10:41 Dmitry Status needs work => needs testing
2009-12-16 10:40 Dmitry New Issue
2009-12-16 10:40 Dmitry Status active => needs work
2009-12-16 10:40 Dmitry Assigned To => !COMMUNITY
2009-12-16 10:40 Dmitry File Added: admin_rewrite_urls.patch
2009-12-16 10:40 Dmitry Reference => Google Groups http://groups.google.com/group/in-portal-bugs/browse_thread/thread/e00f519eaed911f9

Notes
(0001178)
Dmitry   
2009-12-20 02:04   
Tested good!
(0001182)
alex   
2009-12-20 06:51   
Fix committed to 5.0.x branch. Commit Message:

1. Fixes 0000445: Enable Mod-rewrite for Front-end in Admin
2. It seems, that Dmitry haven't found several bugs in this patch:
- phrase editing won't working
- edit block/edit design won't working
- whole template editing (index) won't working in "Design Mode"
(0001313)
Dmitry   
2010-01-11 22:01   
Closing completed tasks.