incubator-adffaces-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Martin Marinschek" <martin.marinsc...@gmail.com>
Subject Re: ADF Faces breaks many Tomahawk components
Date Thu, 27 Jul 2006 10:11:20 GMT
Just discussed with Manfred, and this is how it should work:

if you click on a link, a hidden parameter is set, then the form is
submitted, and then the hidden parameter is cleared out again.

Manfred means that it doesn't matter if there is one parameter per
link or one parameter for the whole form, the renderer needs to work
correctly, though. If we have links from several frameworks in the
game, it should probably be one parameter per link, right?

In any case, I don't see how the current javascript method does
anything even close to this behaviour...

regards,

Martin

On 7/27/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> P.S. I mean the generated html of course, not the jsf source :)
>
>
> 2006/7/27, Cosma Colanicchia <cosmacol@gmail.com>:
> > Can you explain me better how do you mean for "clears all hidden
> > fields for all links, and their parameters"? I'm looking at the source
> > of this page: http://www.irian.at/home.jsf (I don't have a myfaces
> > playground to experiment with at this time), and I don't see an hidden
> > field for *each* commandLink, I see two forms, so two _link_hidden_
> > fields and two clear methods..
> >
> > Cosma
> >
> >
> > 2006/7/27, Martin Marinschek <martin.marinschek@gmail.com>:
> > > Well, as it is implemented in MyFaces currently, it clears all hidden
> > > fields for all links, and their parameters, right?
> > >
> > > regards,
> > >
> > > Martin
> > >
> > > On 7/27/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > Hi Martin,
> > > >
> > > > I'm trying to understand how links work. We have:
> > > >
> > > > - single "_link_hidden_" field in the form
> > > > - a single function in the form that clear _link_hidden_ value and
> > > > reset the form target
> > > > - every link in the onClick call the clear function, set it own id in
> > > > _link_hidden_, then call the onsubmit form handler (checking the
> > > > return value), and finally submit the form
> > > >
> > > > Why the clear method should know about every links? It seems that it
> > > > just clear the _link_hidden_ field.. I must be missing something
> > > >
> > > >
> > > > Cosma
> > > >
> > > >
> > > > 2006/7/27, Martin Marinschek <martin.marinschek@gmail.com>:
> > > > > Hi Cosma,
> > > > >
> > > > > I think it was developed this way cause the clear method needed to
> > > > > know about all links anyways, so it was centrally done.
> > > > >
> > > > > Yes, the component namespace standard is definitely a good idea. If we
> > > > > find a clean solution to do this in MyFaces, I think that both
> > > > > Trinidad and the RI will be happy to build on this as well.
> > > > >
> > > > > regards,
> > > > >
> > > > > Martin
> > > > >
> > > > > On 7/27/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > Using a central way is not my option, I only said that, being it the
> > > > > > way things are currently implemented, there should be a clear
> > > > > > advantage for switching now.
> > > > > >
> > > > > > Thinking about first solution, I think you're right. The thing I
> > > > > > didn't like is the hack of adding hidden form fields to the DOM using
> > > > > > javascript, but I agree on having component's renderer simply write
> > > > > > out the <input type="hidden" .../> along the other html..
> > > > > >
> > > > > > The only thing I could add is some component "namespace" standard to
> > > > > > avoid parameters collision.
> > > > > >
> > > > > > BTW, do you remember why the hidden fields form service was
> > > > > > implemented, instead of going this way?
> > > > > >
> > > > > >
> > > > > > Cosma
> > > > > >
> > > > > > 2006/7/27, Martin Marinschek <martin.marinschek@gmail.com>:
> > > > > > > P.S.: your option - using a central way of doing this - is nothing I
> > > > > > > would fancy. That's where we are right now, and it doesn't help us
> > > > > > > with compatibility, really.
> > > > > > >
> > > > > > > regards,
> > > > > > >
> > > > > > > Martin
> > > > > > >
> > > > > > > On 7/27/06, Martin Marinschek <martin.marinschek@gmail.com> wrote:
> > > > > > > > Yes, that is the second thing - we'll need the component let add
> > > > > > > > hidden parameters to the form.
> > > > > > > >
> > > > > > > > Now, as the component is embedded in the form, why can it not just
> > > > > > > > render out the hidden element?
> > > > > > > >
> > > > > > > > This doesn't remind me of BASIC at all, but of component orientation -
> > > > > > > > everything that the component needs should be wrapped up in the
> > > > > > > > component itself, don't you think so?
> > > > > > > >
> > > > > > > > Where does the need of dynamically added client-side fields stem from
> > > > > > > > again, I don't see it right now?
> > > > > > > >
> > > > > > > > What needs to be added dynamically is the clear-script. And this
> > > > > > > > really is complicated, it touches the base of what dojo has to offer
> > > > > > > > with its event system. I don't know if we can replicate this without
> > > > > > > > having a dependency on dojo in our implementation, and I'm pretty sure
> > > > > > > > we don't want this. Does somebody know how
> > > > > > > >
> > > > > > > > dojo.event.bind()
> > > > > > > >
> > > > > > > > works under the covers, and if it's easy (meaning one or two
> > > > > > > > javascript functions) to replicate?
> > > > > > > >
> > > > > > > > regards,
> > > > > > > >
> > > > > > > > Martin
> > > > > > > >
> > > > > > > > On 7/26/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > > > > Martin,
> > > > > > > > >
> > > > > > > > > I am not sure that the problem is related only to links.. there can be
> > > > > > > > > other components that need an hidden form parameter.
> > > > > > > > >
> > > > > > > > > For example, JSCookMenu encodes the links in the html with some
> > > > > > > > > special prefix syntax, set this value in an hidden form named
> > > > > > > > > "jscook_action" on click, and intercept and decode it in the
> > > > > > > > > server-side decode logic.
> > > > > > > > >
> > > > > > > > > I think that the choice is between letting every component to add
> > > > > > > > > hidden fields dynamically on the DOM (dojo can help? its not that
> > > > > > > > > difficult anyway), or provide some sort of central service to do so,
> > > > > > > > > that's the current approach.
> > > > > > > > >
> > > > > > > > > First solution doesn't create dependency between components, but I
> > > > > > > > > don't like very much.. it reminds me of those BASIC programs that used
> > > > > > > > > variables without declaring them ;) On the other hand second solution
> > > > > > > > > does create dependency, and we must be careful to allow mixing
> > > > > > > > > component libraries.
> > > > > > > > >
> > > > > > > > > Any thoughts?
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > Cosma
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > 2006/7/26, Martin Marinschek <martin.marinschek@gmail.com>:
> > > > > > > > > > That's the clear-problem, right?
> > > > > > > > > >
> > > > > > > > > > For this problem, this is what I was thinking about:
> > > > > > > > > >
> > > > > > > > > > every link renders a javascript which adds a function as an onsubmit-handler
> > > > > > > > > >
> > > > > > > > > > dojo.event.bind(form,"submit",myFunction() { clearLink and param} );
> > > > > > > > > >
> > > > > > > > > > so there would be no central clear script anymore, but each link would
> > > > > > > > > > clear itself, we'd loose the interdependency between link and form.
> > > > > > > > > >
> > > > > > > > > > regards,
> > > > > > > > > >
> > > > > > > > > > Martin
> > > > > > > > > >
> > > > > > > > > > On 7/19/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > > > > > > Take a look at how the addHiddenCommandParameter is implemented in
> > > > > > > > > > > HtmlFormRendererBase:
> > > > > > > > > > >
> > > > > > > > > > >     public static void addHiddenCommandParameter(UIComponent form,
> > > > > > > > > > > String paramName)
> > > > > > > > > > >     {
> > > > > > > > > > >         Set set = (Set)form.getAttributes().get(HIDDEN_COMMAND_INPUTS_SET_ATTR);
> > > > > > > > > > >         if (set == null)
> > > > > > > > > > >         {
> > > > > > > > > > >             set = new HashSet();
> > > > > > > > > > >             form.getAttributes().put(HIDDEN_COMMAND_INPUTS_SET_ATTR, set);
> > > > > > > > > > >         }
> > > > > > > > > > >         set.add(paramName);
> > > > > > > > > > >     }
> > > > > > > > > > >
> > > > > > > > > > > This has no dependency at all on the HtmlFormRendererBase, not even on
> > > > > > > > > > > UIForm. It simply add an attribute with a particular name on a generic
> > > > > > > > > > > UIComponent. Somewhere, something will read that attribute and create
> > > > > > > > > > > the hidden links.
> > > > > > > > > > >
> > > > > > > > > > > We could simply:
> > > > > > > > > > >
> > > > > > > > > > > 1) Move this method and the HIDDEN_COMMAND_INPUTS_SET_ATTR constant
> > > > > > > > > > > outside of the HtmlFormRendererBase, maybe in
> > > > > > > > > > > shared\util\_ComponentUtils.java
> > > > > > > > > > >
> > > > > > > > > > > 2) Define the same constant and create a method that does the same
> > > > > > > > > > > thing in some Trinidad's util class
> > > > > > > > > > >
> > > > > > > > > > > 3) Have the Trinidad form renderer look at that attribute, and render
> > > > > > > > > > > the requested hidden fields just as HtmlFormRendererBase does.
> > > > > > > > > > >
> > > > > > > > > > > This way, any Trinidad and MyFaces/Tomahawk component can find the
> > > > > > > > > > > parent form (without caring if it is a Trinidad or a MyFaces one) and
> > > > > > > > > > > ask for an hidden field with the respective utility method.
> > > > > > > > > > >
> > > > > > > > > > > What do you think about this? The only ungly thing is having the same
> > > > > > > > > > > constant/method in two different places, but as long as we'll keep the
> > > > > > > > > > > two codebases separated I don't think that much can be done on this.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Cosma
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > 2006/7/19, Cosma Colanicchia <cosmacol@gmail.com>:
> > > > > > > > > > > > Any ideas on this? :-)
> > > > > > > > > > > >
> > > > > > > > > > > > Cosma
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > > 2006/7/17, Cosma Colanicchia <cosmacol@gmail.com>:
> > > > > > > > > > > > > Adam, technically it could be done, but I'm wondering where we can do that.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Having some centralized code in a renderer that inject the javascript
> > > > > > > > > > > > > don't bring us again on the same problem? (it would be
> > > > > > > > > > > > > renderer-dependent)
> > > > > > > > > > > > >
> > > > > > > > > > > > > In my hack, is the routine itself that manipulate the DOM, but I don't
> > > > > > > > > > > > > think this solution can be adopted at large scale.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Cosma
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > > 2006/7/17, Adam Winer <awiner@gmail.com>:
> > > > > > > > > > > > > > Modern browsers have no difficulty being told to add hidden fields
> > > > > > > > > > > > > > to the DOM on the fly;  MyFaces could use this solution, which
> > > > > > > > > > > > > > Trinidad uses (at least in some parts of the code).
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > -- Adam
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > On 7/17/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > This makes things more complex.. anyway, I don't clearly see your
> > > > > > > > > > > > > > > solution: will it solve the JSCookMenu problem? There we have a
> > > > > > > > > > > > > > > javascript code that explicitly search a form field with a fixed name
> > > > > > > > > > > > > > > (see MyFacesHack.js), that is added by the form renderer.
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > Question: is there any reason for components that need an hidden
> > > > > > > > > > > > > > > field, to dinamically add one component to the tree that renders it? I
> > > > > > > > > > > > > > > see two problem with this:
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > 1) If it is done in the renderer of component A child of form B, are
> > > > > > > > > > > > > > > we sure that a child added to B at this stage would get rendered?
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > 2) Does a standard component to render an hidden field that is not
> > > > > > > > > > > > > > > value-bounded exists?
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > 2006/7/17, Martin Marinschek <martin.marinschek@gmail.com>:
> > > > > > > > > > > > > > > > Right now, no way. We don't have a common codebase between Trinidad
> > > > > > > > > > > > > > > > and Tomahawk. I wonder if this is a problem that can be solved on a
> > > > > > > > > > > > > > > > purely javascript basis - client side. How about each link attaching
> > > > > > > > > > > > > > > > itself as a submit handler?
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > Werner, what do you think? Javascript nightmares ahead?
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > regards,
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > Martin
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > On 7/17/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > > > > > > > > > > > > First attemp:
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > move addHiddenCommandParameter method in an external interface (such
> > > > > > > > > > > > > > > > > as FormRendererUtils), and make both Trinidad and MyFaces form
> > > > > > > > > > > > > > > > > renderers implement it. Components that need that service must then
> > > > > > > > > > > > > > > > > code against that interface, and deal with renderers that doesn't
> > > > > > > > > > > > > > > > > implement it.
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > At least, this way we can control/check at compile time the usage of
> > > > > > > > > > > > > > > > > those non-standard services.
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > 2006/7/17, Martin Marinschek <martin.marinschek@gmail.com>:
> > > > > > > > > > > > > > > > > > We should fix this. Once and for all.
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > Any suggestions?
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > regards,
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > Martin
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > On 7/17/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > I've only modified the JSCookMenu javascript to create that field
> > > > > > > > > > > > > > > in
> > > > > > > > > > > > > > > > > > > the DOM if it isn't there. It's not an elegant solution, I know..
> > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > 2006/7/17, Cosma Colanicchia <cosmacol@gmail.com>:
> > > > > > > > > > > > > > > > > > > > P.S. I think that a component relying on other renderers isn't a
> > > > > > > > > > > > > > > good
> > > > > > > > > > > > > > > > > > > > design and we should try to avoid it..
> > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > 2006/7/17, Cosma Colanicchia <cosmacol@gmail.com>:
> > > > > > > > > > > > > > > > > > > > > It looks like Martin has done a similar work, fixing all the
> > > > > > > > > > > > > > > form
> > > > > > > > > > > > > > > > > > > > > searching routines, so that part of my patch should be
> > > > > > > > > > > > > > > dropped. The
> > > > > > > > > > > > > > > > > > > > > addHiddenCommandParameter problem seems to be still there.. do
> > > > > > > > > > > > > > > we want
> > > > > > > > > > > > > > > > > > > > > to wait for a complete solution or, in the meantime, apply the
> > > > > > > > > > > > > > > js hack
> > > > > > > > > > > > > > > > > > > > > to make JSCookMenu work?
> > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > 2006/7/17, Cosma Colanicchia <cosmacol@gmail.com>:
> > > > > > > > > > > > > > > > > > > > > > Yes, I mean TOMAHAWK-516, it includes a change that is very
> > > > > > > > > > > > > > > similar to
> > > > > > > > > > > > > > > > > > > > > > the Martin code, with other things:
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > 1) The _ComponentUtils method is not used everywhere. For
> > > > > > > > > > > > > > > example,
> > > > > > > > > > > > > > > > > > > > > > HtmlJSCookMenuRenderer implements one of its own, and other
> > > > > > > > > > > > > > > > > > > > > > rendererers do it as well. I tried to refactor those method
> > > > > > > > > > > > > > > to use the
> > > > > > > > > > > > > > > > > > > > > > _ComponentUtils one, see the JIRA for details.
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > 2) Another potential source of trouble for Trinidad/Tomahawk
> > > > > > > > > > > > > > > > > > > > > > integration is the
> > > > > > > > > > > > > > > HtmlFormRendererBase.addHiddenCommandParameter
> > > > > > > > > > > > > > > > > > > > > > method. This is called at least by the JSCookMenu (but
> > > > > > > > > > > > > > > presumibly also
> > > > > > > > > > > > > > > > > > > > > > by other components) to queue adding hidden form parameters,
> > > > > > > > > > > > > > > but with
> > > > > > > > > > > > > > > > > > > > > > that rendered will not be used in a Trinidad project leading
> > > > > > > > > > > > > > > to
> > > > > > > > > > > > > > > > > > > > > > javascript errors. In my patch for JSCookMenu I workarounded
> > > > > > > > > > > > > > > this
> > > > > > > > > > > > > > > > > > > > > > hacking into MyFacesHack.js, but I think it should be
> > > > > > > > > > > > > > > addressed
> > > > > > > > > > > > > > > > > > > > > > someway.
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > Well I don't know SVN very well, but if it works as CVS we
> > > > > > > > > > > > > > > have two
> > > > > > > > > > > > > > > > > > > > > > different patch applied to the same lines of code (the
> > > > > > > > > > > > > > > > > > > > > > _ComponentUtils.findNestingForm method), so a merge will be
> > > > > > > > > > > > > > > required
> > > > > > > > > > > > > > > > > > > > > > if TOMAHAWK-516 will be commited.
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > 2006/7/17, Matthias Wessendorf <matzew@apache.org>:
> > > > > > > > > > > > > > > > > > > > > > > Yes that stuff is committed. _CompontUtils too.
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > You mean TOMAHAWK-516 ? I just asigned it to me.
> > > > > > > > > > > > > > > > > > > > > > > Will (or must) take a look on it.
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > Can you please give an update of the status ?
> > > > > > > > > > > > > > > > > > > > > > > Also what's the *conflict* now is?
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > Would be great!
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > -Matthias
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > On 7/16/06, Cosma Colanicchia <cosmacol@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > > > > > > Martin,
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > have you already committed that change? As I pointed out
> > > > > > > > > > > > > > > in another
> > > > > > > > > > > > > > > > > > > > > > > > thread, I've submitted a patch that will likely conflict
> > > > > > > > > > > > > > > with yours if
> > > > > > > > > > > > > > > > > > > > > > > > commited.
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > Ciao
> > > > > > > > > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > 2006/7/17, Martin Marinschek <
> > > > > > > > > > > > > > > martin.marinschek@gmail.com>:
> > > > > > > > > > > > > > > > > > > > > > > > > As a workaround for the form problem, I've made the
> > > > > > > > > > > > > > > form searching
> > > > > > > > > > > > > > > > > > > > > > > > > routing of MyFaces aware of the Trinidad form family -
> > > > > > > > > > > > > > > so the trinidad
> > > > > > > > > > > > > > > > > > > > > > > > > form should be found as well.
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > I'd be glad about any test results!
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > regards,
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > Martin
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > On 5/26/06, Cosma Colanicchia <cosmacol@gmail.com>
> > > > > > > > > > > > > > > wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > Thomas, I'm using the MyFaces RI. I think that the
> > > > > > > > > > > > > > > <t:commandLink>
> > > > > > > > > > > > > > > > > > > > > > > > > > problem is related to *not* using the MyFaces
> > > > > > > > > > > > > > > renderer for the
> > > > > > > > > > > > > > > > > > > > > > > > > > <h:form> component (like in TOMAHAWK-416).
> > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > 2006/5/25, Thomas Spiegl <thomas.spiegl@gmail.com>:
> > > > > > > > > > > > > > > > > > > > > > > > > > > Cosma, seems like you are using the Sun RI as JSF
> > > > > > > > > > > > > > > implementation.
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > see
> > > > > > > > > > > > > > > http://issues.apache.org/jira/browse/TOMAHAWK-416
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > On 5/25/06, Martin Marinschek <
> > > > > > > > > > > > > > > martin.marinschek@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > So it is a must that a renderer for a certain
> > > > > > > > > > > > > > > component family needs
> > > > > > > > > > > > > > > > > > > > > > > > > > > > to be able to render out all components
> > > > > > > > > > > > > > > belonging to this family? I
> > > > > > > > > > > > > > > > > > > > > > > > > > > > don't see why you couldn't just use a different
> > > > > > > > > > > > > > > renderer-type for the
> > > > > > > > > > > > > > > > > > > > > > > > > > > > UIXForm component...
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > regards,
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > Martin
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > On 5/24/06, Adam Winer <awiner@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 5/24/06, Martin Marinschek <
> > > > > > > > > > > > > > > martin.marinschek@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Adam,
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I don't remember a thread talking about
> > > > > > > > > > > > > > > component families, we only
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > talked about extending from UIForm - so
> > > > > > > > > > > > > > > UIXForm does not even have the
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > same component family as UIForm?
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > No, it doesn't, which is more-or-less implied
> > > > > > > > > > > > > > > by not extending UIForm.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > In general, component family should follow
> > > > > > > > > > > > > > > subclassing
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > as well, because otherwise you'd get
> > > > > > > > > > > > > > > ClassCastExceptions in
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > Renderers.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- Adam
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > regards,
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Martin
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 5/24/06, Adam Winer <awiner@gmail.com>
> > > > > > > > > > > > > > > wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Actually, probably not...  UIXForm is in a
> > > > > > > > > > > > > > > different component
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > family from UIForm.  (An earlier thread
> > > > > > > > > > > > > > > talked about that.)
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- Adam
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 5/24/06, Martin Marinschek <
> > > > > > > > > > > > > > > martin.marinschek@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Problem 1) should have been fixed by
> > > > > > > > > > > > > > > Mario's recent addition of
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > searching for the component family
> > > > > > > > > > > > > > > instead of the instance of UIForm.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > regards,
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Martin
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 5/24/06, Cosma Colanicchia <
> > > > > > > > > > > > > > > cosmacol@gmail.com> wrote:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I've done some other tries:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 1) When using the <af:form> component,
> > > > > > > > > > > > > > > a dummy form and its clear
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > method are created. Clicking on
> > > > > > > > > > > > > > > <t:commandLink> or
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > <t:commandSortHeader> does trigger a
> > > > > > > > > > > > > > > submit, but the expected
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > behaviour isn't executed. Probably
> > > > > > > > > > > > > > > this is because the _link_hidden_
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > value is set telling JSF that a
> > > > > > > > > > > > > > > "linkDummyForm:commandLinkId" has been
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > triggered, but the actual name of the
> > > > > > > > > > > > > > > component is different (its name
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > contaner has a different id).
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2) When using an <h:form> component,
> > > > > > > > > > > > > > > the dummy form isn't created
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (seems that the <t:commandLink>
> > > > > > > > > > > > > > > recognize its parent form), but no
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > clear_ method is generated in the
> > > > > > > > > > > > > > > page, so the onclick javascript
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > fails to submit the form.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2006/5/23, Cosma Colanicchia <
> > > > > > > > > > > > > > > cosmacol@gmail.com>:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > It would be useful to share info
> > > > > > > > > > > > > > > about making they work togheter, if
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > someone has resolved these problems.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Anyway I'm using the latest
> > > > > > > > > > > > > > > snapshots.. and I tried with both <h:form>
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > and <af:form>. I had some issues
> > > > > > > > > > > > > > > with dummyForm at first, if I can get
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > some detail I'll try to post it
> > > > > > > > > > > > > > > tomorrow, now I'm leaving the office.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Bye
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cosma
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2006/5/23, Matthias Wessendorf <
> > > > > > > > > > > > > > > matzew@apache.org>:
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi-
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > time by time there are people on
> > > > > > > > > > > > > > > the list, using both.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MyFaces (incl. Tomahawk) and ADF
> > > > > > > > > > > > > > > Faces
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 1) <t:commandLink>
> > > > > > > > > > > > > > > components (and <t:commandSortHeader>s) don't work,
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > because the MyFaces
> > > > > > > > > > > > > > > javascript functions such as clear_myFormId don't get
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > generated. Also the
> > > > > > > > > > > > > > > _link_hidden_form isn't generated anymore. The rendered
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > HTML includes the string
> > > > > > > > > > > > > > > <!-- MYFACES JAVASCRIPT -->, and looking at the
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > stack I can see that both
> > > > > > > > > > > > > > > the adfFaces and the ExtensionsFilter are invoked,
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > but no MyFaces script are
> > > > > > > > > > > > > > > actually rendered in the page.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'm not sure why this would
> > > > > > > > > > > > > > > happen, but this may be a MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > bug;  I know there's been some
> > > > > > > > > > > > > > > discussion on the main MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > list about issues like this.
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'll try to search more
> > > > > > > > > > > > > > > carefully in the archives.. do you know if
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > someone had success using
> > > > > > > > > > > > > > > MyFaces, Tomahawk and ADF Faces in a
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > project?
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Which version (of MyFaces) are you
> > > > > > > > > > > > > > > using?
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Can you wrapp <h:form> around
> > > > > > > > > > > > > > > commandLink?
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > There where some issue - discussed
> > > > > > > > > > > > > > > on myfaces dev - regarding "auto
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > generated form" aka dummy form
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -Matthias
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > > > > > > > Matthias Wessendorf
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > > > further stuff:
> > > > > > > > > > > > > > > > > > > > > > > blog: http://jroller.com/page/mwessendorf
> > > > > > > > > > > > > > > > > > > > > > > mail: mwessendorf-at-gmail-dot-com
> > > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > http://www.irian.at
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > > > > > > > Courses in English and German
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > >
> > > > > > > > > > http://www.irian.at
> > > > > > > > > >
> > > > > > > > > > Your JSF powerhouse -
> > > > > > > > > > JSF Consulting, Development and
> > > > > > > > > > Courses in English and German
> > > > > > > > > >
> > > > > > > > > > Professional Support for Apache MyFaces
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > >
> > > > > > > > http://www.irian.at
> > > > > > > >
> > > > > > > > Your JSF powerhouse -
> > > > > > > > JSF Consulting, Development and
> > > > > > > > Courses in English and German
> > > > > > > >
> > > > > > > > Professional Support for Apache MyFaces
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > >
> > > > > > > http://www.irian.at
> > > > > > >
> > > > > > > Your JSF powerhouse -
> > > > > > > JSF Consulting, Development and
> > > > > > > Courses in English and German
> > > > > > >
> > > > > > > Professional Support for Apache MyFaces
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > >
> > > > > http://www.irian.at
> > > > >
> > > > > Your JSF powerhouse -
> > > > > JSF Consulting, Development and
> > > > > Courses in English and German
> > > > >
> > > > > Professional Support for Apache MyFaces
> > > > >
> > > >
> > >
> > >
> > > --
> > >
> > > http://www.irian.at
> > >
> > > Your JSF powerhouse -
> > > JSF Consulting, Development and
> > > Courses in English and German
> > >
> > > Professional Support for Apache MyFaces
> > >
> >
>


-- 

http://www.irian.at

Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German

Professional Support for Apache MyFaces

Mime
View raw message