Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UrbanDictionary.maxNumberOfDefinitions seems to have no effect #40

Open
MetaNova opened this issue Jul 9, 2021 · 0 comments
Open

UrbanDictionary.maxNumberOfDefinitions seems to have no effect #40

MetaNova opened this issue Jul 9, 2021 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@MetaNova
Copy link

MetaNova commented Jul 9, 2021

<@MetaNova> +help ud
<+chat> (ud <a global alias, 0 arguments>) -- Alias for "urbandictionary $*".
<@MetaNova> +cc ##llamas-private supybot.plugins.UrbanDictionary.maxNumberOfDefinitions
<+chat> 10
<@MetaNova> +ud test
<+chat> test :: 1. the main cause of [explosions].2. any thing [dreaded] that your "teachers" say is "good" for you. soon after, you explode for no reason.3. what scientists do to make stuff explode.4. when a sheet of paper explodes into [flames]. [ex:] 1. test [sodium] and water.2. SAT is a test.3. [Monkeys].4. you brought your [lighter] to test. [/ex] | slang for [testosterone]..[steroid] [hormones]. [ex:]  (6 more messages)
<@MetaNova> +cc ##llamas-private supybot.plugins.UrbanDictionary.maxNumberOfDefinitions 1
<+chat> Done.
<@MetaNova> +ud test
<+chat> test :: 1. the main cause of [explosions].2. any thing [dreaded] that your "teachers" say is "good" for you. soon after, you explode for no reason.3. what scientists do to make stuff explode.4. when a sheet of paper explodes into [flames]. [ex:] 1. test [sodium] and water.2. SAT is a test.3. [Monkeys].4. you brought your [lighter] to test. [/ex] | slang for [testosterone]..[steroid] [hormones]. [ex:]  (6 more messages)

However, calling --num 1 seems to work as intended:

<@MetaNova> +urbandictionary --num 1 test
<+chat> test :: 1. the main cause of [explosions].2. any thing [dreaded] that your "teachers" say is "good" for you. soon after, you explode for no reason.3. what scientists do to make stuff explode.4. when a sheet of paper explodes into [flames]. [ex:] 1. test [sodium] and water.2. SAT is a test.3. [Monkeys].4. you brought your [lighter] to test. [/ex]
<@MetaNova> +urbandictionary --disableexamples --num 1 test
<+chat> test :: 1. the main cause of [explosions].2. any thing [dreaded] that your "teachers" say is "good" for you. soon after, you explode for no reason.3. what scientists do to make stuff explode.4. when a sheet of paper explodes into [flames].

Consequently, I've simply changed my ud aka from urbandictionary $* to urbandictionary --disableexamples --num 1 $* and that's good enough for now, but it would be nice if UrbanDictionary.maxNumberOfDefinitions actually worked.

The current (running) version of this Limnoria is 2021.07.04, running on Python 3.7.3 (default, Jan 22 2021, 20:04:44) [GCC 8.3.0]. The newest versions available online are 2021.06.15 (in master), 2021.07.04 (in testing).

Latest git clone of oddluck/limnoria-plugins

@oddluck oddluck added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed labels Aug 15, 2021
@oddluck oddluck closed this as completed Aug 15, 2021
@oddluck oddluck reopened this Aug 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants