Implementation error in Twitter::Status#all_urls #206

Merged
merged 1 commit into from Oct 20, 2011

2 participants

@seanmoon

Yikes. Flubbed the order of 'compact' and 'flatten' in my implementation.

Was getting things like

['http://example.com/', nil]

Corrected it to return

['http://example.com/']

Wrote a test that fails, might be too defensive, your choice if you think it's worth maintaining the test.

Sorry about that :<

@sferik sferik merged commit cacb6ca into sferik:master Oct 20, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment