Support jQuery 1.5.2

As it's drupal_json() that's broken - shouldn't this be posted as a bug towards Drupal core?

Yes, but we can fix it separately too.

To be fair I haven't decided if we *should* fix it separately.

Should be a back port of the functionality in Drupal 7 then I think: http://api.drupal.org/api/function/drupal_json_output/7

+++ includes/ajax.inc	5 Aug 2010 05:58:07 -0000
@@ -580,3 +580,12 @@ function ctools_ajax_associate_url_to_el
+  drupal_set_header('Content-Type: application/json; charset=utf-8');

Remove charset - according to the JSON-spec it shouldn't be there.

+++ includes/ajax.inc	5 Aug 2010 05:58:07 -0000
@@ -580,3 +580,12 @@ function ctools_ajax_associate_url_to_el
+    echo str_replace(array("\r", "\n", "<", ">", "&"), array('\r', '\n', '\u003c', '\u003e', '\u0026'), str_replace("\'","'", json_encode($var)));

Why replace \r and \n? Drupal 7 doesn't.

Powered by Dreditor.

We looked at that. The Drupal 7 version is PHP 5 only, and we continue to support PHP4.

Isn't this patch PHP 5.2+ only as well? It uses json_encode().

If Ctools should support PHP 4 then this patch should look more like what is discussed in #479368: D7: Create RFC compliant HTML safe JSON.

Um. Ok that's not the patch I thought it was. I guess realityloop took the Drupal 7 version after all instead of the Drupal 6 version of drupal_to_js().

I'm torn if we should just point to the core patch or implement our own until core fixes itself.

Just a quick question coming from #1067290: Fix jQuery 1.7 for Drupal 6.

A lot of users report that Panels Drag and Drop is broken after updating to jQuery 1.5.2 ( and applying the json fix to core that comes with it ).

Using Chromes DevInspector it actually seems like some CTools js is breaking after the patch. Maybe its just a small adjustment to ctools that would make it compatible. Help would be much appreciated :)

...subscribing.

I admit I have had no reason to try jquery 1.5 with Drupal 6, and I don't know what the actual differences are.

Same problem with 1.4.4, if that means anything. In 1.5.2 on the panels edit page you can drag the panels but you can't drop them anywhere. But I'm not seeing any errors, unlike #11.

I'm poking at Ctools but I still haven't found the drag and drop js, any help would be appreciated. Probably a simple solution but for now I'm swapping Jquery versions whenever I need to rearrange panels.

+1

Triaging the 6.x issues, it's no longer supported.