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.
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. Regards, Oliver Paukstadt -- Oliver Paukstadt <pst...@sourcentral.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
