On Sep 7, 6:48 am, The Editor <[email protected]> wrote:
> Are you having plugins editing the page? Using the section edit links?
> or uploading changes to the script?

The problem was clicking on Edit.  It would just take me back to the
page.

> If the first, let me know, and I'll make sure you have edit status.
> Looks like you created the page however.
>
> If the second, I think the plugin is broken. Haven't had a chance to
> look into that yet. On my to do list.
>
> For the latter, add &reload=true to the url

Didn't seem to work.

> As for the plugin itself:  Kudos!  Very cool...
>
> 1) Is it possible to get support of php4? Or is it the xml parsing
> that is the issue...

The simplexml_load_file function was added in PHP5.  Really don't want
to parse XML myself like I used to.

Note about PHP versions though... PHP 5.3 is now stable (spent some
time fixing older scripts that used strip() because of it).
The last release of PHP 4 was Aug 2008.  It is no longer on the
download page at PHP.net which makes it very old now.

PHP 5.3.0 Stable was released Jun 2009.

> 2) I think returning the phpinfo() line pose a mild security risk, as
> it could expose your system configuration settings.  You might wrap it
> in something like this (with the proper globals of course):
>
>         if (strpos(",$BOLTadmin,", ",$BOLTid,") === false)
>
> This way only admins get the info. On the other hand, I think just the
> warning message is sufficient.

Nice to know that so that I can use that in the future.  I was only
using phpversion() though, not phpinfo().  Could have just said it was
an older version though without giving out the version.

However, Most web servers expose which version of  PHP you are using
in their headers when they serve pages Even if they are using
ServerTokens Prod which I recommend.  So it's not a big secret from
those that would use that information.

For example:  X-Powered-By: PHP/5.2.9

> 3) I made a few line spacing changes to get it to look how I wanted.
> Other than that, very coolll...
> 4) Oh also, why not rename the page and plugin to something more
> simple, like "weather.

I was going to call it just weather, but I plan on a number of other
weather plugins, some that work with specific Weather reporting
packages like Weather-Display and Cumulus.  So I figured this was a
basic one for those who don't have their own weather station.

> 5) And too, your document is excellent. If only all my plugins were so
> nicely done!
>
> One nifty use of this might be to ask a person's city/state when they
> register and store them on their profile. Then when they go to the
> weather page, it could automatically supply that info to the function
>
> [(wxbasic city="{~city}" state="{~state}")]

Learning something every day.... that would work for everything except
the weather radar image.  I don't have anyway to really look that up
to know which one fits for which city.

>
> This is a GREAT first contribution. I am going to put it on my
> personal site I think!  Thanks...

Thanks!  Was a good learning experience...

> Cheers,
> Dan

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"BoltWire" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/boltwire?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to