Blojsom-257 code review

View: New views
1 Messages — Rating Filter:   Alert me  

Blojsom-257 code review

by Timothy Stone-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

I just created Blojsom-257:

        Twitter Notification Plugin fails to locate configuration properties

During local testing, the TwitterNotificationPlugin would fail to load  
the username/password set per the documentation. It was determined  
that the plugin attempted locate the configuration in the User metadata.

The documentation for the plugin states to set the twitter  
configuration properties in Weblog Settings | Properties | Set  
Property. This will place the keys in the `properties' table of the  
database. This table is loaded by the Blog, not the User, object.

A patch is attached for review.

*This patch also includes a security correction to the  
TWITTER_STATUS_UPDATE_URL. s/http/https. This prevents the twitter  
login information from being passed in the clear.*

Regards,
Tim






------------------------------------------------------------------------------
Create and Deploy Rich Internet Apps outside the browser with Adobe(R)AIR(TM)
software. With Adobe AIR, Ajax developers can use existing skills and code to
build responsive, highly engaging applications that combine the power of local
resources and data with the reach of the web. Download the Adobe AIR SDK and
Ajax docs to start building applications today-http://p.sf.net/sfu/adobe-com
_______________________________________________
Blojsom-developers mailing list
Blojsom-developers@...
https://lists.sourceforge.net/lists/listinfo/blojsom-developers

twitterNotificationPlugin.patch (2K) Download Attachment
PGP.sig (266 bytes) Download Attachment