Many copies fo this error message:

# Notice: Undefined variable: node_type in media_uninstall() (line 24 of /sites/all/modules/media-DRUPAL-7--1/media.install).
# Notice: Undefined index: field_name in media_uninstall() (line 24 of sites/all/modules/media-DRUPAL-7--1/media.install).

Comments

aaron’s picture

not sure of the state of this issue -- there are other issues on disable in any case; they can probably go here as well.

aaron’s picture

Priority: Normal » Critical
Issue tags: +alpha blocker

this really should be fixed before releasing the alpha.

dipen chaudhary’s picture

When you uninstall the module you get a Fatal error for undefined function - media_registration_kinds() (Line 201 media.install).

Also not sure why do we have following:

 variable_del('media_file_extensions');
  variable_del('media_max_filesize');
  variable_del('media_file_progress_image');
  variable_del('media_file_list_size');
  variable_del('media_file_progress_success');

Are we storing these settings somewhere, on a quick look I dont think we are.

Also upon further testing, if we enable the module again db_add_field doesn't add type column to file table.

aaron’s picture

Status: Active » Needs work

i've fixed up the variable deletion.

JacobSingh’s picture

Status: Needs work » Fixed

I think aaron's fix does the trick.

Status: Fixed » Closed (fixed)
Issue tags: -alpha blocker

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