Page MenuHomeDevCentral

[Channel] Ensure we've a video title before print it
ClosedPublic

Authored by dereckson on Jan 9 2016, 01:34.
Tags
None
Referenced Files
F23854199: D191.id452.diff
Sat, Feb 7, 05:53
F23844969: D191.id452.diff
Sat, Feb 7, 04:17
F23844758: D191.diff
Sat, Feb 7, 04:16
F23834949: D191.id452.diff
Sat, Feb 7, 02:49
F23834361: D191.diff
Sat, Feb 7, 02:44
F23815125: D191.diff
Fri, Feb 6, 23:42
F23809364: D191.id.diff
Fri, Feb 6, 22:35
F23808184: D191.id452.diff
Fri, Feb 6, 22:25
Subscribers
None

Details

Summary

Sometimes, a video from Vimeo doesn't have a title.

Before, "PRIVMSG #channel :" were sent to the IRC server, with
an extraneous command sent to the server.

With D166 and a new prefix, we had a "[Vid] " line printed.

We now check the length of the video title.

Issue reported by popolon.

Test Plan

Tested on Freenode #fauve.

Diff Detail

Repository
rVIPERSERV ViperServ scripts [legacy Mercurial repo]
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson retitled this revision from to [Channel] Ensure we've a video title before print it.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson edited edge metadata.
This revision is now accepted and ready to land.Jan 9 2016, 01:35
This revision was automatically updated to reflect the committed changes.