DragnLord's weather script

stoneone · Post by **stoneone** » Fri Dec 19, 2008 7:19 pm

Didn't even notice. Thanks!

Suntop · Post by **Suntop** » Fri Dec 19, 2008 7:20 pm

stoneone wrote:Just make the changes to the .tcl file that I mentioned, .rehash, and it will work just fine.

I just did, thanx, works like a charm now

Suntop · Post by **Suntop** » Fri Dec 19, 2008 7:31 pm

random2 wrote:stoneone, the 745 fix needs to be inside the catch block (yep, in that big now-one-longer-line -- side note, the original coding style and back in weather4 even was better). Otherwise if there's a problem, you'll be writing to the closed socket. It needs to go right after the "GET ...".

The inconsistencies in the coding style here (look at the difference in all three of the places "GET" is in the code makes this rather unmaintainable code.

The script hangtough mentioned is, like this one, also a successor to murf's. If you want the "original" go back to murf's. I've looked at that one as well. I'm not fond of the output format of that one either, but note that the parser on that one is better (it looks at the properties of the html, sort of the way you would in XML [simplified explanation]) rather than the structure of the page. It also didn't need to be updated for this page breakage.

So how should it be now?

GrumpyBear · Post by **GrumpyBear** » Fri Dec 19, 2008 7:43 pm

stoneone wrote:Dragonlords script will work with the fixes mentioned before. My 4.2.4 had the mtemp problem and the outtitle problems also. For clarification the patches I made are shown below:

* SNIP *

Thanks.

AWESOME, its working perfect now, thanks for the hot fix

doubleu · Post by **doubleu** » Fri Dec 19, 2008 7:50 pm

yes thank you stoneone! my channel is happy again

stoneone · Post by **stoneone** » Fri Dec 19, 2008 8:11 pm

Yea, the coding style is a bit inconsistent. Actually, I used line returns and tabbed it out right before I posted my edit, and had to edit my edit when I realized how out of place my formatting seemed.

Sorry for the error, and to those who asked what it should be now, see the edit of my orig. post.

Cheers.

cache · Post by **cache** » Fri Dec 19, 2008 8:18 pm

stoneone wrote:Yea, the coding style is a bit inconsistent. Actually, I used line returns and tabbed it out right before I posted my edit, and had to edit my edit when I realized how out of place my formatting seemed. Sorry for the error, and to those who asked what it should be now, see the edit of my orig. post.

Cheers.

Thank You

pdq · Post by **pdq** » Sat Dec 20, 2008 6:55 am

thanks for fix stoneone.

=]

Suntop · Post by **Suntop** » Sat Dec 20, 2008 8:58 am

Yeah! Thanx for the fix stoneone

Now it really works like a charm

kato · Post by **kato** » Sat Dec 20, 2008 10:58 pm

Ok not sure on where to make changes anyone have a working copy of the tcl to give out, Thanks in advance

Kato

kato · Post by **kato** » Sat Dec 20, 2008 11:28 pm

it took me a little but got it right and its working correctly thanks for all the info

KATO

Meth · Post by **Meth** » Sun Dec 21, 2008 2:17 am

I made the appropriate edits and am still getting this error.

Code: Select all

[01:11] Tcl error [pub_wz]: can't read "wzsock": no such variable

Crow50 · Post by **Crow50** » Sun Dec 21, 2008 8:15 am

I used your edit post and it worked for me. I am a complete idiot when it comes to tcl, but you made it easy, thanks again.

Fl00k · Post by **Fl00k** » Sun Dec 21, 2008 4:46 pm

Can someone please post a fixed full script, i tried doing the fixes and i keep getting..
Tcl error [pub_wz]: can't read "wzsock": no such variable
Im a sure a few others are too.
A fixed version that is complete would be appreciated.

Thanks!

CtrlAltDel · Post by **CtrlAltDel** » Mon Dec 22, 2008 1:45 pm

###### Legal stuff ######
## This script is free to use and modify for personal use
## without warranty of any kind.
## Modified versions of this script may not be distributed
## without author's permission (open source != free domain).

but you can get murf's script @

http://www.egghelp.org/cgi-bin/tcl_arch ... oad&id=785

egghelp/eggheads community

DragnLord's weather script

working copy

I got it

Thanks !!