Add checkbox which enables inline form errors on all forms by default.

Thanks for this patch. I had my own ideas on how to implement a site-wide enable, but I'll definitely have a look at this patch.

- I think there should be a distinction between frontend and backend, maybe integrate with admin_theme somehow
- Individual settings should remain in tact, but I think this patch covers that

Anyway, I hadn't considered a sitewide enable yet, just a front end enable. With the settings from admin_theme I could then check what else would be considered frontend/backend.

Additionaly an option should be provided to 'ignore' the sitewide setting on individual forms allowing to disable IFE on certain forms if desired.

I know, it's for the same reasen filefield/imagefield don't work.

I'm looking in to different approaches, but nothing definite yet.

It would also be nice to allow ife for all forms provided by a specific module (webform for example) or to allow a wildcard in form_if (webform_client_form_* for example)

should that be a separate feature request/patch?

The patch in #3 did not apply cleanly for me (patch complained of a malformed patch in the ife.module section) but once I manually applied the changes, it worked great. Thank you!

this patch breaks tabledrag.js on admin/build/menu-customize/menu-name. probably also on admin/build/block

this makes it impossible to sort menu items with ife enabled.

it would be probably be good to add a form id blacklist so that core drupal functionality isn't broken

Here is a patch against 6.x-1.x-dev that adds a black list of core drupal forms that are incompatible with ife because of tabledrag.

Moving this to D7 branch

Here is a slightly refurbished version of #10 against the latest 7.x-dev.

A separate checkbox for adding ife to all webforms could improve this even more. My main reason for patching ife that it is to complicated for editors who create webforms to get the form id and add the id to the configuration for each webform.

But maybe it is ok to use ife anywhere. It will test this now with the global setting and report back if things break as I am using a lot of inline forms and stuff on the site I am using this and I wouldn't be suprised if the blacklist isn't enough.

Patch works good. Thanks! Can this be committed?

Needs a re-roll. Does not apply to dev.

Patch re-rolled for current dev version.

+1 #16 works for me.

Re-roll does not work for me. The branch in the issue is 7.x-2.x.

#16 does not work me either. I tried applying the patch to the current dev version with no luck.

patching file ife.settings.inc
Hunk #2 FAILED at 154.
1 out of 2 hunks FAILED -- saving rejects to file ife.settings.inc.rej

Rerolled the patch.

Looks like you missed a line.

No the vset part is removed and no longer relevant.