blame: Put the filename in the loadblame result

We are going to use this to say what kind of fish it was, in
~blame #last.

We also need to change the two places where this is destructured.

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

blame #last: Blame the right thing

Previously this would search all the blame databases.  This is wrong,
it should search the one corresponding to the last fish.  Now that we
have last_cfg, and the db info is in the cfg, we can do that.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

blame: Move blame db information into the cfg

These should never have been separate.

We abolish the ad-hoc tbdb, fbdbk etc. variables.  Instead, we put
this all in the cfg.  It is OK to extend the cfg since the code does
not destructure it but instead always indexes it.  So bot instances
without blame will not mind.

blameq needs to fish it out again for the benefit of the open-coded
conditions on "#trouts" etc.  I'm not fixing that right now.

The deleted lines like this
  tbdb,tbdbk=loadblame("trouts")
have different variable names than the inserted lines like this
  tdb,tdbk = cfgs[0][7]
because, unaccountably, the global variable names are different to
the local variables inside blameq.

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

fish: Pass cfgs to blameq

This open-coding of three separate things, with blameq having magic
knowledge of which one is which, is completely wrong.  blameq should
take a list.  The various Xdb Xdbk should in the cfgs too.

A step towards this is for blameq to get the cfgs.

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

fish: track last_cfg too

No-one uses this yet.  So no functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

new flirt from Emperor (apropos remarks made about a cyclist)

new trout from jtn ( https://theoatmeal.com/comics/idiocy )

new trouts from rejs

new flirts from rejs

new trout from jtn

new trout from jtn

Output the date of first mention, not the current date

...this is in the code path where the URL was mentioned on a previous
date, so we should output the date of first mention (self.localfirst)
not the current date (n).

new trout from chelgrian (no, not the album by Bride)

strip more garbage out of twitter urls (esp the ?s=[:digits:]) tracking code - thanks cjwatson

Add blame-filtering

This simply works on the md5 of the offending commit message, and
replaces it with new text.

blame_filter.py is a simple dictionary of md5: replacement pairs; it
is not committed itself (and is ignored by git)

0-pad times (so 13:03 rather than 13:3)

New trout from Diziet (apropos what 🔥 looks like)

Correct tyop in flirt from rjk

new flirt from Diziet

Adjust url-observation, per chiark.irc discussion

Servus now describes URLs without adjective, as these were felt to be
discouraging. Further, it won't mention a URL within 5 minutes of its
previous mention. Time is now given as a timestamp, rather than the
"about X minutes/hours ago"

Don't remark on URLs that are repeated after a short period

If a URL has been mentioned in the recent past, then it'll be
obviously visible in-channel, and it was probably a mistake
(e.g. mentioning a URL you meant to ~twit), so no need to comment on
it.

url observations: Rename `urlcomplaints' variable to `urlinfos'

These are no longer complaints.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
Conflicts:
commands.py

url observations: make the "complaints" less complainy

Based on Ian Jackson's re-wording, but tweaked a little to suit my
taste better.

main: Drop daft try/expect

This serves only to obfuscate.  The result is that, for example, if
you don't have the twitter module, you get this error message:
  acrobat: configuration module Servus-chiark.py not found
which is a lie.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

Remove trout in light of objections

new trout from rjk

new flirt from Emperor (SatW board game)

correct typo (missing : )

Add a limit to tweet expansion recursion.

Signed-off-by: Matthew Vernon <matthewv@chiark.greenend.org.uk>

Add exemplar servus_twapi_oauth.txt file.

Signed-off-by: Matthew Vernon <matthewv@chiark.greenend.org.uk>

With the new embedded tweet code there's no reason not to expand embedded tweets inside embedded tweets.

Signed-off-by: Matthew Vernon <matthewv@chiark.greenend.org.uk>

Fix for embedded tweet support:

Problem:
 * Overly long embedded tweets cause Acrobat to go over the server's
  line length limit which causes the message to be silently discarded
  because of a lack of error handling in automsg.

Solutions:
 * Make each tweet in an embedded tweet chain a seperate message
   - this requires getTweet to now return a list
 * Did not fix automsg at all

Signed-off-by: Matthew Vernon <matthewv@chiark.greenend.org.uk>

new flirts from rejs

new trouts from rejs

New trouts from rjk

new trout from Emperor (based on previous)

new trout from rjk

new trouts from rjk

new flirt from jtn

new trout from jtn

new flirts from rejs

new trouts from rejs

New trouts from jtn

New trout from adam

New trout from Midwinter

New flirt from jtn (after fivemack)

new trout from rjk

new trout from jtn [https://en.wikipedia.org/wiki/Paperclip_maximizer]

new trout from rjk

new trout from adam (bbc.co.uk/news/uk-england-bristol-41167296)

new trout from jtn

new trouts from rejs

new trout from rjk

New trout from Emperor

new trout from Diziet

New trout from rjk

new trout from rjk (ref Alt-J song)

new flirt from fivemack

new flirt from rejs

new trouts from rejs

New trout from Emperor (improved by ceb)

New trout from rjk

Updates for newer version of python-twitter (new twitter API)

These are changes to support what will eventually be version 3.2 of
python-twitter; for the time being we are running of a git clone of
the devel branch, which supports the newer twitter API (so e.g. tweets
quoting other tweets work properly); it doesn't look like an upstream
release of 3.2 is going to happen any time soon.

Author: Adam Bernard
Signed-off-by: Matthew Vernon

new flirt from rjk

new flirt from rjk [ref Better off Dead]

New slash from Diziet

new slash from rejs

New flirt from rejs

New trouts from rejs

new trout from adam

Patch from adam to allow - in video URLs

new trout from rjk

New trouts from Emperor (pp climbing)

new trout from jtn [bbc.co.uk/news/uk-politics-36160135]

new flirt from rejs

new trouts from rejs

New trouts from Emperor

New trout from rjk

Patch from adam to expand quoted tweet (1 level only)

new trout from rjk

new trout from rjk

new flirt from fivemack

new flirt from ceb

Restore correct handling of multi-line tweets

Improve UTF-8 handling, multiple media links, url expanders

This patch has 3 improvements:

* understands links to multiple media items, spits out URLs for each
  picture &c.
* understands a range of URL-shorteners, expands them
* improves UTF-8 handling

signed-off-by: adamb

Expand video URLs

The previous patch gave you the photo link for the video, not the
video itself; this fixes that (and handles non-ascii better again,
though it's a bit of a hack having to UTF-8 it twice).

patch by adam

Expand URLs and unescape &<> in tweets

This expands the URLs in tweets (instead of the t.co shorturls that
you get currently).

Patch by adamb

new flirt from rjk

new slashes from rejs

new flirts from rejs

new trouts from rejs

new trout from jtn [cf hard stare]

new trouts from rjk

new trout from rjk [ref: Truth Coming Out of Her Well to Shame Mankind]

new trout from rjk

new trout from rjk

new trout from rjk

new trout from Emperor

new slash from rjk

new flirt from rjk