Replace jquery_interface with jquery_ui

Yeah, it's in process now.

I really apreciate testing :D

I've got jquery_ui, but it still there is white screen during task adding.

hey you can copy code from list module we have already ported it to use jquery_ui :)
http://drupal.org/project/list

regards
sumitk
http://sumitk.net

Hi sumitk,

Where could I find the code for the list project?... ok scrap that, moment of stupidity. checked the cvs :)

Thanks,
Peter

@ #2

Because there is a if condition in the code that is checking for an input named 'sort' but it doesn't exist!
Have a look in the function 'todolist_create_task()'.

A security issue: $_REQUEST should be replaced by $_POST

Can you explain the fix in more detail for non-programmers to fix?

What needs to happen here, and how can we help?

Hi jeffschuler,

Like it's mentioned, the problem here is that we¡re not using jquery_ui, so patches are _really_ welcome. Take a look to the list project too.

Sorry for log delay here, but I have not so many time now.

So, like I said, patchers are really welcome :D

@palo.. it's a code issue :(

So, if I (following what s8inac said in #5) -- set
$_REQUEST['sort'] = 0;
in todolist_create_task()... I don't get a white screen anymore, but, rather, just see the Javascript from that function printed on the page, by itself.

I'm not familiar with what todolist_ok_js() does, in setting new content headers and sprinkling Javascript in with the PHP.
How is this supposed to work to inject the js into the loaded page? Is there some prerequisite I'm missing?

subscribing

this is a simple fix change line 128 that says

    jquery_interface_add();

to

    jquery_ui_add();

(this)
is just part of the fix sorry

JOsh

@sanguis: exactlly, that's part of the fix. Thanks for the help.

There's still a lot to do here :(

finally, reviewing list module, I arrive to a beta-1, where jquery_ui is used :D

http://drupal.org/cvs?commit=272424