search

clickrain / EE_Twitter

EE Add-on that uses oAuth to display Twitter Feed
Other
48 stars 18 forks source link

Fix missing check that access_token and access_token_secret are set #42

Closed stevegrossi closed 9 years ago

stevegrossi commented 9 years ago

In views/index.php, you check that both $settings['access_token'] and $settings['access_token_secret'] are set, otherwise setting them to false. This check wasn’t being made in mod.twitter.php, which was raising Undefined index: access token and Undefined index: access token_secret for me when using the {exp:twitter:user} tag.

This commit defaults those settings to FALSE to avoid this error, as was already done in views/index.php

bryanburgers commented 9 years ago

Good stuff, @stevegrossi! Thanks for that.