New Posting Features - Append User Agent String to New Threads and Posts

Description
This add-on will append a user's browser's User Agent String to new posts/threads.

Version Information
0.0.1 - Initial Release
0.0.2 - Added ability to only append User Agent Sting to posts/threads in specified forums.
0.0.3 - Fixed bug where previewing a post adds UA multiple times
0.0.3c - Fixed possible HTML injection vulnerability

Options
You can optionally configure it to append alternative text or BBCode (Such as IMG tags) rather than the literal User Agent String for specific User Agent Strings (or parts of).

You can configure it to not appended anything under certain styles

You can change the way the string is displayed with BBCode

(0.0.2) You can specify to only append User Agent String to posts/threads in specific forums.

Limitations
1. The User Agent String (or alternative BBCode/Text) is appended to the post text itself. There is nothing to stop the user from then editing the post and removing it. As such it will also show by default when quoted.

2. Turning the add-on OFF will not stop the display of User Agent Strings for past posts, it will only stop new posts/threads having it appended.

The upside of these is that there are no additional queries added to display this.


Screen Shots

[vbreal]

thanks

that did the job

[vbreal]

few issues

i have my main forum set to be excluded from appending the browser info. it should only show for my mobile theme. BUT it shows for mozilla browser and mac users on the forum that's excluded


also
HTC TOUCH and palm centrio phones don't even show up when they use the mobile theme. but it should, any idea why it isn't

[dartho]

There is no option for excluding forums. By default, all forums are included or you can choose to only have it show in explicitly defined forums only.

Could it be that the Mozilla/Mac users are actually browsing your site using the mobile style and teh HTC/Palm users are using your default style?

I'll check the logic of the product also and see if there may issues with it

[nerofix]

Could you do this to show in the Memberinfo template?

[vbreal]

Originally Posted by dartho

There is no option for excluding forums. By default, all forums are included or you can choose to only have it show in explicitly defined forums only.

Could it be that the Mozilla/Mac users are actually browsing your site using the mobile style and teh HTC/Palm users are using your default style?

I'll check the logic of the product also and see if there may issues with it

your mod has the ability to set what style you do not want the user info to show on.

I made my main theme not show the user agent string. but users who use macs and modzilla are showing up when posting from this style. i am sure they are not using the mobile theme. i had them do screen shots and i made the mobile theme unselectable.

the HTC users were on the mobile style.

No User Agent for these styles
Enter a comma separated list of StyleID numbers to not append User Agent String to.

If a user is using the any of the following styles, the User Agent String will not be appended to their posts.

[vbreal]

any thoughts?

[dartho]

I've looked and can;t see what could be causing your issues - do you have any other addons install which modify styles, or even with new posts/threads? Or, perhaps it is not compatible with 3.5/3.6?

Anyone else have similar issues - or any otehr for that matter?

[dartho]

I've had another look and the *may* be missing parenthesis in a plugin - I say may as I think it should be OK anyway, but if you could edit the Add Useragent String to New Posts and Add Useragent String to New Threads plugins and change

if ((($vbulletin->options['ua37_enabled']) && ($_POST[preview] != "Preview Post")  && (!in_array($styleid,explode(",",$vbulletin->options['ua37_not_this_style'])))) && (in_array($threadinfo[forumid],explode(",",$vbulletin->options['ua37_this_forum_only'])) || (trim($vbulletin->options['ua37_this_forum_only'])=="")))

to

if ((($vbulletin->options['ua37_enabled']) && ($_POST[preview] != "Preview Post")  && (!in_array($styleid,explode(",",$vbulletin->options['ua37_not_this_style'])))) && ((in_array($threadinfo[forumid],explode(",",$vbulletin->options['ua37_this_forum_only']))) || (trim($vbulletin->options['ua37_this_forum_only'])=="")))

and see if that fixes things?

-----

For a reality check - the logic for adding the useragent string is as follow:

IF the product is enabled, AND we are not previewing the post AND the user is not using an excluded style are all true

AND

We are in an forum where UA strings should be appended OR no forums are specified (ie. append in all forums) are true

THEN add the UA string

this sound right?

[CR_TurboGuy]

What about the ability to add this to private messages? I've taken a look at the hooks private_newpm_reply and private_newpm_start, and neither seem to do the trick for adding to a PM.

--JOsh

[dartho]

After a cursory look, I think PM integration would require file edits - hooks don't seem to be in quite the right place ...