Status
Not open for further replies.

Optimus

Member
On the vbulletin's default mobile style (or any mobile style) this modification has issues. The first is, although the PMs show up when someone PMs you, when you click on the actual PM message in order to read or respond nothing but a blank page shows up. Doesn't matter what mobile style is used, on a pc or on a mobile. I don't want to get rid of the modification for everybody, so I would like to disable this modification for mobile styles only. But I don't know the right plugin to add the "AND $vbulletin->userinfo['styleid'] != X" code however. Any suggestions on where I could add that in so that members that use a mobile style are disabled from using this modification?

Also if I could make a suggestion, it might be a good idea to make disabling it for mobile styles an option in the settings for future releases.
 
Last edited:
Sorted, you can re-download the files from here, and upload, dbtech/vbsuper_pm/hooks/reply_all/template_render_output and that should fix it. :)
 
Hi Ozzy. I reuploaded the the file dbtech/vbsuper_pm/hooks/reply_all/template_render_output and it does work as advertised with mobile styles but there are other issues now.

With the new file uploaded, when I view the PM log in the ACP, this error shows up and prevents the log from being seen:

Fatal error: Call to a member function query_write() on a non-object in /home/____/public_html/forum/global.php(29) : eval()'d code on line 332

Also, the same fatal error shows up when another product is trying to install or when a template is being updated. which prevents the styles from refreshing/updating. Once the error shows up the updating/refreshing of a style completely stops and won't go further. I end up having to add the original dbtech/vbsuper_pm/hooks/reply_all/template_render_output file back when I want to edit a template or add a product so that the templates and styles updates completely, then add the new one back again.

Any solution to this?
 
Last edited:
I wanted to add that in addition to the above conflictions that moderating posts or threads from the ACP also produces that same error and doesn't work until the old backup file is reuploaded. Just felt it was important to let you all know about them all as I discover them. If you all would like me to stop, I won't report the errors.
 
Thanks Ozzy. Ok, I will post any other areas I find that error as I discover them. Anything to aid in this great product.
 
Just thought I'd update everybody with a work-around I did for this issue. It makes the mobile styles usable while Ozzy debugs the modification for a permanent fix.

I used your original template_render_output Ozzy since the newer one produced errors. Open the file, find the codes and replace them with ones below. Replace X with the ID of your mobile style then re-upload the file.

Find:
Code:
if (!$vbulletin->options['dbtech_vbsuper_pm_reply_all_active'] AND !$vbulletin->options['dbtech_vbsuper_pm_next_prev_active'])

Change to:
Code:
if (!$vbulletin->options['dbtech_vbsuper_pm_reply_all_active'] AND !$vbulletin->options['dbtech_vbsuper_pm_next_prev_active'] AND $vbulletin->userinfo['styleid'] != X)

Find:
Code:
if (!$_REQUEST['do'] == 'showpm')

Change to:
Code:
if (!$_REQUEST['do'] == 'showpm' AND $vbulletin->userinfo['styleid'] != X)

Find:
Code:
if ($this->template == 'pm_showpm')

Change to:
Code:
if ($this->template == 'pm_showpm' AND $vbulletin->userinfo['styleid'] != X)

I hope it helps :)
 
Last edited:
Status
Not open for further replies.

Legacy vBSuper PM's

vBulletin 4.x.x
Seller
DragonByte Technologies
Release date
Last update
Total downloads
543
Customer rating
0.00 star(s) 0 ratings
Back
Top