[sorry for my english ;-) ]
Hello to everybody,
I've got an issue with webform 5.x in junction with Drupal 5 - 5.1 .

When I try to delete webform's submitted entries, the sistem return an error :

"warning: call_user_func_array(): First argumented is expected to be a valid callback, 'delete_submission' was given in /var/www/vhosts/bioanchio.it/httpdocs/includes/form.inc on line 218."

I've searched on Goglee, but no answer. Also I've searched this on Drupal.org but no lucky!..
Instead of it, I've used webform 4.7 on drupal 4.7 without this problem.

Can Anybody help me?
Thank you very much.
Masismo

CommentFileSizeAuthor
#3 webform_delete_formapi_d5.patch3.38 KBnelson_rp

Comments

nelson_rp’s picture

nelson_rp commented

I've experienced the same problem. I believe that it is due to an incomplete port of this feature to the Drupal 5.x form API. I figure it should be a matter of replicating the confirmation process used in the "clear" results function. Unfortunately, I haven't had the time to look into it too hard. Sorry, that's probably not much help to you.

Deyan’s picture

Deyan commented

I am experiencing the same thing. PLEASE help!

nelson_rp’s picture

nelson_rp commented
Status: Active » Needs review
StatusFileSize
new webform_delete_formapi_d5.patch3.38 KB

This patch updates the "delete" function to work with Drupal 5's FormAPI. I've also included a fix for the "clear" function "cancel" url redirecting to a 404.

Deyan’s picture

Deyan commented

This patch does not seem to work. It is not possible to patch using it. :(

Deyan’s picture

Deyan commented

Just to clear things up - I use patch -p0 FILENAME as I usually do via SSH, but it dies not execute.

Anonymous’s picture

Anonymous (not verified) commented

Thank you for this patch, it was fine for me.
I patched the original files locally on my PC (webform.inc, webform.module with “patch.exe” for Windows than copied to the server.

Now delete feature works well.
Zs

BioALIEN’s picture

BioALIEN commented
Priority: Normal » Critical

This should be committed back into the module itself. Can someone review whether the patch still applies to HEAD and patch accordingly?

JHeffner’s picture

JHeffner commented

patch applies to HEAD. The patch worked for me as well.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

Thanks nelson_rp for the patch! It works great and has been committed to the 5.x branch.

maxp76’s picture

maxp76 commented

Hello to everybody,
I've just tried Drupal 5.1 with Webform 5.x-1.x-dev , on a XAMPP envirorment, and the error does'n appear!
Thanks to everybody.
Massimo

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)