Perhaps if no author then just leave URL null?


On 21 May 2014 14:11, Victor Kareh <notifications@github.com> wrote:

> In lib/fetching/content-services/twitter-content-service.js:
>
> > @@ -71,12 +71,13 @@ TwitterContentService.prototype.addListeners = function() {
> >  };
> >
> >  TwitterContentService.prototype._parse = function(data) {
> > +  var author = data.user ? data.user.screen_name : null
>
> Added a condition to check for the URL. A few times (really rarely, like
> once or twice in a test of hundreds of thousands of reports) I got a
> twitter post without an author. Decided it's worth checking for it than
> having Aggie crash unexpectedly, even if the probability is really low.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/TID-Lab/aggie/pull/46/files#r12911828>
> .
>



-- 
Tom Smyth
Worker-Owner, Sassafras Tech Collective
Specializing in innovative, usable tech for social change
sassafras.coop *·* @sassafrastech