fixes #144

Author: Zirak

master.js

@@ -6598,18 +6598,36 @@ bot.addCommand({
 var cache = {};
 
 function urban ( args, cb ) {
-	if ( !args.length ) {
+	if ( cache[args] ) {
+		return finish( cache[args] );
+	}
+
+	var parts = args.parse(),
+		query, resultIndex;
+
+	if ( !parts.length ) {
 		return 'Y U NO PROVIDE ARGUMENTS!?';
 	}
 
-	if ( cache[args] ) {
-		return finish( cache[args] );
+	// /urban query in several words
+	if ( isNaN(parts[1]) ) {
+		bot.log( '/urban input isNaN' );
+		query = args.toString();
+		resultIndex = 0;
+	}
+	// /urban query index
+	else {
+		bot.log( '/urban input isn\'t NaN' );
+		query = parts[ 0 ];
+		resultIndex = Number( parts[1] );
 	}
 
+	bot.log( query, resultIndex, '/urban input' );
+
 	IO.jsonp({
 		url : 'http://api.urbandictionary.com/v0/define',
 		data : {
-			term : args.content
+			term : query
 		},
 		jsonpName : 'callback',
 		fun : complete
@@ -6619,10 +6637,14 @@ function urban ( args, cb ) {
 		var msg;
 
 		if ( resp.result_type === 'no_results' ) {
-			msg = 'No definition found for ' + args;
+			msg = 'No definition found for ' + query;
+		}
+		else if ( resultIndex > resp.list.length ) {
+			msg = 'Nothing in that index. The last one is:\n' +
+				formatTop( resp.list.pop() );
 		}
 		else {
-			msg = formatTop( resp.list[0] );
+			msg = formatTop( resp.list[resultIndex] );
 		}
 		cache[ args ] = msg;
 
@@ -6659,7 +6681,8 @@ bot.addCommand({
 
 	permissions : { del : 'NONE', use : 'ALL' },
 
-	description : 'Fetches UrbanDictionary definition. `/urban something`',
+	description : 'Fetches UrbanDictionary definition. ' +
+		'`/urban query [resultIndex=0]`',
 	async : true
 });