Hi, On Sun, Jan 6, 2013 at 9:03 PM, Oliver Paukstadt <pst...@sourcentral.org> wrote: > On Sun, 2013-01-06 at 19:19 +0000, Pedro Melo wrote: >> Hi, >> >> On Sun, Jan 6, 2013 at 3:16 PM, Oliver Paukstadt <pst...@sourcentral.org> >> wrote: >> > On Fri, 2013-01-04 at 23:50 +0000, Pedro Melo wrote: >> >> > Adding is_index like is_dhandler to Mason::Component::ClassMeta and >> >> > check for this in _build_match_request_path should fix the real >> >> problem. >> >> >> >> Depends on your definition of what the real problem is :). You use >> >> case is only one situation. Add dhandlers or components with >> >> allow_path_info => 1 and the redirect is no longer the correct >> >> behaviour. >> > The redirect is not the real problem, the real problem is that index.mc >> > without allow_path_info => 1 is ignored if a directory (not root) is >> > accessed with ending slash since the changes in 2.20. >> >> Probably. >> >> I'll see if I can add a failing test case tomorow morning when I get >> to the office, and a patch for it. > There is a test for this case as far as I understand line 104 in > Mason-2.20/lib/Mason/t/ResolveURI.pm correct. But this one is designed > wrong in my eyes. It fits the implementation, not what was reality > before the 2.20 release and what is reasonable and expected.
I'll check it tomorow. > Index files are for being displayed, when a directory is hit. Having > them only displayed when allow_path_info is set is like having to define > allow_path_info inside of all dhandlers. True, my only concern is making sure issue #8 is not re-opened. Bye, -- Pedro Melo @pedromelo http://www.simplicidade.org/ http://about.me/melo xmpp:m...@simplicidade.org mailto:m...@simplicidade.org ------------------------------------------------------------------------------ Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft MVPs and experts. ON SALE this month only -- learn more at: http://p.sf.net/sfu/learnmore_123012 _______________________________________________ Mason-users mailing list Mason-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mason-users
