Comments on: Prevent debug info from messing with your AJAX responses

By: teknoid

teknoid — Mon, 30 Aug 2010 12:13:02 +0000

@ADmad

Yep, I did mention that in the update at the top.
The debug kit, is great… I wish it would work better in IE, because it causes some JS errors, which cross-browser testing a little more complicated.

By: ADmad

ADmad — Sun, 29 Aug 2010 09:46:15 +0000

The timestamp echoing has been removed altogether in 1.3.3. If one needs timestamps just use the awesome DebugKit.

By: teknoid

teknoid — Sat, 19 Jun 2010 21:21:21 +0000

@jblotus

Thank you. Fixed-up.

By: jblotus

jblotus — Sat, 19 Jun 2010 18:22:45 +0000

hey quick note your code in the first example is missing a closing brace
if (Configure::read() > 0) {
if (!env(‘HTTP_X_REQUESTED_WITH’) === “XMLHttpRequest”) {
echo ““;
}
}

By: Joseph Crawford

Joseph Crawford — Sun, 13 Jun 2010 22:07:23 +0000

Like Andy I have also used the RequestHandler in the beforeFilter of the AppController to set layout to false and debug 0. It works to disable the comment but it also disabled debug which can be frustrating. I like your method because it removes the comment but still allows debug ;) Good job!

By: tonie

tonie — Mon, 24 May 2010 20:29:23 +0000

Yay! Congrats on the new design (liked the old one better lol j/k). Nice article as always, thanks.

By: JamesF

JamesF — Sat, 15 May 2010 19:56:36 +0000

@teknoid

I wasn’t even thinking about the DRY part. So many times in cake i end up writing functions to avoid violating DRY, and I only end up using it once, and I spent three times as long on it. I guess I need to get better at planning what things I might use more than once.

By: teknoid

teknoid — Sat, 15 May 2010 19:19:02 +0000

@Andy

What is $this->isAjax()? (You probably meant $this->RequestHandler->isAjax())…
Also, I don’t see how that is more cake… to me app controller has other stuff to “worry” about.

@Jason

It’s the same in case of ‘debug’. (That’s what Configure::read() defaults to).

@JamesF

Not very DRY, other suggestions listed here are at least keeping the code from repeating itself over and over (especially when you have many ajax methods).

By: teknoid

teknoid — Sat, 15 May 2010 19:12:52 +0000

@Daniel Hofstetter

That timestamp is really not necessary, IMO… but it is there for now. So that’s one way to avoid having it during AJAX response.
As far as other output, I believe RequestHandler outputs it properly, at least I never get that timestamp with XML or otherwise.

By: teknoid

teknoid — Sat, 15 May 2010 19:08:59 +0000

@Nick

To me that’s extra code in the layout, which is not necessary.
Also, ajax.ctp may not be the only layout you use, but that largely depends on your app.