Delete Action - Error @ Hotschi's Download MOD 6.3.7

[bog_tom]

I upgraded to 6.3.7 from 6.3.6 and I uploaded by mistake two files under same download. If I try to delete the old file from Moderator Panel (having Delete files too option checked) I get an error message (see below).

Is it a known bug? Thanks !

Code: Alles auswählen

SQL ERROR [ mysql4 ]

[0]

SQL

No values specified for SQL IN comparison

BACKTRACE

FILE: includes/db/dbal.php
LINE: 379
CALL: dbal->sql_error()

FILE: dl_mod/includes/dl_modcp.php
LINE: 88
CALL: dbal->sql_in_set()

FILE: downloads.php
LINE: 1349
CALL: include('dl_mod/includes/dl_modcp.php')

Similar error I'm getting if I try the same action from Administrator Panel:

Code: Alles auswählen

SQL ERROR [ mysql4 ]

[0]

SQL

No values specified for SQL IN comparison

BACKTRACE

FILE: includes/db/dbal.php
LINE: 379
CALL: dbal->sql_error()

FILE: dl_mod/admin/dl_admin_files.php
LINE: 435
CALL: dbal->sql_in_set()

FILE: includes/acp/acp_downloads.php
LINE: 213
CALL: include('dl_mod/admin/dl_admin_files.php')

FILE: includes/functions_module.php
LINE: 507
CALL: acp_downloads->main()

FILE: adm/index.php
LINE: 74
CALL: p_master->load_active()

[oxpus]

Please explain this a little bit more detailed:
You have uploaded a file and in the download the same file as new version, too?
And deleting one of these files the script will just display the error message?

[bog_tom]

Yes, that's the case.

[oxpus]

Okay, I see the error:
You must really select the version in die "dropdown" box you will delete otherwise the MOD will not get the file id to manage it.

But this is a point I must check before building the sql statements at this point.
So thanks for that hint.

[bog_tom]

Am I doing it right or does it mean is a bug?

[oxpus]

You have not selected the dropdown box with the version you wants to delete.
So the script cannot fetch the file id to delete the file!!

I mean this box:

DL_SELECTION.png (45.32 KiB) 4858 mal betrachtet

So just click into this box, then select the option "Delete the following selected versions" and after this submit the download.

Regard:
Only selected versions can be deleted!
All other combinations will currently just produce a sql error like this one you've posted.

[bog_tom]

Yeap, it worked!

So this should be enforced in a future release.

Thanks for your help!

[oxpus]

Code: Alles auswählen

So this should be enforced in a future release.

It's still entered in the todo list.

[bog_tom]

I have to appreciate your very responsive support !

Good job and keep it up!

[bog_tom]

Code: Alles auswählen

So this should be enforced in a future release.

It's still entered in the todo list.

I see that it wasn't implemented yet... at least 6.4.2