Delete function not working

Please confirm you are working with version 2.1.1. The first function in the php, the last lines should look like:

add_filter( 'gform_confirmation', 'gfuea_clean_zips', 10, 4 );
        $notification['attachments'] = $attachments;
    }
    GFCommon::log_debug( $log . 'stopping.' );
    return $notification;
}

I meant version 2.1.2. I am not sure but I think the WP Plugin system has issues with sub versions. If I need to push a 2.2 I will to have updated be better pushed out. But from my testing with the moving of that filter towards the bottom of that function allowed for deletion even without zip.

I am on version 2.1.2.

Adding a log_debug for the $gf_delete_files variable in the gfuea_clean_zips show that it’s always “no”.

I just did a test with writing to the error log and confirmed things are working how they should.
I put the following code in:
‘
error_log(“Delete Files: ” . $gf_delete_files . ” ” . date(“Y-m-d H:i:s”));
‘
and got the following response after checking and unchecking the box.

[25-Aug-2016 20:57:29 UTC] Delete Files: yes 2016-08-25 20:57:29
[25-Aug-2016 20:58:42 UTC] Delete Files: no 2016-08-25 20:58:42

I am going to make a slight modification to something and then give it a version 2.2 to try to force an update for you but I am seeing the delete functionality working for no zip.

I am going to start limiting version numbers to just one decimal place as I think that is how the system fully updates everything. But please give version 2.2 a try and let me know if it starts working for you for some reason. I only changed some wording and also the
‘if ($last7 == “GFUEANZ” || $zip_attachment != ‘yes’){‘
so that it would register the code changing.

I wonder if the globals use is not working. If that is the case then I will have to look into another way of accomplishing the delete after confirmation kind of thing.

I am on version 5.3.3 of PHP, not sure if that makes a difference.

I uninstalled and force reinstall of version 2.1.2, and verified in the readme and code that it was 2.1.2.

I’ll try version 2.2 and see.

I also have multiple notifications, the one that I want to delete the files on, is not the last notification. The last notification does not have file attachments or anything on it. Does that matter?

Bingo, that was it. I didn’t know the notification with the delete has to be the last notification. I mean, it only make sense.

Doh: User Error.

Thanks for the great plugin and helping me with the trouble shoot.

Resolved. Thanks!

I will look into how to make it work no matter where it gets checked of. Thanks for bringing it to my attention.