GJ and thx for the update :)

I need for you to tell me a bit more, as you are only 1 of 2 people (92 people so far have downloaded this) who have reported this issue.

Can you tell me what parts of Read PM's CP you are on about (it is not specific enough), I am trying to find out where it occurring.

Same error...screenshot is in the attachment
vBulletin v3.8.3
PHP v5.2.9

Error come only when you list PMs (show last 100 PMs, or when you search PMs with some word)

Same errors on 2 different installs

One Install...

Shows the TO for the last 55 items then errors.
vBulletin 3.8.1 Patch Level 1
PHP 5.2.5
MySQL Version 5.0.81-community

Second Install...

WHen showing last 250, 2 of the items about midway show the error but the rest are OK.
vBulletin 3.8.2
PHP 5.2.5
MySQL Version 5.0.81-community

Not having any other issues on either site

What I think is happening is that the for some reason the array that holds the To, CC and BCC users for certain PMs is not being picked up, and as a consequence empty arrays are being passed to the function that creates the list of recipients that is displayed in the To field. I have done a change that should display a message Reciepients can not be displayed. if I am right.

[s]Can all four of you try the file attached and let me know if it works (i.e. no error messages and displaying the message above in some fields? Simply over-write your old version with this one.[/s]

If it fixes the issue, I will release a new version :)

EDIT: Removed attachment, as this has now been rolled into the official v3.8.009 release.

For me is working with this fix :up:

Worked for me also

Update: v3.8.009 - Monday 6th July 2009
-- Fixed: Invalid argument supplied for foreach() error a very small percentage of people were experiencing
-- Added: 1 phrase

As a lot of people benefit from this hack, again, if you have time, I would urge you to help me out and give back to the community at the same time, by providing a translation in your native language (as long as there is not an official translation already out there). So, if you read and write a different language other than English, and have a couple of hours to provide a translation in your language, please see the Translations section above for more information.

Is there any way to get your version of this Reading PMs mod to be listed by user name rather than totals. It seems to be a waste of time to first list the PMs by totals. I need it to show a list of usernames.

Anyway to get this to list like Dreams version but with your updates?

The error issue is resolved.

In the individual PM, there is no TO listed. Not sure what is up with that.

Yes, but the reason I did it this way rather than leave it the way Dream does it, is because Dream's version works fine on smaller forums with only a few thousand members on it. But when you get into the tens of thousands of users and they all have at least one PM, then what happens with Dream's version, is that it starts taking thirty plus seconds for the query to complete as Dream's version does not have any form of pagination, thus rendering this hack unusable (I have actually replicated this issue on a Big Board).

That is why it is the way it is in this version.

I may look into doing it via username (no promises), but it would definitely need to have pagination incorporated to prevent making the hack useless. It would be in addition to to via PM totals, not instead of them. But like I said, no promises.

It means that the To array in the individual PM is not being retrieved from the database, and as a consequence it is not being displaced. It has been this way since v3.8.001 of this hack, so it is not the hack doing this.

Also, I have found that this happens with forums that are really old (i.e. they were installed with a vBulletin version below v3.5), as the structure of PMs changed around vBulletin v3.5, so older PMs created in version 3.0.x or below may not show the To and CC and BCC fields correctly with this hack.

I figured your design layout had to do with size of board issues. It would be awesome to have a paginated version that lists user names though. I hope you consider it.

This modification is very useful if members are are abusing or spaming a forum and if used in that context there is no raisng issue of violation of members rights.

thanks for the update

Great hack. This should solve those pesky 'he said, she said' arguments.

Installed.

See the above, but I will tell you only this until you have clicked Mark as Installed: You have not uploaded the files to the correct locations.

OK, now that you have clicked Mark As Installed, the first thing I noticed is that read_pms.php is not in your admincp folder. As print_cp_header() is an AdminCP function, if read_pms.php is not in the admincp folder, then you will have this issue - see the vBulletin API for more info.

nice job...

Thank alot

You know, this is one of those hacks, as an admin... I don't really want to admit I have... but I got a bot that has managed to bypass vBulletin's image verification, etc... to register and then PM me with an advert for his crap that enabled him to register and spam me with PMS !



So... I take the time to go to the database, look up the id and get to deleting all of the spamming he has done to the other members.

So, if I want to read PM's.. I could do it anyway since I have access to the database... no, I do not want to read your PRIVATE message's... but yes, I do want a quicker way than digging through the database to see if I'm the only one getting spammmed... so yes.. I have this installed - not to read PM's - but to lighten the load a bit.

Sorry... I just hate having to actually need this. But thanks, it is useful.

You know you really need to moderate new members by approving them first! That is what Admins and Mods are for! No bot can get in a properly moderated site! :up:

Hi Mosh, and thankyou for another greatfull hack.

Is there the possibility to hide the label 'Read PMs' to the regular admin and to anyone NOT SuperAdmin?

ty 4 hints will be highly appreciated

apologises for broken english

ciao take care

edit: on script line #63 there is a little issue. I immagine the correct one should be :

instead original

Thank you!

Andrea, the only way to do so currently, is to not upload cpnav_rpm.xml and to use a shortcut/bookmark to http://<your-forum-path-here>/admincp/read_pms.php? instead.

Thanks Andrea, well spotted :) Will fix it in the next update ;)

hello!

nice mod

found one mistake in read_pms.php on line 171

you have:


should be:

reg
macc

Thanks macc :) Will fix it in the next version ;)

I think I had better do a code review now. I will post an update in the next hour or so.

Update: v3.8.010 - Saturday 25th July 2009
-- Fixed: Typo on line 63 of 'readpms.php' - thanks to Andrea (Silvio)
-- Fixed: Typo on line 171 of 'readpms.php' - thanks to macc
-- Fixed: Typo on line 476 of 'readpms.php'
-- Fixed: Typo on line 483 of 'readpms.php'

This is just a typo fix update. There is no new functionality, no bug fixes, just fixed four typos. So, only update if you feel the need to.

And yet, again I am requesting this. As a lot of people benefit from this hack, if you have time, I would urge you to help me out and give back to the community at the same time, by providing a translation in your native language (as long as there is not an official translation already out there). So, if you read and write a different language other than English, and have a couple of hours to provide a translation in your language, please see the Translations section in the first post for more information.

Hi Mosh,

I recently installed your mod, but:

1st: I dont have a link in the ACP
2nd: When I try to open the read_pms.php manually, it redirects me to the forum index

=(

Are you sure you followed all the steps in the installation instructions in the readme.txt file? As the first problem and most likely the second are being caused by not following the instructions to the letter.

Question on this MOD. Is it possible to prevent my global moderators, who have access to the user part of the admincp, from seeing/reading this link to the specific part, or concealing it elsewhere? Also, same with a coadmin.

gj and thx for the update.

Update: v3.8.011 - Wednesday 12th August 2009
-- Added: Italian translation - author wishes to remain anonymous
-- Updated: "readpms.php" to accommodate the new language
-- Updated: Instructions on how to change language files

This is just a language update. There is no new functionality, no bug fixes, just added a new language. So, only update if you feel the need to.

As an unnamed contributor got the ball rolling with language translations (thank you once again ;) you know who you are), again I will make the request.

As a lot of people benefit from this hack, if you have time, I would urge you to help me out and give back to the community at the same time, by providing a translation in your native language (as long as there is not an official translation already out there).

So, if you read and write a different language other than English or Italian, and have a couple of hours to provide a translation in your language, please see the Translations section in the first post for more information.

thanks

you are going forward with this :) gj as always

updated ...thanks

thanks

Update: v3.8.012 - Thursday 13th August 2009
-- Added: German language file "read_pms_deu.php" - thanks to Speedy1505
-- Updated: "readpms.php" to accommodate the new language
-- Updated: Instructions

This is just a language update. There is no new functionality, no bug fixes, just added a new language. So, only update if you feel the need to.

As Speedy1505 continued the ball rolling with language translations (thank you once again Speedy ;)), again I will make the request.

As a lot of people benefit from this hack, if you have time, I would urge you to help me out and give back to the community at the same time, by providing a translation in your native language (as long as there is not an official translation already out there).

So, if you read and write a different language other than English, German or Italian, and have a couple of hours to provide a translation in your language, please see the Translations section in the first post for more information.

Update: v3.8.013 - Friday 14th August 2009
-- Added: Brazilian (Portuguese) language file "read_pms_bra.php" - thanks to Setokaiba (SW)
-- Updated: "readpms.php" to accommodate the new language
-- Updated: Instructions

This is just a language update. There is no new functionality, no bug fixes, just added a new language. So, only update if you feel the need to.

As Setokaiba (SW) is continuing to keep the ball rolling with language translations (thank you once again Setokaiba ;)), again I will make the request.

As a lot of people benefit from this hack, if you have time, I would urge you to help me out and give back to the community at the same time, by providing a translation in your native language (as long as there is not an official translation already out there).

So, if you read and write a different language other than Brazilian, English, German or Italian, and have a couple of hours to provide a translation in your language, please see the Translations section in the first post for more information.