egghelp.org community

[madwoota]

Ok, now thats out of the way, time for some begging/suggestions/thoughts/feedback

General suggestions, feel free to ignore them all as if you listen to us damn (ab)users too much, it ends up taking over your life! (And then we whinge & complain when all the free support vanishes and stuff... *cough*)

1 - Include ability to specify language per search instead of only having the default eg: "!g -fr blahblah"

2 - Also part of 1 - allow users to create a default setting for themselves ... perhaps register it in their bot handles, eg: "--XTRA incithgoogle.lang" or something.

3 - It'd be nice to be able to hard code the "allowed" or even "ignored" channels into the tcl/settings file too. While I think the +google setting is fantastic for dynamic channels, it creates a problem when you use a "source channels/settings" file to define the channels layouts in that eggdrop will crash if you define the channels prior to running this script.
(It complains about +google being an invalid channel setting.)

4 - Similar to 1 & 2 above, allow to use the "local" google for searching. Eg: www.google.com.au for me. Its a bit faster and stuff. Would be a lot better for the non-english language users too.

5 - Along the same lines/purposes of the language & local google ideas - allow domain/site narrowing. eg: "!g -.au blahblah". This often returns different results to just using the narrowing the "country" search to say "au". Would use the "site:.blah" modifier as per the below cheatsheets.

6 - More "!g define:" style extras from here: http://www.google.com/help/operators.html and http://www.google.com/help/cheatsheet.html
Stocks: is one of those I can certainly see myself using

PS: If you dont mind these ideas, I'm happy to do some code ideas and send you various bits to add in if you like. Royalty free too !!

PPS: Damn nice addon, it's given me a purpose in life

[incith]

[demond]

a little bit offtopic, but loading chan settings from a script is bad practice - the chanfile is there for a reason, and the reason is that, besides providing dynamic settings modification, it actually gets loaded after the config (and therefore after all scripts), thus allowing all [setudef]s to take place before reading in their actual values for a particular channel

so, +google chan setting should stay; and in general, no script should require configuring its channel functionality not via .chanset

[incith]

Alright, I'll keep the setudef functionality. It seemed cleaner this way as it is anyway. =)
_________________
; Answer a few unanswered posts!

[madwoota]

I wasnt saying to remove that functionality, only *add* the *option* to put it in the tcl/conf file.

It could remain #'d out to ignore it & default to a +google.

And saying you should or shouldnt do something is irrelevant since its dependant entirely on your own personal preference.

I use a setup that works for me, but its also very security concious, and hard codes a lot of variables within the files (eg: static channel definitions).

It used to be more of a problem back when people could more easily spoof hostmasks and join channels and such as a +n.


Anyway, incith, I've added two things already.
1 - Recognition for a "special match" eg: something like this, you will get the first results as a "special match" or whatever google calls it.

2 - groups.google.com search added via !gg !group and !groups

If you want either of these, let me know and I'll send you either the whole thing or a .diff.

[incith]

Oh, yeah. Google groups was another TODO thing that a friend wanted. =) (but I forgot about it..)

I just submitted incith:xrl (xrl.us url shortening service) and incith:filemirrors (filemirrors.com file searching), and I had to update horoscope for a couple folks that emailed me about it.. so at the moment I'm a bit Tcl'd out =)

I think I may try and clean up google before the next release, that while{} loop has grown monstrously scary..

As far as inline settable channels.. I suppose we could have a [check_channel] proc that would check for the setudef flag and the settable channels variable.

[madwoota]

[incith]

I've gone over madwootas changes, they'll 99% most likely be in the next release of incith:google, which will bring google groups support. =)

And the population thing too. *smirk*

Thanks, madwoota. =)
_________________
; Answer a few unanswered posts!

[madwoota]

incith,
news.google.com searching (!gn) is added.

I've emailed you another unified diff, but I couldnt be bothered pulling the previous changes out and just modding in the current bits for news.
So it includes all changes made so far against a default 1.4

I gather this is why cvs and such is handy

Next challenge - movie:, language, locality & stocks:.
... Although, since google stocks: actually uses yahoo, is that even appropriate for a google script ?

[incith]

Version 1.5 is up, support for print.google.com, news.google, groups, and a bunch more..

There's a bug in previous versions which allows a lot of potential channel spam to be generated, so you may want to update. =)
_________________
; Answer a few unanswered posts!

[kayuto]

hi incith

your script works great with "!" as conmmand char.
but if want to use another char there, your script doesn't work for me anymore ;|

for example, i normally use "§" in my other scripts. but i get no response if i use this char in yours ;(

[incith]

[incith]

Ok, well I've coded a solution for this, I'll try and get it into the release after the next one.. it is fixed in the next incith:weather script update.
_________________
; Answer a few unanswered posts!

[kayuto]

Thanks

[cooluser55]

I can't for the life of me find the require package http 2.3 .. after reading these posts here, I saw http-2.5.tcl and don't see that in my scripts file... can someone show me where I can get it? thanks.