Return-Path: Delivered-To: apmail-wicket-dev-archive@www.apache.org Received: (qmail 80793 invoked from network); 3 Oct 2009 05:06:52 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 3 Oct 2009 05:06:52 -0000 Received: (qmail 18598 invoked by uid 500); 3 Oct 2009 05:06:52 -0000 Delivered-To: apmail-wicket-dev-archive@wicket.apache.org Received: (qmail 18523 invoked by uid 500); 3 Oct 2009 05:06:51 -0000 Mailing-List: contact dev-help@wicket.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@wicket.apache.org Delivered-To: mailing list dev@wicket.apache.org Received: (qmail 18513 invoked by uid 99); 3 Oct 2009 05:06:51 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Oct 2009 05:06:51 +0000 X-ASF-Spam-Status: No, hits=3.4 required=10.0 tests=HTML_MESSAGE,SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (nike.apache.org: local policy) Received: from [209.85.212.182] (HELO mail-vw0-f182.google.com) (209.85.212.182) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Oct 2009 05:06:41 +0000 Received: by vws12 with SMTP id 12so956036vws.8 for ; Fri, 02 Oct 2009 22:06:19 -0700 (PDT) MIME-Version: 1.0 Received: by 10.220.101.2 with SMTP id a2mr5310482vco.40.1254546379727; Fri, 02 Oct 2009 22:06:19 -0700 (PDT) In-Reply-To: <23eb48360910022150u17c920b3pb9d2a07eb8c6ba1d@mail.gmail.com> References: <23eb48360910021547w3f87540fje6fcb4f6d40549@mail.gmail.com> <109027740910021837m55f84907q2b520cc358145b69@mail.gmail.com> <3909c0fc0910022005r5cf72c82o3b3a686c467b419d@mail.gmail.com> <23eb48360910022150u17c920b3pb9d2a07eb8c6ba1d@mail.gmail.com> Date: Sat, 3 Oct 2009 00:06:19 -0500 Message-ID: <3909c0fc0910022206y35e687c4o52be031e8dda9538@mail.gmail.com> Subject: Re: [wicket 1.5] url handling refactor preview From: Jeremy Thomerson To: dev@wicket.apache.org Content-Type: multipart/alternative; boundary=0016368329be5169b6047500d718 X-Virus-Checked: Checked by ClamAV on apache.org --0016368329be5169b6047500d718 Content-Type: text/plain; charset=ISO-8859-1 My sincere apologies for not sounding appreciative enough. I think all of us appreciate the work that the two of you have put in to not only this, but day-after-day of supporting this framework. I know I certainly do. On a lighter note, if that's all people have to critique about, it might be a sign that it looks good. (or, that we haven't had sufficient time to look at the rest of it). -- Jeremy Thomerson http://www.wickettraining.com On Fri, Oct 2, 2009 at 11:50 PM, Igor Vaynberg wrote: > it warms my heart to see that matej and i have slaved over a new > design and an initial implementation to eliminate a huge pain point > and the only feedback we get is about method names. > > i would much rather hear feedback about the design, the usecases the > design supports, and more importantly the ones it does not. > > peace, > > -igor > > On Fri, Oct 2, 2009 at 8:05 PM, Jeremy Thomerson > wrote: > > public RequestHandler map(Request request); > > > > Is that one not more of a "resolve" action? > > > > -- > > Jeremy Thomerson > > http://www.wickettraining.com > > > > > > > > On Fri, Oct 2, 2009 at 8:37 PM, Matej Knopp > wrote: > > > >> The methods are > >> public RequestHandler map(Request request); > >> public Url map(RequestHandler requestHandler); > >> > >> is it not obvious from the parameter and return type what is mapped to > >> what? > >> > >> -Matej > >> > >> On Sat, Oct 3, 2009 at 3:31 AM, James Carman > >> wrote: > >> > I'm not jazzed about the fact that both methods are named map() when > >> > they're supposed to be inverses of one another. > >> > > >> > On Fri, Oct 2, 2009 at 6:47 PM, Igor Vaynberg < > igor.vaynberg@gmail.com> > >> wrote: > >> >> as ive mentioned before, the focus of 1.5 will be the overhaul of how > >> >> we handle the urls and process requests. this part of wicket has > grown > >> >> organically and has turned into a bunch of overcomplicated spaghetti > >> >> code. > >> >> > >> >> matej and i (mostly matej) have been working on an clean room > >> >> implementation that we think will be simpler, cleaner, and allow > users > >> >> to mangle their urls as much as they want. > >> >> > >> >> the preview is available here: > >> >> > >> > https://svn.apache.org/repos/asf/wicket/sandbox/knopp/experimental/wicket-ng > >> >> > >> >> with a tiny test webapp here > >> >> > >> >> > >> > https://svn.apache.org/repos/asf/wicket/sandbox/knopp/experimental/wicket-ng-webapp > >> >> > >> >> this is just a small prototype to allow us to feel out the api before > >> >> we begin the nightmarish task of merging this back into wicket > proper. > >> >> > >> >> the main package of interest is org.apache.wicket.request which > >> >> contains the two key players: RequestMapper and RequestHandler. these > >> >> two entities work together to resolve and process urls, facilitated > by > >> >> the new and much simplified yet functional > >> >> org.apache.wicket.requset.cycle.RequestCycle. > >> >> > >> >> the RequestHandler is analogous to IRequestTarget - its job is to > >> >> generate a response to user's request. > >> >> > >> >> the RequestMapper is what maps urls to requesthandlers and back. this > >> >> is the backbone of url handling. the interface is quiet simple, yet > >> >> allows total flexibility. the mappers can be composed and chained, > >> >> producing any kind of url scheme desirable. as well as allowing users > >> >> to build the url hierarchy in any way they want rather then just > >> >> working with absolute mounts like in 1.4. > >> >> > >> >> for example, the ever desirable // url encoding > >> >> scheme is now easily achievable by creating a mapper to take care of > >> >> the /locale/ portion and chaining all other mappers behind that. > >> >> > >> >> and for the other ever desirable usecase of mounting things like so: > >> >> /${user}/profile/${section}/detail there is now the > >> >> org.apache.wicket.request.mapper.MountedMapper > >> >> > >> >> further, because mappers are chainable one can foresee a mapper who > >> >> doesnt manipulate the url, but instead enforces a security constraint > >> >> on a subset of url space. > >> >> > >> >> of interest are also: org/apache/wicket/request/url-format.txt and > >> >> org/apache/wicket/page/page-management.txt > >> >> > >> >> feedback is welcome. above all we would like to hear all your weird > >> >> and interesting url mapping scheme ideas so we can proof the api > >> >> against them. > >> >> > >> >> -igor > >> >> > >> > > >> > > > --0016368329be5169b6047500d718--