search

clickrain / EE_Twitter

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

Prefix Tags #5

Closed nevsie closed 11 years ago

nevsie commented 11 years ago

Great addon and works swimmingly!!!

Would it be possible for you to prefix you variables? In many occasions i am using this plugin with a main EE entries tag, and if a field matches things go a bit screwy!!!

For example a site i inherited has a field text, and this collides with the {text} tag in this module. If it were prefixed like the RSS magpie one does {magpie:title} then this collision would be averted!!!

Short term i just renamed fields, we actuall duplicated text to fulltext and outputted {fulltext} - but that was justa quicker fix for me!

Thanks, N

bryanburgers commented 11 years ago

@nevsie This is a good idea. It's not high on my priority list right now, but it is something I'd like to tackle. How have you seen this done in other plugins? I don't want to just start prefixing all of the variables, because that would break backwards compatibility when the plugin is updated—something I try to avoid if at all possible. Does a prefix variable passed into the tag make sense?