404 error when clicking "Add this task"

I just set up a new Drupal 5.1 installation w/ todolist, jquery_update, jquery_interface installed--and it is working for me with Firefox 2.0 and Windows XP. Could the problem you are experiencing could be a configuration or browser issue?

Suggestions:

• Are clean URLs enabled/disabled? I assume they are enabled since you got the blank screen instead of 'Page not found'.
• Did you copy the jQuery.js file to the /misc directory when you installed jquery_update? I assume you did, because otherwise you wouldn't be able to click on the collapsible fieldset for "Add new task".

The blank screen is by design, if you have not provided all the parameters the /todolist/create_task callback is looking for. Try this, instead:

/todolist/create_task?nid=1&order=1&task=one

You should get a response similar to the following:

TodoList.add_incomplete_list();
$('.todolist.incomplete').append('<li class="task" id="task-5"><input type="checkbox" name="task-5" /></li>');
$('#task-5').each(TodoList.add_nub);
$('#task-5').each(TodoList.bind_task);
$('#task-5').Highlight(1000, '#ff0');
$('.todolist.incomplete').SortableAddItem($('#task-5').get(0));

And this would indicate that the module is working correctly under normal circumstances.

Questions:

• When and where do you see the 404 error? I use the Firebug extension for Firefox to show me the AJAX activity going on behind the scenes. If you don't have this installed, I don't think you'd normally be able to see an error like that. UNLESS it is redirecting the browser to a 404 page when you click the "Add this task" button; in which case, that'd definitely be a problem I can fix.

  The strange part about your experience is that you get 404 when you click on "Add this task" but not when you visit the URL directly in your browser--the blank screen means HTTP Status code 200 OK and no further data (which is the correct response to the wrong parameters)

• Which OS are you using? I am assuming since you have Safari that you're running Mac OS X. Admittedly, I have not tested the code in Safari, and I know it doesn't work in IE7 currently, but Firefox 2.0 should work regardless of the OS...

Hi.
First to say: very nice little module.
The error is located in todolist.js. All urls begin with a slash! This did not work on most installations (for example on sub-dirs or on IIS).
I've created a patch that fixes this by replacing the slash with index.php?q=.

hth,

Stefan

@Stefan: Thanks for the patch! I tested it and it works. There is only one thing I wish we could do to improve it. Currently, with your patch, the XMLHTTPRequest object will try to POST to different paths depending on where you are, like:

http://domain.tld/node/index.php?q=todolist/create_task
http://domain.tld/taxonomy/term/index.php?q=todolist/create_task

...and that works, but the path is not exactly where the file should be found. Technically, if there were a directory that existed in that location, it would fail because mod_rewrite wouldn't redirect to /index.php when the file appears to exist. Ideally, you would always have something like this:

http://domain.tld/index.php?q=todolist/create_task

Except in cases where your Drupal installation resides in a sub-directory. That does indeed throw in a monkey wrench unless we can utilize a Drupal function like url()...

I have tried renaming the .js file to .js.php and bootstrapping drupal when the file is called directly, but then Drupal thinks the base_path() is the module directory, and generates url() incorrectly. So, for now, I'll commit this patch to HEAD because it does work, but hopefully down the road we'll find a way to accomplish the ideal solution I described above.

Patch committed to HEAD. Thanks for your contribution! :)

@christefano: Does this solve your problem?

Well, here's another try.
You don't need to get the base url via drupal, you can do it all with jQuery :-)

Tested with and without sub-dirs and it works fine.
The patch applies to 5.x-1.x-dev.

greetings,

Stefan

Works for me! Committed to CVS.

Still getting this error.

I'm getting a white screen too.

found this code, but seems there is some other problem.

5.x in no more maintained, so, like on 6.x it's solved I'm closing this