Social Media Plugin Code Review
 

Hello friends, I've been learning PHP and creating my first plugin the past few days. I'm using Nexia's OpenGraph Protocol Inserts as a basis for my project, so many thanks and credits for this great modification. I've shot him an email asking permission to perhaps update/release this project and am waiting to hear back.

The plugin accomplishes this so far:

• Adds Open Graph and Twitter Card meta tags to pages for better sharing
• Includes code linking Facebook Domain and app insights to your page
• vbAdvanced Support (the main reason I started modifying this)
• Picks an image from the content with priority being video, then hot linked image, then attached image.
• Grabs thumbnail from youtube video and drives traffic from your social share to your page instead of youtube
• Defaults to a vboptions defined image if no image is found, no more social shares without images or badly resized images
• Tells twitter the aspect ratio of the image for better resizing

This is what is adds to the HTML:
Now the share looks like this:
https://vborg.vbsupport.ru/external/2016/05/11.png
Instead of:
https://vborg.vbsupport.ru/external/2016/05/12.png

Combining this with some simple social share buttons makes my social marketing campaign 2easy

I was hoping to get some feedback on my code from the pros. Currently on vb4.2.3 and vbAdvanced 4.3.0
Please don't install if you don't know what you're doing, I've tested it thoroughly but am not an expert

The code looks fine. The only comment I can make about it is that you should avoid using $GLOBALS. It's bad practice to use that these days.

I agree with Dave on both counts. I've never needed to use $GLOBALS.

But, congratulations on creating your first product. But be warned...it can be addictive. :)

Could you fine fellas enlighten on me on how I would access $GLOBALS['threadinfo']['description']
without using $GLOBALS? Thanks in advance

You've already declared the array "$threadinfo" global in your plugin, so you should be able to access that element with:

$threadinfo['description']

Worked like a charm, thanks