Excellent module! However, the "Delete CSV file from server" option at the end of a node import doesn't work. This is due to a typo in one of the validation functions, which I've corrected in the attached patch (the old validation function checked for a button called "Delete file from server" to be used in the form submission, but that button doesn't exist).

CommentFileSizeAuthor
node_import_delete.patch475 bytesDavid_Rothstein

Comments

Matthew Davidson’s picture

Matthew Davidson commented

Does this fix this bug as well?.

xtfer’s picture

xtfer commented

This works, thanks David, if you use a CSV file... I've taken your idea and applied it to the other end of the function:

http://drupal.org/node/205342

wmostrey’s picture

wmostrey commented
Status: Needs review » Closed (duplicate)

This indeed works for http://drupal.org/node/205342 as well, which has a working patch RTBC. Marking this as a duplicate.

David_Rothstein’s picture

David_Rothstein commented
Status: Closed (duplicate) » Fixed

Ah, nice, thanks! I only use the module for CSV files myself, so when I wrote the patch I never even thought of doing it the other way around, whoops ;)

Anonymous’s picture

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

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