search

primefaces-extensions / primefaces-extensions.github.com

Organization repo, only for homepage, wiki and issue tracker
https://primefaces-extensions.github.io/
70 stars 22 forks source link

AjaxErrorHandler should only handle PF Ajax requests #217

Closed tandraschko closed 9 years ago

tandraschko commented 10 years ago

Currently it globally listens on each jquery AJAX request: (document).ajaxComplete(handlerFunction);

You can use the 'pfAjaxComplete' event, just look e.g. in blockUI.

tandraschko commented 10 years ago

Can you do it Pavol? :)

slany commented 10 years ago

yes :)

On Mon 24 Mar 2014 10:53:25 AM CET, Thomas Andraschko wrote:

Can you do it Pavol? :)

— Reply to this email directly or view it on GitHub https://github.com/primefaces-extensions/primefaces-extensions.github.com/issues/217#issuecomment-38426585.

veenarm commented 10 years ago

Are you sure that it's actually doing it anyway? I've attempted to use JSF AJAX and all i get is the javascript alert error using processError in standard jsf ajaxResponse. I actually want it to handle all ajax requests which is what I'm looking at at the moment :) I guess could make it a param on the widget?

tandraschko commented 10 years ago

I don't think that it actually could handle all ajax requests because we parse our own xml. Isn't it Pavol?

ova2 commented 10 years ago

Thomas, should I close this issue? It was targeted for 2.1.0.

tandraschko commented 10 years ago

@veenarm Actually i mean that it should only handle JSF ajax requests and not e.g. a plain jQuery ajax request. So it should still cover JSF ajax.

@slany what you like to do it? Or should we close it?

ova2 commented 10 years ago

Just a note: a new release 2.1.0 will be done on Friday (22.08). So, we should decide what we have to do with this issue - postpone, close or fix till 22.08. Thanks.

ova2 commented 9 years ago

AjaxErrorHandler has been removed due to bugs and because PrimeFaces has a better functionality with PrimeFaces ExceptionHandler.