This is the error message I get when I click on the "Invisimail Formatter" button on the admin menu....

Notice: Undefined index: invisimail_formatter_settings in drupal_retrieve_form() (line 736 of /var/www/oracletalk.info/public_html/includes/form.inc).
Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'invisimail_formatter_settings' not found or invalid function name in drupal_retrieve_form() (line 771 of /var/www/oracletalk.info/public_html/includes/form.inc).
CommentFileSizeAuthor
#2 1016102-invisimail-remove-invisible-callbacks.patch1.61 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

Hrm, there doesn't seem to be a settings form at all. Should this callback even exist?

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Version: 7.x-1.0 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
1016102-invisimail-remove-invisible-callbacks.patch1.61 KB

Just checked, looks like its a remnant from the D6 version that is no longer necessary.

radoeka’s picture

radoeka CreditAttribution: radoeka commented

Seems to work. The settings page is gone, is that correct?

hgurol’s picture

hgurol CreditAttribution: hgurol commented

patch fails?

patching file invisimail.module
Hunk #1 FAILED at 11.
Hunk #2 FAILED at 71.
Hunk #3 FAILED at 159.

edit: lol, i guess, Im doing something wrong...

radoeka’s picture

radoeka CreditAttribution: radoeka commented

What is the command you use?

sites/all/modules/invisimail> patch -p0 < patch
patching file invisimail.module
hgurol’s picture

hgurol CreditAttribution: hgurol commented

Yes, the patch works, as the settings button on admin menu has gone.

@radoeka
thanks...

Crell’s picture

Crell CreditAttribution: Crell commented
Status: Needs review » Fixed

Crell--

I could have sworn I cleared out all of the legacy crap. Committed #2. Thanks, Dave. I also removed the permission hook, which is not used either.

I'll roll a new stable release momentarily, since I'd call that a critical fail.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.