diff options
| author | Matthias Hecker <mail@apoc.cc> | 2020-04-13 20:48:40 +0200 |
|---|---|---|
| committer | Matthias Hecker <mail@apoc.cc> | 2020-04-13 20:48:40 +0200 |
| commit | cc241a9709593491b6a67810ed8a5a5054e19208 (patch) | |
| tree | 9c5eed30d0547f01002606f15e1299bbaf862f0b | |
| parent | 90656f4203a0a989b6fb110d4a07598dd186b84c (diff) | |
plugin(points): forgot one special case, see #34
Tokens that include itself ++/-- are now ignored as well.
| -rw-r--r-- | data/rbot/plugins/points.rb | 3 | ||||
| -rw-r--r-- | test/plugins/test_points.rb | 2 |
2 files changed, 5 insertions, 0 deletions
diff --git a/data/rbot/plugins/points.rb b/data/rbot/plugins/points.rb index 976594ee..2f67092f 100644 --- a/data/rbot/plugins/points.rb +++ b/data/rbot/plugins/points.rb @@ -85,6 +85,9 @@ class PointsPlugin < Plugin token = $1 # strip ++/-- from token flag = $2 # remember ++/-- + # token must not have more than one ++/-- + next if token.match(/(\+\+|--)/) + # each token must include at least one alphanumerical character next unless token.match /[[:alnum:]]/ diff --git a/test/plugins/test_points.rb b/test/plugins/test_points.rb index d29b6174..b3386ed4 100644 --- a/test/plugins/test_points.rb +++ b/test/plugins/test_points.rb @@ -72,6 +72,8 @@ class PointsPluginTest < Test::Unit::TestCase ' --', '++ --', '-- ++', + '+rw+r++r++', + '-rw-r--r--', 'https://linux.slashdot.org/story/20/04/12/2138205/how-red-hats-new-ceo-handles-life-under-ibm----and-a-global-pandemic' ] ignored.each do |ignore| |