Ill get to work on adding the requested features tomorrow :)
About the limit, currently there's no limit, but I suppose I should add one lol. Thats gonna be a bish to test out tho, havnt got Inline Mod Tools installed on localhost :p

very useful hack

Thanks

Nice work man

v2 has been released, I fulfilled the feature requests of mtha.
The following features was added to the mod:

• Name and link to forum of the thread/post
• Reason for deletion, if any were inputted
• Date of thread/post deletion
• Limit the number of total entries to show per page

To upgrade, simply download the upgrade zip and follow the instructions there.
This zip will be deleted in 1 week, after that you will have to redo the file edits as described in the main zip readme.

Also, if you use this mod,

that's GREAT! Thank you very much.

you may want to consider adding a phrase "del_deleted_by" for Name?

Thanks for the update, running smooth.. :)

In your file,
post.* also has the userid value, this will take over the deletionlog.userid I would bring that line deletionlog.userid AS userid to the end of serquence, and put a link to deletor's profile:

In thread section
<strong>". $vbphrase['soft_deleted_by'].":</strong> <a href='" . $vboptions['bburl'] . "/member.php?u=". $thread['userid'] . "' target='_blank'>" . stripslashes($thread['username']) . "</a><br>


In Post section
<strong>". $vbphrase['soft_deleted_by'].":</strong> <a href='" . $vboptions['bburl'] . "/member.php?u=". $post['userid'] . "' target='_blank'>" . stripslashes($post['username']) . "</a><br>


PS. I added a phrase called soft_deleted_by as GLOBAL with the text: Deleted by (instead of using "NAME" phrase).



adding
print_table_break();

on top of
print_table_header($vbphrase['soft_deleted_posts']);

would also seperate two sections.


also, showing the total of soft deleted posts/threads (and/or the current number of thread) would be nice

something like

Soft Deleted Threads (showing thread 16 to 30 in total of 82 threads)
Soft Deleted Posts (showing post 16 to 30 in total of 511 posts)


here's the code:

Look for:
$countlog = $DB_site->query_first("SELECT COUNT(*) AS total FROM " . TABLE_PREFIX . "deletionlog");
Add after

Look for (the first one)

print_table_header($vbphrase['soft_deleted_threads']);
while ($thread = $DB_site->fetch_array($softdelthreads))


Replace by

print_table_header("$vbphrase[soft_deleted_threads] (Total: $countthreads[total] threads)");
while ($thread = $DB_site->fetch_array($softdelthreads))

Look for

print_table_header($vbphrase['soft_deleted_posts]);
while ($post = $DB_site->fetch_array($softdelposts))

replace by

print_table_header("$vbphrase[soft_deleted_posts] (Total: $countposts[total] posts)");
while ($post = $DB_site->fetch_array($softdelposts))

About the post.*, when I tested it on localhost it always showed the mod name that deleted it, regardless of who the post belonged to.

I am implementing your changes on localhost now, and gonna see if theres a way to optimise it some (I always try my best to keep my query count to an all time low). Thanks alot for your suggestions :) Users like you, that are able to come up with simple yet very useful suggestions (even writing them :p), is what makes hacks great and hack authors happy :)


//peace

Bug ... (that might come out from original version of vB)

If you has a "soft deleted thread" containing some "soft deleted posts"

If you "Hard delete" the thread, the co-responding "soft deleted posts" will be undefined ($post['postid'] will not exists, only $post['primaryid'] exists)

Check that out, and it might be a bug in vB.com too. I dont know if it is, or if it has beed reported over there.




Additional suggestion.

01: This migh not be functionally important, but has "management" cleaner.

Giving the option to manage the thread and post seperately.

Says if you have

/forum/admincp/softdeladmin.php?type=thread, it will only process the deleted threads

/forum/admincp/softdeladmin.php?type=post, it will only process the deleted posts

if no value given, do both (like what you already have)


02: CHECK ALL option. Can you make it to check all thread and check all post seperately?


03. Thread Name/Moderator Name/Thread Link phrase should be changed :) I dont think you need that phrase at all.



... development ...
I just check the moderatorlog table, and dateline information for any moderating action is recored in there (with postid and threadid if applicable).
However, for one post, there could be many record in the log, including editing, moving, deleting/undeleting many times. So I expect (guess) the latest date record for one post should be the deletion time
... well but this will introduce few more searching, sorting ... queries ...

dont know if we should use this one or just use a new date record, for convenient

(just look that back because the new deldate is only applicable for posts/threads that are deleted after this hack installed. ... or we should just update the value from moderatorlog to deletionlog ONCE)

we can also sort them by date (asc or des - dont even need to have deldate value) so all recent deletion log displayed first.

just ideas, I dont know if all of them are useful :D

About that bug, I havnt really tried it. Do you mean if you hard delete a thread through this hack or vB itself?

Sug #1: This Ill do tomorrow. 4.30 AM here XD
Sug #2: Not poss as I am using vB internal JScript (I know no JS myself), and it checks all the elements in a form.